hwdec_d3d11egl: EGL_EXT_device_query is client extension - #11142
Conversation
| gl->es == 200) || | ||
| !gl_check_extension(exts, "EGL_EXT_device_query") || | ||
| !(gl_check_extension(client_exts, "EGL_EXT_device_query") || | ||
| gl_check_extension(exts, "EGL_EXT_device_query")) || |
There was a problem hiding this comment.
if its a client extension why keep the second check?
There was a problem hiding this comment.
If I understand the comments in #10212 correctly, before November 2020 ANGLE exposed this in the old location, while current binaries like the one I just tested this with (https://github.com/google/gfbuild-angle/releases/tag/github%2Fgoogle%2Fgfbuild-angle%2Ff810e998993290f049bbdad4fae975e4867100ad) expose it in the new location.
This PR seems to attempt to support both ages of libraries.
edit: My slight personal opinion is that since it's already been two years and the new location seems to be the technically correct one, we might as well switch to supporting the new version only at this point?
There was a problem hiding this comment.
supporting old version is trivial work. some users may use outdated angle provided by framework(eg. qt?)
There was a problem hiding this comment.
This seems to have gotten completely lost but given that's been yet another two years, I think only supporting the new location at this point is the way to go.
|
I assume this is still an actual issue. Any takers? @kasper93 ? |
|
#18326 (now merged) is exactly the same as this. Not sure why this never got merged. Sorry about that. Closing. |
|
Oh, my bad. I had no recollection about this PR. Sorry, for pushing my version, but I did it completely separately. |
5 participants
close #10212