=== rob_w_away is now known as rob_w === MacSlow is now known as MacSlow|lunch === MacSlow|lunch is now known as MacSlow [13:54] investigating lp:827958 [14:03] bregma, which is that? [14:05] libgrip crash at gdk_x11_window_get_xid() in eog [14:05] looks like a latent bug in eog [14:06] can't be sure 'til I've tracked it down [14:07] yeah [14:07] bregma, seb128 is hounding me about it [14:07] I asked Satoris to take a look [14:07] saw that, that's why I'm looking at it [14:07] cool [14:09] I think a good step would be to reproduce it on a build that does not have the utouch patch. [14:10] indeed [14:10] it would be great if we could do that :) [14:10] I'm working on that now: I can reproduce it with the current binary [14:10] * cnd is selfish [14:10] oh, that's good [14:10] I didn't know how hard it would be to reproduce [14:11] I do note that there are many warnings emitted because some functions return NULL and there are no logic checks for tat [14:11] bregma, in our code? [14:11] no, in eog [14:12] most functions check their parameters and return erly instead of having logic that handles the error condition where it occurs [14:12] libgrip instead assumes it is handed valid data [14:14] I think this is the cause of the problem, libgrip assumes object invariants hold, eog does not and checks validity of everything all the time [14:14] different design philosophies [14:15] ahh === rob_w is now known as rob_w_away [15:34] there is definitely a logic error in libgrip involving maze of twisty little passages, all different === MacSlow is now known as MacSlow|break [16:40] bregma, does that mean you can't repro without the libgrip addition? [16:43] correct [16:43] ok [16:44] it looks like a widget and its toplevel get out of synch somehow during a convoluted sequence of signals [16:44] it's a matter of making sure everything is lined up nicely before trying to get actual X11 info [16:45] it is indeed a little maze of twisty passages, all different === MacSlow|break is now known as MacSlow