early return

Author: alexrudd2

pymodbus_repl/client/mclient.py

@@ -81,30 +81,27 @@ def _wrapper(*args, **kwargs):
     _Base = object
 
 
-class ExtendedRequestSupport(_Base):  # pylint: disable=(too-many-public-methods
+class ExtendedRequestSupport(_Base):  # pylint: disable=too-many-public-methods
     """Extended request support."""
 
     @staticmethod
     def _process_exception(resp, **kwargs):
         """Set internal process exception."""
         if "slave" not in kwargs:
-            err = {"message": "Broadcast message, ignoring errors!!!"}
-        else:
-            if isinstance(resp, ExceptionResponse):  # pylint: disable=else-if-used
-                err = {
-                    "original_function_code": f"{resp.original_code} ({hex(resp.original_code)})",
-                    "error_function_code": f"{resp.function_code} ({hex(resp.function_code)})",
-                    "exception code": resp.exception_code,
-                    "message": ExceptionResponse.decode(resp.exception_code),
-                }
-            elif isinstance(resp, ModbusIOException):
-                err = {
-                    "original_function_code": f"{resp.fcode} ({hex(resp.fcode)})",
-                    "error": resp.message,
-                }
-            else:
-                err = {"error": str(resp)}
-        return err
+            return {"message": "Broadcast message, ignoring errors!!!"}
+        if isinstance(resp, ExceptionResponse):  # pylint: disable=else-if-used
+            return {
+                "original_function_code": f"{resp.original_code} ({hex(resp.original_code)})",
+                "error_function_code": f"{resp.function_code} ({hex(resp.function_code)})",
+                "exception code": resp.exception_code,
+                "message": ExceptionResponse.decode(resp.exception_code),
+            }
+        if isinstance(resp, ModbusIOException):
+            return {
+                "original_function_code": f"{resp.fcode} ({hex(resp.fcode)})",
+                "error": resp.message,
+            }
+        return {"error": str(resp)}
 
     def read_coils(self, address, count=1, slave=0, **kwargs):
         """Read `count` coils from a given slave starting at `address`.