[waffle] [PATCH 08/18] api: make waffle_get_proc_address() display aware
Emil Velikov
emil.l.velikov at gmail.com
Mon Aug 4 09:43:11 PDT 2014
On 04/08/14 08:17, Chad Versace wrote:
> Thunderbird believes this patch is Chinese... :(
>
In that case I will attach the patch (and push branch for-upstream-WGL-1.2
shortly).
> I need more time to look at this patch because of the API break.
>
> Even though eglGetProcAddress and glXGetProcAddress allow queries before the process
> makes any other EGL/GLX calls, in practice I see no harm in requiring Waffle users
> to postpone procAddress queries until after acquiring a display. Except for a few
> special cases, GetProc'd function pointers aren't useful anyway without a current
> context, which requires a display.
>
> So, I don't believe this should cause any real problems. But I want a little time to
> consider all the implications here.
>
On possible issue that I can see is if one uses waffle_get_proc_address()
using a pointer to the first waffle_display, while a second one is active. On
the other hand I assume that waffle itself may explode a bit before that :)
Or in other words I am _not_ a big fan of the two API/ABI changes yet I see no
clear way of solving this without enforcing additional and "interesting" rules
on the user.
> Also, when breaking the ABI, we will need to increment Waffle's major version. It's
> just a number, so no big deal. Waffle 2.0 won't be an earth-shattering change from
> Waffle 1.x (except for WGL support!). It will simply be another incremental improvement
> over its previous release, like Git 1.9 to Git 2.0.
>
And the current versioning scheme makes things even easier wrt having multiple
waffles side by side. I like mine with chocolate :)
-Emil
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0008-api-make-waffle_get_proc_address-display-aware.patch
Type: text/x-patch
Size: 10074 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/waffle/attachments/20140804/3baaae6c/attachment.bin>
More information about the waffle
mailing list