emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

master b8b1703 1/2: Pacify gcc 10.2 -Wanalyzer-null-argument in gtkutil.


From: Paul Eggert
Subject: master b8b1703 1/2: Pacify gcc 10.2 -Wanalyzer-null-argument in gtkutil.c
Date: Fri, 25 Dec 2020 04:40:47 -0500 (EST)

branch: master
commit b8b17038e140fe215a76f2e899c00b9b95614886
Author: Paul Eggert <eggert@cs.ucla.edu>
Commit: Paul Eggert <eggert@cs.ucla.edu>

    Pacify gcc 10.2 -Wanalyzer-null-argument in gtkutil.c
    
    * src/gtkutil.c (xg_item_label_same_p): Simplify.  Without this
    simplification, GCC (Ubuntu 10.2.0-13ubuntu1)
    -Wanalyzer-null-argument complains about use of NULL where
    non-null expected as argument of strcmp.
---
 src/gtkutil.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/gtkutil.c b/src/gtkutil.c
index fafd94c..807ee17 100644
--- a/src/gtkutil.c
+++ b/src/gtkutil.c
@@ -2944,14 +2944,11 @@ xg_get_menu_item_label (GtkMenuItem *witem)
 static bool
 xg_item_label_same_p (GtkMenuItem *witem, const char *label)
 {
-  bool is_same = 0;
   char *utf8_label = get_utf8_string (label);
   const char *old_label = witem ? xg_get_menu_item_label (witem) : 0;
 
-  if (! old_label && ! utf8_label)
-    is_same = 1;
-  else if (old_label && utf8_label)
-    is_same = strcmp (utf8_label, old_label) == 0;
+  bool is_same = (!old_label == !utf8_label
+                 && (!old_label || strcmp (utf8_label, old_label) == 0));
 
   if (utf8_label) g_free (utf8_label);
 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]