From patchwork Fri Oct 27 09:37:00 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Thomas Huth X-Patchwork-Id: 738657 Delivered-To: patch@linaro.org Received: by 2002:a5d:5101:0:b0:32d:baff:b0ca with SMTP id s1csp329217wrt; Fri, 27 Oct 2023 02:38:05 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEfqbH5Q2zxE0WokOkixIBp6qnLQ0M+Nibw2JkF42tK67eWUrzjqkq4ExIg70UgMnSIXxuH X-Received: by 2002:ad4:5c47:0:b0:66d:62b7:53f4 with SMTP id a7-20020ad45c47000000b0066d62b753f4mr3345168qva.45.1698399485294; Fri, 27 Oct 2023 02:38:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1698399485; cv=none; d=google.com; s=arc-20160816; b=l448yC8hb9KOne+E6XBblmElkK+2xn2vj+ylRcPV9eXAxE/IyWl2ppvhd95Tw+bpFp Pc7YE6+OtOm+yBzfeggCOin9UPfRc/4+o7INDcOO4hhEyBgg/2H3Oa6f/oeZM60ii+jE TUymZd+ULmLo+NKXJyxAsjb5ne6kxkrFgdI1QBDpGvAvLV4yKCkwHYMV8JEGRpfnX0wi Jkrz6AUi4dBU0mE2aP9xEZr6EaZ1zLgn/YVCDJp8CHC4OP2e9FUsPYNfG2AYRYECiOZa 5lXkqOH2T3jVxxyBJXjC2/I2ZBLTBwZLda78k5imA4GP2AeR06dTA1zX3EokHwX9SVjM eqAw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=PkDoNhXrEs6DDLniA7+cCqorQe2ZGbpgA9TWmT9RxzA=; fh=mKYVeSjHJKUvhIoxLlaWoOhsu0/LFnbJdOZ+CRgqMEY=; b=xpRaLTnxMBllLkBRDMZ3lAF3AinYWSXf8Xw/5uNfWFs7ke992JnMr/f/0hc906IxYl MD/cRRA+rSYifbcQ4l2CVMGoFOOg4OjVzPj1y3w2zZd3y9jn6yxotR/+KtVqCODk4lh3 yMDa8FQlrHWbi07yOpgzhwGGTYjaMVa+G/klHa5Blzgg5EL9cf9jhn17i8vFkgrD4O8M VrZFitiEpGsXhYE6T72khqcvon7TnM8GheY4/x5VHTpOuAr8Nm67rA8SjGriLVVlaFHt mFQXgpUGyj/BFzZCw5JD4GAFF91Yu389NNSQhOazdWdlIpxwQj85RFeNmrv9KnP4yK0W FI7w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=Yhps1ZGZ; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id bs33-20020a05620a472100b007756a7b3dfasi474770qkb.503.2023.10.27.02.38.05 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 27 Oct 2023 02:38:05 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=Yhps1ZGZ; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qwJHS-0002NY-Fk; Fri, 27 Oct 2023 05:37:26 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qwJHQ-0002MU-Do for qemu-devel@nongnu.org; Fri, 27 Oct 2023 05:37:24 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qwJHO-0002kt-TI for qemu-devel@nongnu.org; Fri, 27 Oct 2023 05:37:24 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1698399442; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=PkDoNhXrEs6DDLniA7+cCqorQe2ZGbpgA9TWmT9RxzA=; b=Yhps1ZGZv1s26+fef83SpRofrMZly3iIKqsGrBIyCfW7N0EK38tMaNdCX7oO8+747ThXow w0MEpqgjCygSFQrgSl02OHFD/YCoRgxgPVyVmnVnCJRYpw2VMf6Fc/O/fWbgqAWw6xlyf4 J6fASfGnn4L5GJQn2yk8GvWLdtPSB4g= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-442-lyjEXZ2hOHm3E2pRISMDAg-1; Fri, 27 Oct 2023 05:37:13 -0400 X-MC-Unique: lyjEXZ2hOHm3E2pRISMDAg-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 0889B88D711; Fri, 27 Oct 2023 09:37:13 +0000 (UTC) Received: from thuth-p1g4.redhat.com (unknown [10.39.192.196]) by smtp.corp.redhat.com (Postfix) with ESMTP id 25A551121319; Fri, 27 Oct 2023 09:37:12 +0000 (UTC) From: Thomas Huth To: qemu-devel@nongnu.org Cc: Stefan Hajnoczi , =?utf-8?q?Philippe_Mathieu-Daud?= =?utf-8?q?=C3=A9?= Subject: [PULL 01/11] system/qtest: Clean up global variable shadowing in qtest_server_init() Date: Fri, 27 Oct 2023 11:37:00 +0200 Message-ID: <20231027093710.273558-2-thuth@redhat.com> In-Reply-To: <20231027093710.273558-1-thuth@redhat.com> References: <20231027093710.273558-1-thuth@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.3 Received-SPF: pass client-ip=170.10.133.124; envelope-from=thuth@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: qemu-devel-bounces+patch=linaro.org@nongnu.org From: Philippe Mathieu-Daudé Rename the variable to fix: softmmu/qtest.c:869:13: error: declaration shadows a variable in the global scope [-Werror,-Wshadow] Object *qtest; ^ softmmu/qtest.c:53:15: note: previous declaration is here static QTest *qtest; ^ Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Thomas Huth Message-ID: <20231009100251.56019-2-philmd@linaro.org> Signed-off-by: Thomas Huth --- system/qtest.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/system/qtest.c b/system/qtest.c index 35b643a274..7964f0b248 100644 --- a/system/qtest.c +++ b/system/qtest.c @@ -866,7 +866,7 @@ void qtest_server_init(const char *qtest_chrdev, const char *qtest_log, Error ** { ERRP_GUARD(); Chardev *chr; - Object *qtest; + Object *qobj; chr = qemu_chr_new("qtest", qtest_chrdev, NULL); if (chr == NULL) { @@ -875,18 +875,18 @@ void qtest_server_init(const char *qtest_chrdev, const char *qtest_log, Error ** return; } - qtest = object_new(TYPE_QTEST); - object_property_set_str(qtest, "chardev", chr->label, &error_abort); + qobj = object_new(TYPE_QTEST); + object_property_set_str(qobj, "chardev", chr->label, &error_abort); if (qtest_log) { - object_property_set_str(qtest, "log", qtest_log, &error_abort); + object_property_set_str(qobj, "log", qtest_log, &error_abort); } - object_property_add_child(qdev_get_machine(), "qtest", qtest); - user_creatable_complete(USER_CREATABLE(qtest), errp); + object_property_add_child(qdev_get_machine(), "qtest", qobj); + user_creatable_complete(USER_CREATABLE(qobj), errp); if (*errp) { - object_unparent(qtest); + object_unparent(qobj); } object_unref(OBJECT(chr)); - object_unref(qtest); + object_unref(qobj); } static bool qtest_server_start(QTest *q, Error **errp)