Repeatable vdr 1.3.35 crash at end of recording

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 04, 2005 at 02:51:01PM +0100, Klaus Schmidinger wrote:
> Marko M?kel? wrote:
> >...
> >BTW, is there a reason why VDR discards the currently displayed menu
> >at the end of a recording?
> 
> Not really, it just arose that way.
> 
> This should fix it:
> 
> 
> --- vdr.c       2005/10/09 10:01:45     1.218
> +++ vdr.c       2005/11/04 13:48:39
> @@ -813,8 +813,14 @@
>          if (Interact) {
>             eOSState state = Interact->ProcessKey(key);
>             if (state == osUnknown && Interact != cControl::Control()) {
> -              if (ISMODELESSKEY(key) && cControl::Control())
> +              if (ISMODELESSKEY(key) && cControl::Control()) {
>                   state = cControl::Control()->ProcessKey(key);
> +                 if (state == osEnd) {
> +                    // let's not close a menu when replay ends:
> +                    cControl::Shutdown();
> +                    continue;
> +                    }
> +                 }
>                else if (time(NULL) - LastActivity > MENUTIMEOUT)
>                   state = osEnd;
>                }

Thanks!  Out of curiosity, I enabled the main menu (the one that shows
"stop playback" as the last entry) during playback.  The item did not
go away when the playback ended.  Of course, I had to select that item
when softdevice was already showing live program.  It went black for
a couple of seconds and then resumed playing the live program.

For me, it's just a cosmetic thing.  It's very nice that I can play with
the menus (such as browse the EPG) while watching a recording that is
about to end.  However, perhaps someone with a hardware MPEG decoder
(FF card or DXR3) should test this patch?

	Marko


[Index of Archives]     [Linux Media]     [Asterisk]     [DCCP]     [Netdev]     [Xorg]     [Util Linux NG]     [Xfree86]     [Big List of Linux Books]     [Fedora Users]     [Fedora Women]     [ALSA Devel]     [Linux USB]

  Powered by Linux