Merge pull request #16 from dhalbert/fix_mouse_no_move

Fix Mouse bug introduced in 3.0.0

Author: tannewt

adafruit_hid/mouse.py

@@ -60,10 +60,10 @@ def __init__(self):
         # Do a no-op to test if HID device is ready.
         # If not, wait a bit and try once more.
         try:
-            self.move(0, 0, 0)
+            self._send_no_move()
         except OSError:
             time.sleep(1)
-            self.move(0, 0, 0)
+            self._send_no_move()
 
     def press(self, buttons):
         """Press the given mouse buttons.
@@ -80,7 +80,7 @@ def press(self, buttons):
             m.press(Mouse.LEFT_BUTTON | Mouse.RIGHT_BUTTON)
         """
         self.report[0] |= buttons
-        self.move(0, 0, 0)
+        self._send_no_move()
 
     def release(self, buttons):
         """Release the given mouse buttons.
@@ -89,12 +89,12 @@ def release(self, buttons):
             ``MIDDLE_BUTTON``, and ``RIGHT_BUTTON``.
         """
         self.report[0] &= ~buttons
-        self.move(0, 0, 0)
+        self._send_no_move()
 
     def release_all(self):
         """Release all the mouse buttons."""
         self.report[0] = 0
-        self.move(0, 0, 0)
+        self._send_no_move()
 
     def click(self, buttons):
         """Press and release the given mouse buttons.
@@ -111,7 +111,6 @@ def click(self, buttons):
             m.click(Mouse.LEFT_BUTTON)
             m.click(Mouse.LEFT_BUTTON)
         """
-
         self.press(buttons)
         self.release(buttons)
 
@@ -140,7 +139,6 @@ def move(self, x=0, y=0, wheel=0):
             # Roll the mouse wheel away from the user.
             m.move(wheel=1)
         """
-
         # Send multiple reports if necessary to move or scroll requested amounts.
         while x != 0 or y != 0 or wheel != 0:
             partial_x = self._limit(x)
@@ -154,6 +152,13 @@ def move(self, x=0, y=0, wheel=0):
             y -= partial_y
             wheel -= partial_wheel
 
+    def _send_no_move(self):
+        """Send a button-only report."""
+        self.report[1] = 0
+        self.report[2] = 0
+        self.report[3] = 0
+        self.hid_mouse.send_report(self.report)
+
     @staticmethod
     def _limit(dist):
         return min(127, max(-127, dist))