public inbox for libabigail@sourceware.org
 help / color / mirror / Atom feed
From: Dodji Seketeli <dodji@redhat.com>
To: libabigail@sourceware.org
Subject: [PATCH, applied] {ctf,btf}-reader: Document the namespace for nicer apidoc
Date: Thu, 18 Apr 2024 18:21:14 +0200	[thread overview]
Message-ID: <87ttjyhcat.fsf@redhat.com> (raw)

Hello,

The apidoc is not generating any documentation for the libabigail::btf
and libabigail::ctf namespaces.

It turns out I need to add some doxygen comment to the these namespaces
in the source code.  Fixed thus.

	* include/abg-btf-reader.h (namespace btf): Add doxygen comment.
	Just adding it to the *.cc won't cut it, I am not sure why.
	* include/abg-ctf-reader.h (namespace ctf): Likewise.
	* src/abg-btf-reader.cc (namespace btf): Add doxygen comment.
	* src/abg-ctf-reader.cc (namespace ctf): Add doxygen commit.
	(create_reader, reset_reader): Fix doxygen comments.

Signed-off-by: Dodji Seketeli <dodji@redhat.com>
Applied to the master branch.
---
 include/abg-btf-reader.h | 45 ++++++++++++++++++++++++++++++++++------
 include/abg-ctf-reader.h |  2 ++
 src/abg-btf-reader.cc    |  1 +
 src/abg-ctf-reader.cc    | 18 +++++++---------
 4 files changed, 49 insertions(+), 17 deletions(-)

diff --git a/include/abg-btf-reader.h b/include/abg-btf-reader.h
index 9239d4f2..e891f268 100644
--- a/include/abg-btf-reader.h
+++ b/include/abg-btf-reader.h
@@ -18,15 +18,48 @@
 namespace abigail
 {
 
+/// Namespace of the reader for the BTF debug information.
 namespace btf
 {
 
-elf_based_reader_sptr
-create_reader(const std::string& elf_path,
-	      const vector<char**>& debug_info_root_paths,
-	      environment& env,
-	      bool load_all_types = false,
-	      bool linux_kernel_mode = false);
+  /// Create and return a BTF reader (or front-end) which is an instance
+  /// of @ref btf::reader.
+  ///
+  /// @param elf_path the path to the path to the elf file the reader is
+  /// to be used for.
+  ///
+  /// @param debug_info_root_paths a vector to the paths to the
+  /// directories under which the debug info is to be found for @p
+  /// elf_path.  Pass an empty vector if th debug info is not in a split
+  /// file.
+  ///
+  /// @param environment the environment used by the current context.
+  /// This environment contains resources needed by the BTF reader and
+  /// by the types and declarations that are to be created later.  Note
+  /// that ABI artifacts that are to be compared all need to be created
+  /// within the same environment.
+  ///
+  /// Please also note that the life time of this environment object
+  /// must be greater than the life time of the resulting @ref
+  /// reader the context uses resources that are allocated in the
+  /// environment.
+  ///
+  /// @param load_all_types if set to false only the types that are
+  /// reachable from publicly exported declarations (of functions and
+  /// variables) are read.  If set to true then all types found in the
+  /// debug information are loaded.
+  ///
+  /// @param linux_kernel_mode if set to true, then consider the special
+  /// linux kernel symbol tables when determining if a symbol is
+  /// exported or not.
+  ///
+  /// @return a smart pointer to the resulting btf::reader.
+  elf_based_reader_sptr
+  create_reader(const std::string& elf_path,
+		const vector<char**>& debug_info_root_paths,
+		environment& env,
+		bool load_all_types = false,
+		bool linux_kernel_mode = false);
 
 }//end namespace btf
 }//end namespace abigail
diff --git a/include/abg-ctf-reader.h b/include/abg-ctf-reader.h
index 58d1bc77..cd9e2dfe 100644
--- a/include/abg-ctf-reader.h
+++ b/include/abg-ctf-reader.h
@@ -23,6 +23,8 @@
 
 namespace abigail
 {
+
+/// Namespace of the reader for the CTF debug information.
 namespace ctf
 {
 
diff --git a/src/abg-btf-reader.cc b/src/abg-btf-reader.cc
index 25fcb685..ac47c2ad 100644
--- a/src/abg-btf-reader.cc
+++ b/src/abg-btf-reader.cc
@@ -35,6 +35,7 @@ namespace abigail
 {
 using namespace ir;
 
+/// Namespace of the reader for the BTF debug information.
 namespace btf
 {
 
diff --git a/src/abg-ctf-reader.cc b/src/abg-ctf-reader.cc
index cc3d2694..a377423a 100644
--- a/src/abg-ctf-reader.cc
+++ b/src/abg-ctf-reader.cc
@@ -42,6 +42,7 @@ ABG_END_EXPORT_DECLARATIONS
 
 namespace abigail
 {
+/// Namespace of the reader for the CTF debug information
 namespace ctf
 {
 using std::dynamic_pointer_cast;
@@ -1671,8 +1672,11 @@ fill_ctf_section(const Elf_Scn *elf_section, ctf_sect_t *ctf_section)
 /// from a given ELF file.
 ///
 /// @param elf_path the patch of some ELF file.
+///
+/// @param debug_info_root_paths the paths to where to find the debug
+/// info.
+///
 /// @param env a libabigail IR environment.
-
 elf_based_reader_sptr
 create_reader(const std::string& elf_path,
 	      const vector<char**>& debug_info_root_paths,
@@ -1692,16 +1696,8 @@ create_reader(const std::string& elf_path,
 /// @param elf_path the path to the elf file the context is to be used
 /// for.
 ///
-/// @param environment the environment used by the current context.
-/// This environment contains resources needed by the reader and by
-/// the types and declarations that are to be created later.  Note
-/// that ABI artifacts that are to be compared all need to be created
-/// within the same environment.
-///
-/// Please also note that the life time of this environment object
-/// must be greater than the life time of the resulting @ref
-/// reader the context uses resources that are allocated in the
-/// environment.
+/// @param debug_info_root_paths the paths pointing to where to find
+/// the debug info.
 void
 reset_reader(elf_based_reader&		rdr,
 	     const std::string&	elf_path,
-- 
2.39.3


-- 
		Dodji


                 reply	other threads:[~2024-04-18 16:21 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=87ttjyhcat.fsf@redhat.com \
    --to=dodji@redhat.com \
    --cc=libabigail@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).