[FFmpeg-devel] [PATCH] libavcodec/qsvdec_h2645.c Bug fixed: wrong ticks_per_frame.
h.leppkes at gmail.com
Mon Apr 25 17:50:18 CEST 2016
On Mon, Apr 25, 2016 at 5:44 PM, Ivan Uskov <ivan.uskov at nablet.com> wrote:
> Hello Derek,
> Monday, April 25, 2016, 4:50:28 PM, you wrote:
> DB> On 4/25/2016 2:14 PM, Ivan Uskov wrote:
>>> The attached patch does fixes the issue of frames duplication when
>>> elementary h.264 stream decodes by qsvdec.
> DB> Could you perhaps elaborate in the commit message, and a code comment, on
> DB> why it must be 2? Where does that value come from, etc.
> I do not see a necessity for additional comments because
> AVCodecContext::ticks_per_frame is public API and already enough documented:
Its not about what the field does, but why this is needed here. So
please explain why its set here, and only for this case.
More information about the ffmpeg-devel