/srv/irclogs.ubuntu.com/2016/12/05/#ubuntu-mir.txt

RAOFYou know what would be ace? If our mutexs had some form of documentation of the invariants they're protecting.00:51
RAOFHm. I might have managed megabytes of valgrind output here!01:37
dufluFinely valground01:50
RAOFHah. That output gets much more readable if you spread it out over the width of two monitors!01:59
RAOFOh!02:24
RAOFYes, that's right.02:24
RAOFMirWaitHandle being terrible.02:24
dufluI'm pretty sure MirWaitHandle works perfectly. If it's not perfect then add a regression test for what fails. But we may also not need it...02:33
RAOFRight, it works, but it's architecturally horrible.02:41
RAOF(Mostly works; it still doesn't distinguish responses)02:41
dufluIndeed, it was invented by someone who only envisioned single responses. Then fixed by someone (me) who realised it failed for multiple responses. Then later probably can simply be not mentioned in the client API in future03:02
dufluAlthough I much prefer using a wait object of some form than callbacks. Anything is nicer than callbacks03:03
dufluAnd you could convert a wait object into a callback if it was well designed03:04
RAOFI was in favour of foo_begin/foo_complete at the beginning.04:09
RAOFThat would be nice wait-objects :)04:09
RAOFOh, dear.04:10
RAOFMocks.04:10
RAOFOh, good.06:35
RAOFPulling in more recent trunk and resolving the conflicts *also* resolves some of the test errors. Yay!06:35
RAOFWoot!06:46
RAOFNow down to 5 sets of failures.06:46
=== seb128_ is now known as seb128
alan_gdandrader: would this address some of your concerns about moving child windows? https://code.launchpad.net/~alan-griffiths/miral/fix-1646735/+merge/31244709:55
dandraderalan_g, will comment on the mp10:11
alan_ggreyback: your position on this? https://code.launchpad.net/~alan-griffiths/miral/fix-1646735/+merge/312447/comments/81108911:08
greybackalan_g: needed to give it a think. Yep I think your suggestion is worth a go (commented as much)11:24
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== hikiko is now known as hikiko|ln
=== hikiko|ln is now known as hikik
=== hikik is now known as hikiko
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== Saviq_ is now known as Saviq
=== Trevinho_ is now known as Trevinho

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!