[pushed] Make value_must_coerce_to_target return a bool

Message ID 20190711015238.7869-1-simon.marchi@polymtl.ca
State New
Headers show
Series
  • [pushed] Make value_must_coerce_to_target return a bool
Related show

Commit Message

Simon Marchi July 11, 2019, 1:52 a.m.
... and move comment to header file.

gdb/ChangeLog:

	* valops.c (value_must_coerce_to_target): Change return type to
	bool.
	* value.h (value_must_coerce_to_target): Likewise.
---
 gdb/ChangeLog |  6 ++++++
 gdb/valops.c  | 11 +++++------
 gdb/value.h   |  5 ++++-
 3 files changed, 15 insertions(+), 7 deletions(-)

-- 
2.22.0

Patch

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 0121791a22b2..49ccba76f149 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@ 
+2019-07-10  Simon Marchi  <simon.marchi@polymtl.ca>
+
+	* valops.c (value_must_coerce_to_target): Change return type to
+	bool.
+	* value.h (value_must_coerce_to_target): Likewise.
+
 2019-07-10  Simon Marchi  <simon.marchi@efficios.com>
 
 	* breakpoint.c (is_hardware_watchpoint): Remove
diff --git a/gdb/valops.c b/gdb/valops.c
index 82b54561c244..0f6ff7b0a572 100644
--- a/gdb/valops.c
+++ b/gdb/valops.c
@@ -1343,10 +1343,9 @@  address_of_variable (struct symbol *var, const struct block *b)
   return val;
 }
 
-/* Return one if VAL does not live in target memory, but should in order
-   to operate on it.  Otherwise return zero.  */
+/* See value.h.  */
 
-int
+bool
 value_must_coerce_to_target (struct value *val)
 {
   struct type *valtype;
@@ -1355,7 +1354,7 @@  value_must_coerce_to_target (struct value *val)
   if (VALUE_LVAL (val) != not_lval
       && VALUE_LVAL (val) != lval_internalvar
       && VALUE_LVAL (val) != lval_xcallable)
-    return 0;
+    return false;
 
   valtype = check_typedef (value_type (val));
 
@@ -1364,9 +1363,9 @@  value_must_coerce_to_target (struct value *val)
     case TYPE_CODE_ARRAY:
       return TYPE_VECTOR (valtype) ? 0 : 1;
     case TYPE_CODE_STRING:
-      return 1;
+      return true;
     default:
-      return 0;
+      return false;
     }
 }
 
diff --git a/gdb/value.h b/gdb/value.h
index f96b095941a2..9f55408dde0b 100644
--- a/gdb/value.h
+++ b/gdb/value.h
@@ -793,7 +793,10 @@  extern struct value *value_ptradd (struct value *arg1, LONGEST arg2);
 
 extern LONGEST value_ptrdiff (struct value *arg1, struct value *arg2);
 
-extern int value_must_coerce_to_target (struct value *arg1);
+/* Return true if VAL does not live in target memory, but should in order
+   to operate on it.  Otherwise return false.  */
+
+extern bool value_must_coerce_to_target (struct value *arg1);
 
 extern struct value *value_coerce_to_target (struct value *arg1);