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

[Carl Eugen Hoyos]

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

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel