[mythtv] [mythtv-commits] mythtv commit: r9291 by danielk

Kevin Kuphal kuphal at dls.net
Wed Mar 8 05:46:13 UTC 2006


Daniel Kristjansson wrote:
> On Tue, 2006-03-07 at 22:10 -0600, Kevin Kuphal wrote:
>   
>> mythtv at cvs.mythtv.org wrote:
>>     
>>>       Author: danielk
>>>         Date: 2006-03-08 01:48:29 +0000 (Wed, 08 Mar 2006)
>>> New Revision: 9291
>>>    Changeset: http://cvs.mythtv.org/trac/changeset/9291
>>>
>>> Modified:
>>>
>>>    trunk/mythtv/libs/libmythtv/DVDRingBuffer.cpp
>>>    trunk/mythtv/libs/libmythtv/NuppelVideoPlayer.cpp
>>>    trunk/mythtv/libs/libmythtv/NuppelVideoPlayer.h
>>>    trunk/mythtv/libs/libmythtv/avformatdecoder.cpp
>>>    trunk/mythtv/libs/libmythtv/avformatdecoder.h
>>>    trunk/mythtv/libs/libmythtv/decoderbase.cpp
>>>    trunk/mythtv/libs/libmythtv/decoderbase.h
>>>    trunk/mythtv/libs/libmythtv/tv_play.cpp
>>>    trunk/mythtv/libs/libmythtv/tv_play.h
>>>
>>> Log:
>>>
>>> References #1095. UI cleanup for captions/subtitle/teletext.
>>>
>>>   
>>>       
>> Daniel,
>>
>> It looks like this has broken some of the OSD handling, specifically now 
>> the
>>
>>     handled = HandleTrackAction(action);
>>
>> in TreeMenuSelected which sets handled to false if the action is not a 
>> track action.  This basically causes the tree to act like no action is 
>> ever handled.  I'm not sure specifically how to fix as I haven't had a 
>> chance to examine the code but if you could...
>>     
> This should probably be:
>     handled |= HandleTrackAction(action);
>
> I'll test this tomorrow morning.
>   
I tested that just now and it didn't seem to work.  In think I got it in 
[9293].  Give it a whirl and verify it.  Thanks,

Kevin


More information about the mythtv-dev mailing list