Re: [FFmpeg-devel] [PATCH 0/2] Support for TrueHD streams with an Atmos substream

2014-09-21 Thread Michael Niedermayer
On Sat, Sep 20, 2014 at 11:07:53PM -0300, James Almer wrote:
> On 20/09/14 10:59 PM, Michael Niedermayer wrote:
> > On Sun, Sep 21, 2014 at 02:29:26AM +0200, Hendrik Leppkes wrote:
> >>
> >> Hey,
> >>
> >> this patchset adds support for playing TrueHD streams on new Blu-ray discs
> >> which carry an additional Atmos substream inside the TrueHD stream.
> >>
> >> The Atmos stream is simply discarded right now, since we don't have any way
> >> to decode or expose it, but it might be feasible to shove it into side data
> >> in the future in case the user app knows what to do with it.
> > 
> > Is there a testcase / (small) sample available somewhere ?
> 
> https://trac.ffmpeg.org/ticket/3960#comment:6

tested

applied

thanks


[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH 0/2] Support for TrueHD streams with an Atmos substream

2014-09-20 Thread James Almer
On 20/09/14 10:59 PM, Michael Niedermayer wrote:
> On Sun, Sep 21, 2014 at 02:29:26AM +0200, Hendrik Leppkes wrote:
>>
>> Hey,
>>
>> this patchset adds support for playing TrueHD streams on new Blu-ray discs
>> which carry an additional Atmos substream inside the TrueHD stream.
>>
>> The Atmos stream is simply discarded right now, since we don't have any way
>> to decode or expose it, but it might be feasible to shove it into side data
>> in the future in case the user app knows what to do with it.
> 
> Is there a testcase / (small) sample available somewhere ?

https://trac.ffmpeg.org/ticket/3960#comment:6

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH 0/2] Support for TrueHD streams with an Atmos substream

2014-09-20 Thread Michael Niedermayer
On Sun, Sep 21, 2014 at 02:29:26AM +0200, Hendrik Leppkes wrote:
> 
> Hey,
> 
> this patchset adds support for playing TrueHD streams on new Blu-ray discs
> which carry an additional Atmos substream inside the TrueHD stream.
> 
> The Atmos stream is simply discarded right now, since we don't have any way
> to decode or expose it, but it might be feasible to shove it into side data
> in the future in case the user app knows what to do with it.

Is there a testcase / (small) sample available somewhere ?

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


[FFmpeg-devel] [PATCH 0/2] Support for TrueHD streams with an Atmos substream

2014-09-20 Thread Hendrik Leppkes

Hey,

this patchset adds support for playing TrueHD streams on new Blu-ray discs
which carry an additional Atmos substream inside the TrueHD stream.

The Atmos stream is simply discarded right now, since we don't have any way
to decode or expose it, but it might be feasible to shove it into side data
in the future in case the user app knows what to do with it.

The changes are in two parts:
1) Our TrueHD sync header parsing was not perfect, it didn't account for an
optional extension header at the end, which was never used before, but now it 
is.
Its simply skipped.

2) Being aware of and discarding the extra substream.

The changes for the first patch are derived from a disasm of a binary encoder.

- Hendrik


___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel