Fix cross-platform build and type errors in tab drag implementation

- Add _glfwPlatformSetDragTitle stub for X11, Wayland, and null platforms
- Fix _glfwInputDrop declaration to match implementation (void return)
- Fix drop_callback return type to match GLFWdropfun typedef
- Fix type error: use 0 instead of None for dragging_tab_id
- Add freeDragMimes cleanup in _glfwPlatformDestroyWindow to prevent memory leak

Author: seijikohara

glfw/cocoa_window.m

@@ -2360,6 +2360,9 @@ void _glfwPlatformDestroyWindow(_GLFWwindow* window)
         window->ns.dragTitle = NULL;
     }
 
+    // Clean up cached drag MIME types
+    freeDragMimes(window);
+
     if (_glfw.ns.disabledCursorWindow == window)
         _glfw.ns.disabledCursorWindow = NULL;
 

glfw/internal.h

@@ -824,7 +824,7 @@ void _glfwInputScroll(_GLFWwindow* window, const GLFWScrollEvent *ev);
 void _glfwInputMouseClick(_GLFWwindow* window, int button, int action, int mods);
 void _glfwInputCursorPos(_GLFWwindow* window, double xpos, double ypos);
 void _glfwInputCursorEnter(_GLFWwindow* window, bool entered);
-int _glfwInputDrop(_GLFWwindow* window, const char *mime, const char *text, size_t sz);
+void _glfwInputDrop(_GLFWwindow* window, const char *mime, const char *text, size_t sz);
 int _glfwInputDragEvent(_GLFWwindow* window, int event, double xpos, double ypos, const char** mime_types, int* mime_count);
 void _glfwInputDragEnd(_GLFWwindow* window, bool accepted, double screen_x, double screen_y);
 void _glfwInputColorScheme(GLFWColorScheme, bool);

glfw/null_window.c

@@ -544,6 +544,11 @@ void _glfwPlatformUpdateDragState(_GLFWwindow* window UNUSED)
     // No-op for null platform
 }
 
+void _glfwPlatformSetDragTitle(_GLFWwindow* window UNUSED, const char* title UNUSED)
+{
+    // No-op for null platform
+}
+
 void _glfwPlatformSetClipboardString(const char* string)
 {
     char* copy = _glfw_strdup(string);

glfw/wl_window.c

@@ -3198,3 +3198,8 @@ _glfwPlatformUpdateDragState(_GLFWwindow* window) {
     }
 }
 
+void _glfwPlatformSetDragTitle(_GLFWwindow* window UNUSED, const char* title UNUSED)
+{
+    // No-op: drag title display not supported on Wayland
+}
+

glfw/x11_window.c

@@ -3868,3 +3868,8 @@ void _glfwPlatformUpdateDragState(_GLFWwindow* window) {
     XFlush(_glfw.x11.display);
 }
 
+void _glfwPlatformSetDragTitle(_GLFWwindow* window UNUSED, const char* title UNUSED)
+{
+    // No-op: drag title display not supported on X11
+}
+

kitty/glfw.c

@@ -757,24 +757,23 @@ drag_callback(GLFWwindow *w, GLFWDragEventType event, double xpos, double ypos,
     return ret;
 }
 
-static int
+static void
 drop_callback(GLFWwindow *w, const char *mime, const char *data, size_t sz) {
-    if (!set_callback_window(w)) return 0;
-#define RETURN(x) { global_state.callback_os_window = NULL; return x; }
+    if (!set_callback_window(w)) return;
+#define RETURN { global_state.callback_os_window = NULL; return; }
 
     // Handle kitty tab drop
     if (is_kitty_tab_mime(mime)) {
-        if (!data) RETURN(1);  // Accept kitty tab MIMEs during ENTER check
+        if (!data) RETURN;  // Accept kitty tab MIMEs during ENTER check
         WINDOW_CALLBACK(on_tab_drop, "s#", data, (Py_ssize_t)sz);
         request_tick_callback();
-        RETURN(1);
+        RETURN;
     }
     // Handle other drops (files, text, etc.)
-    if (!data) RETURN(is_droppable_mime(mime) ? 1 : 0);
+    if (!data) RETURN;
     WINDOW_CALLBACK(on_drop, "sy#", mime, data, (Py_ssize_t)sz);
     request_tick_callback();
     global_state.callback_os_window = NULL;
-    return 1;
 #undef RETURN
 }
 

kitty/tabs.py

@@ -1695,7 +1695,7 @@ def _cancel_tab_drag(self) -> None:
         """Cancel the current tab drag operation and clean up state."""
         if self.tab_drag_state is not None:
             self.tab_drag_state = None
-            self.tab_bar.dragging_tab_id = None
+            self.tab_bar.dragging_tab_id = 0
             self.tab_bar.update_drag_indicator(None, None)
             set_tab_bar_drag_in_progress(False)
             self.mark_tab_bar_dirty()