[FFmpeg-devel] [PATCH] lavf/utils: Fix DTS for short H264 streams.

Sasi Inguva isasi at google.com
Fri Mar 11 23:52:44 CET 2016


We get significant uploads of short timelapse videos which are of very low
frame rate very short length videos, and contain 5-6 frames.
We also have lot of teams using very small test-files, so that they can
test their pipeline very fast.

Sorry for the unintended space insertion.

I couldn't figure out what return value maps to eof and what return value
maps to error , by reading the code of read_frame_internal . If you have an
idea, I can include the return value condition there.

Thanks

On Fri, Mar 11, 2016 at 1:52 PM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:

> Sasi Inguva <isasi-at-google.com <at> ffmpeg.org> writes:
>
> > Fill DTS if all packets have been read in
> > avformat_find_stream_info, and still
> > has_decode_delay_been_guessed returns false.
>
> Could you explain your use-case?
> Are streams with less than seven frames frequent?
>
> >          return 1;
> >  #endif
> > +
>
> No unrelated changes please.
>
> >          if (ret < 0) {
> >              /* EOF or error*/
> > +            eof_reached = 1;
> >              break;
> >          }
>
> Doesn't the comment indicate that you should
> check if you really reached eof?
>
> Carl Eugen
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list