diff options
| author | Chris Dickens <christopher.a.dickens@gmail.com> | 2020-04-28 22:04:11 -0700 |
|---|---|---|
| committer | Chris Dickens <christopher.a.dickens@gmail.com> | 2020-04-28 22:04:11 -0700 |
| commit | e9eec3a680cad3b2c9c5213fb7d60148cf6900da (patch) | |
| tree | f525022f61d24aba50da630ed851807f8c3104d6 /libusb/os/windows_winusb.h | |
| parent | 14a302a2f55cb2e619158854f94845f2ca2c8214 (diff) | |
| download | libusb-e9eec3a680cad3b2c9c5213fb7d60148cf6900da.tar.gz | |
core: Narrow the types passed to certain backend functions
Backend functions dealing with interfaces and alternate settings should
use a type whose range represents that of valid values for interfaces
and alternate settings. Switch to use uint8_t instead of int so that
backends do not have to cast values or do range checks.
Signed-off-by: Chris Dickens <christopher.a.dickens@gmail.com>
Diffstat (limited to 'libusb/os/windows_winusb.h')
| -rw-r--r-- | libusb/os/windows_winusb.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/libusb/os/windows_winusb.h b/libusb/os/windows_winusb.h index 39ce6fe..88863ff 100644 --- a/libusb/os/windows_winusb.h +++ b/libusb/os/windows_winusb.h @@ -98,10 +98,10 @@ struct windows_usb_api_backend { void (*exit)(void); int (*open)(int sub_api, struct libusb_device_handle *dev_handle); void (*close)(int sub_api, struct libusb_device_handle *dev_handle); - int (*configure_endpoints)(int sub_api, struct libusb_device_handle *dev_handle, int iface); - int (*claim_interface)(int sub_api, struct libusb_device_handle *dev_handle, int iface); - int (*set_interface_altsetting)(int sub_api, struct libusb_device_handle *dev_handle, int iface, int altsetting); - int (*release_interface)(int sub_api, struct libusb_device_handle *dev_handle, int iface); + int (*configure_endpoints)(int sub_api, struct libusb_device_handle *dev_handle, uint8_t iface); + int (*claim_interface)(int sub_api, struct libusb_device_handle *dev_handle, uint8_t iface); + int (*set_interface_altsetting)(int sub_api, struct libusb_device_handle *dev_handle, uint8_t iface, uint8_t altsetting); + int (*release_interface)(int sub_api, struct libusb_device_handle *dev_handle, uint8_t iface); int (*clear_halt)(int sub_api, struct libusb_device_handle *dev_handle, unsigned char endpoint); int (*reset_device)(int sub_api, struct libusb_device_handle *dev_handle); int (*submit_bulk_transfer)(int sub_api, struct usbi_transfer *itransfer); |