Use bool in pv_area
authorTom Tromey <tom@tromey.com>
Sun, 8 Oct 2017 02:13:42 +0000 (20:13 -0600)
committerTom Tromey <tom@tromey.com>
Thu, 12 Oct 2017 21:39:24 +0000 (15:39 -0600)
This updates a couple of member functions in pv_area to return bool.

gdb/ChangeLog
2017-10-12  Tom Tromey  <tom@tromey.com>

* prologue-value.h (pv_area::store_would_trash): Return bool.
(pv_area::find_reg): Likewise.
* prologue-value.c (pv_area::store_would_trash): Return bool.
(pv_area::find_reg): Likewise.

gdb/ChangeLog
gdb/prologue-value.c
gdb/prologue-value.h

index 31a9a2687d17e4ea0c86a60faf672da9b9860ee6..65a326573719e16f56e2e142f4be6059dad52657 100644 (file)
@@ -1,3 +1,10 @@
+2017-10-12  Tom Tromey  <tom@tromey.com>
+
+       * prologue-value.h (pv_area::store_would_trash): Return bool.
+       (pv_area::find_reg): Likewise.
+       * prologue-value.c (pv_area::store_would_trash): Return bool.
+       (pv_area::find_reg): Likewise.
+
 2017-10-12  Tom Tromey  <tom@tromey.com>
 
        * s390-linux-tdep.c (s390_store, s390_load)
index c602fe13abc8be37123d77a3b2eca7105213c538..bec8422d3a7ca78fb84df0bfc7bf045f9af48463 100644 (file)
@@ -341,7 +341,7 @@ pv_area::~pv_area ()
 
 /* See prologue-value.h.  */
 
-int
+bool
 pv_area::store_would_trash (pv_t addr)
 {
   /* It may seem odd that pvk_constant appears here --- after all,
@@ -492,7 +492,7 @@ pv_area::fetch (pv_t addr, CORE_ADDR size)
 
 /* See prologue-value.h.  */
 
-int
+bool
 pv_area::find_reg (struct gdbarch *gdbarch, int reg, CORE_ADDR *offset_p)
 {
   struct area_entry *e = m_entry;
@@ -507,14 +507,14 @@ pv_area::find_reg (struct gdbarch *gdbarch, int reg, CORE_ADDR *offset_p)
           {
             if (offset_p)
               *offset_p = e->offset;
-            return 1;
+            return true;
           }
 
         e = e->next;
       }
     while (e != m_entry);
 
-  return 0;
+  return false;
 }
 
 
index e3004aa6adcad11014a7fa75b31d7e53f934bb7c..c6fd34bdab9b37f4fb70b40fa4c86032902e0bba 100644 (file)
@@ -269,7 +269,7 @@ public:
      prologue analysis, and return the information you've gathered,
      instead of losing all that information, most of which is probably
      okay.  */
-  int store_would_trash (pv_t addr);
+  bool store_would_trash (pv_t addr);
 
   /* Search AREA for the original value of REGISTER.  If we can't find
      it, return zero; if we can find it, return a non-zero value, and if
@@ -280,7 +280,7 @@ public:
      items stored in AREA.  If you plan to gather a lot of information
      about registers saved in AREA, consider calling pv_area::scan
      instead, and collecting all your information in one pass.  */
-  int find_reg (struct gdbarch *gdbarch, int reg, CORE_ADDR *offset_p);
+  bool find_reg (struct gdbarch *gdbarch, int reg, CORE_ADDR *offset_p);
 
 
   /* For every part of AREA whose value we know, apply FUNC to CLOSURE,