<div dir='auto'>Indeed, please revert. I'm in the process of<div dir="auto">removing this code altogether.</div><div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">-- </div><div dir="auto">Alexandre Janniaux</div><div dir="auto">Videolabs</div></div><div class="gmail_extra"><br><div class="gmail_quote">Le 26 févr. 2020 15:54, Romain Vimont <rom1v@videolabs.io> a écrit :<br type="attribution" /><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">This reverts commit 77aa424b72d8b6a1e7049c430c90303b9473ed7d.</p>
<p dir="ltr">The replacement was not equivalent:</p>
<p dir="ltr">    strncmp(a, b, strlen(b)) == 0  # test if a starts with b<br>
    strcmp(a, b) == 0              # test if a equals b</p>
<p dir="ltr">In practice, it breaks OpenGL on Android when the vout_modules contains<br>
"gles2,none".<br>
---<br>
 modules/video_output/android/utils.c | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)</p>
<p dir="ltr">diff --git a/modules/video_output/android/utils.c b/modules/video_output/android/utils.c<br>
index 25ac24676f..2010f13976 100644<br>
--- a/modules/video_output/android/utils.c<br>
+++ b/modules/video_output/android/utils.c<br>
@@ -546,8 +546,8 @@ AWindowHandler_new(vout_window_t *wnd, awh_events_t *p_events)<br>
          */<br>
         char *vout_modules = var_InheritString(wnd, "vout");<br>
         if (vout_modules<br>
-         && (strcmp(vout_modules, "gles2") == 0<br>
-          || strcmp(vout_modules, "opengles2") == 0))<br>
+         && (strncmp(vout_modules, "gles2", sizeof("gles2") - 1) == 0<br>
+          || strncmp(vout_modules, "opengles2", sizeof("opengles2") - 1) == 0))<br>
             p_awh->b_has_video_layout_listener = false;<br>
         free(vout_modules);<br>
     }<br>
-- <br>
2.25.1</p>
<p dir="ltr">_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
https://mailman.videolan.org/listinfo/vlc-devel</p>
</blockquote></div><br></div>