[FFmpeg-devel] [RFC] comment mpegvideo_split

[Baptiste Coudurier]

Michael Niedermayer wrote:
> On Tue, Mar 18, 2008 at 01:16:34AM +0100, Baptiste Coudurier wrote:
>> Hi,
>>
>> Currently, mpegvideo split function causes problems with containers
>> setting AVFMT_GLOBAL_HEADER, mov and mp4 in particular, because
>> MPEG-1/2 in those containers are to be muxed normally.
>>
>> I don't know any particular usage of the mpegvideo split function, so I
>> tend to want to comment it, to finally fix this issue.
>>
>> Anyone see another better alternative ?
> 
> After a few more seconds :)
> The muxers could just dump extradata for the case of mpeg1/2 before keyframes,
> that should fix it i think. Its of course a little odd to first extract and
> then put it back in the muxer.
> 

Would the attached patch be ok ?

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG SAS                                     http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mpeg2_mov.patch
Type: text/x-diff
Size: 977 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080319/197efdc4/attachment.patch>