Hi On Tue, Jun 9, 2015 at 11:00 PM, Sergei Zviagintsev <[email protected]> wrote: > Enclose all arguments into parentheses to stay consistent across the > whole macro. > > Signed-off-by: Sergei Zviagintsev <[email protected]> > --- > Documentation/kdbus/kdbus.item.xml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/kdbus/kdbus.item.xml > b/Documentation/kdbus/kdbus.item.xml > index b0eeeef995af..ee09dfa443b8 100644 > --- a/Documentation/kdbus/kdbus.item.xml > +++ b/Documentation/kdbus/kdbus.item.xml > @@ -72,10 +72,10 @@ > (typeof(item))((uint8_t *)(item) + KDBUS_ALIGN8((item)->size))
Reviewed-by: David Herrmann <[email protected]> Thanks David > #define KDBUS_ITEM_FOREACH(item, head, first) \ > - for (item = (head)->first; \ > + for ((item) = (head)->first; \ > ((uint8_t *)(item) < (uint8_t *)(head) + (head)->size) && \ > ((uint8_t *)(item) >= (uint8_t *)(head)); \ > - item = KDBUS_ITEM_NEXT(item)) > + (item) = KDBUS_ITEM_NEXT(item)) > ]]></programlisting> > </refsect2> > </refsect1> > -- > 1.8.3.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
