Hi On Tue, Jun 9, 2015 at 11:00 PM, Sergei Zviagintsev <ser...@s15v.net> wrote: > Enclose all arguments into parentheses to stay consistent across the > whole macro. > > Signed-off-by: Sergei Zviagintsev <ser...@s15v.net> > --- > samples/kdbus/kdbus-api.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: David Herrmann <dh.herrm...@gmail.com> Thanks David > diff --git a/samples/kdbus/kdbus-api.h b/samples/kdbus/kdbus-api.h > index fab873b89d97..7f3abae18396 100644 > --- a/samples/kdbus/kdbus-api.h > +++ b/samples/kdbus/kdbus-api.h > @@ -10,10 +10,10 @@ > #define KDBUS_ITEM_NEXT(item) \ > (typeof(item))((uint8_t *)(item) + KDBUS_ALIGN8((item)->size)) > #define KDBUS_FOREACH(iter, first, _size) \ > - for (iter = (first); \ > + for ((iter) = (first); \ > ((uint8_t *)(iter) < (uint8_t *)(first) + (_size)) && \ > ((uint8_t *)(iter) >= (uint8_t *)(first)); \ > - iter = (void *)(((uint8_t *)iter) + KDBUS_ALIGN8((iter)->size))) > + (iter) = (void *)((uint8_t *)(iter) + > KDBUS_ALIGN8((iter)->size))) > > static inline int kdbus_cmd_bus_make(int control_fd, struct kdbus_cmd *cmd) > { > -- > 1.8.3.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/