I think it's cleaner the way it is, this double return would never happen, it's just another implementation of the function.

On 11/16/2016 01:46 AM, Eric Engestrom wrote:
Signed-off-by: Eric Engestrom <[email protected]>
---
 src/egl/drivers/dri2/platform_wayland.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/egl/drivers/dri2/platform_wayland.c 
b/src/egl/drivers/dri2/platform_wayland.c
index 4fd4289..64eb4ef 100644
--- a/src/egl/drivers/dri2/platform_wayland.c
+++ b/src/egl/drivers/dri2/platform_wayland.c
@@ -693,9 +693,11 @@ try_damage_buffer(struct dri2_egl_surface *dri2_surf,
                                rect[2], rect[3]);
    }
    return EGL_TRUE;
-#endif
+#else
    return EGL_FALSE;
+#endif
 }
+
 /**
  * Called via eglSwapBuffers(), drv->API.SwapBuffers().
  */

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to