[pushed] Make add_separate_debug_objfile static
Commit Message
The original change was created by Tom Tromey.
Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/493
......................................................................
Make add_separate_debug_objfile static
This changes objfile::make to take a "parent" parameter, and makes
add_separate_debug_objfile static.
gdb/ChangeLog
2019-12-12 Tom Tromey <tom@tromey.com>
* symfile.c (symbol_file_add_with_addrs): Pass "parent" to
objfile::make.
* objfiles.h (struct objjfile) <make>: No longer inline.
(add_separate_debug_objfile): Don't declare.
* objfiles.c (add_separate_debug_objfile): Now static.
(objfile::make): Move from objfiles.h. Call
add_separate_debug_objfile. Add "parent" parameter.
Change-Id: I631f43bb71738dea6ae0697317bf8ef4a0db4617
---
M gdb/ChangeLog
M gdb/objfiles.c
M gdb/objfiles.h
M gdb/symfile.c
4 files changed, 26 insertions(+), 11 deletions(-)
@@ -1,5 +1,15 @@
2019-12-12 Tom Tromey <tom@tromey.com>
+ * symfile.c (symbol_file_add_with_addrs): Pass "parent" to
+ objfile::make.
+ * objfiles.h (struct objjfile) <make>: No longer inline.
+ (add_separate_debug_objfile): Don't declare.
+ * objfiles.c (add_separate_debug_objfile): Now static.
+ (objfile::make): Move from objfiles.h. Call
+ add_separate_debug_objfile. Add "parent" parameter.
+
+2019-12-12 Tom Tromey <tom@tromey.com>
+
* symfile.c (symbol_file_add_with_addrs): Use objfile::make.
* objfiles.h (struct objfile): Make constructor private.
<make>: New static method.
@@ -520,7 +520,7 @@
/* Add OBJFILE as a separate debug objfile of PARENT. */
-void
+static void
add_separate_debug_objfile (struct objfile *objfile, struct objfile *parent)
{
gdb_assert (objfile && parent);
@@ -541,6 +541,18 @@
put_objfile_before (objfile, parent);
}
+/* See objfiles.h. */
+
+objfile *
+objfile::make (bfd *bfd_, const char *name_, objfile_flags flags_,
+ objfile *parent)
+{
+ objfile *result = new objfile (bfd_, name_, flags_);
+ if (parent != nullptr)
+ add_separate_debug_objfile (result, parent);
+ return result;
+}
+
/* Free all separate debug objfile of OBJFILE, but don't free OBJFILE
itself. */
@@ -402,10 +402,8 @@
public:
/* Create an objfile. */
- static objfile *make (bfd *bfd_, const char *name_, objfile_flags flags_)
- {
- return new objfile (bfd_, name_, flags_);
- }
+ static objfile *make (bfd *bfd_, const char *name_, objfile_flags flags_,
+ objfile *parent = nullptr);
~objfile ();
@@ -649,8 +647,6 @@
extern void build_objfile_section_table (struct objfile *);
-extern void add_separate_debug_objfile (struct objfile *, struct objfile *);
-
extern void free_objfile_separate_debug (struct objfile *);
extern void free_all_objfiles (void);
@@ -1093,10 +1093,7 @@
if (mainline)
flags |= OBJF_MAINLINE;
- objfile = objfile::make (abfd, name, flags);
-
- if (parent)
- add_separate_debug_objfile (objfile, parent);
+ objfile = objfile::make (abfd, name, flags, parent);
/* We either created a new mapped symbol table, mapped an existing
symbol table file which has not had initial symbol reading