From: Junio C Hamano Date: Mon, 22 Jan 2024 16:09:34 GMT Subject: Re: [PATCH 2/4] completion: complete --patch-with-raw Message-ID: In-Reply-To: <1debe2fe18eec7f431f355241117afb0a5e2bf5f.1705810072.git.gitgitgadget@gmail.com> "Philippe Blain via GitGitGadget" writes: > From: Philippe Blain > > Signed-off-by: Philippe Blain > --- > contrib/completion/git-completion.bash | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash > index 6108d523a11..ccb17f4ad7b 100644 > --- a/contrib/completion/git-completion.bash > +++ b/contrib/completion/git-completion.bash > @@ -1807,7 +1807,7 @@ __git_diff_common_options="--stat --numstat --shortstat --summary > --output= --output-indicator-context= > --output-indicator-new= --output-indicator-old= > --ws-error-highlight= > - --pickaxe-all --pickaxe-regex > + --pickaxe-all --pickaxe-regex --patch-with-raw > " Its ancient company, --patch-with-stat, is listed there, so it is not all that wrong to include it as a suggestion, I guess. But in the longer term, I think we would want to slim "git diff -h" output by hiding them (no reason to touch code to remove the support) from the "common diff options" part. They were added as a kludge before I realized we need more than these two combinations and made the options --patch, --stat, and --raw cumulative. Thanks.