From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 108005 invoked by alias); 21 Oct 2019 02:54:27 -0000 Mailing-List: contact gdb-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: , Sender: gdb-cvs-owner@sourceware.org List-Subscribe: Sender: gdb-cvs-owner@sourceware.org Received: (qmail 107948 invoked by uid 306); 21 Oct 2019 02:54:26 -0000 Date: Mon, 21 Oct 2019 02:54:00 -0000 Message-ID: <20191021025426.107947.qmail@sourceware.org> Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Tom Tromey To: gdb-cvs@sourceware.org Subject: [binutils-gdb] Make unlink_objfile and put_objfile_before static X-Act-Checkin: binutils-gdb X-Git-Author: Tom Tromey X-Git-Refname: refs/heads/master X-Git-Oldrev: fd65fe1a180695a3500135bac6d5ed6a70b874de X-Git-Newrev: 54d83b8d3920eea4a0d545f1ff7ac2923d938e6c X-SW-Source: 2019-10/txt/msg00149.txt.bz2 https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=54d83b8d3920eea4a0d545f1ff7ac2923d938e6c commit 54d83b8d3920eea4a0d545f1ff7ac2923d938e6c Author: Tom Tromey Date: Sun Oct 20 20:41:20 2019 -0600 Make unlink_objfile and put_objfile_before static I noticed an obsolete comment just before unlink_objfile, and then I noticed that both unlink_objfile and put_objfile_before could be static. This patch makes these changes, and also moves unlink_objfile earlier, so that a forward declaration is not needed. Tested by rebuilding. gdb/ChangeLog 2019-10-20 Tom Tromey * objfiles.h (unlink_objfile, put_objfile_before): Don't declare. * objfiles.c (unlink_objfile): Move earlier. Now static. Remove obsolete comment. (put_objfile_before): Now static. Change-Id: I1b5927a60fd1cc59bfc9c6761f61652a01ef13e0 Diff: --- gdb/ChangeLog | 7 +++++++ gdb/objfiles.c | 55 ++++++++++++++++++++++--------------------------------- gdb/objfiles.h | 4 ---- 3 files changed, 29 insertions(+), 37 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 7c737ac..67de848 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2019-10-20 Tom Tromey + + * objfiles.h (unlink_objfile, put_objfile_before): Don't declare. + * objfiles.c (unlink_objfile): Move earlier. Now static. Remove + obsolete comment. + (put_objfile_before): Now static. + 2019-10-19 Simon Marchi * gdbsupport/common-utils.h (startswith): Change return type to diff --git a/gdb/objfiles.c b/gdb/objfiles.c index f1e708d..fd1cbf7 100644 --- a/gdb/objfiles.c +++ b/gdb/objfiles.c @@ -473,10 +473,31 @@ separate_debug_iterator::operator++ () return *this; } +/* Unlink OBJFILE from the list of known objfiles. */ + +static void +unlink_objfile (struct objfile *objfile) +{ + struct objfile **objpp; + + for (objpp = &object_files; *objpp != NULL; objpp = &((*objpp)->next)) + { + if (*objpp == objfile) + { + *objpp = (*objpp)->next; + objfile->next = NULL; + return; + } + } + + internal_error (__FILE__, __LINE__, + _("unlink_objfile: objfile already unlinked")); +} + /* Put one object file before a specified on in the global list. This can be used to make sure an object file is destroyed before another when using objfiles_safe to free all objfiles. */ -void +static void put_objfile_before (struct objfile *objfile, struct objfile *before_this) { struct objfile **objp; @@ -497,38 +518,6 @@ put_objfile_before (struct objfile *objfile, struct objfile *before_this) _("put_objfile_before: before objfile not in list")); } -/* Unlink OBJFILE from the list of known objfiles, if it is found in the - list. - - It is not a bug, or error, to call this function if OBJFILE is not known - to be in the current list. This is done in the case of mapped objfiles, - for example, just to ensure that the mapped objfile doesn't appear twice - in the list. Since the list is threaded, linking in a mapped objfile - twice would create a circular list. - - If OBJFILE turns out to be in the list, we zap it's NEXT pointer after - unlinking it, just to ensure that we have completely severed any linkages - between the OBJFILE and the list. */ - -void -unlink_objfile (struct objfile *objfile) -{ - struct objfile **objpp; - - for (objpp = &object_files; *objpp != NULL; objpp = &((*objpp)->next)) - { - if (*objpp == objfile) - { - *objpp = (*objpp)->next; - objfile->next = NULL; - return; - } - } - - internal_error (__FILE__, __LINE__, - _("unlink_objfile: objfile already unlinked")); -} - /* Add OBJFILE as a separate debug objfile of PARENT. */ void diff --git a/gdb/objfiles.h b/gdb/objfiles.h index 2d921208..aa8e320 100644 --- a/gdb/objfiles.h +++ b/gdb/objfiles.h @@ -637,12 +637,8 @@ extern CORE_ADDR entry_point_address (void); extern void build_objfile_section_table (struct objfile *); -extern void put_objfile_before (struct objfile *, struct objfile *); - extern void add_separate_debug_objfile (struct objfile *, struct objfile *); -extern void unlink_objfile (struct objfile *); - extern void free_objfile_separate_debug (struct objfile *); extern void free_all_objfiles (void);