[minor] Fix linter (isort) errors.

Change the multi line output to keep the line width < 80.

Author: renanrodrigo

pyof/foundation/base.py

@@ -23,8 +23,8 @@
 from enum import Enum, IntEnum
 
 # Local source tree imports
-from pyof.foundation.exceptions import (BadValueException, PackException,
-                                        UnpackException, ValidationError)
+from pyof.foundation.exceptions import (
+    BadValueException, PackException, UnpackException, ValidationError)
 
 # Third-party imports
 

pyof/foundation/network_types.py

@@ -4,8 +4,8 @@
 """
 
 from pyof.foundation.base import GenericStruct
-from pyof.foundation.basic_types import (BinaryData, HWAddress, IPAddress,
-                                         UBInt8, UBInt16)
+from pyof.foundation.basic_types import (
+    BinaryData, HWAddress, IPAddress, UBInt8, UBInt16)
 from pyof.foundation.exceptions import PackException
 
 __all__ = ('Ethernet', 'GenericTLV', 'IPv4', 'TLVWithSubType', 'LLDP')
@@ -140,7 +140,7 @@ def get_size(self, value=None):
 
 
 class IPv4(GenericStruct):
-    """IPv4 packet "struct"
+    """IPv4 packet "struct".
 
     Contains all fields of an IP version 4 packet header, plus the upper layer
     content as binary data.
@@ -197,7 +197,7 @@ def __init__(self, version=4, ihl=5, dscp=0, ecn=0, length=0, # noqa
         self.data = data
 
     def _update_checksum(self):
-        """Updates the packet checksum to enable integrity check."""
+        """Update the packet checksum to enable integrity check."""
         source_list = [int(octet) for octet in self.source.split(".")]
         destination_list = [int(octet) for octet in
                             self.destination.split(".")]
@@ -218,6 +218,10 @@ def _update_checksum(self):
         self.checksum = ~block_sum & 65535
 
     def pack(self, value=None):
+        """Pack the struct in a binary representation.
+
+        Merge some fields to ensure correct packing.
+        """
         # Set the correct IHL based on options size
         if self.options:
             self.ihl += int(len(self.options) / 4)
@@ -235,6 +239,10 @@ def pack(self, value=None):
         return super().pack()
 
     def unpack(self, buff, offset=0):
+        """Unpack a binary struct into this object's attributes.
+
+        Return the values instead of the lib's basic types.
+        """
         super().unpack(buff, offset)
 
         self.version = self._version_ihl.value >> 4

pyof/v0x01/asynchronous/packet_in.py

@@ -4,8 +4,8 @@
 from enum import IntEnum
 
 from pyof.foundation.base import GenericMessage
-from pyof.foundation.basic_types import (BinaryData, Pad, UBInt8, UBInt16,
-                                         UBInt32)
+from pyof.foundation.basic_types import (
+    BinaryData, Pad, UBInt8, UBInt16, UBInt32)
 from pyof.v0x01.common.constants import NO_BUFFER
 from pyof.v0x01.common.header import Header, Type
 

pyof/v0x01/common/action.py

@@ -4,8 +4,8 @@
 
 # Local source tree imports
 from pyof.foundation.base import GenericBitMask, GenericStruct
-from pyof.foundation.basic_types import (FixedTypeList, HWAddress, Pad, UBInt8,
-                                         UBInt16, UBInt32)
+from pyof.foundation.basic_types import (
+    FixedTypeList, HWAddress, Pad, UBInt8, UBInt16, UBInt32)
 from pyof.foundation.constants import UBINT16_MAX_VALUE
 
 # Third-party imports

pyof/v0x01/common/flow_match.py

@@ -6,8 +6,8 @@
 
 # Local source tree imports
 from pyof.foundation.base import GenericBitMask, GenericStruct
-from pyof.foundation.basic_types import (HWAddress, IPAddress, Pad, UBInt8,
-                                         UBInt16, UBInt32)
+from pyof.foundation.basic_types import (
+    HWAddress, IPAddress, Pad, UBInt8, UBInt16, UBInt32)
 
 __all__ = ('Match', 'FlowWildCards')
 

pyof/v0x01/common/phy_port.py

@@ -5,8 +5,8 @@
 
 # Local source tree imports
 from pyof.foundation.base import GenericBitMask, GenericStruct
-from pyof.foundation.basic_types import (Char, FixedTypeList, HWAddress,
-                                         UBInt16, UBInt32)
+from pyof.foundation.basic_types import (
+    Char, FixedTypeList, HWAddress, UBInt16, UBInt32)
 from pyof.foundation.constants import OFP_MAX_PORT_NAME_LEN
 
 # Third-party imports

pyof/v0x01/common/utils.py

@@ -21,10 +21,10 @@
 from pyof.v0x01.controller2switch.get_config_request import GetConfigRequest
 from pyof.v0x01.controller2switch.packet_out import PacketOut
 from pyof.v0x01.controller2switch.port_mod import PortMod
-from pyof.v0x01.controller2switch.queue_get_config_reply import \
-    QueueGetConfigReply
-from pyof.v0x01.controller2switch.queue_get_config_request import \
-    QueueGetConfigRequest
+from pyof.v0x01.controller2switch.queue_get_config_reply import (
+    QueueGetConfigReply)
+from pyof.v0x01.controller2switch.queue_get_config_request import (
+    QueueGetConfigRequest)
 from pyof.v0x01.controller2switch.set_config import SetConfig
 from pyof.v0x01.controller2switch.stats_reply import StatsReply
 from pyof.v0x01.controller2switch.stats_request import StatsRequest

pyof/v0x01/controller2switch/common.py

@@ -4,10 +4,10 @@
 from enum import IntEnum
 
 from pyof.foundation.base import GenericMessage, GenericStruct
-from pyof.foundation.basic_types import (Char, Pad, UBInt8, UBInt16, UBInt32,
-                                         UBInt64)
-from pyof.foundation.constants import (DESC_STR_LEN, OFP_MAX_TABLE_NAME_LEN,
-                                       SERIAL_NUM_LEN)
+from pyof.foundation.basic_types import (
+    Char, Pad, UBInt8, UBInt16, UBInt32, UBInt64)
+from pyof.foundation.constants import (
+    DESC_STR_LEN, OFP_MAX_TABLE_NAME_LEN, SERIAL_NUM_LEN)
 # Local source tree imports
 from pyof.v0x01.common.action import ListOfActions
 from pyof.v0x01.common.flow_match import FlowWildCards, Match

pyof/v0x04/asynchronous/packet_in.py

@@ -4,8 +4,8 @@
 from enum import IntEnum
 
 from pyof.foundation.base import GenericMessage
-from pyof.foundation.basic_types import (BinaryData, Pad, UBInt8, UBInt16,
-                                         UBInt32, UBInt64)
+from pyof.foundation.basic_types import (
+    BinaryData, Pad, UBInt8, UBInt16, UBInt32, UBInt64)
 from pyof.v0x04.common.flow_match import Match
 from pyof.v0x04.common.header import Header, Type
 

pyof/v0x04/common/action.py

@@ -4,8 +4,8 @@
 
 # Local source tree imports
 from pyof.foundation.base import GenericStruct
-from pyof.foundation.basic_types import (FixedTypeList, Pad, UBInt8, UBInt16,
-                                         UBInt32)
+from pyof.foundation.basic_types import (
+    FixedTypeList, Pad, UBInt8, UBInt16, UBInt32)
 
 # Third-party imports