diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2025-12-02 08:39:24 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2025-12-02 08:39:24 +0100 |
| commit | ec782f56b0c30ef493e8356b46e1131612f01d9f (patch) | |
| tree | faf0c5226db71ec50e79978135f493d853882b76 /src | |
| parent | 376c649634bb3fbfad79940f996e2fcb4c2ea6bf (diff) | |
Replace pointer comparisons and assignments to literal zero with NULL
While 0 is technically correct, NULL is the semantically appropriate
choice for pointers.
Author: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Discussion: https://www.postgresql.org/message-id/aS1AYnZmuRZ8g%2B5G%40ip-10-97-1-34.eu-west-3.compute.internal
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/nodes/copyfuncs.c | 2 | ||||
| -rw-r--r-- | src/backend/postmaster/postmaster.c | 2 | ||||
| -rw-r--r-- | src/backend/utils/adt/pg_locale.c | 4 | ||||
| -rw-r--r-- | src/backend/utils/adt/timestamp.c | 2 | ||||
| -rw-r--r-- | src/interfaces/ecpg/ecpglib/prepare.c | 2 |
5 files changed, 6 insertions, 6 deletions
diff --git a/src/backend/nodes/copyfuncs.c b/src/backend/nodes/copyfuncs.c index 475693b08bc..efd02eb01c4 100644 --- a/src/backend/nodes/copyfuncs.c +++ b/src/backend/nodes/copyfuncs.c @@ -204,7 +204,7 @@ copyObjectImpl(const void *from) default: elog(ERROR, "unrecognized node type: %d", (int) nodeTag(from)); - retval = 0; /* keep compiler quiet */ + retval = NULL; /* keep compiler quiet */ break; } diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c index 7c064cf9fbb..a956db4ad27 100644 --- a/src/backend/postmaster/postmaster.c +++ b/src/backend/postmaster/postmaster.c @@ -3391,7 +3391,7 @@ LaunchMissingBackgroundProcesses(void) Shutdown <= SmartShutdown) { WalReceiverPMChild = StartChildProcess(B_WAL_RECEIVER); - if (WalReceiverPMChild != 0) + if (WalReceiverPMChild != NULL) WalReceiverRequested = false; /* else leave the flag set, so we'll try again later */ } diff --git a/src/backend/utils/adt/pg_locale.c b/src/backend/utils/adt/pg_locale.c index b02e7fa4f18..b26257c0a8d 100644 --- a/src/backend/utils/adt/pg_locale.c +++ b/src/backend/utils/adt/pg_locale.c @@ -1222,10 +1222,10 @@ pg_newlocale_from_collation(Oid collid) * Make sure cache entry is marked invalid, in case we fail before * setting things. */ - cache_entry->locale = 0; + cache_entry->locale = NULL; } - if (cache_entry->locale == 0) + if (cache_entry->locale == NULL) { cache_entry->locale = create_pg_locale(collid, CollationCacheContext); } diff --git a/src/backend/utils/adt/timestamp.c b/src/backend/utils/adt/timestamp.c index af48527d436..2dc90a2b8a9 100644 --- a/src/backend/utils/adt/timestamp.c +++ b/src/backend/utils/adt/timestamp.c @@ -5161,7 +5161,7 @@ interval_trunc(PG_FUNCTION_ARGS) errmsg("unit \"%s\" not supported for type %s", lowunits, format_type_be(INTERVALOID)), (val == DTK_WEEK) ? errdetail("Months usually have fractional weeks.") : 0)); - result = 0; + result = NULL; } } diff --git a/src/interfaces/ecpg/ecpglib/prepare.c b/src/interfaces/ecpg/ecpglib/prepare.c index 4b1ae839506..a3f51993d70 100644 --- a/src/interfaces/ecpg/ecpglib/prepare.c +++ b/src/interfaces/ecpg/ecpglib/prepare.c @@ -509,7 +509,7 @@ ecpg_freeStmtCacheEntry(int lineno, int compat, if (entry->ecpgQuery) { ecpg_free(entry->ecpgQuery); - entry->ecpgQuery = 0; + entry->ecpgQuery = NULL; } return entNo; |