Add detailed explanations in Fl_Wayland_Gl_Window_Driver::make_current_before()

1 files changed, 7 insertions, 3 deletions

diff --git a/src/drivers/Wayland/Fl_Wayland_Gl_Window_Driver.cxx b/src/drivers/Wayland/Fl_Wayland_Gl_Window_Driver.cxx
index d6748842e..66183d647 100644
--- a/src/drivers/Wayland/Fl_Wayland_Gl_Window_Driver.cxx
+++ b/src/drivers/Wayland/Fl_Wayland_Gl_Window_Driver.cxx
@@ -231,9 +231,13 @@ void Fl_Wayland_Gl_Window_Driver::make_current_before() {
     egl_surface = eglCreateWindowSurface(egl_display, g->egl_conf, egl_window, NULL);
 //fprintf(stderr, "Created egl surface=%p at scale=%d\n", egl_surface, win->scale);
     wl_surface_set_buffer_scale(surface, win->scale);
-//TODO It's still not entirely clear how to justify the value 5 below
-    int count = 0;
-    while (count++ < 5) {
+    // Tested apps: shape, glpuzzle, cube, fractals, gl_overlay, fullscreen,
+    //   OpenGL3-glut-test, OpenGL3test
+    // Tested wayland compositors: mutter, kde-plasma, weston, sway on FreeBSD
+    // Origin of the value 3 below :
+    // All tests run OK with value 3 whereas some tests fail with 2, e.g.,
+    // glpuzzle + KDE, glpuzzle + sway sometimes.
+    for (int count = 0; count < 3; count++) {
       wl_display_roundtrip(Fl_Wayland_Screen_Driver::wl_display);
       eglSwapBuffers(Fl_Wayland_Gl_Window_Driver::egl_display, egl_surface);
     }