Rename struct attribute accessors

This removes the "value_" prefix from the struct value accessors.
This seemed unnecessarily wordy to me.

gdb/ChangeLog
2020-09-29  Tom Tromey  <tom@tromey.com>

	* dwarf2/read.c (dwarf2_find_base_address, read_call_site_scope)
	(dwarf2_get_pc_bounds, dwarf2_record_block_ranges)
	(partial_die_info::read, dwarf2_string_attr, new_symbol): Update.
	* dwarf2/attribute.h (struct attribute): Rename methods.
	* dwarf2/attribute.c (attribute::as_address): Rename from
	value_as_address.
	(attribute::as_string): Rename from value_as_string.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index cd5958ccc4459fed0bac8c0cff5ea775693800bd..b561f8a8302aa8fb225af2510292ac758793479b 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,13 @@
+2020-09-29  Tom Tromey  <tom@tromey.com>
+
+       * dwarf2/read.c (dwarf2_find_base_address, read_call_site_scope)
+       (dwarf2_get_pc_bounds, dwarf2_record_block_ranges)
+       (partial_die_info::read, dwarf2_string_attr, new_symbol): Update.
+       * dwarf2/attribute.h (struct attribute): Rename methods.
+       * dwarf2/attribute.c (attribute::as_address): Rename from
+       value_as_address.
+       (attribute::as_string): Rename from value_as_string.
+
 2020-09-29  Tom Tromey  <tom@tromey.com>
 
        * dwarf2/read.c (partial_die_info::read) <case

diff --git a/gdb/dwarf2/attribute.c b/gdb/dwarf2/attribute.c
index b39cfe2c00dbd74c6d31df8d2fc26815be56be13..1345e0a3f867e1dd6b1d2c05b13d3112a1cecef6 100644 (file)
--- a/gdb/dwarf2/attribute.c
+++ b/gdb/dwarf2/attribute.c
@@ -32,7 +32,7 @@
 /* See attribute.h.  */
 
 CORE_ADDR
-attribute::value_as_address () const
+attribute::as_address () const
 {
   CORE_ADDR addr;
 
@@ -62,7 +62,7 @@ attribute::value_as_address () const
 /* See attribute.h.  */
 
 const char *
-attribute::value_as_string () const
+attribute::as_string () const
 {
   if (form == DW_FORM_strp || form == DW_FORM_line_strp
       || form == DW_FORM_string

diff --git a/gdb/dwarf2/attribute.h b/gdb/dwarf2/attribute.h
index ffb91e878f108216f823bf1afdb9c2f746bce39e..86b9136fc13af1e1b94c2d803a8f3050615b0a01 100644 (file)
--- a/gdb/dwarf2/attribute.h
+++ b/gdb/dwarf2/attribute.h
@@ -44,11 +44,11 @@ struct attribute
 {
   /* Read the given attribute value as an address, taking the
      attribute's form into account.  */
-  CORE_ADDR value_as_address () const;
+  CORE_ADDR as_address () const;
 
   /* If the attribute has a string form, return the string value;
      otherwise return NULL.  */
-  const char *value_as_string () const;
+  const char *as_string () const;
 
   /* Return non-zero if ATTR's value is a section offset --- classes
      lineptr, loclistptr, macptr or rangelistptr --- or zero, otherwise.

diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c
index 3a2a918f4ed42fbc9587355ec7a8e5d90fbd650f..2e5b7e28deb7d6728ebf2b2219e4b1e78babb9d7 100644 (file)
--- a/gdb/dwarf2/read.c
+++ b/gdb/dwarf2/read.c
@@ -6160,12 +6160,12 @@ dwarf2_find_base_address (struct die_info *die, struct dwarf2_cu *cu)
 
   attr = dwarf2_attr (die, DW_AT_entry_pc, cu);
   if (attr != nullptr)
-    cu->base_address = attr->value_as_address ();
+    cu->base_address = attr->as_address ();
   else
     {
       attr = dwarf2_attr (die, DW_AT_low_pc, cu);
       if (attr != nullptr)
-       cu->base_address = attr->value_as_address ();
+       cu->base_address = attr->as_address ();
     }
 }
 
@@ -13829,7 +13829,7 @@ read_call_site_scope (struct die_info *die, struct dwarf2_cu *cu)
                 sect_offset_str (die->sect_off), objfile_name (objfile));
       return;
     }
-  pc = attr->value_as_address () + baseaddr;
+  pc = attr->as_address () + baseaddr;
   pc = gdbarch_adjust_dwarf2_addr (gdbarch, pc);
 
   if (cu->call_site_htab == NULL)
@@ -14568,8 +14568,8 @@ dwarf2_get_pc_bounds (struct die_info *die, CORE_ADDR *lowpc,
       attr = dwarf2_attr (die, DW_AT_low_pc, cu);
       if (attr != nullptr)
         {
-         low = attr->value_as_address ();
-         high = attr_high->value_as_address ();
+         low = attr->as_address ();
+         high = attr_high->as_address ();
          if (cu->header.version >= 4 && attr_high->form_is_constant ())
            high += low;
        }
@@ -14748,8 +14748,8 @@ dwarf2_record_block_ranges (struct die_info *die, struct block *block,
       attr = dwarf2_attr (die, DW_AT_low_pc, cu);
       if (attr != nullptr)
         {
-          CORE_ADDR low = attr->value_as_address ();
-         CORE_ADDR high = attr_high->value_as_address ();
+         CORE_ADDR low = attr->as_address ();
+         CORE_ADDR high = attr_high->as_address ();
 
          if (cu->header.version >= 4 && attr_high->form_is_constant ())
            high += low;
@@ -19030,15 +19030,15 @@ partial_die_info::read (const struct die_reader_specs *reader,
          /* Note that both forms of linkage name might appear.  We
             assume they will be the same, and we only store the last
             one we see.  */
-         linkage_name = attr.value_as_string ();
+         linkage_name = attr.as_string ();
          break;
        case DW_AT_low_pc:
          has_low_pc_attr = 1;
-         lowpc = attr.value_as_address ();
+         lowpc = attr.as_address ();
          break;
        case DW_AT_high_pc:
          has_high_pc_attr = 1;
-         highpc = attr.value_as_address ();
+         highpc = attr.as_address ();
          if (cu->header.version >= 4 && attr.form_is_constant ())
                high_pc_relative = 1;
          break;
@@ -20276,7 +20276,7 @@ dwarf2_string_attr (struct die_info *die, unsigned int name, struct dwarf2_cu *c
 
   if (attr != NULL)
     {
-      str = attr->value_as_string ();
+      str = attr->as_string ();
       if (str == nullptr)
         complaint (_("string type expected for attribute %s for "
                     "DIE at %s in module %s"),
@@ -21453,7 +21453,7 @@ new_symbol (struct die_info *die, struct type *type, struct dwarf2_cu *cu,
            {
              CORE_ADDR addr;
 
-             addr = attr->value_as_address ();
+             addr = attr->as_address ();
              addr = gdbarch_adjust_dwarf2_addr (gdbarch, addr + baseaddr);
              SET_SYMBOL_VALUE_ADDRESS (sym, addr);
              SYMBOL_ACLASS_INDEX (sym) = LOC_LABEL;