From: Simon Marchi <simon.marchi@polymtl.ca>
Date: Thu, 11 Jul 2019 01:49:32 +0000 (-0400)
Subject: Make value_must_coerce_to_target return a bool
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=00db9531969db8b6ab984da996d0411fad938589;p=binutils-gdb.git

Make value_must_coerce_to_target return a bool

... 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.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 0121791a22b..49ccba76f14 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 82b54561c24..0f6ff7b0a57 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 f96b095941a..9f55408dde0 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);