Fix -Werror=declaration-after-statement and add flag #3995
Conversation
marc-hb
force-pushed
the
C90-decl
branch
2 times, most recently
from
986d972 to
d0803e8
Compare
marc-hb
requested review from
akloniex,
bkokoszx,
cujomalainey,
dbaluta,
dgreid,
lbetlej,
lgirdwood,
mmaka1,
plbossart and
singalsu
as code owners
|
Thanks for the fixes, i just have one nit, that the fixes and the flag change be two separate patches. Otherwise LGTM |
Will make git commands (blame, merge, revert, rebase, cherry-pick,...) much less likely to fail. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Do not add the option yet. Automate current (and unwritten?) coding guidelines. Let's not waste considerable time in code reviews discussing problems computers can catch automatically. Note this goes against CERT-C recommendations DCL00-C, DCL19-C and many others but it's the current policy so let's apply and automate it. Can be changed later. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Automate current (and unwritten?) coding guidelines. Let's not waste considerable time in code reviews discussing problems computers can catch automatically. Note this goes against CERT-C recommendations DCL00-C, DCL19-C and many others but it's the current policy so let's apply and automate it. Can be changed later. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
3 commits for you sir! |
cujomalainey
left a comment
cujomalainey
left a comment
There was a problem hiding this comment.
Thanks more bot info always a win 👍
|
@marc-hb can you check CI, this should not break UT (unless it break UT build) ? |
|
Downgrading to draft for now. |
|
Can one of the admins verify this patch? |
|
@marc-hb Over a year with no activity, keep open? |
|
The Linux kernel has moved to C99 (even higher actually but not relevant here), we should do the same. Closing |
7 participants
Automate current (and unwritten?) coding guidelines. Let's not waste
considerable time in code reviews discussing problems computers can
catch automatically.
Note this goes against CERT-C recommendations DCL00-C, DCL19-C and many
others but it's the current policy so let's apply and automate it. Can
be changed later.
Signed-off-by: Marc Herbert marc.herbert@intel.com