diff options
author | Andrew Burgess <andrew.burgess@embecosm.com> | 2015-08-31 21:19:49 +0100 |
---|---|---|
committer | Andrew Burgess <andrew.burgess@embecosm.com> | 2015-09-04 16:49:54 +0100 |
commit | be9033582599e50521feb879d7476756c2c8c017 (patch) | |
tree | 5dbfd6d2c54c2b04bed8146aa5dc873e4b109044 /gdb/xcoffread.c | |
parent | 02f024f9a527012938e978c233fef39f439a5013 (diff) | |
download | gdb-be9033582599e50521feb879d7476756c2c8c017.zip gdb-be9033582599e50521feb879d7476756c2c8c017.tar.gz gdb-be9033582599e50521feb879d7476756c2c8c017.tar.bz2 |
gdb: Remove casts of NULL during assignment.
In the following code:
struct symbol *wsym = (struct symbol *) NULL;
the cast of NULL is redundant, it adds noise, and is just one more thing
to change if the type of wsym ever changes. There are a relatively
small number of places in gdb where the above code pattern is used.
Usually the cast is removed like this:
struct symbol *wsym = NULL;
This commit updates all the places within the gdb/ directory where we
cast NULL during assignment, removing the cast.
gdb/ChangeLog:
* c-valprint.c (print_unpacked_pointer): Remove cast of NULL
pointer.
* dbxread.c (dbx_end_psymtab): Likewise.
* gnu-nat.c (gnu_write_inferior): Likewise.
* mdebugread.c (cross_ref): Likewise.
* p-valprint.c (pascal_val_print): Likewise.
* xcoffread.c (xcoff_end_psymtab): Likewise.
Diffstat (limited to 'gdb/xcoffread.c')
-rw-r--r-- | gdb/xcoffread.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/xcoffread.c b/gdb/xcoffread.c index 4695801..0d49751 100644 --- a/gdb/xcoffread.c +++ b/gdb/xcoffread.c @@ -2108,7 +2108,7 @@ xcoff_end_psymtab (struct objfile *objfile, struct partial_symtab *pst, discard_psymtab (objfile, pst); /* Indicate that psymtab was thrown away. */ - pst = (struct partial_symtab *) NULL; + pst = NULL; } return pst; } |