On Fri, Aug 4, 2017 at 9:44 AM, Amit Kapila <amit.kapila16@gmail.com> wrote:
> There is no need to use Parentheses around opaque. I mean there is no
> problem with that, but it is redundant and makes code less readable.
Amit, I'm sure you know this, but just for the benefit of anyone who doesn't:
We often include these kinds of extra parentheses in macros, for good
reason. Suppose you have:
#define mul(x,y) x * y
If the user says mul(2+3,5), it will expand to 2 + 3 * 5 = 17, which
is wrong. If you instead do this:
#define mul(x,y) (x) * (y)
...then mul(2+3,5) expands to (2 + 3) * (5) = 25, which is what the
user of the macro is expecting to get.
Outside of macro definitions, as you say, there's no point and we
should avoid it.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company