diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/backend/port/dynloader/bsdi.h | 6 | ||||
-rw-r--r-- | src/backend/port/dynloader/linux.h | 11 | ||||
-rw-r--r-- | src/backend/regex/engine.c | 7 | ||||
-rw-r--r-- | src/backend/regex/regexec.c | 17 | ||||
-rw-r--r-- | src/backend/utils/adt/pg_lzcompress.c | 22 | ||||
-rw-r--r-- | src/interfaces/odbc/convert.c | 9 | ||||
-rw-r--r-- | src/interfaces/odbc/iodbc.h | 6 | ||||
-rwxr-xr-x | src/tools/find_baddefs | 3 |
8 files changed, 58 insertions, 23 deletions
diff --git a/src/backend/port/dynloader/bsdi.h b/src/backend/port/dynloader/bsdi.h index 834dff0d705..2ea466317cc 100644 --- a/src/backend/port/dynloader/bsdi.h +++ b/src/backend/port/dynloader/bsdi.h @@ -27,7 +27,11 @@ #else /* not HAVE_DLOPEN */ #define pg_dlsym(handle, funcname) ((PGFunction) dld_get_func((funcname))) -#define pg_dlclose(handle) ({ dld_unlink_by_file(handle, 1); free(handle); }) +#define pg_dlclose(handle) \ +do { \ + dld_unlink_by_file(handle, 1); \ + free(handle); \ +} while (0) #endif /* not HAVE_DLOPEN */ diff --git a/src/backend/port/dynloader/linux.h b/src/backend/port/dynloader/linux.h index d6873763b4b..290771cfc05 100644 --- a/src/backend/port/dynloader/linux.h +++ b/src/backend/port/dynloader/linux.h @@ -6,7 +6,7 @@ * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $Id: linux.h,v 1.12 2001/05/14 21:45:53 petere Exp $ + * $Id: linux.h,v 1.13 2001/10/25 01:29:37 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -23,10 +23,15 @@ #ifndef HAVE_DLD_H #define pg_dlsym(handle, funcname) (NULL) -#define pg_dlclose(handle) ({}) +#define pg_dlclose(handle) {} #else #define pg_dlsym(handle, funcname) ((PGFunction) dld_get_func((funcname))) -#define pg_dlclose(handle) ({ dld_unlink_by_file(handle, 1); free(handle); }) +#define pg_dlclose(handle) \ +do { \ + dld_unlink_by_file(handle, 1); \ + free(handle); \ +} while (0) + #endif #else /* HAVE_DLOPEN */ diff --git a/src/backend/regex/engine.c b/src/backend/regex/engine.c index 8bf0ee68954..2d7f4ee92d0 100644 --- a/src/backend/regex/engine.c +++ b/src/backend/regex/engine.c @@ -129,7 +129,12 @@ static int pg_isprint(int c); #ifdef REDEBUG #define SP(t, s, c) print(m, t, s, c, stdout) #define AT(t, p1, p2, s1, s2) at(m, t, p1, p2, s1, s2) -#define NOTE(str) { if (m->eflags®_TRACE) printf("=%s\n", (str)); } +#define NOTE(str) \ +do { \ + if (m->eflags®_TRACE) \ + printf("=%s\n", (str)); \ +} while (0) + #else #define SP(t, s, c) /* nothing */ #define AT(t, p1, p2, s1, s2) /* nothing */ diff --git a/src/backend/regex/regexec.c b/src/backend/regex/regexec.c index cb52ad9afe4..b1733a9d40d 100644 --- a/src/backend/regex/regexec.c +++ b/src/backend/regex/regexec.c @@ -114,10 +114,19 @@ static int nope = 0; /* for use in asserts; shuts lint up */ #define ASSIGN(d, s) memcpy(d, s, m->g->nstates) #define EQ(a, b) (memcmp(a, b, m->g->nstates) == 0) #define STATEVARS int vn; char *space -#define STATESETUP(m, nv) { (m)->space = malloc((nv)*(m)->g->nstates); \ - if ((m)->space == NULL) return(REG_ESPACE); \ - (m)->vn = 0; } -#define STATETEARDOWN(m) { free((m)->space); } +#define STATESETUP(m, nv) \ +do { \ + (m)->space = malloc((nv)*(m)->g->nstates); \ + if ((m)->space == NULL) \ + return(REG_ESPACE); \ + (m)->vn = 0; \ +} while (0) + +#define STATETEARDOWN(m) \ +do { \ + free((m)->space); \ +} while (0) + #define SETUP(v) ((v) = &m->space[m->vn++ * m->g->nstates]) #define onestate int #define INIT(o, n) ((o) = (n)) diff --git a/src/backend/utils/adt/pg_lzcompress.c b/src/backend/utils/adt/pg_lzcompress.c index d92d969d40e..5c6cc6e1e4a 100644 --- a/src/backend/utils/adt/pg_lzcompress.c +++ b/src/backend/utils/adt/pg_lzcompress.c @@ -1,7 +1,7 @@ /* ---------- * pg_lzcompress.c - * - * $Header: /cvsroot/pgsql/src/backend/utils/adt/pg_lzcompress.c,v 1.11 2001/03/22 06:16:17 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/utils/adt/pg_lzcompress.c,v 1.12 2001/10/25 01:29:37 momjian Exp $ * * This is an implementation of LZ compression for PostgreSQL. * It uses a simple history table and generates 2-3 byte tags @@ -283,7 +283,8 @@ static PGLZ_HistEntry hist_entries[PGLZ_HISTORY_SIZE]; * Adds a new entry to the history table. * ---------- */ -#define pglz_hist_add(_hs,_he,_hn,_s,_e) { \ +#define pglz_hist_add(_hs,_he,_hn,_s,_e) \ +do { \ int __hindex = pglz_hist_idx((_s),(_e)); \ if ((_he)[(_hn)].prev == NULL) { \ (_hs)[__hindex] = (_he)[(_hn)].next; \ @@ -303,7 +304,7 @@ static PGLZ_HistEntry hist_entries[PGLZ_HISTORY_SIZE]; if (++(_hn) >= PGLZ_HISTORY_SIZE) { \ (_hn) = 0; \ } \ - } +} while (0) /* ---------- @@ -312,7 +313,8 @@ static PGLZ_HistEntry hist_entries[PGLZ_HISTORY_SIZE]; * Outputs the last and allocates a new control byte if needed. * ---------- */ -#define pglz_out_ctrl(__ctrlp,__ctrlb,__ctrl,__buf) { \ +#define pglz_out_ctrl(__ctrlp,__ctrlb,__ctrl,__buf) \ +do { \ if ((__ctrl & 0xff) == 0) \ { \ *__ctrlp = __ctrlb; \ @@ -320,7 +322,7 @@ static PGLZ_HistEntry hist_entries[PGLZ_HISTORY_SIZE]; __ctrlb = 0; \ __ctrl = 1; \ } \ -} +} while (0) /* ---------- @@ -330,11 +332,12 @@ static PGLZ_HistEntry hist_entries[PGLZ_HISTORY_SIZE]; * appropriate control bit. * ---------- */ -#define pglz_out_literal(_ctrlp,_ctrlb,_ctrl,_buf,_byte) { \ +#define pglz_out_literal(_ctrlp,_ctrlb,_ctrl,_buf,_byte) \ +do { \ pglz_out_ctrl(_ctrlp,_ctrlb,_ctrl,_buf); \ *_buf++ = (unsigned char)(_byte); \ _ctrl <<= 1; \ -} +} while (0) /* ---------- @@ -345,7 +348,8 @@ static PGLZ_HistEntry hist_entries[PGLZ_HISTORY_SIZE]; * appropriate control bit. * ---------- */ -#define pglz_out_tag(_ctrlp,_ctrlb,_ctrl,_buf,_len,_off) { \ +#define pglz_out_tag(_ctrlp,_ctrlb,_ctrl,_buf,_len,_off) \ +do { \ pglz_out_ctrl(_ctrlp,_ctrlb,_ctrl,_buf); \ _ctrlb |= _ctrl; \ _ctrl <<= 1; \ @@ -360,7 +364,7 @@ static PGLZ_HistEntry hist_entries[PGLZ_HISTORY_SIZE]; _buf[1] = (unsigned char)((_off) & 0xff); \ _buf += 2; \ } \ -} +} while (0) /* ---------- diff --git a/src/interfaces/odbc/convert.c b/src/interfaces/odbc/convert.c index 6210d030bf0..2afc079dfd8 100644 --- a/src/interfaces/odbc/convert.c +++ b/src/interfaces/odbc/convert.c @@ -817,7 +817,10 @@ do { \ * Terminate the stmt_with_params string with NULL. *---------- */ -#define CVT_TERMINATE { new_statement[npos] = '\0'; } +#define CVT_TERMINATE \ +do { \ + new_statement[npos] = '\0'; \ +} while (0) /*---------- * Append a data. @@ -1434,7 +1437,7 @@ copy_statement_with_parameters(StatementClass *stmt) /* error */ stmt->errormsg = "Unrecognized C_parameter type in copy_statement_with_parameters"; stmt->errornumber = STMT_NOT_IMPLEMENTED_ERROR; - CVT_TERMINATE /* just in case */ + CVT_TERMINATE; /* just in case */ SC_log_error(func, "", stmt); return SQL_ERROR; } @@ -1684,7 +1687,7 @@ copy_statement_with_parameters(StatementClass *stmt) } /* end, for */ /* make sure new_statement is always null-terminated */ - CVT_TERMINATE + CVT_TERMINATE; if (conn->DriverToDataSource != NULL) { diff --git a/src/interfaces/odbc/iodbc.h b/src/interfaces/odbc/iodbc.h index 21091de90f9..99ed5520550 100644 --- a/src/interfaces/odbc/iodbc.h +++ b/src/interfaces/odbc/iodbc.h @@ -8,7 +8,11 @@ #include <sys/types.h> #define MEM_ALLOC(size) (malloc((size_t)(size))) -#define MEM_FREE(ptr) {if(ptr) free(ptr);} +#define MEM_FREE(ptr) \ +do { \ + if(ptr) \ + free(ptr); \ +} while (0) #define STRCPY(t, s) (strcpy((char*)(t), (char*)(s))) #define STRNCPY(t,s,n) (strncpy((char*)(t), (char*)(s), (size_t)(n))) diff --git a/src/tools/find_baddefs b/src/tools/find_baddefs index 07f6fa2c614..8a12b59becf 100755 --- a/src/tools/find_baddefs +++ b/src/tools/find_baddefs @@ -14,6 +14,7 @@ do was_define = "Y" else was_define = "N" - }' $FILE + }' "$FILE" + grep -on '^#define.*{' "$FILE" | grep -v 'do[ ]*{' done |