From 967748e489083c41a1ab31b805c637ecef29308b Mon Sep 17 00:00:00 2001 From: Vojtech Trefny Date: Wed, 22 May 2024 10:21:20 +0200 Subject: [PATCH] fix: Fix 'possibly-used-before-assignment' pylint issues Latest pylint added a new check for values used before assignment. This fixes these issues found in the blivet module. Some of these are false positives, some real potential issues. --- library/blivet.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/library/blivet.py b/library/blivet.py index d82b86b5..a6715d91 100644 --- a/library/blivet.py +++ b/library/blivet.py @@ -642,6 +642,9 @@ def _look_up_device(self): self._device = None return # TODO: see if we can create this device w/ the specified name + # pylint doesn't understand that "luks_fmt" is always set when "encrypted" is true + # pylint: disable=unknown-option-value + # pylint: disable=possibly-used-before-assignment def _update_from_device(self, param_name): """ Return True if param_name's value was retrieved from a looked-up device. """ log.debug("Updating volume settings from device: %r", self._device) @@ -1717,6 +1720,8 @@ def str_to_size(spec, hundredpercent=None): if auto_size_dev_count > 0: calculated_thinlv_size = available_space / auto_size_dev_count + else: + calculated_thinlv_size = available_space for thinlv in thinlvs_to_create: