No. This is consistant with a couple hundred others in the tree, which were originally copied from sys/param.h before we cleaned userland from using the file. The brackets were good enough for CSRG, and they are good enough for me...
> This patch removes the redundant brackets form the MINIMUM macro in ber.c > > Cheers > > Fred > > Index: ber.c > =================================================================== > RCS file: /cvs/src/usr.sbin/ldapd/ber.c,v > retrieving revision 1.12 > diff -u -p -u -p -r1.12 ber.c > --- ber.c 11 Feb 2017 20:40:03 -0000 1.12 > +++ ber.c 30 May 2017 17:34:14 -0000 > @@ -31,7 +31,7 @@ > > #include "ber.h" > > -#define MINIMUM(a, b) (((a) < (b)) ? (a) : (b)) > +#define MINIMUM(a, b) ((a) < (b) ? (a) : (b)) > > #define BER_TYPE_CONSTRUCTED 0x20 /* otherwise primitive */ > #define BER_TYPE_SINGLE_MAX 30 >
