Re: [PATCH v3 13/13] some document update

From: Jakub Narebski <jnareb@gmail.com>
Date: Jul 11, 2010, 08:27 UTC
Message-ID: <m3hbk6l901.fsf@localhost.localdomain>
In-Reply-To: <1278829141-11900-13-git-send-email-struggleyb.nku@gmail.com>

Bo Yang <struggleyb.nku@gmail.com> writes:

>  Documentation/git-log.txt |   30 ++++++++++++++++++++++++++++++
>  1 files changed, 30 insertions(+), 0 deletions(-)

[...]

Show 21 quoted lines

> +-L <start>,<end>::
> +	The line range.  <start> and <end> can take one of these forms:
> +
> +- number
> ++
> +If <start> or <end> is a number, it specifies an
> +absolute line number (lines count from 1).
> ++
> +
> +- /regex/
> ++
> +This form will use the first line matching the given
> +POSIX regex.  If <end> is a regex, it will search
> +starting at the line given by <start>.
> ++
> +
> +- +offset or -offset
> ++
> +This is only valid for <end> and will specify a number
> +of lines before or after the line given by <start>.
> ++

If the parsing code for -L <start>,<end> is the same for git-blame and for git-log, and therefore documentation is the same or nearly the same for this option, wouldn't it be better to separate this documentation into separate file, e.g. line-range-option.txt, and include it both in git-blame and git-log manpages? If there are minor differences, they can be covered by ifdefs.

-- 
Jakub Narebski
Poland
ShadeHawk on #git