[Mplayer-dvb] DVB OSD Menu bug hunting

[petri.nykanen at suomi24.fi]

I guess this is question you Nico - saw your name in the OSD code :)

I installed the latest CVS version today, and got the DVB OSD menu to work,
however there are a couple of items that potentially are bugs waiting for
fixing:

I have the following more or less standard menu.conf:

....

<dvbsel title="DVB Channels" name="dvbsel" />
 
<cmdlist name="main" title="MPlayer 0.90" ptr="<>" >
      <e name="DVB" ok="set_menu dvbsel"/>
      <e name="Pause" ok="pause"/>
      <e name="Prev/Next" ok="pt_step 1" cancel="pt_step -1"/>
      <e name="Jump to ..." ok="set_menu jump_to"/>
      <e name="Open ..." ok="set_menu open_file"/>
      <e name="Open playlist ..." ok="set_menu open_list"/>
      <e name="Help" ok="set_menu man"/>
      <e name="Pref" ok="set_menu pref_main"/>
      <e name="Console" ok="set_menu console0"/>
      <e name="Quit" ok="quit"/>
</cmdlist>

Now, when opening OSD and entering the DVB Channels menu:

1) if you try to go up from the first menu channel item mplayer crashes.

2) if you select any channel item, the channel changes ok, but

     the OSD stays on, the mplayer (full screen at least) loses keyboard
focus, meaning that cursor keys do not work in the menu until I select it
again with mouse (also the mouse0 key is my command to bring up the menu).

3) Feature, not bug: when changing channel the underlying xterm is flashed/workbench
shown before opening the new selected channel.

Here maybe the function could be either to have full screen image (std theme
image?) to default to or to set black screen while changing to make the channel
transition more appealing?

Anyhow, hope these help anyone with familiarity with the codebase.

Petri


_____________________________________________________________
Kuukausimaksuton nettiyhteys: http://www.suomi24.fi/liittyma/
Yli 12000 logoa ja soittoääntä: http://sms.suomi24.fi/