Minor ecpg tweak: the return value of calloc() is guaranteed to be NULL

or zero-filled; therefore zero-filling it via memset() is pointless.
(I think setting `errno' is probably a waste of cycles as well, but I
haven't changed that.)

1 files changed, 1 insertions, 5 deletions

diff --git a/src/interfaces/ecpg/pgtypeslib/common.c b/src/interfaces/ecpg/pgtypeslib/common.c
index d4b40098089..903013a2e3a 100644
--- a/src/interfaces/ecpg/pgtypeslib/common.c
+++ b/src/interfaces/ecpg/pgtypeslib/common.c
@@ -2,18 +2,14 @@
 
 #include "extern.h"
 
+/* Return value is zero-filled. */
 char *
 pgtypes_alloc(long size)
 {
 	char	   *new = (char *) calloc(1L, size);
 
 	if (!new)
-	{
 		errno = ENOMEM;
-		return NULL;
-	}
-
-	memset(new, '\0', size);
 	return (new);
 }