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 /gcc/gcc.c | |
| parent | ff3aed6233b16f02b26f82b3dac7d604aac42641 (diff) | |
| download | gcc-dd045aeeddc65c15840ac25e90f729e2805dfc17.tar.gz | |
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 'gcc/gcc.c')
| -rw-r--r-- | gcc/gcc.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/gcc/gcc.c b/gcc/gcc.c index 7ad959aa8da..f9a2bb2105c 100644 --- a/gcc/gcc.c +++ b/gcc/gcc.c @@ -4806,8 +4806,7 @@ do_spec_1 (const char *spec, int inswitch, const char *soft_matched_part) t->filename_length = temp_filename_length; } - if (saved_suffix) - free (saved_suffix); + free (saved_suffix); obstack_grow (&obstack, t->filename, t->filename_length); delete_this_arg = 1; @@ -6724,12 +6723,10 @@ warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.\n\n" { if (compare_debug) { - if (debug_check_temp_file[0]) - free (debug_check_temp_file[0]); + free (debug_check_temp_file[0]); debug_check_temp_file[0] = NULL; - if (debug_check_temp_file[1]) - free (debug_check_temp_file[1]); + free (debug_check_temp_file[1]); debug_check_temp_file[1] = NULL; } @@ -6773,12 +6770,10 @@ warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.\n\n" if (compare_debug) { - if (debug_check_temp_file[0]) - free (debug_check_temp_file[0]); + free (debug_check_temp_file[0]); debug_check_temp_file[0] = NULL; - if (debug_check_temp_file[1]) - free (debug_check_temp_file[1]); + free (debug_check_temp_file[1]); debug_check_temp_file[1] = NULL; } } |