Fix most -Wundef warnings

In some cases #if was used instead of #ifdef in an inconsistent style.
Cleaning this up also helps when analyzing cases like
38d8dce61f where this makes a
difference.

There are no behavior changes here, but the change in pg_bswap.h would
prevent possible accidental misuse by third-party code.

Discussion: https://www.postgresql.org/message-id/flat/3b615ca5-c595-3f1d-fdf7-a429e564f614%402ndquadrant.com
This commit is contained in:
Peter Eisentraut 2019-10-19 18:21:58 +02:00
parent 48cc59ed24
commit 5d3587d14b
11 changed files with 27 additions and 19 deletions

View File

@ -299,7 +299,7 @@ hstoreUpgrade(Datum orig)
if (valid_new) if (valid_new)
{ {
#if HSTORE_IS_HSTORE_NEW #ifdef HSTORE_IS_HSTORE_NEW
elog(WARNING, "ambiguous hstore value resolved as hstore-new"); elog(WARNING, "ambiguous hstore value resolved as hstore-new");
/* /*

View File

@ -145,7 +145,7 @@ CustomizableInitialize(void)
switch (restoreCommandType) switch (restoreCommandType)
{ {
case RESTORE_COMMAND_LINK: case RESTORE_COMMAND_LINK:
#if HAVE_WORKING_LINK #ifdef HAVE_WORKING_LINK
SET_RESTORE_COMMAND("ln -s -f", WALFilePath, xlogFilePath); SET_RESTORE_COMMAND("ln -s -f", WALFilePath, xlogFilePath);
break; break;
#endif #endif

View File

@ -2361,12 +2361,12 @@ s_ucmp(mp_int a, mp_int b)
static int static int
s_vcmp(mp_int a, mp_small v) s_vcmp(mp_int a, mp_small v)
{ {
#if _MSC_VER #ifdef _MSC_VER
#pragma warning(push) #pragma warning(push)
#pragma warning(disable: 4146) #pragma warning(disable: 4146)
#endif #endif
mp_usmall uv = (v < 0) ? -(mp_usmall) v : (mp_usmall) v; mp_usmall uv = (v < 0) ? -(mp_usmall) v : (mp_usmall) v;
#if _MSC_VER #ifdef _MSC_VER
#pragma warning(pop) #pragma warning(pop)
#endif #endif

View File

@ -95,7 +95,7 @@ be_lo_open(PG_FUNCTION_ARGS)
LargeObjectDesc *lobjDesc; LargeObjectDesc *lobjDesc;
int fd; int fd;
#if FSDB #ifdef FSDB
elog(DEBUG4, "lo_open(%u,%d)", lobjId, mode); elog(DEBUG4, "lo_open(%u,%d)", lobjId, mode);
#endif #endif
@ -118,7 +118,7 @@ be_lo_close(PG_FUNCTION_ARGS)
(errcode(ERRCODE_UNDEFINED_OBJECT), (errcode(ERRCODE_UNDEFINED_OBJECT),
errmsg("invalid large-object descriptor: %d", fd))); errmsg("invalid large-object descriptor: %d", fd)));
#if FSDB #ifdef FSDB
elog(DEBUG4, "lo_close(%d)", fd); elog(DEBUG4, "lo_close(%d)", fd);
#endif #endif

View File

@ -3247,7 +3247,7 @@ typedef struct RewriteMappingFile
char fname[MAXPGPATH]; char fname[MAXPGPATH];
} RewriteMappingFile; } RewriteMappingFile;
#if NOT_USED #ifdef NOT_USED
static void static void
DisplayMapping(HTAB *tuplecid_data) DisplayMapping(HTAB *tuplecid_data)
{ {

View File

@ -738,7 +738,7 @@ durable_link_or_rename(const char *oldfile, const char *newfile, int elevel)
if (fsync_fname_ext(oldfile, false, false, elevel) != 0) if (fsync_fname_ext(oldfile, false, false, elevel) != 0)
return -1; return -1;
#if HAVE_WORKING_LINK #ifdef HAVE_WORKING_LINK
if (link(oldfile, newfile) < 0) if (link(oldfile, newfile) < 0)
{ {
ereport(elevel, ereport(elevel,

View File

@ -243,7 +243,7 @@ struct HTAB
*/ */
#define MOD(x,y) ((x) & ((y)-1)) #define MOD(x,y) ((x) & ((y)-1))
#if HASH_STATISTICS #ifdef HASH_STATISTICS
static long hash_accesses, static long hash_accesses,
hash_collisions, hash_collisions,
hash_expansions; hash_expansions;
@ -706,7 +706,7 @@ init_htab(HTAB *hashp, long nelem)
/* Choose number of entries to allocate at a time */ /* Choose number of entries to allocate at a time */
hctl->nelem_alloc = choose_nelem_alloc(hctl->entrysize); hctl->nelem_alloc = choose_nelem_alloc(hctl->entrysize);
#if HASH_DEBUG #ifdef HASH_DEBUG
fprintf(stderr, "init_htab:\n%s%p\n%s%ld\n%s%ld\n%s%d\n%s%ld\n%s%u\n%s%x\n%s%x\n%s%ld\n", fprintf(stderr, "init_htab:\n%s%p\n%s%ld\n%s%ld\n%s%d\n%s%ld\n%s%u\n%s%x\n%s%x\n%s%ld\n",
"TABLE POINTER ", hashp, "TABLE POINTER ", hashp,
"DIRECTORY SIZE ", hctl->dsize, "DIRECTORY SIZE ", hctl->dsize,
@ -832,7 +832,7 @@ hash_destroy(HTAB *hashp)
void void
hash_stats(const char *where, HTAB *hashp) hash_stats(const char *where, HTAB *hashp)
{ {
#if HASH_STATISTICS #ifdef HASH_STATISTICS
fprintf(stderr, "%s: this HTAB -- accesses %ld collisions %ld\n", fprintf(stderr, "%s: this HTAB -- accesses %ld collisions %ld\n",
where, hashp->hctl->accesses, hashp->hctl->collisions); where, hashp->hctl->accesses, hashp->hctl->collisions);
@ -933,7 +933,7 @@ hash_search_with_hash_value(HTAB *hashp,
HASHBUCKET *prevBucketPtr; HASHBUCKET *prevBucketPtr;
HashCompareFunc match; HashCompareFunc match;
#if HASH_STATISTICS #ifdef HASH_STATISTICS
hash_accesses++; hash_accesses++;
hctl->accesses++; hctl->accesses++;
#endif #endif
@ -988,7 +988,7 @@ hash_search_with_hash_value(HTAB *hashp,
break; break;
prevBucketPtr = &(currBucket->link); prevBucketPtr = &(currBucket->link);
currBucket = *prevBucketPtr; currBucket = *prevBucketPtr;
#if HASH_STATISTICS #ifdef HASH_STATISTICS
hash_collisions++; hash_collisions++;
hctl->collisions++; hctl->collisions++;
#endif #endif
@ -1130,7 +1130,7 @@ hash_update_hash_key(HTAB *hashp,
HASHBUCKET *oldPrevPtr; HASHBUCKET *oldPrevPtr;
HashCompareFunc match; HashCompareFunc match;
#if HASH_STATISTICS #ifdef HASH_STATISTICS
hash_accesses++; hash_accesses++;
hctl->accesses++; hctl->accesses++;
#endif #endif
@ -1204,7 +1204,7 @@ hash_update_hash_key(HTAB *hashp,
break; break;
prevBucketPtr = &(currBucket->link); prevBucketPtr = &(currBucket->link);
currBucket = *prevBucketPtr; currBucket = *prevBucketPtr;
#if HASH_STATISTICS #ifdef HASH_STATISTICS
hash_collisions++; hash_collisions++;
hctl->collisions++; hctl->collisions++;
#endif #endif

View File

@ -164,7 +164,7 @@ static void FreePagePushSpanLeader(FreePageManager *fpm, Size first_page,
static Size FreePageManagerLargestContiguous(FreePageManager *fpm); static Size FreePageManagerLargestContiguous(FreePageManager *fpm);
static void FreePageManagerUpdateLargest(FreePageManager *fpm); static void FreePageManagerUpdateLargest(FreePageManager *fpm);
#if FPM_EXTRA_ASSERTS #ifdef FPM_EXTRA_ASSERTS
static Size sum_free_pages(FreePageManager *fpm); static Size sum_free_pages(FreePageManager *fpm);
#endif #endif

View File

@ -139,7 +139,14 @@ pg_bswap64(uint64 x)
* the same result as a memcmp() of the corresponding original Datums, but can * the same result as a memcmp() of the corresponding original Datums, but can
* be much cheaper. It's generally safe to do this on big-endian systems * be much cheaper. It's generally safe to do this on big-endian systems
* without any special transformation occurring first. * without any special transformation occurring first.
*
* If SIZEOF_DATUM is not defined, then postgres.h wasn't included and these
* macros probably shouldn't be used, so we define nothing. Note that
* SIZEOF_DATUM == 8 would evaluate as 0 == 8 in that case, potentially
* leading to the wrong implementation being selected and confusing errors, so
* defining nothing is safest.
*/ */
#ifdef SIZEOF_DATUM
#ifdef WORDS_BIGENDIAN #ifdef WORDS_BIGENDIAN
#define DatumBigEndianToNative(x) (x) #define DatumBigEndianToNative(x) (x)
#else /* !WORDS_BIGENDIAN */ #else /* !WORDS_BIGENDIAN */
@ -149,5 +156,6 @@ pg_bswap64(uint64 x)
#define DatumBigEndianToNative(x) pg_bswap32(x) #define DatumBigEndianToNative(x) pg_bswap32(x)
#endif /* SIZEOF_DATUM == 8 */ #endif /* SIZEOF_DATUM == 8 */
#endif /* WORDS_BIGENDIAN */ #endif /* WORDS_BIGENDIAN */
#endif /* SIZEOF_DATUM */
#endif /* PG_BSWAP_H */ #endif /* PG_BSWAP_H */

View File

@ -1052,7 +1052,7 @@ fmtint(long long value, char type, int forcesign, int leftjust,
} }
/* disable MSVC warning about applying unary minus to an unsigned value */ /* disable MSVC warning about applying unary minus to an unsigned value */
#if _MSC_VER #ifdef _MSC_VER
#pragma warning(push) #pragma warning(push)
#pragma warning(disable: 4146) #pragma warning(disable: 4146)
#endif #endif
@ -1061,7 +1061,7 @@ fmtint(long long value, char type, int forcesign, int leftjust,
uvalue = -(unsigned long long) value; uvalue = -(unsigned long long) value;
else else
uvalue = (unsigned long long) value; uvalue = (unsigned long long) value;
#if _MSC_VER #ifdef _MSC_VER
#pragma warning(pop) #pragma warning(pop)
#endif #endif

View File

@ -188,7 +188,7 @@ _dosmaperr(unsigned long e)
ereport(DEBUG5, ereport(DEBUG5,
(errmsg_internal("mapped win32 error code %lu to %d", (errmsg_internal("mapped win32 error code %lu to %d",
e, doserr))); e, doserr)));
#elif FRONTEND_DEBUG #elif defined(FRONTEND_DEBUG)
fprintf(stderr, "mapped win32 error code %lu to %d", e, doserr); fprintf(stderr, "mapped win32 error code %lu to %d", e, doserr);
#endif #endif
errno = doserr; errno = doserr;