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

Re: none

From
Junio C Hamano <gitster@pobox.com>
Date
Oct 10, 2018, 22:19 UTC
Message-ID
<xmqq36tdla1a.fsf@gitster-ct.c.googlers.com>
In-Reply-To
<20181010152007.14441-1-mihir@cs.utexas.edu>
Mihir Mehta <mihir@cs.utexas.edu> writes:
> Thanks, Junio. Instead of removing that part of the patch, I opted to
> expand it to make it a little clearer (in my opinion) than it was
> before. Let me know if this works.

I am mildly negative on that change. "Omitting both would give an empty diff" would be understandable to anybody who understands that an omitted end of dot-dot is substituted with HEAD *and* thinks what range HEAD..HEAD means, so it is just an additional noise to them, and to those who do not want to waste time on thinking, it is a statement that reads as if "it will be an error" without saying why it is an error. So overall, it seems, at least to me, that the additional text adds negative value.

So, I dunno.
Previous: Mihir MehtaNext: Mihir Mehta
Message 5 of 7 in “doc: fix a typo and clarify a sentence”
  1. doc: fix a typo and clarify a sentenceMihir Mehta, Oct 3, 2018
  2. Junio C HamanoOct 5, 2018
  3. Mihir Mehta, Oct 10, 2018
  4. doc: fix a typo and clarify a sentenceMihir Mehta, Oct 10, 2018
  5. Junio C HamanoOct 10, 2018
  6. doc: fix a typo and clarify a sentenceMihir Mehta, Oct 10, 2018
  7. Junio C HamanoOct 10, 2018

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.