Refactored 'ui_tidy_wins()' to return bool and not output text, Corrected underline, removed call to function 'wins_resize_all()'.

2 files changed, 7 insertions, 4 deletions

diff --git a/src/command/command.c b/src/command/command.c
index 25571e81..ea6df2ba 100644
--- a/src/command/command.c
+++ b/src/command/command.c
@@ -688,7 +688,7 @@ static struct cmd_t command_defs[] =
         cmd_winstidy, parse_args, 1, 1, &cons_winstidy_setting,
         { "/winstidy on|off", "Auto tidy windows.",
         { "/winstidy on|off",
-          "------------",
+          "---------------",
           "Enable or disable auto window tidy.",
           NULL } } },
 
diff --git a/src/command/commands.c b/src/command/commands.c
index 89ded432..86781330 100644
--- a/src/command/commands.c
+++ b/src/command/commands.c
@@ -689,7 +689,11 @@ cmd_wins(gchar **args, struct cmd_help_t help)
     if (args[0] == NULL) {
         cons_show_wins();
     } else if (strcmp(args[0], "tidy") == 0) {
-        ui_tidy_wins();
+        if ( ui_tidy_wins() ) {
+            cons_show("Windows tidied.");
+        } else {
+            cons_show("No tidy needed.");
+        }
     } else if (strcmp(args[0], "prune") == 0) {
         ui_prune_wins();
     } else if (strcmp(args[0], "swap") == 0) {
@@ -723,8 +727,6 @@ cmd_winstidy(gchar **args, struct cmd_help_t help)
 {
     gboolean result = _cmd_set_boolean_preference(args[0], help, "Wins Auto Tidy", PREF_WINS_AUTO_TIDY);
 
-    wins_resize_all();
-
     return result;
 }
 
@@ -3254,6 +3256,7 @@ cmd_close(gchar **args, struct cmd_help_t help)
 
     // Tidy up the window list.
     if (prefs_get_boolean(PREF_WINS_AUTO_TIDY)) {
+        // TODO: Any benefit of checking the return value?
         ui_tidy_wins();
     }