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

Re: [PATCH 1/2] Fix generic typos in comments

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 21, 2013, 22:29 UTC
Message-ID
<7vy58zr1hc.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20130720190707.GA30044@domone>
Ondřej Bílka <neleai@seznam.cz> writes:
Show 11 quoted lines
> Hi, I wrote a tool that effectively fixes typos in comments and only in
> comments.
> It can be downloaded here:
>
> https://github.com/neleai/stylepp
>
> For typos you need identify them, write replacement dictionary and run
> STYLEPP/script/stylepp_skeleton stylepp_fix_comment
>
> Here generic dictionary is used that was taken from wikipedia.
> https://github.com/neleai/stylepp/blob/master/maintained/dictionary_wiki

Thanks, but sorry we cannot use this patch as-is (even if it were signed-off and came with a proper log message).

The patch touches phrases somebody may find questionable at many different levels.

 - "lowercase ok" and "fall thru" are quite acceptable colloquial
   forms; if the person who DID the real work chose to write these
   in such a way, I do not see a reason for us to update it to a
   different preference.
 - There are outright typoes that noboddy would disagree with,
   e.g. s/sinse/since/.
Previous: Junio C Hamano
Message 8 of 8 in “Fix generic typos in comments”
  1. 1/2 Fix generic typos in commentsOndřej Bílka, Jul 20, 2013
  2. Ondřej BílkaJul 20, 2013
  3. brian m. carlsonJul 20, 2013
  4. American or British English?Ondřej Bílka, Jul 20, 2013
  5. Ramkumar RamachandraJul 20, 2013
  6. Ondřej BílkaJul 20, 2013
  7. Junio C HamanoJul 21, 2013
  8. Junio C HamanoJul 21, 2013

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.