[FFmpeg-devel] [PATCH 1/2] examples/muxing: misc style fixes

Steven Liu lingjiujianke at gmail.com
Sat Sep 26 11:40:05 EEST 2020


Jun Zhao <mypopydev at gmail.com> 于2020年9月25日周五 下午8:24写道:
>
> From: Jun Zhao <barryjzhao at tencent.com>
>
> misc style fixes.
>
> Signed-off-by: Jun Zhao <barryjzhao at tencent.com>
> ---
>  doc/examples/muxing.c | 47 +++++++++++++++++++++++------------------------
>  1 file changed, 23 insertions(+), 24 deletions(-)
>
> diff --git a/doc/examples/muxing.c b/doc/examples/muxing.c
> index bd16486..42f704c 100644
> --- a/doc/examples/muxing.c
> +++ b/doc/examples/muxing.c
> @@ -200,7 +200,7 @@ static void add_stream(OutputStream *ost, AVFormatContext *oc,
>               * the motion of the chroma plane does not match the luma plane. */
>              c->mb_decision = 2;
>          }
> -    break;
> +        break;
>
>      default:
>          break;
> @@ -284,25 +284,25 @@ static void open_audio(AVFormatContext *oc, AVCodec *codec, OutputStream *ost, A
>      }
>
>      /* create resampler context */
> -        ost->swr_ctx = swr_alloc();
> -        if (!ost->swr_ctx) {
> -            fprintf(stderr, "Could not allocate resampler context\n");
> -            exit(1);
> -        }
> +    ost->swr_ctx = swr_alloc();
> +    if (!ost->swr_ctx) {
> +        fprintf(stderr, "Could not allocate resampler context\n");
> +        exit(1);
> +    }
>
> -        /* set options */
> -        av_opt_set_int       (ost->swr_ctx, "in_channel_count",   c->channels,       0);
> -        av_opt_set_int       (ost->swr_ctx, "in_sample_rate",     c->sample_rate,    0);
> -        av_opt_set_sample_fmt(ost->swr_ctx, "in_sample_fmt",      AV_SAMPLE_FMT_S16, 0);
> -        av_opt_set_int       (ost->swr_ctx, "out_channel_count",  c->channels,       0);
> -        av_opt_set_int       (ost->swr_ctx, "out_sample_rate",    c->sample_rate,    0);
> -        av_opt_set_sample_fmt(ost->swr_ctx, "out_sample_fmt",     c->sample_fmt,     0);
> -
> -        /* initialize the resampling context */
> -        if ((ret = swr_init(ost->swr_ctx)) < 0) {
> -            fprintf(stderr, "Failed to initialize the resampling context\n");
> -            exit(1);
> -        }
> +    /* set options */
> +    av_opt_set_int       (ost->swr_ctx, "in_channel_count",   c->channels,       0);
> +    av_opt_set_int       (ost->swr_ctx, "in_sample_rate",     c->sample_rate,    0);
> +    av_opt_set_sample_fmt(ost->swr_ctx, "in_sample_fmt",      AV_SAMPLE_FMT_S16, 0);
> +    av_opt_set_int       (ost->swr_ctx, "out_channel_count",  c->channels,       0);
> +    av_opt_set_int       (ost->swr_ctx, "out_sample_rate",    c->sample_rate,    0);
> +    av_opt_set_sample_fmt(ost->swr_ctx, "out_sample_fmt",     c->sample_fmt,     0);
> +
> +    /* initialize the resampling context */
> +    if ((ret = swr_init(ost->swr_ctx)) < 0) {
> +        fprintf(stderr, "Failed to initialize the resampling context\n");
> +        exit(1);
> +    }
>  }
>
>  /* Prepare a 16 bit dummy audio frame of 'frame_size' samples and
> @@ -349,10 +349,10 @@ static int write_audio_frame(AVFormatContext *oc, OutputStream *ost)
>
>      if (frame) {
>          /* convert samples from native format to destination codec format, using the resampler */
> -            /* compute destination number of samples */
> -            dst_nb_samples = av_rescale_rnd(swr_get_delay(ost->swr_ctx, c->sample_rate) + frame->nb_samples,
> -                                            c->sample_rate, c->sample_rate, AV_ROUND_UP);
> -            av_assert0(dst_nb_samples == frame->nb_samples);
> +        /* compute destination number of samples */
> +        dst_nb_samples = av_rescale_rnd(swr_get_delay(ost->swr_ctx, c->sample_rate) + frame->nb_samples,
> +                                        c->sample_rate, c->sample_rate, AV_ROUND_UP);
> +        av_assert0(dst_nb_samples == frame->nb_samples);
>
>          /* when we pass a frame to the encoder, it may keep a reference to it
>           * internally;
> @@ -519,7 +519,6 @@ static AVFrame *get_video_frame(OutputStream *ost)
>  static int write_video_frame(AVFormatContext *oc, OutputStream *ost)
>  {
>      return write_frame(oc, ost->enc, ost->st, get_video_frame(ost));
> -
>  }
>
>  static void close_stream(AVFormatContext *oc, OutputStream *ost)
> --
> 2.7.4
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
lgtm


More information about the ffmpeg-devel mailing list