public inbox for libabigail@sourceware.org
 help / color / mirror / Atom feed
* [PATCH, applied] Bug 29693 - clang-libs from f37 fails self test
@ 2023-05-17 15:30 Dodji Seketeli
  0 siblings, 0 replies; only message in thread
From: Dodji Seketeli @ 2023-05-17 15:30 UTC (permalink / raw)
  To: libabigail

Hello,

To reproduce the issue reported in the bug above, here is the relevant
command line:

    $ tools/fedabipkgdiff --self-compare -a --from fc37 clang-libs

When the abixml reader encounters a decl-only class, it wrongly avoids
trying to read its members.  This is wrong because a decl-only class
can still have member types.  By doing so, the abixml might "forget"
some member types, leading to self comparison errors as the original
class from DWARF (for instance) might have those member types.

Fixed thus.

	* src/abg-reader.cc (build_class_decl): Read member types even
	when we are looking at a decl-only class.

Signed-off-by: Dodji Seketeli <dodji@redhat.com>
---
 src/abg-reader.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/abg-reader.cc b/src/abg-reader.cc
index f318dbb9..f54e22aa 100644
--- a/src/abg-reader.cc
+++ b/src/abg-reader.cc
@@ -4825,7 +4825,7 @@ build_class_decl(reader&		rdr,
   maybe_set_naming_typedef(rdr, node, decl);
 
   for (xmlNodePtr n = xmlFirstElementChild(node);
-       !is_decl_only && n;
+       n;
        n = xmlNextElementSibling(n))
     {
       if (xmlStrEqual(n->name, BAD_CAST("base-class")))
-- 
2.39.1


-- 
		Dodji


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-05-17 15:31 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-17 15:30 [PATCH, applied] Bug 29693 - clang-libs from f37 fails self test Dodji Seketeli

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).