Remove if before free

diff --git a/src/main.c b/src/main.c
index 265a4a3bba4b6f082fb7c9a2162be1e72dd3e116..3015dd0a3722520cbaf9b0d43167c250d35fd63f 100644 (file)
--- a/src/main.c
+++ b/src/main.c
@@ -760,8 +760,7 @@ warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.\n"),
 
     if (!recode_scan_request (request, user_request))
       error (EXIT_FAILURE, 0, _("Request `%s' is erroneous"), user_request);
-    if (rewritten_request)
-      free (rewritten_request);
+    free (rewritten_request);
   }
 
   /* If we merely want source code, do it and get out.  */

diff --git a/src/outer.c b/src/outer.c
index 336a2825fd3303e1dcdb1f345b893f341dd13d2d..438e9d901171e185f5cb0e0b33444aadadcb2325 100644 (file)
--- a/src/outer.c
+++ b/src/outer.c
@@ -562,8 +562,7 @@ recode_delete_outer (RECODE_OUTER outer)
       outer->number_of_singles--;
       free (single);
     }
-  if (outer->pair_restriction)
-    free (outer->pair_restriction);
+  free (outer->pair_restriction);
   if (outer->alias_table)
     hash_free ((Hash_table *) outer->alias_table);
   if (outer->argmatch_charset_array)
@@ -576,8 +575,7 @@ recode_delete_outer (RECODE_OUTER outer)
        free (*cursor);
       free (outer->argmatch_charset_array);
     }
-  if (outer->one_to_same)
-    free ((void *) outer->one_to_same);
+  free ((void *) outer->one_to_same);
   free (outer);
   return true;
 }

diff --git a/src/request.c b/src/request.c
index 95b1a3c52a363958cd12fed5ba028167c32a18eb..13676d2c3572bf935198e07416afb191c4e53511 100644 (file)
--- a/src/request.c
+++ b/src/request.c
@@ -1096,10 +1096,8 @@ recode_new_request (RECODE_OUTER outer)
 bool
 recode_delete_request (RECODE_REQUEST request)
 {
-  if (request->sequence_array)
-    free (request->sequence_array);
-  if (request->work_string)
-    free (request->work_string);
+  free (request->sequence_array);
+  free (request->work_string);
   free (request);
   return true;
 }

diff --git a/src/task.c b/src/task.c
index 0e9b7ee284160bce3148c0441de33019df5cbada..1075525fc36949a0936f94dfbc2969ef756e3047 100644 (file)
--- a/src/task.c
+++ b/src/task.c
@@ -324,10 +324,8 @@ perform_memory_sequence (RECODE_TASK task)
   /* FIXME: Maybe we should manage this so it could wait, instead, for
      avoiding buffer reallocation each time a new recoding is done?  */
 
-  if (input.buffer)
-    free (input.buffer);
-  if (output.buffer)
-    free (output.buffer);
+  free (input.buffer);
+  free (output.buffer);
 
   SUBTASK_RETURN (subtask);
 }