git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2] blame: add a range option to -L

From
Jakub Narebski <jnareb@gmail.com>
Date
May 4, 2010, 17:48 UTC
Message-ID
<m3bpcvftjk.fsf@localhost.localdomain>
In-Reply-To
<7vd3xbmv4w.fsf@alter.siamese.dyndns.org>
Junio C Hamano <gitster@pobox.com> writes:
Show 11 quoted lines
> I wonder if it makes the interface more consistent to rewrite the
> above comment like this:
> 
> 	/*
> 	 * Allow "-L <something>,+20" to mean starting at <something>
> 	 * for 20 lines; "-L <something>,-5" for 5 lines ending at
> 	 * <something>; and "-L <something>,+-5" for 5 lines around
>          * <something>.
> 	 */
> 
> and the match the code.
I like this.

The other approach would be to use -B <num> / -A <num> / -C [num], -<num> convention from 'grep'... but git-blame uses -C and -<num> for other things.

-- 
Jakub Narebski
Poland
ShadeHawk on #git
Previous: Junio C HamanoNext: Matthieu Moy
Message 4 of 5 in “blame: add a range option to -L”
  1. blame: add a range option to -LBill Pemberton, May 3, 2010
  2. Michael WittenMay 3, 2010
  3. Junio C HamanoMay 4, 2010
  4. Jakub NarebskiMay 4, 2010
  5. Matthieu MoyMay 4, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.