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

Re: [PATCH] Fixed pluralization in diff reports

From
Jon Forrest <nobozo@gmail.com>
Date
Aug 1, 2011, 04:11 UTC
Message-ID
<4E36277B.9020208@gmail.com>
In-Reply-To
<7v8vrdkc19.fsf@alter.siamese.dyndns.org>
On 7/31/2011 9:09 PM, Junio C Hamano wrote:
Show 7 quoted lines
> Jon Forrest<nobozo@gmail.com>  writes:
>
>> I got irritated by the
>>
>> 	 1 files changed, 0 insertions(+), 1 deletions(-)
>
> This is how the other diff implementation has always showed this line, no?
Maybe, but it's not grammatical English no matter who does it.

But, I'm in the process of asking you to disregard this change because I noticed that the diff of my change to diff.c was

	1 file changed, 6 insertions(+), 4 deletion(-)

The last part is clearly wrong, but I don't see why my change didn't work. It's such a simple change.

Back to the drawing board.
Sorry for the bother.
Jon
Previous: Junio C HamanoNext: Junio C Hamano
Message 3 of 5 in “Fixed pluralization in diff reports”
  1. Fixed pluralization in diff reportsJon Forrest, Aug 1, 2011
  2. Junio C HamanoAug 1, 2011
  3. Jon ForrestAug 1, 2011
  4. Junio C HamanoAug 1, 2011
  5. Jon ForrestAug 1, 2011

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.