diff options
| author | meyering <meyering@138bc75d-0d04-0410-961f-82ee72b054a4> | 2011-04-20 18:19:03 +0000 |
|---|---|---|
| committer | meyering <meyering@138bc75d-0d04-0410-961f-82ee72b054a4> | 2011-04-20 18:19:03 +0000 |
| commit | dd045aeeddc65c15840ac25e90f729e2805dfc17 (patch) | |
| tree | e56a024bf4863c42aaa3f941f466a849bb9d3b82 /lto-plugin | |
| parent | ff3aed6233b16f02b26f82b3dac7d604aac42641 (diff) | |
remove useless if-before-free tests
Change "if (E) free (E);" to "free (E);" everywhere except in the
libgo/, intl/, zlib/ and classpath/ directories.
Also transform equivalent variants like
"if (E != NULL) free (E);" and allow an extra cast on the
argument to free. Otherwise, the tested and freed "E"
expressions must be identical, modulo white space.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@172785 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'lto-plugin')
| -rw-r--r-- | lto-plugin/ChangeLog | 4 | ||||
| -rw-r--r-- | lto-plugin/lto-plugin.c | 6 |
2 files changed, 6 insertions, 4 deletions
diff --git a/lto-plugin/ChangeLog b/lto-plugin/ChangeLog index 2e63db35451d..8ceb13c4dad0 100644 --- a/lto-plugin/ChangeLog +++ b/lto-plugin/ChangeLog @@ -1,3 +1,7 @@ +2011-04-20 Jim Meyering <meyering@redhat.com> + + * lto-plugin.c (free_1, free_2): Remove useless if-before-free. + 2011-04-05 Richard Guenther <rguenther@suse.de> PR bootstrap/48431 diff --git a/lto-plugin/lto-plugin.c b/lto-plugin/lto-plugin.c index 661f10cdf989..4b5828b3912b 100644 --- a/lto-plugin/lto-plugin.c +++ b/lto-plugin/lto-plugin.c @@ -312,8 +312,7 @@ free_1 (void) { struct ld_plugin_symbol *s = &symtab->syms[j]; free (s->name); - if (s->comdat_key) - free (s->comdat_key); + free (s->comdat_key); } free (symtab->syms); symtab->syms = NULL; @@ -342,8 +341,7 @@ free_2 (void) claimed_files = NULL; num_claimed_files = 0; - if (arguments_file_name) - free (arguments_file_name); + free (arguments_file_name); arguments_file_name = NULL; } |