DRY

jeremypw · jeremypw · commit 3d806bb1dcfc · 2026-01-03T11:47:55.000Z
diff --git a/src/Application.vala b/src/Application.vala
@@ -216,12 +216,12 @@ namespace Scratch {
 
         public async bool handle_quit_window (MainWindow window_to_close) {
             unowned List<Gtk.Window> windows = get_windows ();
-            var n_windows = windows.length ();
-            if (!yield window_to_close.check_unsaved_changes (n_windows == 1)) {
+            var is_last_window = windows.length () == 1;
+            if (!yield window_to_close.check_unsaved_changes (is_last_window)) {
                 return false;
             }
 
-            if (n_windows == 1) {
+            if (is_last_window) {
                 window_to_close.before_quit (); // Update settings
             }
             // Just destroy window - we have already checked whether any docs need saving

Original file line number	Diff line number	Diff line change
`@@ -216,12 +216,12 @@ namespace Scratch {`
`216`	`216`
`217`	`217`	`public async bool handle_quit_window (MainWindow window_to_close) {`
`218`	`218`	`unowned List<Gtk.Window> windows = get_windows ();`
`219`		`- var n_windows = windows.length ();`
`220`		`- if (!yield window_to_close.check_unsaved_changes (n_windows == 1)) {`
	`219`	`+ var is_last_window = windows.length () == 1;`
	`220`	`+ if (!yield window_to_close.check_unsaved_changes (is_last_window)) {`
`221`	`221`	`return false;`
`222`	`222`	`}`
`223`	`223`
`224`		`- if (n_windows == 1) {`
	`224`	`+ if (is_last_window) {`
`225`	`225`	`window_to_close.before_quit (); // Update settings`
`226`	`226`	`}`
`227`	`227`	`// Just destroy window - we have already checked whether any docs need saving`