On Thu, Apr 21, 2011 at 02:18, Jonathan Nieder <jrnieder@xxxxxxxxx> wrote: > Michael Witten wrote: > >> You just need an editor that [...] > > which many contributors don't have (or haven't configured that way). > Really. to which I already said: "if your text editor can't do that, then any style of indentation/alignment is going to be painful." > > [...] >> However, with a tabstop of 2, it looks like this: >> >> Â if (foo && bar && baz && >> Â Â Â qux && quux) { >> Â Â ... >> Â } else if (quuux(quuuux, quuuuux, >> Â Â Â Â long_expresion_comes_here(quuuuuux))) { >> Â Â ... >> Â } >> >> Notice that your `qux && quux' line (which follows the method we both like) >> remained nicely aligned, but your `long_expression_comes_here' line has >> moved back so much that it doesn't even satisfy the `right margin' rule >> anymore. > > Oh, I still think a tabstop of 2 is insane (for various reasons, some > explained in the thread you mentioned). That's not really the point, though, is it. > No matter --- Junio will use --whitespace=fixup to convert your spaces > to tabs anway. Good! That means I can be satisfied with having written sane patches, and Junio can be satisfied with having forced the insane use of tabs. It's a win-win :-D -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html