From: Alan Modra <amodra@sourceware.org>
To: bfd-cvs@sourceware.org
Subject: [binutils-gdb] Re: Move bfd_init to bfd.c
Date: Tue, 10 Jan 2023 10:12:37 +0000 (GMT) [thread overview]
Message-ID: <20230110101237.34EF3385840D@sourceware.org> (raw)
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=56b3aa508cdbabddd4f46357c8382fe7dca35249
commit 56b3aa508cdbabddd4f46357c8382fe7dca35249
Author: Alan Modra <amodra@gmail.com>
Date: Tue Jan 10 20:28:18 2023 +1030
Re: Move bfd_init to bfd.c
Commit b1c95bc4dd73 resulted in
...bfd.texi:246: @include: could not find init.texi
which went unnoticed due to not building in a clean directory.
This fixes the problem by moving bfd_init earlier, giving it a
doc node, and stitching the nodes back together.
* bfd.c (bfd_init): Move earlier. Give it a doc inode.
Adjust other inodes to suit.
* doc/bfd.texi: Don't include init.texi. Adjust nodes to suit.
Diff:
---
bfd/bfd-in2.h | 10 ++++----
bfd/bfd.c | 69 +++++++++++++++++++++++++++++---------------------------
bfd/doc/bfd.texi | 9 +++-----
3 files changed, 44 insertions(+), 44 deletions(-)
diff --git a/bfd/bfd-in2.h b/bfd/bfd-in2.h
index bb2b30200dc..b6e417f26ec 100644
--- a/bfd/bfd-in2.h
+++ b/bfd/bfd-in2.h
@@ -7105,6 +7105,11 @@ typedef void (*bfd_assert_handler_type) (const char *bfd_formatmsg,
bfd_assert_handler_type bfd_set_assert_handler (bfd_assert_handler_type);
+unsigned int bfd_init (void);
+
+/* Value returned by bfd_init. */
+#define BFD_INIT_MAGIC (sizeof (struct bfd_section))
+
long bfd_get_reloc_upper_bound (bfd *abfd, asection *sect);
long bfd_canonicalize_reloc
@@ -7256,11 +7261,6 @@ bfd_vma bfd_emul_get_commonpagesize (const char *);
char *bfd_demangle (bfd *, const char *, int);
-unsigned int bfd_init (void);
-
-/* Value returned by bfd_init. */
-#define BFD_INIT_MAGIC (sizeof (struct bfd_section))
-
/* Extracted from archive.c. */
symindex bfd_get_next_mapent
(bfd *abfd, symindex previous, carsym **sym);
diff --git a/bfd/bfd.c b/bfd/bfd.c
index 1d1c4498938..c59e31d99e2 100644
--- a/bfd/bfd.c
+++ b/bfd/bfd.c
@@ -643,7 +643,7 @@ CODE_FRAGMENT
\f
/*
INODE
-Error reporting, Miscellaneous, typedef bfd, BFD front end
+Error reporting, Initialization, typedef bfd, BFD front end
SECTION
Error reporting
@@ -1628,10 +1628,44 @@ bfd_set_assert_handler (bfd_assert_handler_type pnew)
_bfd_assert_handler = pnew;
return pold;
}
+
+/*
+INODE
+Initialization, Miscellaneous, Error reporting, BFD front end
+
+FUNCTION
+ bfd_init
+
+SYNOPSIS
+ unsigned int bfd_init (void);
+
+DESCRIPTION
+ This routine must be called before any other BFD function to
+ initialize magical internal data structures.
+ Returns a magic number, which may be used to check
+ that the bfd library is configured as expected by users.
+
+.{* Value returned by bfd_init. *}
+.#define BFD_INIT_MAGIC (sizeof (struct bfd_section))
+.
+*/
+
+unsigned int
+bfd_init (void)
+{
+ bfd_error = bfd_error_no_error;
+ input_bfd = NULL;
+ input_error = bfd_error_no_error;
+ _bfd_error_program_name = NULL;
+ _bfd_error_internal = error_handler_fprintf;
+ _bfd_assert_handler = _bfd_default_assert_handler;
+
+ return BFD_INIT_MAGIC;
+}
\f
/*
INODE
-Miscellaneous, Memory Usage, Error reporting, BFD front end
+Miscellaneous, Memory Usage, Initialization, BFD front end
SECTION
Miscellaneous
@@ -2605,34 +2639,3 @@ _bfd_get_link_info (bfd *abfd)
return elf_link_info (abfd);
}
-
-/*
-FUNCTION
- bfd_init
-
-SYNOPSIS
- unsigned int bfd_init (void);
-
-DESCRIPTION
- This routine must be called before any other BFD function to
- initialize magical internal data structures.
- Returns a magic number, which may be used to check
- that the bfd library is configured as expected by users.
-
-.{* Value returned by bfd_init. *}
-.#define BFD_INIT_MAGIC (sizeof (struct bfd_section))
-.
-*/
-
-unsigned int
-bfd_init (void)
-{
- bfd_error = bfd_error_no_error;
- input_bfd = NULL;
- input_error = bfd_error_no_error;
- _bfd_error_program_name = NULL;
- _bfd_error_internal = error_handler_fprintf;
- _bfd_assert_handler = _bfd_default_assert_handler;
-
- return BFD_INIT_MAGIC;
-}
diff --git a/bfd/doc/bfd.texi b/bfd/doc/bfd.texi
index 7374e37f8cf..f348710845f 100644
--- a/bfd/doc/bfd.texi
+++ b/bfd/doc/bfd.texi
@@ -198,9 +198,9 @@ IEEE-695.
@menu
* typedef bfd::
* Error reporting::
+* Initialization::
* Miscellaneous::
* Memory Usage::
-* Initialization::
* Sections::
* Symbols::
* Archives::
@@ -219,7 +219,7 @@ IEEE-695.
@include bfdt.texi
@include bfdio.texi
-@node Memory Usage, Initialization, Miscellaneous, BFD front end
+@node Memory Usage, Sections, Miscellaneous, BFD front end
@section Memory Usage
BFD keeps all of its internal structures in obstacks. There is one obstack
per open BFD file, into which the current state is stored. When a BFD is
@@ -242,10 +242,7 @@ select the greediest open BFD, close it to reclaim the memory, perform
some operation and reopen the BFD again, to get a fresh copy of the data
structures.
-@node Initialization, Sections, Memory Usage, BFD front end
-@include init.texi
-
-@node Sections, Symbols, Initialization, BFD front end
+@node Sections, Symbols, Memory Usage, BFD front end
@include section.texi
@node Symbols, Archives, Sections, BFD front end
reply other threads:[~2023-01-10 10:12 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230110101237.34EF3385840D@sourceware.org \
--to=amodra@sourceware.org \
--cc=bfd-cvs@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).