Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: Fix declaration-after-statement warning

2021-07-19 Thread Nicolas George
Andreas Rheinhardt (12021-07-18): > I pushed this to fix the warning, although I'd like to hear other > peoples' opinion on whether the warning should no longer be enabled. I personally consider it is good code hygiene to group variable declarations. It reduces diff noise when we need to use the

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: Fix declaration-after-statement warning

2021-07-19 Thread Nicolas George
"zhilizhao(赵志立)" (12021-07-19): > For clang, `-Wdeclaration-after-statement` has no effect with -std=c99. Looks like a clang bug: you asked for the warning, it should give it. Regards, -- Nicolas George signature.asc Description: PGP signature ___

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: Fix declaration-after-statement warning

2021-07-19 Thread zhilizhao(赵志立)
> On Jul 19, 2021, at 3:23 AM, Andreas Rheinhardt > wrote: > > Andreas Rheinhardt: >> Signed-off-by: Andreas Rheinhardt >> --- >> What is actually the reason that we stick to this C90 rule? >> Is it because of compability with ancient compilers? (Given that we >> already require several C99 f

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: Fix declaration-after-statement warning

2021-07-19 Thread Gyan Doshi
On 2021-07-19 00:53, Andreas Rheinhardt wrote: Andreas Rheinhardt: Signed-off-by: Andreas Rheinhardt --- What is actually the reason that we stick to this C90 rule? Is it because of compability with ancient compilers? (Given that we already require several C99 features, I doubt that there ar

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: Fix declaration-after-statement warning

2021-07-18 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Signed-off-by: Andreas Rheinhardt > --- > What is actually the reason that we stick to this C90 rule? > Is it because of compability with ancient compilers? (Given that we > already require several C99 features, I doubt that there are compilers > which would fail if we stoppe

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: Fix declaration-after-statement warning

2021-07-18 Thread Paul B Mahol
LGTM ___ 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".

[FFmpeg-devel] [PATCH] fftools/ffmpeg: Fix declaration-after-statement warning

2021-07-17 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- What is actually the reason that we stick to this C90 rule? Is it because of compability with ancient compilers? (Given that we already require several C99 features, I doubt that there are compilers which would fail if we stopped adhering to the declaration-be