[PATCH 6/7] netlist.py: remove the forth 'None' return in get_network_selection()
Daniel Henrique Barboza
danielhb413 at gmail.com
Tue Jun 2 20:39:37 UTC 2020
Commit v2.2.1-200-g4c3c53f7 ("virtManager: Remove network portgroup
UI") removed 'portgroup', the forth value of the returned tuple in
get_network_selection(), and all related code that was using this
forth value.
That change forgot to change the line where, if no rows are found, a
tuple with "None" values is returned. The "None" tuple is still
returning 4 "None" values. Since no remaining code is checking for
a forth value, this is benign and has no impact in the logic.
'pylint' does not seem to care though, and it is complaining about
'unbalanced-tuple-unpacking' because, in the condition mentioned above,
a forth "None" value is returned and no one is bothering checking for
it.
Signed-off-by: Daniel Henrique Barboza <danielhb413 at gmail.com>
---
virtManager/device/netlist.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/virtManager/device/netlist.py b/virtManager/device/netlist.py
index 2b53dda9..0e70b6d4 100644
--- a/virtManager/device/netlist.py
+++ b/virtManager/device/netlist.py
@@ -267,7 +267,7 @@ class vmmNetworkList(vmmGObjectUI):
def get_network_selection(self):
row = self._get_network_row()
if not row:
- return None, None, None, None
+ return None, None, None
net_type = row[NET_ROW_TYPE]
net_src = row[NET_ROW_SOURCE]
--
2.26.2
More information about the virt-tools-list
mailing list