[PATCH v2] help: add prompt-yes setting for autocorrect

From: Chris Howlett <chowlett09@gmail.com>

The help.autocorrect functionality is really useful, saving frustration when a dev fat-fingers a command, and git has a pretty good idea what was originally intended. The config settings are a nice selection, with "prompt" asking the user to confirm that they want to run the assumed command.

However, with "prompt", the choice defaults to "No" - that is, hitting return will _not_ run the command. For me at least, if git is confident it knows which command I wanted, it's usually right, and the golden path would be to run the command.

Therefore this patch adds "prompt-yes" as a counterpart config setting for help.autocorrect, which does the same as "prompt", but defaults to "Yes" - hitting return will run the assumed command.

I have not added any tests because the test suite doesn't have any tests (that I could find) for the "prompt" behaviour - I'm assuming this is because it's hard/impossible to simulate the interactive terminal prompt

Signed-off-by: Chris Howlett <chowlett09@gmail.com>
---
    Add prompt-yes config setting for help.autocorrect
    
    Changes since v1:
    
     * PR rebased against master and fixed up, to account for e21bf2c and
       e4542d8
    
    ------------------------------------------------------------------------
    
    This is my first patch request to git - please do let me know if I
    should be doing something differently!

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1852%2Fasilano%2Fautocorrect-allow-prompt-default-yes-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1852/asilano/autocorrect-allow-prompt-default-yes-v2
Pull-Request: https://github.com/gitgitgadget/git/pull/1852

Range-diff vs v1:

 1:  b87c772089e ! 1:  cade4f1719e help: add prompt-yes setting for autocorrect
     @@ Commit message
      
          Signed-off-by: Chris Howlett <chowlett09@gmail.com>
      
     - ## Documentation/config/help.txt ##
     -@@ Documentation/config/help.txt: help.autoCorrect::
     + ## Documentation/config/help.adoc ##
     +@@ Documentation/config/help.adoc: immediately.
       deciseconds (0.1 sec).
     - 	 - "immediate": run the suggested command immediately.
     + 	 - "never": don't run or show any suggested command.
       	 - "prompt": show the suggestion and prompt for confirmation to run
      -the command.
      +the command. The default choice at the prompt is "No"
      +	 - "prompt-yes": show the suggestion and prompt for confirmation to run
      +the command. The default choice at the prompt is "Yes"
     - 	 - "never": don't run or show any suggested command.
       
       help.htmlPath::
     + 	Specify the path where the HTML documentation resides. File system paths
      
       ## help.c ##
      @@ help.c: struct help_unknown_cmd_config {
       	struct cmdnames aliases;
       };
       
     -+#define AUTOCORRECT_PROMPT_YES (-4)
     ++#define AUTOCORRECT_PROMPT_YES (-5)
     + #define AUTOCORRECT_SHOW (-4)
       #define AUTOCORRECT_PROMPT (-3)
       #define AUTOCORRECT_NEVER (-2)
     - #define AUTOCORRECT_IMMEDIATELY (-1)
     -@@ help.c: static int git_unknown_cmd_config(const char *var, const char *value,
     - 			cfg->autocorrect = AUTOCORRECT_IMMEDIATELY;
     - 		} else if (!strcmp(value, "prompt")) {
     - 			cfg->autocorrect = AUTOCORRECT_PROMPT;
     -+		} else if (!strcmp(value, "prompt-yes")) {
     -+			cfg->autocorrect = AUTOCORRECT_PROMPT_YES;
     - 		} else {
     - 			int v = git_config_int(var, value, ctx->kvi);
     - 			cfg->autocorrect = (v < 0)
     +@@ help.c: static int parse_autocorrect(const char *value)
     + 
     + 	if (!strcmp(value, "prompt"))
     + 		return AUTOCORRECT_PROMPT;
     ++	if (!strcmp(value, "prompt-yes"))
     ++		return AUTOCORRECT_PROMPT_YES;
     + 	if (!strcmp(value, "never"))
     + 		return AUTOCORRECT_NEVER;
     + 	if (!strcmp(value, "immediate"))
      @@ help.c: char *help_unknown_cmd(const char *cmd)
       	if ((cfg.autocorrect == AUTOCORRECT_PROMPT) && (!isatty(0) || !isatty(2)))
       		cfg.autocorrect = AUTOCORRECT_NEVER;

 Documentation/config/help.adoc |  4 +++-
 help.c                         | 15 +++++++++++++++
 2 files changed, 18 insertions(+), 1 deletion(-)