[RFA/commit] Remove use of stdbool.h in GDB sources.
Commit Message
> > gdb/ChangeLog:
> >
> > * utils.h: Remove <stdbool.h> #include.
> > (producer_is_gcc): Change return type to "int".
> > * utils.c (producer_is_gcc): Change return type to int.
> > Return 1 instead of true, and 0 instead of false.
> > Adjust function documentation accordingly.
> >
> > I will push this patch in the next few days, unless there are
> > objections.
>
> No objections here.
Thanks, Pedro. I just pushed the patch.
> (guess we should reinstate the ARI bool rule too then.)
Absolutely. Attached is the patch that I just pushed reinstating
them.
gdb/ChangeLog:
* contrib/ari/gdb_ari.sh: Reinstate checks for "true" and "false".
From 00e474c2e96f9a56f128ee29fbdc36ac32d5e14a Mon Sep 17 00:00:00 2001
From: Joel Brobecker <brobecker@adacore.com>
Date: Mon, 2 Mar 2015 06:05:01 -0800
Subject: [PATCH] Revert "Remove true and false ARI checks now that we use
stdbool.h."
As we cannot use type bool until conversion to C++ is official,
this patch re-instates the ARI checks for "true/false".
gdb/ChangeLog:
* contrib/ari/gdb_ari.sh: Reinstate checks for "true" and "false".
---
gdb/ChangeLog | 4 ++++
gdb/contrib/ari/gdb_ari.sh | 20 ++++++++++++++++++++
2 files changed, 24 insertions(+)
@@ -1,5 +1,9 @@
2015-03-02 Joel Brobecker <brobecker@adacore.com>
+ * contrib/ari/gdb_ari.sh: Reinstate checks for "true" and "false".
+
+2015-03-02 Joel Brobecker <brobecker@adacore.com>
+
* utils.h: Remove <stdbool.h> #include.
(producer_is_gcc): Change return type to "int".
* utils.c (producer_is_gcc): Change return type to int.
@@ -1145,6 +1145,26 @@ Do not use `boolean'\'', use `int'\'' instead"
}
}
+BEGIN { doc["false"] = "\
+Definitely do not use `false'\'' in boolean expressions"
+ category["false"] = ari_regression
+}
+/(^|[^_[:alnum:]])false([^_[:alnum:]]|$)/ {
+ if (is_yacc_or_lex == 0) {
+ fail("false")
+ }
+}
+
+BEGIN { doc["true"] = "\
+Do not try to use `true'\'' in boolean expressions"
+ category["true"] = ari_regression
+}
+/(^|[^_[:alnum:]])true([^_[:alnum:]]|$)/ {
+ if (is_yacc_or_lex == 0) {
+ fail("true")
+ }
+}
+
# Typedefs that are either redundant or can be reduced to `struct
# type *''.
# Must be placed before if assignment otherwise ARI exceptions
--
1.9.1