From 317f712722993375935ff8f992410435fb8e2b27 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Thu, 27 Feb 2020 14:05:49 -0700 Subject: [PATCH] Remove field_info::nfields member I noticed that there's no real reason to have field_info::nfields in the DWARF reader. It simply mirrors information that is already available. This patch removes it, in favor of a convenience method. gdb/ChangeLog 2020-02-27 Tom Tromey * dwarf2/read.c (struct field_info) : Now a method, not a member. (dwarf2_add_field): Don't update nfields. (dwarf2_attach_fields_to_type, process_structure_scope): Update. --- gdb/ChangeLog | 7 +++++++ gdb/dwarf2/read.c | 15 ++++++++------- 2 files changed, 15 insertions(+), 7 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 4376161673b..d99c1784f8b 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2020-02-27 Tom Tromey + + * dwarf2/read.c (struct field_info) : Now a method, not a + member. + (dwarf2_add_field): Don't update nfields. + (dwarf2_attach_fields_to_type, process_structure_scope): Update. + 2020-02-27 Andrew Burgess * gdbtypes.c (create_array_type_with_stride): Use std::abs not diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c index c5f0ffa0d9d..f52b1dd3150 100644 --- a/gdb/dwarf2/read.c +++ b/gdb/dwarf2/read.c @@ -1129,9 +1129,6 @@ struct field_info std::vector fields; std::vector baseclasses; - /* Number of fields (including baseclasses). */ - int nfields = 0; - /* Set if the accessibility of one of the fields is not public. */ int non_public_fields = 0; @@ -1147,6 +1144,12 @@ struct field_info /* Nested types defined by this class and the number of elements in this list. */ std::vector nested_types_list; + + /* Return the total number of fields (including baseclasses). */ + int nfields () const + { + return fields.size () + baseclasses.size (); + } }; /* Loaded secondary compilation units are kept in memory until they @@ -14207,8 +14210,6 @@ dwarf2_add_field (struct field_info *fip, struct die_info *die, new_field = &fip->fields.back (); } - fip->nfields++; - attr = dwarf2_attr (die, DW_AT_accessibility, cu); if (attr != nullptr) new_field->accessibility = DW_UNSND (attr); @@ -14468,7 +14469,7 @@ static void dwarf2_attach_fields_to_type (struct field_info *fip, struct type *type, struct dwarf2_cu *cu) { - int nfields = fip->nfields; + int nfields = fip->nfields (); /* Record the field count, allocate space for the array of fields, and create blank accessibility bitfields if necessary. */ @@ -15336,7 +15337,7 @@ process_structure_scope (struct die_info *die, struct dwarf2_cu *cu) } /* Attach fields and member functions to the type. */ - if (fi.nfields) + if (fi.nfields () > 0) dwarf2_attach_fields_to_type (&fi, type, cu); if (!fi.fnfieldlists.empty ()) { -- 2.30.2