From: Pierre Muller <muller@sourceware.org>
Date: Tue, 19 Apr 2011 14:12:50 +0000 (+0000)
Subject: 	* xcoffread.c (process_xcoff_symbol): ARI fix: Avoid assignment
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=9b13a2db10f7fd19574ef9fb5af6aa2c3351d419;p=binutils-gdb.git

	* xcoffread.c (process_xcoff_symbol): ARI fix: Avoid assignment
	inside if clause.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index c4a98be285f..dfee35abde2 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2011-04-19  Pierre Muller  <muller@ics.u-strasbg.fr>
+
+	* xcoffread.c (process_xcoff_symbol): ARI fix: Avoid assignment
+	inside if clause.
+
 2011-04-19  Pierre Muller  <muller@ics.u-strasbg.fr>
 	    Pedro Alves  <pedro@codesourcery.com>
 
diff --git a/gdb/xcoffread.c b/gdb/xcoffread.c
index fd60447210d..4dd048d5d96 100644
--- a/gdb/xcoffread.c
+++ b/gdb/xcoffread.c
@@ -1585,7 +1585,11 @@ process_xcoff_symbol (struct coff_symbol *cs, struct objfile *objfile)
 	     where we need to, which is not necessarily super-clean,
 	     but seems workable enough.  */
 
-	  if (*name == ':' || (pp = (char *) strchr (name, ':')) == NULL)
+	  if (*name == ':')
+	    return NULL;
+
+	  pp = (char *) strchr (name, ':');
+	  if (pp == NULL)
 	    return NULL;
 
 	  ++pp;