From 35848f58690e4719e0e7a5ee95e7a83db01e1e84 Mon Sep 17 00:00:00 2001
From: Love Hornquist Astrand <lha@h5l.org>
Date: Tue, 22 Nov 2011 13:58:41 -0800
Subject: [PATCH] use low match distance for helping, or use ask user to use
 "help"

---
 lib/sl/sl.c | 31 +++++++++++++++++++++++--------
 1 file changed, 23 insertions(+), 8 deletions(-)

diff --git a/lib/sl/sl.c b/lib/sl/sl.c
index fb05cfe87..f6f4bc52e 100644
--- a/lib/sl/sl.c
+++ b/lib/sl/sl.c
@@ -443,10 +443,18 @@ osad(const char *s1, const char *s2)
     return cost;
 }
 
+/**
+ * Will propose a list of command that are almost matching the command
+ * used, if there is no matching, will ask the user to use "help".
+ *
+ * @param cmds command array to use for matching
+ * @param match the command that didn't exists
+ */
+
 void
 sl_did_you_mean(SL_cmd *cmds, const char *match)
 {
-    int *metrics, best_match = INT_MAX, print = 0;
+    int *metrics, best_match = INT_MAX;
     SL_cmd *c;
     size_t n;
 
@@ -467,14 +475,21 @@ sl_did_you_mean(SL_cmd *cmds, const char *match)
         return;
     }
 
-    fprintf(stderr, "%s is not a known command, did you mean:", match);
-    for (n = 0; cmds[n].name; n++) {
-      if (metrics[n] == best_match) {
-	  fprintf(stderr, "%s %s", print ? "," : "", cmds[n].name);
-	  print = 1;
-      }
+    /* if match distance is low, propose that for the user */
+    if (best_match < 7) {
+
+	fprintf(stderr, "error: %s is not a known command, did you mean ?\n", match);
+	for (n = 0; cmds[n].name; n++) {
+	    if (metrics[n] == best_match) {
+		fprintf(stderr, "\t%s\n", cmds[n].name);
+	    }
+	}
+	fprintf(stderr, "\n");
+
+    } else {
+
+	fprintf(stderr, "error: %s is not a command, use \"help\" for more list of commands.\n", match);
     }
-    fprintf(stderr, " ?\n");
 
     free(metrics);