Re: [PATCH v5 0/3] MR5923: winegstreamer: Use affinity to determine 'max-threads' value.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/unixlib.c:
>  return true;
>
> }
>
> +static ULONG popcount( ULONG val )
> +{
> +#if defined(__MINGW32__)
> - return __builtin_popcount( val );
>
> +#else
> - val -= val >> 1 & 0x55555555;
> - val = (val & 0x33333333) + (val >> 2 & 0x33333333);
> - return ((val + (val >> 4)) & 0x0f0f0f0f) * 0x01010101 >> 24;
>
> +#endif
> +}
> -

Well, now the __builtin_popcount path is never taken, because on the unix side it's not built with MinGW. There's a compiler check available already though. (Also mind the style change to match winegstreamer)

```suggestion:-10+0
static ULONG popcount(ULONG val)
{
#if HAVE___BUILTIN_POPCOUNT
    return __builtin_popcount(val);
#else
    val -= val >> 1 & 0x55555555;
    val = (val & 0x33333333) + (val >> 2 & 0x33333333);
    return ((val + (val >> 4)) & 0x0f0f0f0f) * 0x01010101 >> 24;
#endif
}

```