[FFmpeg-devel] [PATCH] allow ac3 parser to modify codec_id

Justin Ruggles justin.ruggles
Thu Feb 19 01:05:02 CET 2009


Michael Niedermayer wrote:
> On Wed, Feb 18, 2009 at 05:55:15PM -0500, Justin Ruggles wrote:
>> Hi,
>>
>> elupus wrote:
>>> Attached patch makes the AC3 parser modify codec_id so that eac3 streams
>>> are properly marked in for example some mpeg-ts stream from blueray and
>>> hddvd
>> The potential problem I see with this is that AC3 and E-AC3 frames can
>> be mixed in the same stream.  So with this patch, the codec_id would be
>> changed after every frame for such streams.  
> 
>> If the parser were modified
>> so that it groups together an independent frame with its subsequent
>> dependent frames and returns them as a single frame, 
> 
> i thought the parser did that already ...

Hmm... it seems you're right. The decoder just doesn't handle it
properly.  I'll work on that.  The patch is ok then and I'll apply it soon.

Thanks,
Justin





More information about the ffmpeg-devel mailing list