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.

1 files changed, 5 insertions, 6 deletions

diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
index 9d648b8067d..be645c34325 100644
--- a/src/backend/postmaster/postmaster.c
+++ b/src/backend/postmaster/postmaster.c
@@ -37,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/postmaster/postmaster.c,v 1.354 2004/01/06 23:15:22 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/postmaster/postmaster.c,v 1.355 2004/01/07 18:56:27 neilc Exp $
  *
  * NOTES
  *
@@ -167,7 +167,7 @@ int			MaxBackends;
 int			ReservedBackends;
 
 
-static char *progname = (char *) NULL;
+static char *progname = NULL;
 
 /* The socket(s) we're listening to. */
 #define MAXLISTEN	10
@@ -1082,8 +1082,7 @@ ServerLoop(void)
 
 		PG_SETMASK(&UnBlockSig);
 
-		if (select(nSockets, &rmask, (fd_set *) NULL,
-				   (fd_set *) NULL, &timeout) < 0)
+		if (select(nSockets, &rmask, NULL, NULL, &timeout) < 0)
 		{
 			PG_SETMASK(&BlockSig);
 			if (errno == EINTR || errno == EWOULDBLOCK)
@@ -2583,7 +2582,7 @@ BackendRun(Port *port)
 	if (port->cmdline_options)
 		split_opts(av, &ac, port->cmdline_options);
 
-	av[ac] = (char *) NULL;
+	av[ac] = NULL;
 
 	Assert(ac < maxac);
 
@@ -3038,7 +3037,7 @@ SSDataBase(int xlop)
 		av[ac++] = "template1";
 #endif
 
-		av[ac] = (char *) NULL;
+		av[ac] = NULL;
 
 		Assert(ac < lengthof(av));