fix flake8 error

hthiery · hthiery · commit d888f474f1cb · 2026-07-06T10:59:06.000+02:00
E128 continuation line under-indented for visual indent

Signed-off-by: Heiko Thiery &lt;heiko.thiery@gmail.com&gt;
diff --git a/examples/interface_rmcp.py b/examples/interface_rmcp.py
@@ -5,9 +5,9 @@
 
 
 intf = pyipmi.interfaces.create_interface('rmcp',
-                                               slave_address=0x81,
-                                               host_target_address=0x20,
-                                               keep_alive_interval=0)
+                                          slave_address=0x81,
+                                          host_target_address=0x20,
+                                          keep_alive_interval=0)
 sess = pyipmi.Session()
 sess.set_session_type_rmcp('10.0.114.116', 623)
 sess.set_auth_type_user('admin', 'admin')
diff --git a/pyipmi/__init__.py b/pyipmi/__init__.py
@@ -98,7 +98,7 @@ class Target(object):
     ipmb_address = None
 
     def __init__(self, ipmb_address: int | None = None,
-                routing: str | list[tuple] | None = None) -> None:
+                 routing: str | list[tuple] | None = None) -> None:
         """Initializer for the Target class.
 
         `ipmb_address` is the IPMB target address
@@ -165,8 +165,8 @@ class Ipmi(bmc.Bmc, chassis.Chassis, dcmi.Dcmi, fru.Fru, picmg.Picmg, hpm.Hpm,
            messaging.Messaging):
 
     def __init__(self, interface: Any = None, target: Target | None = None,
-                session: Session = Session(),
-                requester: Any = NullRequester()) -> None:
+                 session: Session = Session(),
+                 requester: Any = NullRequester()) -> None:
         self._interface = interface
 
         # we need a session, set if not passed
@@ -187,7 +187,7 @@ def __enter__(self) -> Ipmi:
         return self
 
     def __exit__(self, exception_type: Any, exception_value: Any,
-                traceback: Any) -> bool:
+                 traceback: Any) -> bool:
         self.close()
         return False
 
diff --git a/pyipmi/bmc.py b/pyipmi/bmc.py
@@ -53,11 +53,11 @@ def i2c_write_read(self, bus_type: int, bus_id: int, channel: int,
         return rsp.data
 
     def i2c_write(self, bus_type: int, bus_id: int, channel: int,
-                 address: int, data: bytes) -> None:
+                  address: int, data: bytes) -> None:
         self.i2c_write_read(bus_type, bus_id, channel, address, 0, data)
 
     def i2c_read(self, bus_type: int, bus_id: int, channel: int,
-                address: int, count: int) -> array:
+                 address: int, count: int) -> array:
         return self.i2c_write_read(bus_type, bus_id, channel,
                                    address, count, None)
 
diff --git a/pyipmi/errors.py b/pyipmi/errors.py
@@ -38,8 +38,8 @@ class CompletionCodeError(Exception):
     """IPMI completion code not OK."""
 
     def __init__(self, cc: int, cmdid: int | None = None,
-                netfn: int | None = None,
-                group_extension: int | None = None) -> None:
+                 netfn: int | None = None,
+                 group_extension: int | None = None) -> None:
         self.cc = cc
         self.cc_desc = self.find_cc_desc(cc, cmdid, netfn, group_extension)
 
diff --git a/pyipmi/fields.py b/pyipmi/fields.py
@@ -80,7 +80,7 @@ class TypeLengthString(object):
     TYPE_ASCII_OR_UTF16 = 3
 
     def __init__(self, data: bytes | None = None, offset: int = 0,
-                force_lang_eng: bool = False, sdr: bool = False) -> None:
+                 force_lang_eng: bool = False, sdr: bool = False) -> None:
         if data:
             self._from_data(data, offset, force_lang_eng)
 
@@ -110,7 +110,7 @@ def _from_data(self, data: bytes, offset: int = 0,
 class FruTypeLengthString(TypeLengthString):
 
     def __init__(self, data: bytes | None = None, offset: int = 0,
-                force_lang_eng: bool = False) -> None:
+                 force_lang_eng: bool = False) -> None:
         super(FruTypeLengthString, self).__init__(data, offset,
                                                   force_lang_eng,
                                                   sdr=False)
@@ -119,5 +119,5 @@ def __init__(self, data: bytes | None = None, offset: int = 0,
 class SdrTypeLengthString(TypeLengthString):
 
     def __init__(self, data: bytes | None = None, offset: int = 0,
-                force_lang_eng: bool = False) -> None:
+                 force_lang_eng: bool = False) -> None:
         super(SdrTypeLengthString, self).__init__(data, sdr=True)
diff --git a/pyipmi/fru.py b/pyipmi/fru.py
@@ -54,7 +54,7 @@ def write_fru_data(self, data: bytes, offset: int = 0,
             offset += len(chunk)
 
     def read_fru_data(self, offset: int | None = None,
-                     count: int | None = None, fru_id: int = 0) -> bytes:
+                      count: int | None = None, fru_id: int = 0) -> bytes:
         req_size = 32
         data = array.array('B')
 
@@ -153,7 +153,7 @@ def get_fru_multirecord_area(self, fru_id: int = 0,
         return InventoryMultiRecordArea(data, ignore_checksum=ignore_checksum)
 
     def get_fru_inventory(self, fru_id: int = 0,
-                         ignore_checksum: bool = False) -> FruInventory:
+                          ignore_checksum: bool = False) -> FruInventory:
         """
         Get the full parsed FRU inventory data.
         """
@@ -192,7 +192,7 @@ def get_fru_inventory(self, fru_id: int = 0,
 
 
 def get_fru_inventory_from_file(filename: str,
-                               ignore_checksum: bool = False) -> FruInventory:
+                                ignore_checksum: bool = False) -> FruInventory:
     try:
         file = open(filename, "rb")
     except IOError:
@@ -222,7 +222,7 @@ def _decode_custom_fields(data: bytes) -> list[FruTypeLengthString]:
 
 class FruData(object):
     def __init__(self, data: bytes | str | None = None,
-                ignore_checksum: bool = False) -> None:
+                 ignore_checksum: bool = False) -> None:
         if data:
             if isinstance(data, str):
                 data = [ord(c) for c in data]
@@ -456,7 +456,7 @@ def _from_data(self, data: bytes, ignore_checksum: bool = False) -> None:
 
 class FruInventory(object):
     def __init__(self, data: bytes | None = None,
-                ignore_checksum: bool = False) -> None:
+                 ignore_checksum: bool = False) -> None:
         self.chassis_info_area = None
         self.board_info_area = None
         self.product_info_area = None
diff --git a/pyipmi/hpm.py b/pyipmi/hpm.py
@@ -77,7 +77,7 @@ def get_target_upgrade_capabilities(self) -> TargetUpgradeCapabilities:
         return TargetUpgradeCapabilities(rsp)
 
     def get_component_property(self, component_id: int,
-                              property_id: int) -> ComponentProperty | None:
+                               property_id: int) -> ComponentProperty | None:
         rsp = self.send_message_with_name('GetComponentProperties',
                                           id=component_id,
                                           selector=property_id)
diff --git a/pyipmi/interfaces/ipmbdev.py b/pyipmi/interfaces/ipmbdev.py
@@ -19,7 +19,7 @@ class IpmbDev(object):
     NAME = 'ipmbdev'
 
     def __init__(self, slave_address: int = 0x20,
-                port: str = '/dev/ipmb-0') -> None:
+                 port: str = '/dev/ipmb-0') -> None:
         # TODO: slave address is currently not defined here
         self.slave_address = slave_address
         self.port = port
diff --git a/pyipmi/interfaces/rmcp.py b/pyipmi/interfaces/rmcp.py
@@ -233,7 +233,7 @@ class IpmiMsg(object):
     HEADER_FORMAT_AUTH = '!BII16BB'
 
     def __init__(self, session: Session | None = None,
-                ignore_sdu_length: bool = False) -> None:
+                 ignore_sdu_length: bool = False) -> None:
         self.session = session
         self.ignore_sdu_length = ignore_sdu_length
 
@@ -366,9 +366,9 @@ class Rmcp(object):
     _session: Session | None = None
 
     def __init__(self, slave_address: int = 0x81,
-                host_target_address: int = 0x20,
-                keep_alive_interval: int = 1, max_retries: int = 0,
-                quirks_cfg: dict = dict()) -> None:
+                 host_target_address: int = 0x20,
+                 keep_alive_interval: int = 1, max_retries: int = 0,
+                 quirks_cfg: dict = dict()) -> None:
         """Native RMCP interface constructor
 
         Parameter `quirks_cfg`: a dict of additional configuration parameters
diff --git a/pyipmi/msgs/message.py b/pyipmi/msgs/message.py
@@ -27,7 +27,7 @@
 
 class BaseField(object):
     def __init__(self, name: str, length: int | None,
-                default: Any = None) -> None:
+                 default: Any = None) -> None:
         self.name = name
         self.length = length
         self.default = default
@@ -46,7 +46,7 @@ def create(self) -> Any:
 
 class ByteArray(BaseField):
     def __init__(self, name: str, length: int,
-                default: bytes | None = None) -> None:
+                 default: bytes | None = None) -> None:
         BaseField.__init__(self, name, length)
         if default is not None:
             self.default = array('B', default)
@@ -140,7 +140,7 @@ def decode(self, obj: Message, data: ByteBuffer) -> None:
 
 class UnsignedIntMask(UnsignedInt):
     def __init__(self, name: str, length: int, mask: int,
-                default: int | None = None) -> None:
+                 default: int | None = None) -> None:
         UnsignedInt.__init__(self, name, length, default)
 
 
@@ -151,7 +151,7 @@ def __init__(self, name: str) -> None:
 
 class Conditional(object):
     def __init__(self, cond_fn: Callable[[Message], bool],
-                field: BaseField) -> None:
+                 field: BaseField) -> None:
         self._condition_fn = cond_fn
         self._field = field
 
@@ -210,7 +210,7 @@ def create(self) -> array:
 class Bitfield(BaseField):
     class Bit(object):
         def __init__(self, name: str, width: int = 1,
-                    default: int | None = None) -> None:
+                     default: int | None = None) -> None:
             self.name = name
             self._width = width
             self.default = default
@@ -226,7 +226,7 @@ def __init__(self, width: int, default: int = 0) -> None:
 
     class BitWrapper(object):
         def __init__(self, bits: tuple[Bitfield.Bit, ...],
-                    length: int) -> None:
+                     length: int) -> None:
             self._bits = bits
             self._length = length
             for bit in bits:
@@ -307,7 +307,7 @@ def create(self) -> Bitfield.BitWrapper:
 
 class GroupExtensionIdentifier(UnsignedInt):
     def __init__(self, name: str = 'picmg_identifier',
-                value: int | None = None) -> None:
+                 value: int | None = None) -> None:
         UnsignedInt.__init__(self, name, 1, value)
 
 
diff --git a/pyipmi/msgs/registry.py b/pyipmi/msgs/registry.py
@@ -64,7 +64,7 @@ def register_class(self, cls: type[Message]) -> type[Message]:
         return cls
 
     def create(self, netfn: int, cmdid: int, group_extension: int | None,
-              *args: Any, **kwargs: Any) -> Message:
+               *args: Any, **kwargs: Any) -> Message:
         return self.registry[(netfn, cmdid, group_extension)](*args, **kwargs)
 
     def create_response(self, req: Message) -> Message:
diff --git a/pyipmi/picmg.py b/pyipmi/picmg.py
@@ -149,7 +149,7 @@ def set_port_state(self, link_descr: LinkDescriptor, state: int) -> None:
         check_completion_code(rsp.completion_code)
 
     def get_port_state(self, channel_number: int,
-                      channel_interface: int) -> tuple[LinkDescriptor, int]:
+                       channel_interface: int) -> tuple[LinkDescriptor, int]:
         req = create_request_by_name('GetPortState')
         req.channel.number = channel_number
         req.channel.interface = channel_interface
@@ -377,8 +377,8 @@ class LedState(State):
     ]
 
     def __init__(self, rsp: Message | None = None, fru_id: int | None = None,
-                led_id: int | None = None, color: int | None = None,
-                function: int | None = None) -> None:
+                 led_id: int | None = None, color: int | None = None,
+                 function: int | None = None) -> None:
         super(LedState, self).__init__(rsp)
         if fru_id is not None:
             self.fru_id = fru_id
diff --git a/pyipmi/sdr.py b/pyipmi/sdr.py
@@ -192,7 +192,7 @@ def _from_response(self, rsp: Message) -> None:
 
 class SdrCommon(object):
     def __init__(self, data: bytes | None = None,
-                next_id: int | None = None) -> None:
+                 next_id: int | None = None) -> None:
         if data:
             self.data = data
             self._common_header(data)
@@ -274,7 +274,7 @@ class SdrFullSensorRecord(SdrCommon):
     DATA_FMT_NONE = 3
 
     def __init__(self, data: bytes | None = None,
-                next_id: int | None = None) -> None:
+                 next_id: int | None = None) -> None:
         super(SdrFullSensorRecord, self).__init__(data, next_id)
 
     def __str__(self) -> str:
@@ -502,7 +502,7 @@ def _from_data(self, data: bytes) -> None:
 ##################################################
 class SdrCompactSensorRecord(SdrCommon):
     def __init__(self, data: bytes | None = None,
-                next_id: int | None = None) -> None:
+                 next_id: int | None = None) -> None:
         super(SdrCompactSensorRecord, self).__init__(data, next_id)
 
     def __str__(self) -> str:
@@ -543,7 +543,7 @@ def _from_data(self, data: bytes) -> None:
 ##################################################
 class SdrEventOnlySensorRecord(SdrCommon):
     def __init__(self, data: bytes | None = None,
-                next_id: int | None = None) -> None:
+                 next_id: int | None = None) -> None:
         super(SdrEventOnlySensorRecord, self).__init__(data, next_id)
 
     def __str__(self) -> str:
@@ -571,7 +571,7 @@ def _from_data(self, data: bytes) -> None:
 ##################################################
 class SdrFruDeviceLocator(SdrCommon):
     def __init__(self, data: bytes | None = None,
-                next_id: int | None = None) -> None:
+                 next_id: int | None = None) -> None:
         super(SdrFruDeviceLocator, self).__init__(data, next_id)
 
     def __str__(self) -> str:
@@ -599,7 +599,7 @@ def _from_data(self, data: bytes) -> None:
 ##################################################
 class SdrManagementControllerDeviceLocator(SdrCommon):
     def __init__(self, data: bytes | None = None,
-                next_id: int | None = None) -> None:
+                 next_id: int | None = None) -> None:
         super(SdrManagementControllerDeviceLocator, self).__init__(
                 data, next_id)
 
@@ -627,7 +627,7 @@ def _from_data(self, data: bytes) -> None:
 ##################################################
 class SdrManagementControllerConfirmationRecord(SdrCommon):
     def __init__(self, data: bytes | None = None,
-                next_id: int | None = None) -> None:
+                 next_id: int | None = None) -> None:
         super(SdrManagementControllerConfirmationRecord, self).__init__(
                 data, next_id)
 
@@ -649,7 +649,7 @@ def _from_data(self, data: bytes) -> None:
 ##################################################
 class SdrOEMSensorRecord(SdrCommon):
     def __init__(self, data: bytes | None = None,
-                next_id: int | None = None) -> None:
+                 next_id: int | None = None) -> None:
         super(SdrOEMSensorRecord, self).__init__(data, next_id)
 
     def __str__(self) -> str:
@@ -665,7 +665,7 @@ def _from_data(self, data: bytes) -> None:
 # Any SDR type not known or not implemented
 class SdrUnknownSensorRecord(SdrCommon):
     def __init__(self, data: bytes | None = None,
-                next_id: int | None = None) -> None:
+                 next_id: int | None = None) -> None:
         super(SdrUnknownSensorRecord, self).__init__(data, next_id)
 
     def __str__(self) -> str:
