set_interface: Raise ValueError if no match is found #350

tormodvolden · 2021-01-13T17:58:47Z

Otherwise the type check for claim_interface will fail and the
error message is less helpful.

Signed-off-by: Tormod Volden [email protected]

BEFORE:

>>> dev.set_interface_altsetting(0, 1)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "pyusb/usb/core.py", line 936, in set_interface_altsetting
    self._ctx.managed_set_interface(self, interface, alternate_setting)
  File "pyusb/usb/core.py", line 113, in wrapper
    return f(self, *args, **kwargs)
  File "pyusb/usb/core.py", line 210, in managed_set_interface
    self.managed_claim_interface(device, i)
  File "pyusb/usb/core.py", line 113, in wrapper
    return f(self, *args, **kwargs)
  File "pyusb/usb/core.py", line 178, in managed_claim_interface
    self.backend.claim_interface(self.handle, i)
  File "pyusb/usb/backend/libusb1.py", line 829, in claim_interface
    _check(self.lib.libusb_claim_interface(dev_handle.handle, intf))
ctypes.ArgumentError: argument 2: <class 'TypeError'>: wrong type

AFTER:

...
ValueError: No matching interface (0,1)

Otherwise the type check for claim_interface will fail and the error message is less helpful. Signed-off-by: Tormod Volden <[email protected]>

jonasmalacofilho · 2021-01-20T06:18:48Z

Thanks!

set_interface: Raise ValueError if no match is found

2c3c07b

Otherwise the type check for claim_interface will fail and the error message is less helpful. Signed-off-by: Tormod Volden <[email protected]>

jonasmalacofilho merged commit a54ebc7 into pyusb:master Jan 20, 2021

tormodvolden deleted the intf_alt branch January 20, 2021 07:14

mcuee added the core label Jul 31, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

set_interface: Raise ValueError if no match is found #350

set_interface: Raise ValueError if no match is found #350

tormodvolden commented Jan 13, 2021 •

edited by jonasmalacofilho

Loading

jonasmalacofilho commented Jan 20, 2021

set_interface: Raise ValueError if no match is found #350

set_interface: Raise ValueError if no match is found #350

Conversation

tormodvolden commented Jan 13, 2021 • edited by jonasmalacofilho Loading

jonasmalacofilho commented Jan 20, 2021

tormodvolden commented Jan 13, 2021 •

edited by jonasmalacofilho

Loading