<div dir="ltr">Hi Pekka,<div class="gmail_extra"><br><div class="gmail_quote">On 28 May 2018 at 09:24, Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5"><span style="color:rgb(34,34,34)">do not use strstr() for matching extensions strings. It does a</span><br></div></div>
sub-string match, which may not be what you want. What if there was an<br>
extension called "platform_gbm_unixmem" or such?<br></blockquote><div><br></div><div>The substring match is what we want though, as the extension is one of the multiple extensions in the string, and using strstr() seems to be a fairly common practice (in glamor, glx and elsewhere, this is what is in the documentation for EGL_MESA_platform_gbm [1]  for example)</div><div><br></div><div>But yes, you're right, checking for the complete extension name "EGL_MESA_platform_gbm" might be preferable here.</div><div><br></div><div>Cheers,</div><div>Olivier.</div><div><br></div><div>[1] <a href="https://www.khronos.org/registry/EGL/extensions/MESA/EGL_MESA_platform_gbm.txt">https://www.khronos.org/registry/EGL/extensions/MESA/EGL_MESA_platform_gbm.txt</a></div></div></div></div>