https://bugzilla.gnome.org/show_bug.cgi?id=766566

Emmanuele Bassi (:ebassi) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #328081|none                        |reviewed
             status|                            |

--- Comment #4 from Emmanuele Bassi (:ebassi) <[email protected]> ---
Review of attachment 328081:

::: gdk/wayland/gdkdisplay-wayland.c
@@ +853,3 @@
+
+  output = gdk_wayland_window_get_wl_output (window);
+  struct wl_output *output;

Instead of nesting the for loop under the if, you can flip the condition check
and bail out early:

  if (output == NULL)
    return NULL;

  for (i = 0; ...
    {
      ...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
wayland-bugs mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-bugs

Reply via email to