override: Convert to list comprehension

:%s/retlist = list()\n\s*\(for \w\+ in ret\):\n\s*retlist.append(\(.*\))\n\n\s*return retlist/return [\2 \1]/

Signed-off-by: Philipp Hahn <hahn@univention.de>

libvirt-override-virConnect.py

@@ -467,11 +467,7 @@
         if ret is None:
             raise libvirtError("virConnectListAllDomains() failed")
 
-        retlist = list()
+        return [virDomain(self, _obj=domptr) for domptr in ret]
-        for domptr in ret:
-            retlist.append(virDomain(self, _obj=domptr))
-
-        return retlist
 
     def listAllStoragePools(self, flags: int = 0) -> List['virStoragePool']:
         """Returns a list of storage pool objects"""
@@ -479,11 +475,7 @@
         if ret is None:
             raise libvirtError("virConnectListAllStoragePools() failed")
 
-        retlist = list()
+        return [virStoragePool(self, _obj=poolptr) for poolptr in ret]
-        for poolptr in ret:
-            retlist.append(virStoragePool(self, _obj=poolptr))
-
-        return retlist
 
     def listAllNetworks(self, flags: int = 0) -> List['virNetwork']:
         """Returns a list of network objects"""
@@ -491,11 +483,7 @@
         if ret is None:
             raise libvirtError("virConnectListAllNetworks() failed")
 
-        retlist = list()
+        return [virNetwork(self, _obj=netptr) for netptr in ret]
-        for netptr in ret:
-            retlist.append(virNetwork(self, _obj=netptr))
-
-        return retlist
 
     def listAllInterfaces(self, flags: int = 0) -> List['virInterface']:
         """Returns a list of interface objects"""
@@ -503,11 +491,7 @@
         if ret is None:
             raise libvirtError("virConnectListAllInterfaces() failed")
 
-        retlist = list()
+        return [virInterface(self, _obj=ifaceptr) for ifaceptr in ret]
-        for ifaceptr in ret:
-            retlist.append(virInterface(self, _obj=ifaceptr))
-
-        return retlist
 
     def listAllDevices(self, flags: int = 0) -> List['virNodeDevice']:
         """Returns a list of host node device objects"""
@@ -515,11 +499,7 @@
         if ret is None:
             raise libvirtError("virConnectListAllNodeDevices() failed")
 
-        retlist = list()
+        return [virNodeDevice(self, _obj=devptr) for devptr in ret]
-        for devptr in ret:
-            retlist.append(virNodeDevice(self, _obj=devptr))
-
-        return retlist
 
     def listAllNWFilters(self, flags: int = 0) -> List['virNWFilter']:
         """Returns a list of network filter objects"""
@@ -527,11 +507,7 @@
         if ret is None:
             raise libvirtError("virConnectListAllNWFilters() failed")
 
-        retlist = list()
+        return [virNWFilter(self, _obj=filter_ptr) for filter_ptr in ret]
-        for filter_ptr in ret:
-            retlist.append(virNWFilter(self, _obj=filter_ptr))
-
-        return retlist
 
     def listAllNWFilterBindings(self, flags: int = 0) -> List['virNWFilterBinding']:
         """Returns a list of network filter binding objects"""
@@ -539,11 +515,7 @@
         if ret is None:
             raise libvirtError("virConnectListAllNWFilterBindings() failed")
 
-        retlist = list()
+        return [virNWFilterBinding(self, _obj=filter_ptr) for filter_ptr in ret]
-        for filter_ptr in ret:
-            retlist.append(virNWFilterBinding(self, _obj=filter_ptr))
-
-        return retlist
 
     def listAllSecrets(self, flags: int = 0) -> List['virSecret']:
         """Returns a list of secret objects"""
@@ -551,11 +523,7 @@
         if ret is None:
             raise libvirtError("virConnectListAllSecrets() failed")
 
-        retlist = list()
+        return [virSecret(self, _obj=secret_ptr) for secret_ptr in ret]
-        for secret_ptr in ret:
-            retlist.append(virSecret(self, _obj=secret_ptr))
-
-        return retlist
 
     def _dispatchCloseCallback(self, reason: int, cbData: Dict[str, Any]) -> int:
         """Dispatches events to python user close callback"""
@@ -657,12 +625,7 @@
         if ret is None:
             raise libvirtError("virConnectGetAllDomainStats() failed")
 
-        retlist = list()
+        return [(virDomain(self, _obj=elem[0]), elem[1]) for elem in ret]
-        for elem in ret:
-            record = (virDomain(self, _obj=elem[0]), elem[1])
-            retlist.append(record)
-
-        return retlist
 
     def domainListGetStats(self, doms: List['virDomain'], stats: int = 0, flags: int = 0) -> List[Tuple['virDomain', Dict[str, Any]]]:
         """ Query statistics for given domains.
@@ -704,9 +667,4 @@
         if ret is None:
             raise libvirtError("virDomainListGetStats() failed")
 
-        retlist = list()
+        return [(virDomain(self, _obj=elem[0]), elem[1]) for elem in ret]
-        for elem in ret:
-            record = (virDomain(self, _obj=elem[0]), elem[1])
-            retlist.append(record)
-
-        return retlist

libvirt-override-virDomain.py

@@ -4,11 +4,7 @@
         if ret is None:
             raise libvirtError("virDomainListAllSnapshots() failed")
 
-        retlist = list()
+        return [virDomainSnapshot(self, _obj=snapptr) for snapptr in ret]
-        for snapptr in ret:
-            retlist.append(virDomainSnapshot(self, _obj=snapptr))
-
-        return retlist
 
     def listAllCheckpoints(self, flags: int = 0) -> List['virDomainCheckpoint']:
         """List all checkpoints and returns a list of checkpoint objects"""
@@ -16,11 +12,7 @@
         if ret is None:
             raise libvirtError("virDomainListAllCheckpoints() failed")
 
-        retlist = list()
+        return [virDomainCheckpoint(self, _obj=chkptr) for chkptr in ret]
-        for chkptr in ret:
-            retlist.append(virDomainCheckpoint(self, _obj=chkptr))
-
-        return retlist
 
     def createWithFiles(self, files: List[int], flags: int = 0) -> 'virDomain':
         """Launch a defined domain. If the call succeeds the domain moves from the

libvirt-override-virDomainCheckpoint.py

@@ -12,8 +12,4 @@
         if ret is None:
             raise libvirtError("virDomainCheckpointListAllChildren() failed")
 
-        retlist = list()
+        return [virDomainCheckpoint(self.domain(), _obj=chkptr) for chkptr in ret]
-        for chkptr in ret:
-            retlist.append(virDomainCheckpoint(self.domain(), _obj=chkptr))
-
-        return retlist

libvirt-override-virDomainSnapshot.py

@@ -12,8 +12,4 @@
         if ret is None:
             raise libvirtError("virDomainSnapshotListAllChildren() failed")
 
-        retlist = list()
+        return [virDomainSnapshot(self.domain(), _obj=snapptr) for snapptr in ret]
-        for snapptr in ret:
-            retlist.append(virDomainSnapshot(self.domain(), _obj=snapptr))
-
-        return retlist

libvirt-override-virNetwork.py

@@ -4,8 +4,4 @@
         if ret is None:
             raise libvirtError("virNetworkListAllPorts() failed")
 
-        retlist = list()
+        return [virNetworkPort(self, _obj=domptr) for domptr in ret]
-        for domptr in ret:
-            retlist.append(virNetworkPort(self, _obj=domptr))
-
-        return retlist

libvirt-override-virStoragePool.py

@@ -4,8 +4,4 @@
         if ret is None:
             raise libvirtError("virStoragePoolListAllVolumes() failed")
 
-        retlist = list()
+        return [virStorageVol(self._conn, _obj=volptr) for volptr in ret]
-        for volptr in ret:
-            retlist.append(virStorageVol(self._conn, _obj=volptr))
-
-        return retlist