[12:59] <alan_g> greyback: happy yet? https://code.launchpad.net/~alan-griffiths/miral/encapsulate-mir-PromptSession/+merge/307823
[13:00] <greyback> alan_g: there is still style mismatch in promptsessionmanager.cpp
[13:01] <greyback> and its header
[13:05]  * alan_g tries again
[13:46] <greyback> alan_g: can you repro this bug 1631958 ?
[13:49] <alan_g> greyback: will check shortly
[13:50] <greyback> thanks
[13:55] <alan_g> looking at the line of code in the backtrace it seems that there's a recursive parent-child relationship. What fun!
[14:05] <alan_g> greyback: confirmed
[14:05] <greyback> alan_g: thanks. Been a while since i've seen a stack overflow :)
[14:13] <alan_g> greyback: I want to add a test before proposing, but you can preview: lp:~alan-griffiths/miral/fix-1631958
[14:13] <greyback> on it
[14:16] <greyback> alan_g: yep that fixes it
[16:23] <greyback> it is made clear anywhere exactly what events are passed through the mir_surface_event_callback ?
[16:26] <greyback> ever mind
[16:26] <greyback> never
[16:41] <alan_g> greyback: MP'd
[16:41] <greyback> ack