[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