]> granicus.if.org Git - recode/commitdiff
Remove redundant memset calls zeroing memory
authorReuben Thomas <rrt@sc3d.org>
Sat, 27 Jan 2018 13:57:24 +0000 (13:57 +0000)
committerReuben Thomas <rrt@sc3d.org>
Sun, 28 Jan 2018 22:41:22 +0000 (22:41 +0000)
Now that ALLOC uses calloc.

Also replace a use of malloc then memset with calloc.

src/outer.c
src/request.c
src/task.c

index 848db8104873ea30ea02bf5c9c3d68e33a6e9da0..3f11ea8d90872db00ad60a531ada0a56bbd1eb23 100644 (file)
@@ -480,7 +480,7 @@ unregister_all_modules (RECODE_OUTER outer)
 RECODE_OUTER
 recode_new_outer (unsigned flags)
 {
-  RECODE_OUTER outer = (RECODE_OUTER) malloc (sizeof (struct recode_outer));
+  RECODE_OUTER outer = (RECODE_OUTER) calloc (1, sizeof (struct recode_outer));
 
   if (!outer)
     {
@@ -490,7 +490,6 @@ recode_new_outer (unsigned flags)
       return NULL;
     }
 
-  memset (outer, 0, sizeof (struct recode_outer));
   outer->auto_abort = (flags & RECODE_AUTO_ABORT_FLAG) != 0;
   outer->use_iconv = (flags & RECODE_NO_ICONV_FLAG) == 0;
 
index 644e11902aad0e581ccd24d7aa660c88abf84c3e..cc976a7193f10d48dcb9c4cc6d79b39145f1468a 100644 (file)
@@ -1123,7 +1123,6 @@ recode_new_request (RECODE_OUTER outer)
   if (!ALLOC (request, 1, struct recode_request))
     return NULL;
 
-  memset (request, 0, sizeof (struct recode_request));
   request->outer = outer;
   request->diaeresis_char = '"';
 
index d7050997de3bd0f5550d4f253a9ff94452d1b22f..57a54a9b66c79bac38ab74613725412a3bb943c9 100644 (file)
@@ -509,7 +509,6 @@ recode_new_task (RECODE_CONST_REQUEST request)
   if (!ALLOC (task, 1, struct recode_task))
     return NULL;
 
-  memset (task, 0, sizeof (struct recode_task));
   task->request = request;
   task->strategy = RECODE_STRATEGY_UNDECIDED;
   task->fail_level = RECODE_NOT_CANONICAL;