https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
--- Comment #12 from Aldy Hernandez ---
Author: aldyh
Date: Wed Sep 13 16:20:27 2017
New Revision: 252253
URL: https://gcc.gnu.org/viewcvs?rev=252253=gcc=rev
Log:
PR c/81448
PR c/81306
* c-warn.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
--- Comment #11 from Marek Polacek ---
Cool, thanks for confirming.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
--- Comment #10 from Bernd Edlinger ---
Confirmed.
Thanks!
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
Marek Polacek changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
--- Comment #8 from Marek Polacek ---
Author: mpolacek
Date: Wed Aug 2 11:56:54 2017
New Revision: 250822
URL: https://gcc.gnu.org/viewcvs?rev=250822=gcc=rev
Log:
PR c/81448
PR c/81306
* c-warn.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
Marek Polacek changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
Marek Polacek changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
--- Comment #6 from Bernd Edlinger ---
(In reply to Marek Polacek from comment #5)
> PR81364 is fixed, do you still see any false positives?
Well adding braces would fix the warning,
but consider this slightly reduced test case:
void
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
--- Comment #5 from Marek Polacek ---
PR81364 is fixed, do you still see any false positives?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
--- Comment #4 from Bernd Edlinger ---
(In reply to Marek Polacek from comment #3)
> Guess we'll need the PR81364 fix for that.
Yes, although it would be good to require
a "{" only if cprefix##_ecb_encrypt(...) actually expands
to multiple
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
--- Comment #3 from Marek Polacek ---
Guess we'll need the PR81364 fix for that.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
--- Comment #2 from Bernd Edlinger ---
Not really:
diff --git a/crypto/evp/evp_locl.h b/crypto/evp/evp_locl.h
index 2bb709a..cb44ed8 100644
--- a/crypto/evp/evp_locl.h
+++ b/crypto/evp/evp_locl.h
@@ -71,8 +71,9 @@
#define
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81448
Marek Polacek changed:
What|Removed |Added
CC||mpolacek at gcc dot gnu.org
--- Comment
13 matches
Mail list logo