[FFmpeg-devel] [PATCH] mov.c: read fragment start dts from fragmented mp4

Carl Eugen Hoyos cehoyos at ag.or.at
Sat Oct 11 15:41:28 CEST 2014


Mika Raento <mikie <at> iki.fi> writes:

> +    if (ret < 0)
> +        av_log(c->fc, AV_LOG_ERROR,
> +               "failed to seek back after looking for mfra\n");
> +    else
> +        ret = 0;

If you resubmit please consider making this:
if (ret < 0) {
  avl_log();
} else {
  ret = 0;
}

This makes future patches much easier to read and costs only 
a line.

Thank you, Carl Eugen



More information about the ffmpeg-devel mailing list