summary refs log tree commit diff stats
path: root/scripts/simplebench/table_templater.py
diff options
context:
space:
mode:
authorHanna Reitz <hreitz@redhat.com>2021-09-02 11:40:15 +0200
committerHanna Reitz <hreitz@redhat.com>2021-09-15 15:54:07 +0200
commitd8c2e47dbe998d00dcfe0ca5fc0766a98b69a591 (patch)
treef551123e225cea4739fe5f2ca9f078795a13e67d /scripts/simplebench/table_templater.py
parente2ad17a62d9da45fbcddc3faa3d6ced519a9453a (diff)
downloadfocaccia-qemu-d8c2e47dbe998d00dcfe0ca5fc0766a98b69a591.tar.gz
focaccia-qemu-d8c2e47dbe998d00dcfe0ca5fc0766a98b69a591.zip
migrate-bitmaps-test: Fix pylint warnings
There are a couple of things pylint takes issue with:
- The "time" import is unused
- The import order (iotests should come last)
- get_bitmap_hash() doesn't use @self and so should be a function
- Semicolons at the end of some lines
- Parentheses after "if"
- Some lines are too long (80 characters instead of 79)
- inject_test_case()'s @name parameter shadows a top-level @name
  variable
- "lambda self: mc(self)" were equivalent to just "mc", but in
  inject_test_case(), it is not equivalent, so add a comment and disable
  the warning locally
- Always put two empty lines after a function
- f'exec: cat > /dev/null' does not need to be an f-string

Fix them.

Signed-off-by: Hanna Reitz <hreitz@redhat.com>
Message-Id: <20210902094017.32902-4-hreitz@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Diffstat (limited to 'scripts/simplebench/table_templater.py')
0 files changed, 0 insertions, 0 deletions