[FFmpeg-devel] [PATCH 2/2] avfilter/vf_cas: Remove superfluous ;

Paul B Mahol onemda at gmail.com
Sat Feb 29 00:30:43 EET 2020


lgtm

On 2/28/20, Andreas Rheinhardt <andreas.rheinhardt at gmail.com> wrote:
> The second ; in a double ;; is actually a null statement. It triggers
> the typical declaration-after-statement compiler-warnings if it occurs
> in the middle of several declarations (like here).
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavfilter/vf_cas.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavfilter/vf_cas.c b/libavfilter/vf_cas.c
> index a07d51a84e..7c32ed5f9b 100644
> --- a/libavfilter/vf_cas.c
> +++ b/libavfilter/vf_cas.c
> @@ -58,7 +58,7 @@ static int cas_slice8(AVFilterContext *avctx, void *arg,
> int jobnr, int nb_jobs)
>          const int linesize = out->linesize[p];
>          const int in_linesize = in->linesize[p];
>          const int w = s->planewidth[p];
> -        const int w1 = w - 1;;
> +        const int w1 = w - 1;
>          const int h = s->planeheight[p];
>          const int h1 = h - 1;
>          uint8_t *dst = out->data[p] + slice_start * linesize;
> @@ -125,7 +125,7 @@ static int cas_slice16(AVFilterContext *avctx, void
> *arg, int jobnr, int nb_jobs
>          const int linesize = out->linesize[p] / 2;
>          const int in_linesize = in->linesize[p] / 2;
>          const int w = s->planewidth[p];
> -        const int w1 = w - 1;;
> +        const int w1 = w - 1;
>          const int h = s->planeheight[p];
>          const int h1 = h - 1;
>          uint16_t *dst = (uint16_t *)out->data[p] + slice_start * linesize;
> --
> 2.20.1
>
> _______________________________________________
> 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".


More information about the ffmpeg-devel mailing list