diff options
| author | Tom Lane | 2007-09-22 04:41:19 +0000 |
|---|---|---|
| committer | Tom Lane | 2007-09-22 04:41:19 +0000 |
| commit | 571340a00ed58ed179a9063ef77002b34afc7aca (patch) | |
| tree | 9e06765ced5d45413f53735c76fac1ea831a57a8 | |
| parent | 5e87ebb0c30fc859d927ef1bac0b006aa37e0e51 (diff) | |
Parenthesize macro arguments safely. I see no bug among the current
uses of PG_DETOAST_DATUM_SLICE, but it's clearly trouble waiting to
happen.
| -rw-r--r-- | src/include/fmgr.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/include/fmgr.h b/src/include/fmgr.h index d436ba1be0c..e18624efc3e 100644 --- a/src/include/fmgr.h +++ b/src/include/fmgr.h @@ -11,7 +11,7 @@ * Portions Copyright (c) 1996-2007, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $PostgreSQL: pgsql/src/include/fmgr.h,v 1.53 2007/09/21 22:52:52 tgl Exp $ + * $PostgreSQL: pgsql/src/include/fmgr.h,v 1.54 2007/09/22 04:41:19 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -179,7 +179,7 @@ extern struct varlena *pg_detoast_datum_packed(struct varlena * datum); pg_detoast_datum_copy((struct varlena *) DatumGetPointer(datum)) #define PG_DETOAST_DATUM_SLICE(datum,f,c) \ pg_detoast_datum_slice((struct varlena *) DatumGetPointer(datum), \ - (int32) f, (int32) c) + (int32) (f), (int32) (c)) /* WARNING -- unaligned pointer */ #define PG_DETOAST_DATUM_PACKED(datum) \ pg_detoast_datum_packed((struct varlena *) DatumGetPointer(datum)) |