On Sat, Mar 05, 2016 at 01:41:32PM +0100, Michael Niedermayer wrote: > + * Note, for variable framerate material this has undefined meaning > currently > + * and is not set to the actual framerate nor {0,1} but can be set to > + * 1/timebase (FIXME) the code in some parts assumes framerate == > 1/timebase > + * which is generally not true but these parts need to be fixed before > framerate > + * can be fixed.
This is too complex, maybe it can be split in multiple sentences. Suggestion (but I took a lot of liberty and it might not say what you meant!): Note: meaning is not yet defined for variable frame rate content. Ideally it should be set to either the current frame rate or {0,1} for that case. However for this some code assuming framerate == 1/timebase needs to be fixed first - until then this field may incorrectly be set to 1/timebase. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel