[FFmpeg-devel] [PATCH 1/2] avcodec_string: remove the "(hq)" listing

[Michael Niedermayer]

On Fri, Apr 26, 2013 at 07:42:38PM +0000, Paul B Mahol wrote:
> On 4/26/13, Michael Niedermayer <michaelni at gmx.at> wrote:
> > printing "hq" depending on a single video encoding parameter makes
> > little sense
> >
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  libavcodec/utils.c |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> > index 1f4793c..6c87d15 100644
> > --- a/libavcodec/utils.c
> > +++ b/libavcodec/utils.c
> > @@ -2505,8 +2505,8 @@ void avcodec_string(char *buf, int buf_size,
> > AVCodecContext *enc, int encode)
> >              profile = av_get_profile_name(p, enc->profile);
> >      }
> >
> > -    snprintf(buf, buf_size, "%s: %s%s", codec_type ? codec_type :
> > "unknown",
> > -             codec_name, enc->mb_decision ? " (hq)" : "");
> > +    snprintf(buf, buf_size, "%s: %s", codec_type ? codec_type : "unknown",
> > +             codec_name);
> >      buf[0] ^= 'a' ^ 'A'; /* first letter in uppercase */
> >      if (profile)
> >          snprintf(buf + strlen(buf), buf_size - strlen(buf), " (%s)",
> > profile);
> > --
> > 1.7.9.5
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> 
> lgtm

applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130427/54d07b2b/attachment.asc>