[00:14] <seif_> vish, u there?
[00:24] <Cimi> bratsche: are you here? :)
[03:25] <bratsche> Cimi: I am now.
[03:40] <Cimi> bratsche: why there's 1px of menubar in every window when the menubar is in the appmenu
[03:41] <Cimi> the lowest pixel
[03:45] <Cimi> bratsche: sleeping again
[03:45] <Cimi> bratsche: good night you too
[18:21] <chrisccoulson> klattimer, did you figure out the g-s-d 100% CPU issue?
[18:33] <klattimer> chrisccoulson: nope
[18:33] <klattimer> :/
[18:33] <chrisccoulson> klattimer, i think i've got it, just about to test a patch now
[18:34] <klattimer> go for it
[18:34] <klattimer> what is it?
[18:34] <klattimer> chrisccoulson: ?
[18:34] <chrisccoulson> klattimer, calling xkl_engine_lock_group in popup_menu_set_group generates another XKB event
[18:35] <klattimer> hmm
[18:35] <chrisccoulson> which just starts the whole cycle over and over
[18:35] <chrisccoulson> i can see that with xtrace
[18:36] <klattimer> chrisccoulson: that's from the original code though, so it should be upstream?
[18:37] <chrisccoulson> klattimer, no, the issue is that we seem to be calling popup_menu_set_group from different locations, and expecting it to do more than it was ever intended to do
[18:37] <klattimer> yeah, maybe
[18:37] <chrisccoulson> it only ever used to be called in response to a menu actication
[18:37] <chrisccoulson> but now we're calling it on XKB events too
[18:37] <klattimer> now it's also responding to xkb events too
[18:37] <klattimer> this is what the NULL item was for
[18:38] <klattimer> if item is null then we've come in from xkb
[18:38] <chrisccoulson> yeah, i've just added another NULL item check
[18:38] <chrisccoulson> just about to test that now
[18:38] <klattimer> cool
[18:38] <chrisccoulson> but we should really split this function up, as it seems to be doing 2 entirely different things depending on what you pass to it
[18:39] <klattimer> chrisccoulson: https://blueprints.edge.launchpad.net/ubuntu/+spec/dx-n-indicator-keyboard
[18:39] <klattimer> it's mostly a kludge atm
[18:39] <klattimer> the whole thing needs to be rewritten both upstream and down
[18:39] <klattimer> there's some ui code in libgnomekbd ffs
[18:40] <klattimer> half of the status icon
[18:44] <chrisccoulson> yeah, it is quite a mess