qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Emanuele Giuseppe Esposito <eesposit@redhat.com>
To: qemu-block@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>,
	Emanuele Giuseppe Esposito <eesposit@redhat.com>,
	Hanna Reitz <hreitz@redhat.com>,
	qemu-devel@nongnu.org
Subject: [PATCH v2 2/2] qemu-iotests: fix image-fleecing pylint errors
Date: Fri,  8 Oct 2021 02:28:21 -0400	[thread overview]
Message-ID: <20211008062821.1010967-3-eesposit@redhat.com> (raw)
In-Reply-To: <20211008062821.1010967-1-eesposit@redhat.com>

The problem here is that some variables are formatted with
unnecessary spaces to make it prettier and easier to read.

However, pylint complains about those additional spaces.
Temporarly remove the check, but just on those variables.

Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
---
 tests/qemu-iotests/tests/image-fleecing | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tests/qemu-iotests/tests/image-fleecing b/tests/qemu-iotests/tests/image-fleecing
index 8c5472f421..99101fe902 100755
--- a/tests/qemu-iotests/tests/image-fleecing
+++ b/tests/qemu-iotests/tests/image-fleecing
@@ -30,6 +30,9 @@ iotests.script_initialize(
     supported_platforms=['linux'],
 )
 
+# Temporarly disable bad-whitespace pylint check to keep
+# this formatting.
+# pylint: disable=bad-whitespace
 patterns = [('0x5d', '0',         '64k'),
             ('0xd5', '1M',        '64k'),
             ('0xdc', '32M',       '64k'),
@@ -47,6 +50,7 @@ zeroes = [('0', '0x00f8000', '32k'), # Left-end of partial-left (1M-32K)
 remainder = [('0xd5', '0x108000',  '32k'), # Right-end of partial-left [1]
              ('0xdc', '32M',       '32k'), # Left-end of partial-right [2]
              ('0xcd', '0x3ff0000', '64k')] # patterns[3]
+# pylint: enable=bad-whitespace
 
 def do_test(use_cbw, base_img_path, fleece_img_path, nbd_sock_path, vm):
     log('--- Setting up images ---')
-- 
2.27.0



  parent reply	other threads:[~2021-10-08  6:32 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-08  6:28 [PATCH v2 0/2] pylint: fix new errors and warnings in qemu-iotests Emanuele Giuseppe Esposito
2021-10-08  6:28 ` [PATCH v2 1/2] pylint: fix errors and warnings generated by tests/qemu-iotests/297 Emanuele Giuseppe Esposito
2021-10-08  6:28 ` Emanuele Giuseppe Esposito [this message]
2021-10-11  9:29 ` [PATCH v2 0/2] pylint: fix new errors and warnings in qemu-iotests Hanna Reitz
2021-10-11  9:58   ` Emanuele Giuseppe Esposito
2021-10-11 11:04     ` Hanna Reitz
2021-10-26 18:50     ` John Snow

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211008062821.1010967-3-eesposit@redhat.com \
    --to=eesposit@redhat.com \
    --cc=hreitz@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).