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

Paul B Mahol onemda at gmail.com
Fri Apr 26 21:42:38 CEST 2013


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


More information about the ffmpeg-devel mailing list