[Wayland-bugs] [Bug 764374] Busy loop while "Displays" page is active

[gtk+ (GNOME Bugzilla)]

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

Olivier Fourdan <ofourdan at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #325339|reviewed                    |none
             status|                            |
 Attachment #325339|0                           |1
        is obsolete|                            |

--- Comment #13 from Olivier Fourdan <ofourdan at redhat.com> ---
Created attachment 325357
  --> https://bugzilla.gnome.org/attachment.cgi?id=325357&action=edit
[PATCH v4] wayland: Do not resize with the same size

(In reply to Matthias Clasen from comment #12)
> Review of attachment 325339 [details] [review]:
> 
> I must say that I find the "from_configure" parameter naming mildly
> confusing in a function called gdk_wayland_window_configure, and looking at
> the callers, the extra TRUE/FALSE doesn't really communicate what is going
> on. An alternative approach might be to have a separate function, say
> gdk_wayland_window_maybe_configure() which would do the shortcut, and then
> call gdk_wayland_window_configure().

Sure, makes sense, updated patch attached.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/wayland-bugs/attachments/20160404/f282c56f/attachment.html>