[14:09] <dandrader> bregma, would you like to review that or can I just give to the autolanding machinery? https://code.launchpad.net/~dandrader/frame/non_x11_accept/+merge/138159
[14:09] <bregma> dandrader, I'll just take a look right now
[14:09] <dandrader> bregma, awesome, thanks!
[14:12] <bregma> dandrader, are you using constructs like " (void)window_id;" instead of __attribute__((unused)) ?
[14:12] <dandrader> bregma, yes, so the latter is the C++ way of doing it?
[14:13] <bregma> the latter is a GCC extension
[14:13] <bregma> it's used elsewhere in frame but it's not pretty
[14:13] <bregma> I was just confirming that's what you were doing
[14:13] <dandrader> ok
[16:56] <dandrader> bregma, and that one: https://code.launchpad.net/~dandrader/grail/convert_tests/+merge/138253
[16:56]  * dandrader has a lot of spare time for OIF housekeeping
[17:09] <bregma> lucky you
[17:15] <dandrader> well, actually that's not a good thing.