[FFmpeg-devel] [PATCH] MXF index table based seeking

Carl Eugen Hoyos cehoyos at ag.or.at
Sun Aug 7 19:49:34 CEST 2011


Georg Lippitsch <georg.lippitsch <at> gmx.at> writes:

> -    av_dlog(s, "size %"PRIu64" offset %#"PRIx64"\n", klv.length, klv.offset);
> +    av_dlog(s, "size %lld offset %#llx\n", klv.length, klv.offset);

This looks unintended, or am I missing something?

> -            return 0;
> +            tr->current_sample_index++;

> +            return 0; 

Trailing whitespace cannot be committed to the FFmpeg repository, please
consider using tools/patcheck.

Possibly no need to resend before a real review, Carl Eugen



More information about the ffmpeg-devel mailing list