[FFmpeg-devel] [PATCH] Bug fixed: wrong ticks_per_frame. For H.264 stream ticks_per_frame should be 2
ivan.uskov at nablet.com
Tue Apr 26 16:21:10 CEST 2016
Monday, April 25, 2016, 6:50:18 PM, you wrote:
HL> 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:
HL> Its not about what the field does, but why this is needed here. So
HL> please explain why its set here, and only for this case.
A have added brief comment to patch and to code, please review the attached patch.
Ivan mailto:ivan.uskov at nablet.com
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1017 bytes
Desc: not available
More information about the ffmpeg-devel