From nobody Sun Feb 8 18:31:08 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1550668834883133.50622978920683; Wed, 20 Feb 2019 05:20:34 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id A049FC0BFBC4; Wed, 20 Feb 2019 13:20:31 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 5851B272BC; Wed, 20 Feb 2019 13:20:31 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 1288C3FB15; Wed, 20 Feb 2019 13:20:31 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id x1KDKOfU005198 for ; Wed, 20 Feb 2019 08:20:24 -0500 Received: by smtp.corp.redhat.com (Postfix) id 8AFB85D70E; Wed, 20 Feb 2019 13:20:24 +0000 (UTC) Received: from moe.brq.redhat.com (unknown [10.43.2.30]) by smtp.corp.redhat.com (Postfix) with ESMTP id DE4BB5D706; Wed, 20 Feb 2019 13:20:23 +0000 (UTC) From: Michal Privoznik To: libvir-list@redhat.com Date: Wed, 20 Feb 2019 14:20:10 +0100 Message-Id: <68320eb3d517e4c18d9c7d5eaed03687026bfdb0.1550668692.git.mprivozn@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-loop: libvir-list@redhat.com Cc: abologna@redhat.com Subject: [libvirt] [PATCH v2 3/3] testutils: Explicitly name virTestCompare*() arguments X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.32]); Wed, 20 Feb 2019 13:20:33 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Currently, some arguments are called strcontent and strsrc, or content and src or some other combination. This makes it impossible to see at the first glance what argument is supposed to represent 'expected' value and which one represents 'actual' value. Rename the arguments to make it obvious. Signed-off-by: Michal Privoznik Reviewed-by: John Ferlan Reviewed-by: Andrea Bolognani Reviewed-by: J=C3=A1n Tomko --- tests/testutils.c | 34 +++++++++++++--------------------- tests/testutils.h | 10 +++++----- 2 files changed, 18 insertions(+), 26 deletions(-) diff --git a/tests/testutils.c b/tests/testutils.c index 01f3e8bb93..13bb9630df 100644 --- a/tests/testutils.c +++ b/tests/testutils.c @@ -767,19 +767,19 @@ int virTestDifferenceBin(FILE *stream, } =20 /* - * @param strcontent: String input content - * @param filename: File to compare strcontent against + * @param actual: String input content + * @param filename: File to compare @actual against * - * If @strcontent is NULL, it's treated as an empty string. + * If @actual is NULL, it's treated as an empty string. */ int -virTestCompareToFile(const char *strcontent, +virTestCompareToFile(const char *actual, const char *filename) { int ret =3D -1; char *filecontent =3D NULL; char *fixedcontent =3D NULL; - const char *cmpcontent =3D strcontent; + const char *cmpcontent =3D actual; =20 if (!cmpcontent) cmpcontent =3D ""; @@ -814,36 +814,28 @@ virTestCompareToFile(const char *strcontent, return ret; } =20 -/* - * @param content: Input content - * @param src: Source to compare @content against - */ int -virTestCompareToULL(unsigned long long content, - unsigned long long src) +virTestCompareToULL(unsigned long long expect, + unsigned long long actual) { VIR_AUTOFREE(char *) expectStr =3D NULL; VIR_AUTOFREE(char *) actualStr =3D NULL; =20 - if (virAsprintf(&expectStr, "%llu", content) < 0) + if (virAsprintf(&expectStr, "%llu", expect) < 0) return -1; =20 - if (virAsprintf(&actualStr, "%llu", src) < 0) + if (virAsprintf(&actualStr, "%llu", actual) < 0) return -1; =20 return virTestCompareToString(expectStr, actualStr); } =20 -/* - * @param strcontent: String input content - * @param strsrc: String source to compare strcontent against - */ int -virTestCompareToString(const char *strcontent, - const char *strsrc) +virTestCompareToString(const char *expect, + const char *actual) { - if (STRNEQ_NULLABLE(strcontent, strsrc)) { - virTestDifference(stderr, strcontent, strsrc); + if (STRNEQ_NULLABLE(expect, actual)) { + virTestDifference(stderr, expect, actual); return -1; } =20 diff --git a/tests/testutils.h b/tests/testutils.h index 658f9053ad..d0a259e2d4 100644 --- a/tests/testutils.h +++ b/tests/testutils.h @@ -76,12 +76,12 @@ int virTestDifferenceBin(FILE *stream, const char *expect, const char *actual, size_t length); -int virTestCompareToFile(const char *strcontent, +int virTestCompareToFile(const char *actual, const char *filename); -int virTestCompareToString(const char *strcontent, - const char *strsrc); -int virTestCompareToULL(unsigned long long content, - unsigned long long src); +int virTestCompareToString(const char *expect, + const char *actual); +int virTestCompareToULL(unsigned long long expect, + unsigned long long actual); =20 unsigned int virTestGetDebug(void); unsigned int virTestGetVerbose(void); --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list