Re: [PATCH v2 01/19] test: Restore test behaviour on failure

On Tue, Jun 20, 2023 at 4:23 PM Simon Glass sjg@chromium.org wrote:

A recent change makes test continue to run after failure. This results in a lot of useless output and may lead to a segfault. Fix this by adding back the 'return' statement.

Note that this is not a full revert of the original commit. It only removes the unwanted 'return' statements in each macro.

Fixes: fa847bb409d ("test: Wrap assert macros in ({ ... }) and fix")

Signed-off-by: Simon Glass sjg@chromium.org

Changes in v2:

• Add a comment that this is not a full revert

include/test/ut.h | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-)

Reviewed-by: Bin Meng bmeng.cn@gmail.com