On Mon, Jan 30, 2012 at 07:34:43AM +0100, Anton Khirnov wrote: > > On Mon, 30 Jan 2012 00:34:12 +0100, Diego Biurrun <[email protected]> wrote: > > On Sat, Jan 28, 2012 at 11:29:26PM -0500, Rafaël Carré wrote: > > > Give the exact aspect ratios when mismatch between encoder and muxer > > > > These sentences no verbs. > > > > > --- a/libavformat/utils.c > > > +++ b/libavformat/utils.c > > > @@ -2782,7 +2782,9 @@ int avformat_write_header(AVFormatContext *s, > > > AVDictionary **options) > > > if(av_cmp_q(st->sample_aspect_ratio, > > > st->codec->sample_aspect_ratio)){ > > > - av_log(s, AV_LOG_ERROR, "Aspect ratio mismatch between > > > encoder and muxer layer\n"); > > > + av_log(s, AV_LOG_ERROR, "Aspect ratio mismatch between > > > encoder (%d/%d) and muxer layer (%d/%d)\n", > > > + st->sample_aspect_ratio.num, > > > st->sample_aspect_ratio.den, > > > + st->codec->sample_aspect_ratio.num, > > > st->codec->sample_aspect_ratio.den); > > > > Indentation is off and the lines are loooong. > > Pushed with those fixed.
You missed the missing verb in the subject line. Whatever ... Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
