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

Re: [PATCH] Add line-wrapping guidelines to the coding style documentation

From
Junio C Hamano <gitster@pobox.com>
Date
Nov 14, 2007, 10:37 UTC
Message-ID
<7vsl39i041.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<56A87A65-3C2E-4E10-84D4-4470879EE466@wincent.com>
Wincent Colaiuta <win@wincent.com> writes:
Show 6 quoted lines
> A statistician could probably make some interesting comments about the  
> results, but the basic trend is that, while there are plenty of  
> examples of isolated long lines in the source tree (the longest is a  
> 287-character line in one of the perl scripts), the frequency starts  
> to drop off pretty rapidly once you pass 70 columns and start climbing  
> towards 80.
Gaah.  287???
Show 6 quoted lines
> + - In the case of documentation, mixing excessively long and short
> +   lines may make the AsciiDoc source harder to read, so try to
> +   keep line lengths consistent.
> +
> + - When submitting patches use common sense to decide whether to
> +   rewrap, avoiding gratuitous changes.

Hmph. The last item applies only to the documentation because it uses the word "rewrap", but otherwise applies equally well to the sources (re-indenting). So probably "whether to rewrap or reindent" would be a good change there.

Previous: Wincent ColaiutaNext: Johannes Schindelin
Message 2 of 7 in “Add line-wrapping guidelines to the coding style documentation”
  1. Add line-wrapping guidelines to the coding style documentationWincent Colaiuta, Nov 14, 2007
  2. Junio C HamanoNov 14, 2007
  3. Johannes SchindelinNov 14, 2007
  4. Add line-wrapping guidelines to the coding style documentationWincent Colaiuta, Nov 14, 2007
  5. Andreas EricssonNov 14, 2007
  6. Junio C HamanoNov 14, 2007
  7. Wincent ColaiutaNov 14, 2007

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.