Re: [PATCH v2 00/10] kdbus: macros fixes

From: David Herrmann
Date: Wed Jun 10 2015 - 10:17:27 EST


Hi

On Tue, Jun 9, 2015 at 10:59 PM, Sergei Zviagintsev <sergei@xxxxxxxx> wrote:
> Fix type conversion and style issues in item macros.
>
> First submission was in 2 separate emails:
> https://lkml.kernel.org/g/1433414370-17557-1-git-send-email-sergei@xxxxxxxx
> https://lkml.kernel.org/g/1433414370-17557-2-git-send-email-sergei@xxxxxxxx
>
> Changes in v2:
>
> - fixed similar type cast issue in KDBUS_ITEMS_END macro
> - all arguments in iteration macros are enclosed in parentheses,
> instead of using loop cursor without them
> - summary phrases and commit messages updated
> - documentation updates are excluded to their own patches
> - added fixes for samples/kdbus
> - added fixes for selftests/kdbus
>
>
> Sergei Zviagintsev (10):
> kdbus: fix operator precedence issues in item macros
> kdbus: use parentheses uniformly in KDBUS_ITEMS_FOREACH macro
> Documentation/kdbus: fix operator precedence issue in KDBUS_ITEM_NEXT
> macro
> Documentation/kdbus: use parentheses uniformly in KDBUS_ITEM_FOREACH
> macro
> selftests/kdbus: fix trivial style issues
> selftests/kdbus: fix precedence issues in macros
> selftests/kdbus: use parentheses in iteration macros uniformly
> samples/kdbus: add whitespace
> samples/kdbus: fix operator precedence issue in KDBUS_ITEM_NEXT macro
> samples/kdbus: use parentheses uniformly in KDBUS_FOREACH macro

Next time, please merge most of these changes to reduce the number of
patches and review-overhead. But looks all good to me, much
appreciated.

Thanks
David
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/