Re: dinput: Count should not be 0-based.

[Vitaliy Margolen]

Paul Vriens wrote:
> Vitaliy Margolen wrote:
> > Fixes bug 9439.
> >
> > dlls/dinput/joystick_linux.c |    7 ++++---
> >  1 files changed, 4 insertions(+), 3 deletions(-)
>
> Hi Vitaliy,
>
> doesn't this patch leave it 0-based?
>  if (joystick_devices_count != -1) return joystick_devices_count;
>
> - joystick_devices_count = 0;
> for (i = 0; i < MAX_JOYSTICKS; i++)
> {
>     CHAR device_name[MAX_PATH], *str;
>
> @@ -154,7 +155,7 @@ static INT find_joystick_devices(void)
>          if (!(str = HeapAlloc(GetProcessHeap(), 0, len))) break;
>          memcpy(str, device_name, len);
> -
>    joystick_devices[++joystick_devices_count] = str;
> -
>    joystick_devices[joystick_devices_count++] = str;
> }
>
> AFAIK this doesn't change anything. Before it was:
>
> joystick_devices_count = -1
> ++joystick_devices_count -> 0 (for the first entry)
>
> now:
>
> joystick_devices_count = 0
> joystick_devices_count++ -> 0 (for the first entry)

I didn't say "index", I said "count" <g>. Count should never be "0-based" as
in "0" means "count of one". Index should be 0-based, as in "0" means first.

If the count is -1 then it haven't been initialized yet. If it's 0, then
there are no devices found. 1+ one or more devices have been found.

Where it does mater is in joydev_create_device[A|W], where I assumed that
count was 1 based.

Vitaliy.