Comment on why the tuple needs to be made of 6 elements

gvalkov · gvalkov · commit 62aacba7939b · 2018-05-31T23:42:50.000+02:00
diff --git a/evdev/uinput.py b/evdev/uinput.py
@@ -139,11 +139,13 @@ def __init__(self,
                     # Flatten (ABS_Y, (0, 255, 0, 0, 0, 0)) to (ABS_Y, 0, 255, 0, 0, 0, 0).
                     f = [code[0]]
                     f.extend(code[1])
+                    # Ensure the tuple is always 6 ints long, since uinput.c:uinput_create
+                    # does little in the way of checking the length.
                     f.extend([0] * (6 - len(code[1])))
                     absinfo.append(f)
                     code = code[0]
 
-                # TODO: a lot of unnecessary packing/unpacking
+                # TODO: remove a lot of unnecessary packing/unpacking
                 _uinput.enable(self.fd, etype, code)
 
         # Create the uinput device.
