From 672c84a059c0e6383df39d4449f3ac4e8da16761 Mon Sep 17 00:00:00 2001 From: mmcc Date: Wed, 16 Mar 2016 04:33:02 +0000 Subject: Wrap a macro's args in parentheses. No binary change. I'm discussing with deraadt@ whether it's a good idea to convert some of these to functions. The one changed by this commit probably isn't eligible because it defines only a for loop's condition, but many others in less(1) should probably be converted. ok millert@ --- usr.bin/less/ch.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/usr.bin/less/ch.c b/usr.bin/less/ch.c index 86500972b4d..7262ca87932 100644 --- a/usr.bin/less/ch.c +++ b/usr.bin/less/ch.c @@ -103,8 +103,8 @@ struct filestate { * Macros to manipulate the list of buffers in thisfile->hashtbl[n]. */ #define FOR_BUFS_IN_CHAIN(h, bn) \ - for (bn = thisfile->hashtbl[h].hnext; \ - bn != END_OF_HCHAIN(h); bn = bn->hnext) + for ((bn) = thisfile->hashtbl[h].hnext; \ + (bn) != END_OF_HCHAIN(h); (bn) = (bn)->hnext) #define BUF_HASH_RM(bn) \ (bn)->hnext->hprev = (bn)->hprev; \ -- cgit v1.2.3