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