[FFmpeg-devel] TrueHD track in EVO not playable/testable with ffplay

Michael Niedermayer michaelni
Fri Jul 10 03:55:03 CEST 2009

On Thu, Jul 09, 2009 at 06:47:19PM -0700, Baptiste Coudurier wrote:
> On 07/09/2009 06:45 PM, Michael Niedermayer wrote:
>> On Thu, Jul 09, 2009 at 03:24:24PM +0100, M?ns Rullg?rd wrote:
>>> Michael Niedermayer<michaelni at gmx.at>  writes:
>>>> On Wed, Jul 08, 2009 at 11:01:57PM -0700, Baptiste Coudurier wrote:
>>>>> On 07/08/2009 07:47 PM, Michael Niedermayer wrote:
>>>>>> my tired mind says we need a
>>>>>> coded_channels representing the number of channels stored in the 
>>>>>> stream
>>>>>> and a seperate field that represents the decoder output channels
>>>>> I suggest to keep ->channels as the coded channels and to use another 
>>>>> field
>>>>> like output_channels.
>>>> I wouldnt mind, if we wouldnt have
>>>> coded_width&  coded_height in there
>>> This is different.  Unless I am mistaken, coded_width and coded_height
>>> indicate the encoded width/height of a video including any padding
>>> (e.g. mod-16) required by the codec but never meant to be displayed.
>> I think you are mistaken
>> if my memory doesnt fail me coded_width/height do not take the padding
>> into account.
>> width/height take lowres into account coded_widt/height do not
>> this can be seen in avcodec_set_dimensions()
> Hint: something is weird and definitely not obvious somewhere ;)

so you should fix it or tell us what is where not obvious ...

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090710/6c7d4c6b/attachment.pgp>

More information about the ffmpeg-devel mailing list