Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752261AbdL0PUW (ORCPT ); Wed, 27 Dec 2017 10:20:22 -0500 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:61245 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752097AbdL0PUU (ORCPT ); Wed, 27 Dec 2017 10:20:20 -0500 X-IronPort-AV: E=Sophos;i="5.45,466,1508796000"; d="scan'208";a="249567228" From: Julia Lawall To: Peter Zijlstra Cc: kernel-janitors@vger.kernel.org, Ingo Molnar , Arnaldo Carvalho de Melo , Alexander Shishkin , Jiri Olsa , Namhyung Kim , linux-kernel@vger.kernel.org Subject: [PATCH 08/12] perf test: drop unneeded newline Date: Wed, 27 Dec 2017 15:51:41 +0100 Message-Id: <1514386305-7402-9-git-send-email-Julia.Lawall@lip6.fr> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1514386305-7402-1-git-send-email-Julia.Lawall@lip6.fr> References: <1514386305-7402-1-git-send-email-Julia.Lawall@lip6.fr> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1768 Lines: 47 TEST_ASSERT_VAL prints a newline at the end of the message string, so the message string does not need to include a newline explicitly. Done using Coccinelle. There are some white-space adjustments in the calls to avoid exceeding 80 characters and to put continuation line arguments to the right of the (. Signed-off-by: Julia Lawall --- tools/perf/tests/dso-data.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c index 7f6c520..f514515 100644 --- a/tools/perf/tests/dso-data.c +++ b/tools/perf/tests/dso-data.c @@ -150,7 +150,7 @@ int test__dso_data(struct test *test __maybe_unused, int subtest __maybe_unused) u8 *buf; buf = malloc(TEST_FILE_SIZE); - TEST_ASSERT_VAL("ENOMEM\n", buf); + TEST_ASSERT_VAL("ENOMEM", buf); /* First iteration to fill caches, second one to read them. */ for (c = 0; c < 2; c++) { @@ -265,8 +265,8 @@ int test__dso_data_cache(struct test *test __maybe_unused, int subtest __maybe_u /* and this is now our dso open FDs limit */ dso_cnt = limit / 2; - TEST_ASSERT_VAL("failed to create dsos\n", - !dsos__create(dso_cnt, TEST_FILE_SIZE)); + TEST_ASSERT_VAL("failed to create dsos", + !dsos__create(dso_cnt, TEST_FILE_SIZE)); for (i = 0; i < (dso_cnt - 1); i++) { struct dso *dso = dsos[i]; @@ -336,7 +336,8 @@ int test__dso_data_reopen(struct test *test __maybe_unused, int subtest __maybe_ TEST_ASSERT_VAL("failed to set file limit", !set_fd_limit((nr + 3))); - TEST_ASSERT_VAL("failed to create dsos\n", !dsos__create(3, TEST_FILE_SIZE)); + TEST_ASSERT_VAL("failed to create dsos", + !dsos__create(3, TEST_FILE_SIZE)); /* open dso_0 */ fd = dso__data_fd(dso_0, &machine);