refactoring

kvmd/mouse.py

@@ -36,3 +36,13 @@ def remap(cls, value: int, out_min: int, out_max: int) -> int:
     @classmethod
     def normalize(cls, value: int) -> int:
         return min(max(cls.MIN, value), cls.MAX)
+
+
+class MouseDelta:
+    MIN = -127
+    MAX = 127
+    RANGE = (MIN, MAX)
+
+    @classmethod
+    def normalize(cls, value: int) -> int:
+        return min(max(cls.MIN, value), cls.MAX)

kvmd/plugins/hid/_mcu/proto.py

@@ -26,6 +26,7 @@
 from ....keyboard.mappings import KEYMAP
 
 from ....mouse import MouseRange
+from ....mouse import MouseDelta
 
 from .... import tools
 from .... import bitbang
@@ -162,8 +163,8 @@ class MouseRelativeEvent(BaseEvent):
     delta_y: int
 
     def __post_init__(self) -> None:
-        assert -127 <= self.delta_x <= 127
-        assert -127 <= self.delta_y <= 127
+        assert MouseDelta.MIN <= self.delta_x <= MouseDelta.MAX
+        assert MouseDelta.MIN <= self.delta_y <= MouseDelta.MAX
 
     def make_request(self) -> bytes:
         return _make_request(struct.pack(">Bbbxx", 0x15, self.delta_x, self.delta_y))
@@ -175,8 +176,8 @@ class MouseWheelEvent(BaseEvent):
     delta_y: int
 
     def __post_init__(self) -> None:
-        assert -127 <= self.delta_x <= 127
-        assert -127 <= self.delta_y <= 127
+        assert MouseDelta.MIN <= self.delta_x <= MouseDelta.MAX
+        assert MouseDelta.MIN <= self.delta_y <= MouseDelta.MAX
 
     def make_request(self) -> bytes:
         # Горизонтальная прокрутка пока не поддерживается

kvmd/plugins/hid/ch9329/mouse.py

@@ -23,6 +23,7 @@
 import math
 
 from ....mouse import MouseRange
+from ....mouse import MouseDelta
 
 
 # =====
@@ -79,7 +80,7 @@ def __fix_absolute(self, value: int) -> tuple[int, int]:
 
     def process_wheel(self, delta_x: int, delta_y: int) -> bytes:
         _ = delta_x
-        assert -127 <= delta_y <= 127
+        assert MouseDelta.MIN <= delta_y <= MouseDelta.MAX
         self.__wheel_y = (1 if delta_y > 0 else 255)
         if not self.__absolute:
             return self.__make_relative_cmd()
@@ -110,6 +111,6 @@ def __make_relative_cmd(self) -> bytes:
         ])
 
     def __fix_relative(self, value: int) -> int:
-        assert -127 <= value <= 127
+        assert MouseDelta.MIN <= value <= MouseDelta.MAX
         value = math.ceil(value / 3)
         return (value if value >= 0 else (255 + value))

kvmd/plugins/hid/otg/events.py

@@ -27,6 +27,7 @@
 from ....keyboard.mappings import KEYMAP
 
 from ....mouse import MouseRange
+from ....mouse import MouseDelta
 
 
 # =====
@@ -144,8 +145,8 @@ class MouseRelativeEvent(BaseEvent):
     delta_y: int
 
     def __post_init__(self) -> None:
-        assert -127 <= self.delta_x <= 127
-        assert -127 <= self.delta_y <= 127
+        assert MouseDelta.MIN <= self.delta_x <= MouseDelta.MAX
+        assert MouseDelta.MIN <= self.delta_y <= MouseDelta.MAX
 
 
 @dataclasses.dataclass(frozen=True)
@@ -154,8 +155,8 @@ class MouseWheelEvent(BaseEvent):
     delta_y: int
 
     def __post_init__(self) -> None:
-        assert -127 <= self.delta_x <= 127
-        assert -127 <= self.delta_y <= 127
+        assert MouseDelta.MIN <= self.delta_x <= MouseDelta.MAX
+        assert MouseDelta.MIN <= self.delta_y <= MouseDelta.MAX
 
 
 def make_mouse_report(

kvmd/validators/hid.py

@@ -25,6 +25,7 @@
 from ..keyboard.mappings import KEYMAP
 
 from ..mouse import MouseRange
+from ..mouse import MouseDelta
 
 from . import check_string_in_list
 
@@ -46,7 +47,7 @@ def valid_hid_key(arg: Any) -> str:
 
 def valid_hid_mouse_move(arg: Any) -> int:
     arg = valid_number(arg, name="Mouse move")
-    return min(max(MouseRange.MIN, arg), MouseRange.MAX)
+    return MouseRange.normalize(arg)
 
 
 def valid_hid_mouse_button(arg: Any) -> str:
@@ -55,4 +56,4 @@ def valid_hid_mouse_button(arg: Any) -> str:
 
 def valid_hid_mouse_delta(arg: Any) -> int:
     arg = valid_number(arg, name="Mouse delta")
-    return min(max(-127, arg), 127)
+    return MouseDelta.normalize(arg)