diff options
| author | Pierre Muller <muller@sourceware.org> | 2011-04-19 14:12:50 +0000 |
|---|---|---|
| committer | Pierre Muller <muller@sourceware.org> | 2011-04-19 14:12:50 +0000 |
| commit | 9b13a2db10f7fd19574ef9fb5af6aa2c3351d419 (patch) | |
| tree | 8e543dfb1c0218ad55c6d18dd49e6d9859ade236 /gdb | |
| parent | 1c6e1b0de23f49e732b0ce5ed4373b43ebeb401f (diff) | |
| download | fsf-binutils-gdb-9b13a2db10f7fd19574ef9fb5af6aa2c3351d419.zip fsf-binutils-gdb-9b13a2db10f7fd19574ef9fb5af6aa2c3351d419.tar.gz fsf-binutils-gdb-9b13a2db10f7fd19574ef9fb5af6aa2c3351d419.tar.bz2 | |
* xcoffread.c (process_xcoff_symbol): ARI fix: Avoid assignment
inside if clause.
Diffstat (limited to 'gdb')
| -rw-r--r-- | gdb/ChangeLog | 5 | ||||
| -rw-r--r-- | gdb/xcoffread.c | 6 |
2 files changed, 10 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index c4a98be..dfee35a 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,4 +1,9 @@ 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> * xstormy16-tdep.c (xstormy16_push_dummy_call): Add local diff --git a/gdb/xcoffread.c b/gdb/xcoffread.c index fd60447..4dd048d 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; |