Re: [PATCH 2/2] diff: test and document -W interaction with -U<n>

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am 15.02.21 um 16:50 schrieb Ævar Arnfjörð Bjarmason:
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>
---
  Documentation/diff-options.txt | 8 ++++++++
  t/t4018-diff-funcname.sh       | 5 +++++
  2 files changed, 13 insertions(+)

diff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt
index 8ca59effa7..3c19c78616 100644
--- a/Documentation/diff-options.txt
+++ b/Documentation/diff-options.txt
@@ -88,6 +88,11 @@ endif::git-log[]
  --unified=<n>::
  	Generate diffs with <n> lines of context instead of
  	the usual three.
++
+Under `-W` generates diffs with at least <n> lines of context, if the
+number is lower than the context `-U<n>` would extend the diff to then
+`-U<n>` takes precedence.
+

How about (not as separate paragraph):

When combined with `--function-context`, this specifies the minimum of context lines.

  ifndef::git-format-patch[]
  	Implies `--patch`.
  endif::git-format-patch[]
@@ -763,6 +768,9 @@ endif::git-format-patch[]
  When showing the whole function for context the "@@" context line
  itself will always be empty, since the context that would otherwise be
  shown there will be the first line of the hunk being shown.
++
+See the documentation for `-U<n>` above for how the two options
+interact.

How about

Use `-U<n>` to specify a minimum of context (default three lines).

so that readers do not have to search.

-- Hannes



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux