[FFmpeg-devel] [PATCH] ffplay: group together and vertically align correlated parameters in log function

Marton Balint cus at passwd.hu
Fri Jun 29 21:26:21 CEST 2012



On Fri, 29 Jun 2012, Stefano Sabatini wrote:

> Possibly improve readability.
> ---
> ffplay.c |    8 ++------
> 1 files changed, 2 insertions(+), 6 deletions(-)

LGTM, thanks.

>
> diff --git a/ffplay.c b/ffplay.c
> index e8f6e6b..d00a0a0 100644
> --- a/ffplay.c
> +++ b/ffplay.c
> @@ -1979,12 +1979,8 @@ static int audio_decode_frame(VideoState *is, double *pts_ptr)
>                                                  0, NULL);
>                 if (!is->swr_ctx || swr_init(is->swr_ctx) < 0) {
>                     fprintf(stderr, "Cannot create sample rate converter for conversion of %d Hz %s %d channels to %d Hz %s %d channels!\n",
> -                        dec->sample_rate,
> -                        av_get_sample_fmt_name(dec->sample_fmt),
> -                        dec->channels,
> -                        is->audio_tgt.freq,
> -                        av_get_sample_fmt_name(is->audio_tgt.fmt),
> -                        is->audio_tgt.channels);
> +                        dec->sample_rate,   av_get_sample_fmt_name(dec->sample_fmt),   dec->channels,
> +                        is->audio_tgt.freq, av_get_sample_fmt_name(is->audio_tgt.fmt), is->audio_tgt.channels);
>                     break;
>                 }
>                 is->audio_src.channel_layout = dec_channel_layout;
> -- 
> 1.7.5.4
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list