[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