Re: [PATCH v3 0/4] MR2091: windows.media.speech: Initial Unixlib work.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/windows.media.speech/vosk.c:
> return STATUS_DLL_NOT_FOUND;
>      }
>
> +#define LOAD_FUNCPTR( f ) \
> +    do if((p##f = dlsym(libvosk_handle, #f)) == NULL) \
> +    { \
> +        ERR("Failed to load symbol %s\n", #f); \
> +        goto error; \
> +    } while(0)

You don't need the `do while (0)` for this macro. Also maybe `!` instead of `== NULL`?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2091#note_22798