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

Re: Rebase & Trailing Whitespace

From
Jeff King <peff@peff.net>
Date
Sep 9, 2011, 19:38 UTC
Message-ID
<20110909193842.GA31109@sigill.intra.peff.net>
In-Reply-To
<20110903201832.GB8362@ecki>
On Sat, Sep 03, 2011 at 10:18:32PM +0200, Clemens Buchacher wrote:
Show 10 quoted lines
> I am looking at the output of:
> 
>  git diff --check 4b825 -- $(git ls-files '*.[ch]'|grep -v ^compat)
> 
> Most of those are false positives from multiline function argument
> lists, which are aligned with the first line. So the above command
> would break more than it would fix.
> 
> I don't care either way, but for future reference: Do we or do we
> not want indentation with spaces in this case?

I thought our policy was that a tab is 8 spaces, and that indentations of size N should be floor(N/8) tabs and (N%8) spaces. I know other projects disagree (e.g., they want tabs for structural indentation and spaces for all other alignment), but that seems to be the general use in git.

-Peff
Previous: Clemens Buchacher
Message 8 of 8 in “Rebase & Trailing Whitespace”
  1. Hilco WijbengaAug 31, 2011
  2. Jeff KingSep 1, 2011
  3. Philip OakleySep 1, 2011
  4. Jeff KingSep 1, 2011
  5. Michael J GruberSep 2, 2011
  6. Jeff KingSep 2, 2011
  7. Clemens BuchacherSep 3, 2011
  8. Jeff KingSep 9, 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.