Re: Remove extra brackets from MINIMUM definition ber.c

2017-05-30 Thread Fred
On 05/30/17 20:15, Ted Unangst wrote: Fred wrote: I was greping the tree for MINIMUM and this one stood out as it was (((a) rather than ((a). not sure why this one stood out. the 16 definitions of MAXIMUM in bin and sbin are all the same. i don't think we really need churn changing either

Re: Remove extra brackets from MINIMUM definition ber.c

2017-05-30 Thread Ted Unangst
Fred wrote: > I was greping the tree for MINIMUM and this one stood out as it was > (((a) rather than ((a). not sure why this one stood out. the 16 definitions of MAXIMUM in bin and sbin are all the same. i don't think we really need churn changing either way.

Re: Remove extra brackets from MINIMUM definition ber.c

2017-05-30 Thread Fred
On 05/30/17 19:57, Theo de Raadt wrote: 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

Re: Remove extra brackets from MINIMUM definition ber.c

2017-05-30 Thread Theo de Raadt
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

Remove extra brackets from MINIMUM definition ber.c

2017-05-30 Thread Fred
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