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

Reply via email to