public inbox for libabigail@sourceware.org
 help / color / mirror / Atom feed
From: Sinny Kumari <sinny@redhat.com>
To: libabigail@sourceware.org
Cc: Dodji Seketeli <dodji@redhat.com>
Subject: [Patch] Adding new methods in corpus_diff class
Date: Wed, 01 Jan 2014 00:00:00 -0000	[thread overview]
Message-ID: <377331953.19841118.1418819554407.JavaMail.zimbra@redhat.com> (raw)
In-Reply-To: <1717902824.19835801.1418819010677.JavaMail.zimbra@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 411 bytes --]

Hi,

Public methods are needed to access added functions and variables obtained outside libabigail
while calculating abi diff between libraries with no debug information available.

This patch (available in attachment) adds new public methods in corpus_diff class- 
added_unrefed_function_symbols() and added_unrefed_variable_symbols() to fix same.

Thanks
-- 
     
     Sinny Kumari
(sinny at redhat dot com)

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Add-new-methods-in-corpus_diff-class.patch --]
[-- Type: text/x-patch; name=0001-Add-new-methods-in-corpus_diff-class.patch, Size: 2800 bytes --]

From 201cbcf668e01790faf8c51238cdbf857577a9c8 Mon Sep 17 00:00:00 2001
From: Sinny Kumari <skumari@redhat.com>
Date: Wed, 17 Dec 2014 17:26:26 +0530
Subject: [PATCH 1/1] Add new methods in corpus_diff class

Functions added_unrefed_function_symbols() and
added_unrefed_variable_symbols() are required in order to access
added functions and variables obtained while calculating abi diff
between libraries with no debug information available

	* include/abg-comparison.h
	(corpus_diff::added_unrefed_function_symbols):
	Declare new member function
	(corpus_diff::added_unrefed_variable_symbols):
	Declare new member function
	* src/abg-comparison.cc
	(corpus_diff::added_unrefed_function_symbols):
	Define new member function
	(corpus_diff::added_unrefed_variable_symbols):
	Define new member function

Signed-off-by: Sinny Kumari <skumari@redhat.com>
---
 include/abg-comparison.h |  6 ++++++
 src/abg-comparison.cc    | 18 ++++++++++++++++++
 2 files changed, 24 insertions(+)

diff --git a/include/abg-comparison.h b/include/abg-comparison.h
index ec89cc6..6ed33ca 100644
--- a/include/abg-comparison.h
+++ b/include/abg-comparison.h
@@ -2057,8 +2057,14 @@ public:
   deleted_unrefed_function_symbols() const;
 
   const string_elf_symbol_map&
+  added_unrefed_function_symbols() const;
+
+  const string_elf_symbol_map&
   deleted_unrefed_variable_symbols() const;
 
+  const string_elf_symbol_map&
+  added_unrefed_variable_symbols() const;
+
   const diff_context_sptr
   context() const;
 
diff --git a/src/abg-comparison.cc b/src/abg-comparison.cc
index fd7e3dd..665d12c 100644
--- a/src/abg-comparison.cc
+++ b/src/abg-comparison.cc
@@ -10225,6 +10225,15 @@ const string_elf_symbol_map&
 corpus_diff::deleted_unrefed_function_symbols() const
 {return priv_->deleted_unrefed_fn_syms_;}
 
+/// Getter for function symbols not referenced by any debug info and
+/// that got added.
+///
+/// @return a map of elf function symbols not referenced by any debug
+/// info and that got added.
+const string_elf_symbol_map&
+corpus_diff::added_unrefed_function_symbols() const
+{return priv_->added_unrefed_fn_syms_;}
+
 /// Getter for variable symbols not referenced by any debug info and
 /// that got deleted.
 ///
@@ -10234,6 +10243,15 @@ const string_elf_symbol_map&
 corpus_diff::deleted_unrefed_variable_symbols() const
 {return priv_->deleted_unrefed_var_syms_;}
 
+/// Getter for variable symbols not referenced by any debug info and
+/// that got added.
+///
+/// @return a map of elf variable symbols not referenced by any debug
+/// info and that got added.
+const string_elf_symbol_map&
+corpus_diff::added_unrefed_variable_symbols() const
+{return priv_->added_unrefed_var_syms_;}
+
 /// Getter of the diff context of this diff
 ///
 /// @return the diff context for this diff.
-- 
2.1.0


       reply	other threads:[~2014-12-17 12:32 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1717902824.19835801.1418819010677.JavaMail.zimbra@redhat.com>
2014-01-01  0:00 ` Sinny Kumari [this message]
2014-01-01  0:00   ` Dodji Seketeli

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=377331953.19841118.1418819554407.JavaMail.zimbra@redhat.com \
    --to=sinny@redhat.com \
    --cc=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).