[mythtv-commits] Ticket #6395: AVFormatDecoder Language/codec detection improvements
MythTV
mythtv at cvs.mythtv.org
Sat Mar 28 03:43:25 UTC 2009
#6395: AVFormatDecoder Language/codec detection improvements
---------------------------------------+------------------------------------
Reporter: robert.mcnamara at gmail.com | Owner: ijr
Type: patch | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - Video Playback | Version: head
Severity: medium | Mlocked: 0
---------------------------------------+------------------------------------
Two small improvements for AVFormatDecoder/The internal player.
The first improvement pertains to HD-DVD and Blu Ray support. TrueHD
audio is a mandatory primary audio format on HD-DVD (although not all
disks include it as required). It's a non-mandatory codec on Blu-ray.
That said, it is guaranteed to be more likely to be the primary audio
track than DTS or AC3 when present. This adds a preference for TrueHD
over those two codecs. This makes the audio track selection for all my
HD-DVD rips correct (Trunk has some other issues with this content but
those appear to be resolved in ffmpeg SVN so the next resync should fix
them).
Note that in FFMpeg trunk the MLP codec has been split out into two
codecs, CODEC_ID_MLP and CODEC_ID_TRUEHD. This patch works with current
trunk but on the next ffmpeg sync it will need to be changed to
CODEC_ID_TRUEHD.
The second improvement pertains to Language selection. Myth attempts to
select an audio track first by language, then without. Regrettably, the
setting for which language it chooses is hidden in appearance and called
"Guide Language Preference" #1 and #1. It doesn't even indicate that it
pertains to audio track selection. A complaint from a user today got me
thinking about this.
As a solution, I have changed the primary selection method to first try a
track matching the Frontend language, then to fall back to the guide
language preferences. I think this is much more intuitive and saves
having to shuffle those settings around for no good reason.
--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6395>
MythTV <http://www.mythtv.org/>
MythTV
More information about the mythtv-commits
mailing list