[PATCH 1/2] test: Wrap assert macros in ({ ... }) and fix missing semicolons

Simon Glass sjg at chromium.org
Mon Mar 6 18:53:07 CET 2023


On Wed, 1 Mar 2023 at 20:05, Marek Vasut
<marek.vasut+renesas at mailbox.org> wrote:
>
> Wrap the assert macros in ({ ... }) so they can be safely used both as
> right side argument as well as in conditionals without curly brackets
> around them. In the process, find a bunch of missing semicolons, fix
> them.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas at mailbox.org>
> ---
> Cc: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> Cc: Simon Glass <sjg at chromium.org>
> Cc: Tom Rini <trini at konsulko.com>
> ---
>  include/test/ut.h  | 152 ++++++++++++++++++++++++++++++---------------
>  test/cmd/pwm.c     |   4 +-
>  test/dm/acpigen.c  |   2 +-
>  test/dm/misc.c     |   4 +-
>  test/dm/phy.c      |   8 +--
>  test/dm/scmi.c     |   4 +-
>  test/lib/kconfig.c |  10 +--
>  test/unicode_ut.c  |   6 +-
>  8 files changed, 121 insertions(+), 69 deletions(-)
>

Gosh. That was pretty bad. Thanks for fixing.

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list