[PATCH 5/5] wined3d: Prefer the core glActiveTexture function.

Henri Verbeet hverbeet at gmail.com
Mon Jan 5 11:32:45 CST 2015


On 5 January 2015 at 17:17, Matteo Bruni <mbruni at codeweavers.com> wrote:
> +#define MAP_GL_FUNCTION(core_func, ext_func)                                      \
> +    {                                                                             \
> +        if (!gl_info->gl_ops.ext.p_##core_func)                                   \
> +            gl_info->gl_ops.ext.p_##core_func = gl_info->gl_ops.ext.p_##ext_func; \
> +    }
> +
> +    MAP_GL_FUNCTION(glActiveTexture, glActiveTextureARB);
> +#undef MAP_GL_FUNCTION
> +
This kind of thing doesn't exactly fill me with joy, but I guess the
alternatives wouldn't be much better.

Any reason this can't be part of load_gl_funcs()? (And while we're
touching load_gl_funcs(), I don't see much of a reason we couldn't
just inline GL_EXT_FUNCS_GEN these days.)

> +    /* Newer core functions */ \
> +    USE_GL_FUNC(glActiveTexture) \
It would probably be nice to note the exact GL version where the
function entered core GL.



More information about the wine-devel mailing list