The test application calls printf(...) with the suite->suite_name argument. The intent (based on whitespace) in the printf is to check suite->suite_name first and then apply the printf. This doesn't happen due to missing brackets.
Signed-off-by: Aaron Conole <aconole at redhat.com> Acked-by: Panu Matilainen <pmatilai at redhat.com> --- v2: * No change (apart from adding ACKs) app/test/test.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/test/test.c b/app/test/test.c index f35b304..ccad0e3 100644 --- a/app/test/test.c +++ b/app/test/test.c @@ -162,9 +162,10 @@ unit_test_suite_runner(struct unit_test_suite *suite) int test_success; unsigned total = 0, executed = 0, skipped = 0, succeeded = 0, failed = 0; - if (suite->suite_name) + if (suite->suite_name) { printf(" + ------------------------------------------------------- +\n"); printf(" + Test Suite : %s\n", suite->suite_name); + } if (suite->setup) if (suite->setup() != 0) -- 2.8.0.rc2.35.gc2c5f6b.dirty