From 192ad63bd765d448e91389c6ff1d75f8b18bb268 Mon Sep 17 00:00:00 2001 From: Neil Conway Date: Wed, 7 Jan 2004 18:56:30 +0000 Subject: More janitorial work: remove the explicit casting of NULL literals to a pointer type when it is not necessary to do so. For future reference, casting NULL to a pointer type is only necessary when (a) invoking a function AND either (b) the function has no prototype OR (c) the function is a varargs function. --- src/backend/utils/adt/numutils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/backend/utils/adt/numutils.c') diff --git a/src/backend/utils/adt/numutils.c b/src/backend/utils/adt/numutils.c index 2f004af2871..d169be8bd09 100644 --- a/src/backend/utils/adt/numutils.c +++ b/src/backend/utils/adt/numutils.c @@ -11,7 +11,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/adt/numutils.c,v 1.59 2003/11/29 19:51:59 pgsql Exp $ + * $PostgreSQL: pgsql/src/backend/utils/adt/numutils.c,v 1.60 2004/01/07 18:56:28 neilc Exp $ * *------------------------------------------------------------------------- */ @@ -64,7 +64,7 @@ pg_atoi(char *s, int size, int c) * Some versions of strtol treat the empty string as an error, but * some seem not to. Make an explicit test to be sure we catch it. */ - if (s == (char *) NULL) + if (s == NULL) elog(ERROR, "NULL pointer"); if (*s == 0) ereport(ERROR, -- cgit v1.2.3