some bugs

[vlad]

hello,

yesterday i've installed the new 3.4.6 version. i haven't noticed that
there was a new version on the way, else i would have it installed and tested
earlier. it would be useful if testing versions are also announced on
this ml. 
first bug:
i've tried the <bar> and <raise> options with the NextWindow entries in
rc.xml. this leads to an openbox crash. 
in my rc.xml i have following: 
    <keybind key="A-Tab">
      <action name="NextWindow">
      <bar>no</bar>
      <raise>yes</raise>
        <finalactions>
          <action name="Focus"/>
          <action name="Raise"/>
        </finalactions>
      </action>
    </keybind>
now when pressing alt-tab ob crashes. sometimes it crashes directly,
sometimes after 2-3 times. 
i start openbox through startx running "ssh-agent /usr/bin/openbox" in my .xinitrc. 
console output after crashing:
"
How are you gentlemen? All your base are belong to us. (Openbox received signal 11)
/home/vlad/.xinitrc: line 82: 20533 Aborted                 ssh-agent /usr/bin/openbox

waiting for X server to shut down kdeinit: Fatal IO error: client killed
kdeinit: sending SIGHUP to children.
klauncher: Exiting on signal 1
XIO:  fatal IO error 4 (Interrupted system call) on X server ":0.0"
      after 187 requests (183 known processed) with 0 events remaining.
XIO:  fatal IO error 4 (Interrupted system call) on X server ":0.0"
      after 60 requests (58 known processed) with 0 events remaining.
kdeinit: sending SIGTERM to children.
kdeinit: Exit.
yeahconsole:  fatal IO error 11 (Resource temporarily unavailable) or KillClient on X server ":0.0"
urxvt: X connection to ':0.0' broken, unable to recover, exiting.
xterm:  fatal IO error 11 (Resource temporarily unavailable) or KillClient on X server ":0.0"
"
i guess only the first two lines are of some importance. 

second: 
the client-combined-menu doesn't show the desktop names but only "manage
desktops" (see screenshot: http://img248.imageshack.us/img248/9868/screenshotfrom20080203afq2.png). 
i don't think that's intended. 

now a feature request: 
could you please make the "manage desktop"-stuff in the client-combined-menu optional.
for now i comment following lines in client_list_menu.c out:
lines 155-159:
"
    /*
    menu_add_separator(menu, SEPARATOR, NULL);
    menu_add_normal(menu, ADD_DESKTOP, _("_Add new desktop"), NULL, TRUE);
    menu_add_normal(menu, REMOVE_DESKTOP, _("_Remove last desktop"),
                    NULL, TRUE);
    */
"
but that's only a dirty workaround.

that's all and thank you,

vlad 

--