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-reader: Fix GCC 13 warnings
Date: Tue, 21 Feb 2023 16:40:01 +0100	[thread overview]
Message-ID: <871qmjt4by.fsf@redhat.com> (raw)

Hello,

GCC 13 was recently introduced to Fedora Rawhide and it's complaining
(and rightly so) about some missing parenthesis.  Fixed thus.

	* src/abg-ctf-reader.cc (process_ctf_typedef)
	(process_ctf_base_type, process_ctf_forward_type)
	(process_ctf_struct_type, process_ctf_union_type)
	(process_ctf_enum_type): Add missing parenthesis.

Signed-off-by: Dodji Seketeli <dodji@redhat.com>
Applied to master.
---
 src/abg-ctf-reader.cc | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/abg-ctf-reader.cc b/src/abg-ctf-reader.cc
index 7159a578..54257d11 100644
--- a/src/abg-ctf-reader.cc
+++ b/src/abg-ctf-reader.cc
@@ -731,7 +731,7 @@ process_ctf_typedef(reader *rdr,
 
   const char *typedef_name = ctf_type_name_raw(ctf_dictionary, ctf_type);
   if (corpus_sptr corp = rdr->should_reuse_type_from_corpus_group())
-    if (result = lookup_typedef_type(typedef_name, *corp))
+    if ((result = lookup_typedef_type(typedef_name, *corp)))
       return result;
 
   type_base_sptr utype = rdr->build_type(ctf_dictionary, ctf_utype);
@@ -816,7 +816,7 @@ process_ctf_base_type(reader *rdr,
           integral_type int_type;
           if (parse_integral_type(type_name, int_type))
             normalized_type_name = int_type.to_string();
-          if (result = lookup_basic_type(normalized_type_name, *corp))
+          if ((result = lookup_basic_type(normalized_type_name, *corp)))
             return result;
         }
 
@@ -1038,7 +1038,7 @@ process_ctf_forward_type(reader *rdr,
     {
       if (!type_is_anonymous)
         if (corpus_sptr corp = rdr->should_reuse_type_from_corpus_group())
-          if (result = lookup_class_type(type_name, *corp))
+          if ((result = lookup_class_type(type_name, *corp)))
             return is_type(result);
 
       class_decl_sptr
@@ -1083,7 +1083,7 @@ process_ctf_struct_type(reader *rdr,
 
   if (!struct_type_is_anonymous)
     if (corpus_sptr corp = rdr->should_reuse_type_from_corpus_group())
-      if (result = lookup_class_type(struct_type_name, *corp))
+      if ((result = lookup_class_type(struct_type_name, *corp)))
         return result;
 
   /* The libabigail IR encodes C struct types in `class' IR nodes.  */
@@ -1136,7 +1136,7 @@ process_ctf_union_type(reader *rdr,
 
   if (!union_type_is_anonymous)
     if (corpus_sptr corp = rdr->should_reuse_type_from_corpus_group())
-      if (result = lookup_union_type(union_type_name, *corp))
+      if ((result = lookup_union_type(union_type_name, *corp)))
         return result;
 
   /* Create the corresponding libabigail union IR node.  */
@@ -1477,7 +1477,7 @@ process_ctf_enum_type(reader *rdr,
 
   if (!enum_name.empty())
     if (corpus_sptr corp = rdr->should_reuse_type_from_corpus_group())
-      if (result = lookup_enum_type(enum_name, *corp))
+      if ((result = lookup_enum_type(enum_name, *corp)))
         return result;
 
   /* Build a signed integral type for the type of the enumerators, aka
-- 
2.39.1


-- 
		Dodji


                 reply	other threads:[~2023-02-22 11:08 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=871qmjt4by.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).