From nobody Sun Feb 8 14:13:00 2026 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 99A502F7478 for ; Mon, 19 Jan 2026 21:00:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768856447; cv=none; b=FF33t7vXdv/4YUBpNiSaNzFk5dEmscHsuBUVzIWHxWjJL0+qeE2XwcIEAzNZB8dKU1DuX2WsC7dEAO7pBLIg7xszBWNflmn76sqBg9NywEXzTHAlTiF4j1pXb4liNveWk0emAMgZBdXVhUA2Qt8yz6Nm6/zTCpWcCb4bm0Z8JjM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768856447; c=relaxed/simple; bh=8DkqGNezhZS+jns6pl7LkC98k3QrxCSUlXLi95NECOc=; h=From:To:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=OSaaSFx6vD1Yyu5G0+cDLfNWgVDlVUyL30/xn5P6CG76izhrtGijOzPSJNMYmaQ2tWoXLXRJIgpkDczvWdau3q/Mg+JDNYfwWvq7tbI0jtJ0bArarlwzRPo4550b+asLHtTFsTOryxmckA0yX8C8+xROPnCqGbw2/lWHQmn0ewY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=dG0FFSlY; arc=none smtp.client-ip=170.10.129.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="dG0FFSlY" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1768856444; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=PDv8xxnb+/FccRewmtuSoSQQbd73qMaYdppoDFSJu5Y=; b=dG0FFSlY75zqM0UxYxM8yEHqV+bRNNmJUaJAzgTbquXCRsQeJaQCLbPTaXcZ+JjwGaRp9z aKCChcRxxIECPKf890QnIG5C1WJN+DVCHvSaax5NDkHjJ2MelIJptfPauHwR+UuK5mXhbE 4MzymuVW0P5dNOemCweG+cqezC03bGo= Received: from mx-prod-mc-08.mail-002.prod.us-west-2.aws.redhat.com (ec2-35-165-154-97.us-west-2.compute.amazonaws.com [35.165.154.97]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-587-xnge12KDOD-KRF_wthjfQg-1; Mon, 19 Jan 2026 16:00:41 -0500 X-MC-Unique: xnge12KDOD-KRF_wthjfQg-1 X-Mimecast-MFC-AGG-ID: xnge12KDOD-KRF_wthjfQg_1768856440 Received: from mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.17]) (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 mx-prod-mc-08.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 27F421800473; Mon, 19 Jan 2026 21:00:40 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.22.80.137]) by mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 9FF6E19560AB; Mon, 19 Jan 2026 21:00:37 +0000 (UTC) From: Wander Lairson Costa To: Steven Rostedt , Gabriele Monaco , Nam Cao , Wander Lairson Costa , linux-kernel@vger.kernel.org (open list), linux-trace-kernel@vger.kernel.org (open list:RUNTIME VERIFICATION (RV)) Subject: [PATCH 05/26] rv/rvgen: remove unnecessary semicolons Date: Mon, 19 Jan 2026 17:45:41 -0300 Message-ID: <20260119205601.105821-6-wander@redhat.com> In-Reply-To: <20260119205601.105821-1-wander@redhat.com> References: <20260119205601.105821-1-wander@redhat.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.0 on 10.30.177.17 Content-Type: text/plain; charset="utf-8" Remove unnecessary semicolons from Python code in the rvgen tool. Python does not require semicolons to terminate statements, and their presence goes against PEP 8 style guidelines. These semicolons were likely added out of habit from C-style languages. The changes affect four instances across two files. In dot2c.py, one semicolon is removed from a boolean assignment. In dot2k.py, three semicolons are removed from string append operations that build generated C code. Note that the semicolons inside the string literals themselves are correctly preserved as they are part of the C code being generated, not Python syntax. This cleanup improves consistency with Python coding standards and aligns with the recent improvements to remove other Python anti-patterns from the codebase. Signed-off-by: Wander Lairson Costa Reviewed-by: Gabriele Monaco Reviewed-by: Nam Cao --- tools/verification/rvgen/rvgen/dot2c.py | 2 +- tools/verification/rvgen/rvgen/dot2k.py | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/verification/rvgen/rvgen/dot2c.py b/tools/verification/r= vgen/rvgen/dot2c.py index 0fb3617ad8ce9..b9a2c009a9246 100644 --- a/tools/verification/rvgen/rvgen/dot2c.py +++ b/tools/verification/rvgen/rvgen/dot2c.py @@ -120,7 +120,7 @@ class Dot2c(Automata): for entry in buff: if first: string =3D string + "\t\t\"" + entry - first =3D False; + first =3D False else: string =3D string + "\",\n\t\t\"" + entry string =3D string + "\"" diff --git a/tools/verification/rvgen/rvgen/dot2k.py b/tools/verification/r= vgen/rvgen/dot2k.py index 1c0d0235bdf62..291385adb2c20 100644 --- a/tools/verification/rvgen/rvgen/dot2k.py +++ b/tools/verification/rvgen/rvgen/dot2k.py @@ -37,10 +37,10 @@ class dot2k(Monitor, Dot2c): buff.append("\t/* XXX: validate that this event is only va= lid in the initial state */") handle =3D "handle_start_run_event" if self.monitor_type =3D=3D "per_task": - buff.append("\tstruct task_struct *p =3D /* XXX: how do I = get p? */;"); - buff.append(f"\tda_{handle}_{self.name}(p, {event}{self.en= um_suffix});"); + buff.append("\tstruct task_struct *p =3D /* XXX: how do I = get p? */;") + buff.append(f"\tda_{handle}_{self.name}(p, {event}{self.en= um_suffix});") else: - buff.append(f"\tda_{handle}_{self.name}({event}{self.enum_= suffix});"); + buff.append(f"\tda_{handle}_{self.name}({event}{self.enum_= suffix});") buff.append("}") buff.append("") return '\n'.join(buff) --=20 2.52.0