Minor cleanup
authorReto Buerki <reet@codelabs.ch>
Fri, 29 Sep 2017 14:09:12 +0000 (16:09 +0200)
committerReto Buerki <reet@codelabs.ch>
Fri, 29 Sep 2017 15:03:53 +0000 (17:03 +0200)
12 files changed:
src/acpi.py
src/bindings.py
src/creator.py
src/devicecap.py
src/extractor.py
src/fadt.py
src/message.py
src/output.py
src/parseutil.py
src/schemadata.py
src/update.py
src/util.py

index ccbec70..aed36da 100644 (file)
@@ -22,7 +22,6 @@
 import os
 import shutil
 import subprocess
-import sys
 import uuid
 
 import extractor
index 182c89f..f8bb8c2 100644 (file)
@@ -21,7 +21,6 @@
 # Module to handle generation of PyXB binding file
 import os
 import subprocess
-import shutil
 import customExceptions
 import paths
 import sys
@@ -79,7 +78,7 @@ def createBindings(schemafilepath, outpath, outname, pyxbgenpath):
 
     print "Running PyXB 'pyxbgen' command..."
     try:
-        proc = subprocess.check_call(
+        subprocess.check_call(
             [pyxbgenpath, "-u", infile, "-m", outname, "--binding-root",
              outpath], env=myenv)
     except subprocess.CalledProcessError:
index d33add9..3b9a86f 100644 (file)
@@ -31,10 +31,7 @@ import message
 import devicecap
 import extractor
 import subprocess
-import shutil
-import mmap
-import struct
-from collections import namedtuple, OrderedDict, deque
+from collections import namedtuple, OrderedDict
 import schemadata
 
 Address = namedtuple("Address", "start end")
@@ -774,7 +771,7 @@ class IommuDevicesCreator():
                                                    address,
                                                    regsize),
                        16)
-        except IOError, e:
+        except IOError:
             message.addError("Could not access file: %s - %s" %
                              (devmem, hex(address)), False)
             return -1
index 5453b59..00f092e 100644 (file)
@@ -26,7 +26,6 @@ import os
 import customExceptions
 import util
 import struct
-import message
 from collections import namedtuple
 
 # CAPABILITY CODES
@@ -95,6 +94,7 @@ def get_cap_msix(fileobj):
     value = CAP_MSIX_VALUE(enable=enablebit)
     return value
 
+
 capswitcher = {
     CAP_MSI: get_cap_msi,
     CAP_MSIX: get_cap_msix
@@ -216,7 +216,8 @@ class DevicecapManager():
     def _getCapValue(self, capcode, fileobj):
         """Gets the extra information for capabilities in form of tuple.
         Returns None if no Value found."""
-        result = lambda x: None
+        def result(x):
+            return None
         try:
             result = capswitcher[capcode]
         except KeyError:
index 1bf492b..4122fe8 100644 (file)
@@ -20,6 +20,7 @@
 
 # Module that handles extraction of data
 import customExceptions
+import struct
 
 
 def extractData(loc):
@@ -36,8 +37,6 @@ def extractData(loc):
 
     return trimmedDataString
 
-import struct
-
 
 def extractBinaryData(file, start, bytes, endian="BIG_ENDIAN", chunks=False):
 
index a704dc9..d36a8ea 100644 (file)
@@ -19,8 +19,6 @@
 
 # Module that handles extraction of data from the FADT ACPI table
 
-from collections import namedtuple
-
 import re
 
 import acpi
index 5250a6a..f12bbdb 100644 (file)
@@ -19,7 +19,6 @@
 
 
 # Module to contain functions to handle messages
-import sys
 import customExceptions
 import textwrap
 
index 64124c1..c84ea28 100644 (file)
@@ -24,7 +24,6 @@ import paths
 import message
 import subprocess
 import parseutil
-import util
 import customExceptions
 import importlib
 
index 42f6ef3..03801e6 100644 (file)
@@ -22,9 +22,6 @@
 import util
 import extractor
 import os
-import shutil
-import subprocess
-import message
 import customExceptions
 
 
index 24ea89d..cb644b7 100644 (file)
@@ -22,7 +22,6 @@
 import sys
 import os
 import paths
-import subprocess
 import importlib
 
 # Global variable declaring binding module
index c1ce349..ace0584 100644 (file)
@@ -25,9 +25,6 @@ import urllib2
 import shutil
 import customExceptions
 import parseutil
-import extractor
-import bindings
-import os
 
 PCI_IDS = "https://pci-ids.ucw.cz/v2.2/pci.ids"
 
index f5551ab..20805f7 100644 (file)
@@ -21,7 +21,6 @@
 # Module to contain utility functions
 import math
 import os
-import customExceptions
 
 # == Misc functions ==
 
@@ -176,7 +175,7 @@ def unwrapWord64(word64):
         result = "0x0"
     try:
         int(result, 16)  # Check if result is a hex string
-    except ValueError as e:
+    except ValueError:
         raise ValueError(
             "unwrapWord64: Invalid word64 format to unwrap: %s" % word64)
     return result