On 12/17/2019 12:07 AM, Philippe Blain via GitGitGadget wrote: > From: Philippe Blain <levraiphilippeblain@xxxxxxxxx> > > Currently the line-log functionality (git log -L) only supports > displaying patch output (`-p`, its default behavior) and suppressing it > (`-s`). A check was added in the code to that effect in 5314efaea (line-log: > detect unsupported formats, 2019-03-10) but the documentation was not > updated. > > Explicitly mention that `-L` implies `-p`, that patch output can be > suppressed using `-s`, and that all other diff formats are not allowed. > > Additionnally, mention that the ':<funcname>' form implies `--function-context`. s/Additionnally/Additionally/ > Signed-off-by: Philippe Blain <levraiphilippeblain@xxxxxxxxx> > --- > Documentation/git-log.txt | 6 +++++- > Documentation/gitk.txt | 6 +++++- > 2 files changed, 10 insertions(+), 2 deletions(-) > > diff --git a/Documentation/git-log.txt b/Documentation/git-log.txt > index b406bc4c48..1c52bf184d 100644 > --- a/Documentation/git-log.txt > +++ b/Documentation/git-log.txt > @@ -77,7 +77,11 @@ produced by `--stat`, etc. > not give any pathspec limiters. This is currently limited to > a walk starting from a single revision, i.e., you may only > give zero or one positive revision arguments. > - You can specify this option more than once. > + You can specify this option more than once. Implies `--patch`. > + If ``:<funcname>'' is given, implies `--function-context`. > + Patch output can be suppressed using `-s`, but other diff formats I would prefer the longer "--no-patch" over "-s", especially because you follow with several options in this style. > + (namely `--raw`, `--numstat`, `--shortstat`, `--dirstat`, `--summary`, > + `--name-only`, `--name-status`, `--check`) are not currently implemented. > + > include::line-range-format.txt[] > > diff --git a/Documentation/gitk.txt b/Documentation/gitk.txt > index 1eabb0aaf3..91993b9d4c 100644 > --- a/Documentation/gitk.txt > +++ b/Documentation/gitk.txt > @@ -106,7 +106,11 @@ linkgit:git-rev-list[1] for a complete list. > not give any pathspec limiters. This is currently limited to > a walk starting from a single revision, i.e., you may only > give zero or one positive revision arguments. > - You can specify this option more than once. > + You can specify this option more than once. Implies `--patch`. > + If ``:<funcname>'' is given, implies `--function-context`. > + Patch output can be suppressed using `-s`, but other diff formats (same) > + (namely `--raw`, `--numstat`, `--shortstat`, `--dirstat`, `--summary`, > + `--name-only`, `--name-status`, `--check`) are not currently implemented. Thanks, -Stolee
