| seif_ | vish, u there? | 00:14 |
|---|---|---|
| Cimi | bratsche: are you here? :) | 00:24 |
| === dennis_ is now known as GreySim | ||
| bratsche | Cimi: I am now. | 03:25 |
| Cimi | bratsche: why there's 1px of menubar in every window when the menubar is in the appmenu | 03:40 |
| Cimi | the lowest pixel | 03:41 |
| Cimi | bratsche: sleeping again | 03:45 |
| Cimi | bratsche: good night you too | 03:45 |
| === MacSlow is now known as MacSlow|lunch | ||
| === MacSlow|lunch is now known as MacSlow | ||
| chrisccoulson | klattimer, did you figure out the g-s-d 100% CPU issue? | 18:21 |
| === oubiwann is now known as oubiwann-away | ||
| === oubiwann-away is now known as oubiwann | ||
| klattimer | chrisccoulson: nope | 18:33 |
| klattimer | :/ | 18:33 |
| chrisccoulson | klattimer, i think i've got it, just about to test a patch now | 18:33 |
| 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:34 |
| klattimer | hmm | 18:35 |
| chrisccoulson | which just starts the whole cycle over and over | 18:35 |
| chrisccoulson | i can see that with xtrace | 18:35 |
| klattimer | chrisccoulson: that's from the original code though, so it should be upstream? | 18:36 |
| 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:37 |
| 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:38 |
| 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:39 |
| klattimer | half of the status icon | 18:40 |
| chrisccoulson | yeah, it is quite a mess | 18:44 |
| === oubiwann is now known as oubiwann-away | ||
| === oubiwann-away is now known as oubiwann | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!