From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id AAC662BCC for ; Thu, 25 Feb 2016 19:48:44 +0100 (CET) Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (Postfix) with ESMTPS id 310B78E3DF; Thu, 25 Feb 2016 18:48:44 +0000 (UTC) Received: from aconole-fed23.bos.redhat.com (dhcp-25-213.bos.redhat.com [10.18.25.213]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u1PImfwk000303; Thu, 25 Feb 2016 13:48:43 -0500 From: Aaron Conole To: dev@dpdk.org Date: Thu, 25 Feb 2016 13:48:35 -0500 Message-Id: <1456426121-21423-3-git-send-email-aconole@redhat.com> In-Reply-To: <1456426121-21423-1-git-send-email-aconole@redhat.com> References: <1456426121-21423-1-git-send-email-aconole@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.22 Subject: [dpdk-dev] [PATCH 2/8] app/test/test: Fix missing brackets X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 25 Feb 2016 18:48:44 -0000 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 --- 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.5.0