[FFmpeg-devel] [PATCH] Re: movenc.c: Allow MPEG2.mov playable under QuickTime Player
Takashi Mochizuki
mochi
Sat Feb 12 23:37:20 CET 2011
>>Thanks, could you please split the patch and just have this one hunk.
>>It can be applied right now.
>
>OK. Here it is.
Hi, is there any issue still now prior to commit?
>Since we parse the mpeg2 frame and retrieve the picture type,
>you can set pts == dts for b frames, for P and I you can compute the pts
>from either the temp_ref or by counting the number of b frames after,
>although I think the temp_ref is the correct way.
I think it is not good way. If I use temporal_reference value to calculate frame pts,
it also requires each frame duration is constant value.
mp4/mov container can handle VFR - thus each sample can have different duration.
To make delayed frame's pts value such like P frame or OpenGop's I frame,
I have to evaluate it later - add up each B-frame's duration, then delayed frame can
be fixed. (that is the way my previous patch does)
Takashi Mochizuki
More information about the ffmpeg-devel
mailing list