[PATCH] winebus.sys: Use the ARRAY_SIZE() macro
Aric Stewart
aric at codeweavers.com
Fri Aug 10 12:47:04 CDT 2018
Signed-off-by: Aric Stewart <aric at codeweavers.com>
On 8/9/18 1:59 PM, Michael Stefaniuc wrote:
> Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>
> ---
> dlls/winebus.sys/bus_iohid.c | 2 +-
> dlls/winebus.sys/main.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/dlls/winebus.sys/bus_iohid.c b/dlls/winebus.sys/bus_iohid.c
> index 501a40d26e..ac706f5265 100644
> --- a/dlls/winebus.sys/bus_iohid.c
> +++ b/dlls/winebus.sys/bus_iohid.c
> @@ -295,7 +295,7 @@ static void handle_DeviceMatchingCallback(void *context, IOReturn result, void *
> pid = CFNumberToDWORD(IOHIDDeviceGetProperty(IOHIDDevice, CFSTR(kIOHIDProductIDKey)));
> version = CFNumberToDWORD(IOHIDDeviceGetProperty(IOHIDDevice, CFSTR(kIOHIDVersionNumberKey)));
> str = IOHIDDeviceGetProperty(IOHIDDevice, CFSTR(kIOHIDSerialNumberKey));
> - if (str) CFStringToWSTR(str, serial_string, sizeof(serial_string) / sizeof(WCHAR));
> + if (str) CFStringToWSTR(str, serial_string, ARRAY_SIZE(serial_string));
>
> if (IOHIDDeviceConformsTo(IOHIDDevice, kHIDPage_GenericDesktop, kHIDUsage_GD_GamePad) ||
> IOHIDDeviceConformsTo(IOHIDDevice, kHIDPage_GenericDesktop, kHIDUsage_GD_Joystick))
> diff --git a/dlls/winebus.sys/main.c b/dlls/winebus.sys/main.c
> index 2f3c05aa96..deb2c86d96 100644
> --- a/dlls/winebus.sys/main.c
> +++ b/dlls/winebus.sys/main.c
> @@ -696,7 +696,7 @@ BOOL is_xbox_gamepad(WORD vid, WORD pid)
> if (vid != VID_MICROSOFT)
> return FALSE;
>
> - for (i = 0; i < sizeof(PID_XBOX_CONTROLLERS)/sizeof(*PID_XBOX_CONTROLLERS); i++)
> + for (i = 0; i < ARRAY_SIZE(PID_XBOX_CONTROLLERS); i++)
> if (pid == PID_XBOX_CONTROLLERS[i]) return TRUE;
>
> return FALSE;
>
More information about the wine-devel
mailing list