On Thu, Feb 27, 2020 at 06:15:26AM +0000, Fu, Linjie wrote: > > -----Original Message----- > > From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of > > Andreas Rheinhardt > > Sent: Thursday, February 27, 2020 13:34 > > To: ffmpeg-devel@ffmpeg.org > > Cc: Andreas Rheinhardt <andreas.rheinha...@gmail.com> > > Subject: [FFmpeg-devel] [PATCH 2/2] avcodec/cdtoons: Remove > > superfluous ; > > > > 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.rheinha...@gmail.com> > > --- > > libavcodec/cdtoons.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavcodec/cdtoons.c b/libavcodec/cdtoons.c > > index d5dce6351f..13f9a60f0f 100644 > > --- a/libavcodec/cdtoons.c > > +++ b/libavcodec/cdtoons.c > > @@ -61,7 +61,7 @@ static int cdtoons_render_sprite(AVCodecContext > > *avctx, const uint8_t *data, > > { > > CDToonsContext *c = avctx->priv_data; > > const uint8_t *next_line = data; > > - const uint8_t *end = data + data_size;; > > + const uint8_t *end = data + data_size; > > uint16_t line_size; > > uint8_t *dest; > > int skip = 0, to_skip, x; > > -- > > lgtm, same issue exists in vf_cas.c maybe could be fixed in one commit.
will apply, dont see a reason why not to apply. thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No human being will ever know the Truth, for even if they happen to say it by chance, they would not even known they had done so. -- Xenophanes
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".