Re: DVB OSD Menu bug hunting

[email protected] wrote:

>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.
>  
>
confirm

>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).
>  
>
maybe I should reset the osd

>3) Feature, not bug: when changing channel the underlying xterm is flashed/workbench
>shown before opening the new selected channel.
>  
>
I can't confirm this, it doesn't happen to me.

>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
>
>
>  
>
I'll fix as soon as possible, but at the moment I'm busy with the demuxer.

            Nico