Re: [PATCH 0/4] MR5933: opengl32: Add default implementation for wglChoosePixelFormatARB.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/opengl32/wgl.c:
> +    case WGL_STENCIL_BITS_ARB: return 9;
> +    case WGL_ACCUM_BITS_ARB: return 10;
> +    case WGL_DOUBLE_BUFFER_ARB: return 11;
> +    case WGL_SWAP_METHOD_ARB: return 12;
> +    default: return 100;
> +    }
> +}
> +
> +static int compare_attribs( const void *a, const void *b )
> +{
> +    int prio_a = wgl_attrib_sort_priority( *(int *)a );
> +    int prio_b = wgl_attrib_sort_priority( *(int *)b );
> +
> +    if (prio_a == prio_b) return 0;
> +    else if (prio_a < prio_b) return -1;
> +    else return 1;
> Any reason not to do `return prio_a - prio_b`?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5933#note_74549