From 30347ea701456a12111ca106dac5e53c85989b9d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20M=C3=BCllner?= <fmuellner@gnome.org>
Date: Sun, 2 Nov 2025 19:53:44 +0100
Subject: [PATCH] js: Always use SOURCE_CONTINUE/SOURCE_REMOVE in source
 functions

The constants are more self-explanatory (and therefore readable)
than plain true/false.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell-extensions/-/merge_requests/435>
---
 extensions/places-menu/placeDisplay.js | 2 +-
 extensions/window-list/extension.js    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/extensions/places-menu/placeDisplay.js b/extensions/places-menu/placeDisplay.js
index 72e859ae..8e195366 100644
--- a/extensions/places-menu/placeDisplay.js
+++ b/extensions/places-menu/placeDisplay.js
@@ -267,7 +267,7 @@ export class PlacesManager extends EventEmitter {
                     GLib.PRIORITY_DEFAULT, 100, () => {
                         this._bookmarkTimeoutId = 0;
                         this._reloadBookmarks();
-                        return false;
+                        return GLib.SOURCE_REMOVE;
                     });
             });
 
diff --git a/extensions/window-list/extension.js b/extensions/window-list/extension.js
index 4c65b999..dad85d54 100644
--- a/extensions/window-list/extension.js
+++ b/extensions/window-list/extension.js
@@ -1476,7 +1476,7 @@ class WindowList extends St.Widget {
         this._dndWindow = null;
         this._dndTimeoutId = 0;
 
-        return false;
+        return GLib.SOURCE_REMOVE;
     }
 
     _onDestroy() {