[mythtv-commits] Ticket #7638: DVD: wrong audio track selected when switching
MythTV
mythtv at cvs.mythtv.org
Sat Dec 5 19:16:53 UTC 2009
#7638: DVD: wrong audio track selected when switching
------------------------------------+---------------------------------------
Reporter: linux@… | Owner: skamithi
Type: defect | Status: accepted
Priority: minor | Milestone: unknown
Component: MythTV - General | Version: unknown
Severity: medium | Mlocked: 0
------------------------------------+---------------------------------------
Comment(by linux@…):
Hi,
dvd-audio-stream-ordering-v2.patch does not work for me, namely because
dvdnav_get_audio_logical_stream does not work as expected. If you feed it
the stream id the output will always be -1 (from looking at the code I
guess it only maps some internal numbers as the input stream id must be <=
8 to get anything besides -1). So in fact your patch will behave exactly
as before my original patch as dvd_track_num will always be set to -1.
In my second patch (dvd-audio-stream-ordering.dpatch) I have actually put
the functionality that one thinks dvdnav_get_audio_logical_stream provides
in the function dvd_get_logical_audio_stream_id. It will map the stream
ids to their logical track nos (the disc I am always having problems with
has a mix of AC3 and DTS tracks and the problem is that the AC3 tracks
have a stream id range between 128 and 135 while DTS tracks start a 136;
so in my example, the second logical track has stream id 137 which used to
be put at the end of the track map by simple sorting).
I personally think that providing the dvd-specific comparator for the
sorting a simpler solution now as it keeps irrelevant dvd-specific stuff
out of StreamInfo and its operator which seems to be a more generic
structure. The sorting only happens in the DVD case anyway so having a DVD
specific comparator should not be a problem.
As a sidenote, I was also wondering of whether the helper functions I
added belong into avformatdecoder.cpp or DVDRingbuffer.h and it should be
no problem to move them there.
Best wishes,
Martin.
--
Ticket URL: <http://svn.mythtv.org/trac/ticket/7638#comment:6>
MythTV <http://www.mythtv.org/>
MythTV
More information about the mythtv-commits
mailing list