From nobody Mon Feb 9 13:22:29 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1619060945; cv=none; d=zohomail.com; s=zohoarc; b=XNPoHaG9/4IDdH1V2dYASB5RVfHrrrWCI/doOsn2EhEXbCcR4OQtCK5w00m1CpL75ui+pKHopR+ZNE+b0ehT7tT8gmjJ8u3A8zDrbo+Lpy27uToMYyCxYBS9YsvNRRq81VMI5GyjwcrjeXcVXLxn5bhtGg95X9eckh/V48jTzuM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1619060945; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=I7gEPdebCE6paPsNskhQIQjVSM3mJoBcyzJdr5zoIOc=; b=eU07J+RvetNYCVIx3kp9mKZ88TLhv4M2ZrhtAbm9330x0JvpLdnolD7tNynemXbvndD5NkMmMFHyXr/DvGc7HjfJlfu1kDPf2i8WXbbfzu/svtSKFwmZhjVK1et2srV0e3cCNjdb1LA8+3vm9XODuiUNNqiHgD/kp4t/w7jxyEg= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1619060945435577.2547801706214; Wed, 21 Apr 2021 20:09:05 -0700 (PDT) Received: from localhost ([::1]:42044 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lZPiC-00082k-4h for importer@patchew.org; Wed, 21 Apr 2021 23:09:04 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:49022) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lZPgr-0006LH-9U for qemu-devel@nongnu.org; Wed, 21 Apr 2021 23:07:41 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]:36361) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lZPgh-0006Ro-HV for qemu-devel@nongnu.org; Wed, 21 Apr 2021 23:07:38 -0400 Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-216-VT97YbiIMr-3bPzbwYNgqQ-1; Wed, 21 Apr 2021 23:07:24 -0400 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 0EE3B801814; Thu, 22 Apr 2021 03:07:24 +0000 (UTC) Received: from scv.redhat.com (ovpn-118-152.rdu2.redhat.com [10.10.118.152]) by smtp.corp.redhat.com (Postfix) with ESMTP id 54364607CB; Thu, 22 Apr 2021 03:07:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1619060847; 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=I7gEPdebCE6paPsNskhQIQjVSM3mJoBcyzJdr5zoIOc=; b=DuVdFvpevaKHXRJpXMbX7ovXNuGMpu+BAtZcYMnLI1PGn7IfbnLEsYgmpPc5xZzLAICjpO H6s1ZV9yNlViB5hnGRH4P2YwpWG2zLpWxC6qN7dijpkj9WDa3NgkAXGG5IcuRIZzmHWVax FY6vtKWVi7jNgEf7RMupEyIB6yqUeIo= X-MC-Unique: VT97YbiIMr-3bPzbwYNgqQ-1 From: John Snow To: qemu-devel@nongnu.org Subject: [PATCH 03/22] qapi/source: Remove line number from QAPISourceInfo initializer Date: Wed, 21 Apr 2021 23:07:01 -0400 Message-Id: <20210422030720.3685766-4-jsnow@redhat.com> In-Reply-To: <20210422030720.3685766-1-jsnow@redhat.com> References: <20210422030720.3685766-1-jsnow@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=jsnow@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=170.10.133.124; envelope-from=jsnow@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 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_LOW=-0.7, RCVD_IN_MSPIKE_H4=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.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Michael Roth , Cleber Rosa , John Snow , Markus Armbruster , Eduardo Habkost Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" With the QAPISourceInfo(None, None, None) construct gone, there's not really any reason to have to specify that a file starts on the first line. Remove it from the initializer and have it default to 1. Remove the last vestiges where we check for 'line' being unset. That won't happen again, now! Signed-off-by: John Snow --- scripts/qapi/parser.py | 2 +- scripts/qapi/source.py | 12 +++--------- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/scripts/qapi/parser.py b/scripts/qapi/parser.py index b378fa33807..edd0af33ae0 100644 --- a/scripts/qapi/parser.py +++ b/scripts/qapi/parser.py @@ -47,7 +47,7 @@ def __init__(self, fname, previously_included=3DNone, inc= l_info=3DNone): if self.src =3D=3D '' or self.src[-1] !=3D '\n': self.src +=3D '\n' self.cursor =3D 0 - self.info =3D QAPISourceInfo(fname, 1, incl_info) + self.info =3D QAPISourceInfo(fname, incl_info) self.line_pos =3D 0 self.exprs =3D [] self.docs =3D [] diff --git a/scripts/qapi/source.py b/scripts/qapi/source.py index 21090b9fe78..afa21518974 100644 --- a/scripts/qapi/source.py +++ b/scripts/qapi/source.py @@ -37,10 +37,9 @@ def __init__(self) -> None: class QAPISourceInfo: T =3D TypeVar('T', bound=3D'QAPISourceInfo') =20 - def __init__(self, fname: str, line: int, - parent: Optional['QAPISourceInfo']): + def __init__(self, fname: str, parent: Optional['QAPISourceInfo'] =3D = None): self.fname =3D fname - self.line =3D line + self.line =3D 1 self._column: Optional[int] =3D None self.parent =3D parent self.pragma: QAPISchemaPragma =3D ( @@ -59,12 +58,7 @@ def next_line(self: T) -> T: return info =20 def loc(self) -> str: - # column cannot be provided meaningfully when line is absent. - assert self.line or self._column is None - - ret =3D self.fname - if self.line is not None: - ret +=3D ':%d' % self.line + ret =3D f"{self.fname}:{self.line}" if self._column is not None: ret +=3D ':%d' % self._column return ret --=20 2.30.2