[Wayland-bugs] [Bug 719836] Client side decorations cannot be deactivated with wayland backend

[gtk+ (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=719836
  gtk+ | wayland | 3.11.x

Matthias Clasen <mclasen> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #263503|none                        |needs-work
             status|                            |

--- Comment #2 from Matthias Clasen <mclasen at redhat.com> 2013-12-04 15:10:59 UTC ---
Review of attachment 263503:
 --> (https://bugzilla.gnome.org/review?bug=719836&attachment=263503)

::: gtk/gtkwindow.c
@@ +5407,3 @@
+  csd_env = g_getenv ("GTK_CSD");
+  csd_described = csd_env != NULL;
+  csd_enabled = g_strcmp0 (g_getenv ("GTK_CSD"), "1") == 0;

Cosmetics, but I think csd_described is an odd name. I'd just use

g_strcmp0 (csd_env, "0") == 0

directly in the wayland clause below

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.