Re: [PATCH] winepulse.drv: Use the ARRAY_SIZE() macro

[Andrew Eikum]

Signed-off-by: Andrew Eikum

On Thu, Aug 09, 2018 at 08:59:35PM +0200, Michael Stefaniuc wrote:
> Signed-off-by: Michael Stefaniuc
>
> dlls/winepulse.drv/mmdevdrv.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/dlls/winepulse.drv/mmdevdrv.c b/dlls/winepulse.drv/mmdevdrv.c
> index 32f7acb91d..8d57f1a45a 100644
> --- a/dlls/winepulse.drv/mmdevdrv.c
> +++ b/dlls/winepulse.drv/mmdevdrv.c
> @@ -469,7 +469,7 @@ static HRESULT pulse_connect(void)
>      if (pulse_ctx)
>          pa_context_unref(pulse_ctx);
> - GetModuleFileNameW(NULL, path, sizeof(path)/sizeof(*path));
> - GetModuleFileNameW(NULL, path, ARRAY_SIZE(path));
> name = strrchrW(path, '\');
> if (!name)
>     name = path;
>
> @@ -537,7 +537,7 @@ static HRESULT pulse_test_connect(void)
>  pa_mainloop_set_poll_func(pulse_ml, pulse_poll_func, NULL);
>
> - GetModuleFileNameW(NULL, path, sizeof(path)/sizeof(*path));
> - GetModuleFileNameW(NULL, path, ARRAY_SIZE(path));
> name = strrchrW(path, '\');
> if (!name)
>     name = path;
>
> @@ -1283,7 +1283,7 @@ static HRESULT pulse_spec_from_waveformat(ACImpl *This, const WAVEFORMATEX *fmt)
>          This->map.channels = fmt->nChannels;
>          if (!mask || (mask & (SPEAKER_ALL|SPEAKER_RESERVED)))
>              mask = get_channel_mask(fmt->nChannels);
> -
>    for (j = 0; j < sizeof(pulse_pos_from_wfx)/sizeof(*pulse_pos_from_wfx) && i < fmt->nChannels; ++j) {
> -
>    for (j = 0; j < ARRAY_SIZE(pulse_pos_from_wfx) && i < fmt->nChannels; ++j) {
>        if (mask & (1 << j))
>            This->map.map[i++] = pulse_pos_from_wfx[j];
>    }
>
> -- 
> 2.14.4