diff options
| author | Tom Tromey <tom@tromey.com> | 2018-04-29 21:02:19 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2018-04-30 11:33:11 -0600 |
| commit | 1831a9f9d3346dbf61202d2aba6935f0093487dd (patch) | |
| tree | 2a57de7d2ba7e20e443021aa7262f66ab9c830de /gdb/gdb_bfd.c | |
| parent | 7c1b5f3db73d7ecab03dc4e866e291582935fb04 (diff) | |
| download | gdb-1831a9f9d3346dbf61202d2aba6935f0093487dd.zip gdb-1831a9f9d3346dbf61202d2aba6935f0093487dd.tar.gz gdb-1831a9f9d3346dbf61202d2aba6935f0093487dd.tar.bz2 | |
Remove new_bfd_ref
For gdb_bfd_ref_ptr, gdb already had a convenience function like the
new gdb_ref_ptr::new_reference -- called new_bfd_ref. This patch
removes it in favor of the new common function.
While doing this I also noticed that the comment for gdb_bfd_open was
incorrect (in a way related to reference counting), so this patch
updates the comment as well.
ChangeLog
2018-04-30 Tom Tromey <tom@tromey.com>
* symfile-mem.c (symbol_file_add_from_memory): Use new_reference.
* gdb_bfd.h (new_bfd_ref): Remove.
(gdb_bfd_open): Update comment.
* gdb_bfd.c (gdb_bfd_open, gdb_bfd_fopen, gdb_bfd_openr)
(gdb_bfd_openw, gdb_bfd_openr_iovec, gdb_bfd_record_inclusion)
(gdb_bfd_fdopenr): Use new_reference.
* exec.c (exec_file_attach): Use new_reference.
Diffstat (limited to 'gdb/gdb_bfd.c')
| -rw-r--r-- | gdb/gdb_bfd.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/gdb/gdb_bfd.c b/gdb/gdb_bfd.c index 02b1375..8fedeb4 100644 --- a/gdb/gdb_bfd.c +++ b/gdb/gdb_bfd.c @@ -450,7 +450,7 @@ gdb_bfd_open (const char *name, const char *target, int fd) host_address_to_string (abfd), bfd_get_filename (abfd)); close (fd); - return new_bfd_ref (abfd); + return gdb_bfd_ref_ptr::new_reference (abfd); } abfd = bfd_fopen (name, target, FOPEN_RB, fd); @@ -470,7 +470,7 @@ gdb_bfd_open (const char *name, const char *target, int fd) *slot = abfd; } - return new_bfd_ref (abfd); + return gdb_bfd_ref_ptr::new_reference (abfd); } /* A helper function that releases any section data attached to the @@ -781,7 +781,7 @@ gdb_bfd_fopen (const char *filename, const char *target, const char *mode, { bfd *result = bfd_fopen (filename, target, mode, fd); - return new_bfd_ref (result); + return gdb_bfd_ref_ptr::new_reference (result); } /* See gdb_bfd.h. */ @@ -791,7 +791,7 @@ gdb_bfd_openr (const char *filename, const char *target) { bfd *result = bfd_openr (filename, target); - return new_bfd_ref (result); + return gdb_bfd_ref_ptr::new_reference (result); } /* See gdb_bfd.h. */ @@ -801,7 +801,7 @@ gdb_bfd_openw (const char *filename, const char *target) { bfd *result = bfd_openw (filename, target); - return new_bfd_ref (result); + return gdb_bfd_ref_ptr::new_reference (result); } /* See gdb_bfd.h. */ @@ -826,7 +826,7 @@ gdb_bfd_openr_iovec (const char *filename, const char *target, open_func, open_closure, pread_func, close_func, stat_func); - return new_bfd_ref (result); + return gdb_bfd_ref_ptr::new_reference (result); } /* See gdb_bfd.h. */ @@ -871,7 +871,7 @@ gdb_bfd_record_inclusion (bfd *includer, bfd *includee) struct gdb_bfd_data *gdata; gdata = (struct gdb_bfd_data *) bfd_usrdata (includer); - gdata->included_bfds.push_back (new_bfd_ref (includee)); + gdata->included_bfds.push_back (gdb_bfd_ref_ptr::new_reference (includee)); } /* See gdb_bfd.h. */ @@ -881,7 +881,7 @@ gdb_bfd_fdopenr (const char *filename, const char *target, int fd) { bfd *result = bfd_fdopenr (filename, target, fd); - return new_bfd_ref (result); + return gdb_bfd_ref_ptr::new_reference (result); } |