From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay6-d.mail.gandi.net (relay6-d.mail.gandi.net [217.70.183.198]) by sourceware.org (Postfix) with ESMTPS id 9CF0E3858C50 for ; Sun, 1 Jan 2023 17:28:59 +0000 (GMT) Received: (Authenticated sender: dodji@seketeli.org) by mail.gandi.net (Postfix) with ESMTPSA id 54E2EC0004 for ; Sun, 1 Jan 2023 17:28:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=seketeli.org; s=gm1; t=1672594138; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type; bh=rLsAtTDcXPzqi5YkOX7m2Ft/w8Pao31YHErZrdRKFZs=; b=MyfyR9idUR8ffo1sPfx9HZSynY/q4MXGL3pecaFZzuzEci39hBmjB8g17Vd1GmU+Ss1oPB xwgQ9Iux07WjX1j0HafOCjW4anS2fwqZB8YIgBm+Bj/Az7OkcmLjcWyUy8QZ/oeHOUItY8 Lr+qxcYKokfLAO9gYRbspAOQEoFCu0BVGV/2gpDyMo9BUI0aPvgRp8HfzbyQgRc7VINrwO lqE6xNkhFtUtUBNC+jlXzE5P4TqjiOy33qNh3Ot9NDUykAD1oOiG5M0ltu6uUJs+do4L5G sCqsIEU/d3wfaW4zQYQHvmdLBdp7Tgs85lfl1z9rAcxeRRque/GqwMaCqq0ceg== Received: by localhost (Postfix, from userid 1000) id A3E24581C59; Sun, 1 Jan 2023 18:28:57 +0100 (CET) From: Dodji Seketeli To: libabigail@sourceware.org Subject: [PATCH, applied] Don't use the "infinite" keyword for arrays of unknown size Organization: Me, myself and I X-Operating-System: Fedora 38 X-URL: http://www.seketeli.net/~dodji Date: Sun, 01 Jan 2023 18:28:57 +0100 Message-ID: <87r0wemaee.fsf@seketeli.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-10.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,JMQ_SPF_NEUTRAL,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Hello, In the ABIXML format and in diff reports, array dimensions of unknown size are described with the "infinite" keyword. This is not explicit enough. This patch uses the keyword "unknown" instead. Note that the keyword "infinite" is still recognized by the ABIXML reader. * src/abg-reader.cc (build_subrange_type, build_array_type_def): Support the "unknown" keyword, as well as the "infinite" keyword. * src/abg-reporter-priv.cc (report_size_and_alignment_changes): Emit the "unknown" keyword, not the "infinity" one. * src/abg-writer.cc (write_array_size_and_alignment): Likewise. * tests/data/test-annotate/libtest24-drop-fns-2.so.abi: Adjust. * tests/data/test-annotate/libtest24-drop-fns.so.abi: Likewise. * tests/data/test-annotate/test14-pr18893.so.abi: Likewise. * tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi: Likewise. * tests/data/test-annotate/test7.so.abi: Likewise. * tests/data/test-read-ctf/test-array-size.abi: Likewise. * tests/data/test-read-ctf/test-dynamic-array.o.abi: Likewise. * tests/data/test-read-dwarf/PR25007-sdhci.ko.abi: Likewise. * tests/data/test-read-dwarf/libtest24-drop-fns-2.so.abi: Likewise. * tests/data/test-read-dwarf/libtest24-drop-fns.so.abi: Likewise. * tests/data/test-read-dwarf/test-libandroid.so.abi: Likewise. * tests/data/test-read-dwarf/test10-pr18818-gcc.so.abi: Likewise. * tests/data/test-read-dwarf/test11-pr18828.so.abi: Likewise. * tests/data/test-read-dwarf/test14-pr18893.so.abi: Likewise. * tests/data/test-read-dwarf/test16-pr18904.so.abi: Likewise. * tests/data/test-read-dwarf/test19-pr19023-libtcmalloc_and_profiler.so.abi: Likewise. * tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi: Likewise. * tests/data/test-read-dwarf/test7.so.abi: Likewise. * tests/data/test-read-dwarf/test7.so.hash.abi: Likewise. * tests/data/test-read-dwarf/test9-pr18818-clang.so.abi: Likewise. * tests/data/test-read-write/test25.xml: Likewise. Signed-off-by: Dodji Seketeli --- src/abg-reader.cc | 5 +++-- src/abg-reporter-priv.cc | 8 ++++---- src/abg-writer.cc | 4 ++-- .../test-annotate/libtest24-drop-fns-2.so.abi | 4 ++-- .../data/test-annotate/libtest24-drop-fns.so.abi | 4 ++-- tests/data/test-annotate/test14-pr18893.so.abi | 4 ++-- ...est19-pr19023-libtcmalloc_and_profiler.so.abi | 4 ++-- tests/data/test-annotate/test7.so.abi | 4 ++-- tests/data/test-read-ctf/test-array-size.abi | 4 ++-- .../data/test-read-ctf/test-dynamic-array.o.abi | 4 ++-- tests/data/test-read-dwarf/PR25007-sdhci.ko.abi | 12 ++++++------ .../test-read-dwarf/libtest24-drop-fns-2.so.abi | 4 ++-- .../test-read-dwarf/libtest24-drop-fns.so.abi | 4 ++-- .../data/test-read-dwarf/test-libandroid.so.abi | 16 ++++++++-------- .../test-read-dwarf/test10-pr18818-gcc.so.abi | 12 ++++++------ tests/data/test-read-dwarf/test11-pr18828.so.abi | 4 ++-- tests/data/test-read-dwarf/test14-pr18893.so.abi | 4 ++-- tests/data/test-read-dwarf/test16-pr18904.so.abi | 4 ++-- ...est19-pr19023-libtcmalloc_and_profiler.so.abi | 4 ++-- .../test22-pr19097-libstdc++.so.6.0.17.so.abi | 12 ++++++------ tests/data/test-read-dwarf/test7.so.abi | 4 ++-- tests/data/test-read-dwarf/test7.so.hash.abi | 4 ++-- .../test-read-dwarf/test9-pr18818-clang.so.abi | 8 ++++---- tests/data/test-read-write/test25.xml | 4 ++-- 24 files changed, 71 insertions(+), 70 deletions(-) diff --git a/src/abg-reader.cc b/src/abg-reader.cc index b71ca9c2..5a4fb3c0 100644 --- a/src/abg-reader.cc +++ b/src/abg-reader.cc @@ -4128,7 +4128,7 @@ build_subrange_type(reader& rdr, bool is_infinite = false; if (xml_char_sptr s = XML_NODE_GET_ATTRIBUTE(node, "length")) { - if (string(CHAR_STR(s)) == "infinite") + if (string(CHAR_STR(s)) == "infinite" || string(CHAR_STR(s)) == "unknown") is_infinite = true; else length = strtoull(CHAR_STR(s), NULL, 0); @@ -4259,7 +4259,8 @@ build_array_type_def(reader& rdr, size_in_bits = strtoull(CHAR_STR(s), &endptr, 0); if (*endptr != '\0') { - if (!strcmp(CHAR_STR(s), "infinite")) + if (!strcmp(CHAR_STR(s), "infinite") + ||!strcmp(CHAR_STR(s), "unknown")) size_in_bits = (size_t) -1; else return nil; diff --git a/src/abg-reporter-priv.cc b/src/abg-reporter-priv.cc index a913fa69..075d089f 100644 --- a/src/abg-reporter-priv.cc +++ b/src/abg-reporter-priv.cc @@ -763,12 +763,12 @@ report_size_and_alignment_changes(type_or_decl_base_sptr first, // arrays ... out << indent << "array type size changed from "; if (first_array->is_infinite()) - out << "infinity"; + out << "\'unknown\'"; else emit_num_value(first_array->get_size_in_bits(), *ctxt, out); out << " to "; if (second_array->is_infinite()) - out << "infinity"; + out << "\'unknown\'"; else emit_num_value(second_array->get_size_in_bits(), *ctxt, out); out << "\n"; @@ -797,14 +797,14 @@ report_size_and_alignment_changes(type_or_decl_base_sptr first, << " changed length from "; if ((*i)->is_infinite()) - out << "infinity"; + out << "\'unknown\'"; else out << (*i)->get_length(); out << " to "; if ((*j)->is_infinite()) - out << "infinity"; + out << "\'unknown\'"; else out << (*j)->get_length(); out << "\n"; diff --git a/src/abg-writer.cc b/src/abg-writer.cc index f30ce389..1fb067b8 100644 --- a/src/abg-writer.cc +++ b/src/abg-writer.cc @@ -1470,7 +1470,7 @@ static void write_array_size_and_alignment(const shared_ptr decl, ostream& o) { if (decl->is_infinite()) - o << " size-in-bits='" << "infinite" << "'"; + o << " size-in-bits='" << "unknown" << "'"; else { size_t size_in_bits = decl->get_size_in_bits(); if (size_in_bits) @@ -2857,7 +2857,7 @@ write_array_subrange_type(const array_type_def::subrange_sptr& decl, o << " length='"; if (decl->is_infinite()) - o << "infinite"; + o << "unknown"; else o << decl->get_length(); diff --git a/tests/data/test-annotate/libtest24-drop-fns-2.so.abi b/tests/data/test-annotate/libtest24-drop-fns-2.so.abi index 374376ae..919e06d3 100644 --- a/tests/data/test-annotate/libtest24-drop-fns-2.so.abi +++ b/tests/data/test-annotate/libtest24-drop-fns-2.so.abi @@ -255,9 +255,9 @@ - + - + diff --git a/tests/data/test-annotate/libtest24-drop-fns.so.abi b/tests/data/test-annotate/libtest24-drop-fns.so.abi index 374376ae..919e06d3 100644 --- a/tests/data/test-annotate/libtest24-drop-fns.so.abi +++ b/tests/data/test-annotate/libtest24-drop-fns.so.abi @@ -255,9 +255,9 @@ - + - + diff --git a/tests/data/test-annotate/test14-pr18893.so.abi b/tests/data/test-annotate/test14-pr18893.so.abi index 9259db72..08f9024f 100644 --- a/tests/data/test-annotate/test14-pr18893.so.abi +++ b/tests/data/test-annotate/test14-pr18893.so.abi @@ -6793,9 +6793,9 @@ - + - + diff --git a/tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi b/tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi index 64461110..cdecfae7 100644 --- a/tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi +++ b/tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi @@ -4160,9 +4160,9 @@ - + - + diff --git a/tests/data/test-annotate/test7.so.abi b/tests/data/test-annotate/test7.so.abi index ea7ee86c..f85708ae 100644 --- a/tests/data/test-annotate/test7.so.abi +++ b/tests/data/test-annotate/test7.so.abi @@ -17,9 +17,9 @@ - + - + diff --git a/tests/data/test-read-ctf/test-array-size.abi b/tests/data/test-read-ctf/test-array-size.abi index 3fbc65b6..cd1bafe6 100644 --- a/tests/data/test-read-ctf/test-array-size.abi +++ b/tests/data/test-read-ctf/test-array-size.abi @@ -12,8 +12,8 @@ - - + + diff --git a/tests/data/test-read-ctf/test-dynamic-array.o.abi b/tests/data/test-read-ctf/test-dynamic-array.o.abi index a8978c0c..4a919cae 100644 --- a/tests/data/test-read-ctf/test-dynamic-array.o.abi +++ b/tests/data/test-read-ctf/test-dynamic-array.o.abi @@ -4,8 +4,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/PR25007-sdhci.ko.abi b/tests/data/test-read-dwarf/PR25007-sdhci.ko.abi index 5d773b00..4e4d96f6 100644 --- a/tests/data/test-read-dwarf/PR25007-sdhci.ko.abi +++ b/tests/data/test-read-dwarf/PR25007-sdhci.ko.abi @@ -236,14 +236,14 @@ - - + + - - + + @@ -9171,8 +9171,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/libtest24-drop-fns-2.so.abi b/tests/data/test-read-dwarf/libtest24-drop-fns-2.so.abi index ff8b1d34..01bb1a4c 100644 --- a/tests/data/test-read-dwarf/libtest24-drop-fns-2.so.abi +++ b/tests/data/test-read-dwarf/libtest24-drop-fns-2.so.abi @@ -177,8 +177,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/libtest24-drop-fns.so.abi b/tests/data/test-read-dwarf/libtest24-drop-fns.so.abi index ff8b1d34..01bb1a4c 100644 --- a/tests/data/test-read-dwarf/libtest24-drop-fns.so.abi +++ b/tests/data/test-read-dwarf/libtest24-drop-fns.so.abi @@ -177,8 +177,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/test-libandroid.so.abi b/tests/data/test-read-dwarf/test-libandroid.so.abi index bb60027b..8b15c0e9 100644 --- a/tests/data/test-read-dwarf/test-libandroid.so.abi +++ b/tests/data/test-read-dwarf/test-libandroid.so.abi @@ -3290,8 +3290,8 @@ - - + + @@ -3313,8 +3313,8 @@ - - + + @@ -4262,8 +4262,8 @@ - - + + @@ -21259,8 +21259,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/test10-pr18818-gcc.so.abi b/tests/data/test-read-dwarf/test10-pr18818-gcc.so.abi index 4b876c45..3cbf69c5 100644 --- a/tests/data/test-read-dwarf/test10-pr18818-gcc.so.abi +++ b/tests/data/test-read-dwarf/test10-pr18818-gcc.so.abi @@ -729,8 +729,8 @@ - - + + @@ -3180,11 +3180,11 @@ - - + + - - + + diff --git a/tests/data/test-read-dwarf/test11-pr18828.so.abi b/tests/data/test-read-dwarf/test11-pr18828.so.abi index 6217455d..925643af 100644 --- a/tests/data/test-read-dwarf/test11-pr18828.so.abi +++ b/tests/data/test-read-dwarf/test11-pr18828.so.abi @@ -1010,8 +1010,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/test14-pr18893.so.abi b/tests/data/test-read-dwarf/test14-pr18893.so.abi index 4b15b739..5ef20ee9 100644 --- a/tests/data/test-read-dwarf/test14-pr18893.so.abi +++ b/tests/data/test-read-dwarf/test14-pr18893.so.abi @@ -4341,8 +4341,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/test16-pr18904.so.abi b/tests/data/test-read-dwarf/test16-pr18904.so.abi index a425d776..f98bbdc5 100644 --- a/tests/data/test-read-dwarf/test16-pr18904.so.abi +++ b/tests/data/test-read-dwarf/test16-pr18904.so.abi @@ -2174,8 +2174,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/test19-pr19023-libtcmalloc_and_profiler.so.abi b/tests/data/test-read-dwarf/test19-pr19023-libtcmalloc_and_profiler.so.abi index 85a85a46..92a0b775 100644 --- a/tests/data/test-read-dwarf/test19-pr19023-libtcmalloc_and_profiler.so.abi +++ b/tests/data/test-read-dwarf/test19-pr19023-libtcmalloc_and_profiler.so.abi @@ -2377,8 +2377,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi b/tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi index dabc6417..e62b5985 100644 --- a/tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi +++ b/tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi @@ -10839,8 +10839,8 @@ - - + + @@ -37743,11 +37743,11 @@ - - + + - - + + diff --git a/tests/data/test-read-dwarf/test7.so.abi b/tests/data/test-read-dwarf/test7.so.abi index af702077..7068711a 100644 --- a/tests/data/test-read-dwarf/test7.so.abi +++ b/tests/data/test-read-dwarf/test7.so.abi @@ -10,8 +10,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/test7.so.hash.abi b/tests/data/test-read-dwarf/test7.so.hash.abi index 72223a95..0a53f0dc 100644 --- a/tests/data/test-read-dwarf/test7.so.hash.abi +++ b/tests/data/test-read-dwarf/test7.so.hash.abi @@ -10,8 +10,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/test9-pr18818-clang.so.abi b/tests/data/test-read-dwarf/test9-pr18818-clang.so.abi index 0c56c603..3557763c 100644 --- a/tests/data/test-read-dwarf/test9-pr18818-clang.so.abi +++ b/tests/data/test-read-dwarf/test9-pr18818-clang.so.abi @@ -1806,11 +1806,11 @@ - - + + - - + + diff --git a/tests/data/test-read-write/test25.xml b/tests/data/test-read-write/test25.xml index 8f6ad94f..04a3cb29 100644 --- a/tests/data/test-read-write/test25.xml +++ b/tests/data/test-read-write/test25.xml @@ -3,8 +3,8 @@ - - + + -- 2.39.0 -- Dodji