X11: Fix function returning before cleanup

The _glfwPlatformSetWindowFloating function would return without freeing
the state array if the window was already in the requested state.

README.md

@@ -230,6 +230,7 @@ information on what to include when reporting a bug.
    reported (#1112,#1415,#1472,#1616)
  - [X11] Bugfix: Some window attributes were not applied on leaving fullscreen
    (#1863)
+ - [X11] Bugfix: Changing `GLFW_FLOATING` could leak memory
  - [Wayland] Added dynamic loading of all Wayland libraries
  - [Wayland] Removed support for `wl_shell` (#1443)
  - [Wayland] Bugfix: The `GLFW_HAND_CURSOR` shape used the wrong image (#1432)

src/x11_window.c

@@ -2671,15 +2671,15 @@ void _glfwPlatformSetWindowFloating(_GLFWwindow* window, GLFWbool enabled)
                     break;
             }
 
-            if (i < count)
-                return;
-
+            if (i == count)
+            {
                 XChangeProperty(_glfw.x11.display, window->x11.handle,
                                 _glfw.x11.NET_WM_STATE, XA_ATOM, 32,
                                 PropModeAppend,
                                 (unsigned char*) &_glfw.x11.NET_WM_STATE_ABOVE,
                                 1);
             }
+        }
         else if (states)
         {
             for (i = 0;  i < count;  i++)
@@ -2688,9 +2688,8 @@ void _glfwPlatformSetWindowFloating(_GLFWwindow* window, GLFWbool enabled)
                     break;
             }
 
-            if (i == count)
-                return;
-
+            if (i < count)
+            {
                 states[i] = states[count - 1];
                 count--;
 
@@ -2698,6 +2697,7 @@ void _glfwPlatformSetWindowFloating(_GLFWwindow* window, GLFWbool enabled)
                                 _glfw.x11.NET_WM_STATE, XA_ATOM, 32,
                                 PropModeReplace, (unsigned char*) states, count);
             }
+        }
 
         if (states)
             XFree(states);