[sldev] Patchy whitespace
Melinda Green
melinda at superliminal.com
Wed May 6 22:26:33 PDT 2009
I think that could help developers who don't have complete control of
how their editors handle whitespace, but it doesn't help with the
problem of rectifying inconsistent whitespace in the code base. I
suppose we could blindly run a good pretty-printer on all the code but
it almost certainly wouldn't do a perfect job (especially with comments)
and would carry other costs and risks. Weather or not it'd be worth the
costs would depend upon the state of the art which I don't currently know.
-Melinda
Ricky wrote:
> I've been seeing a bit of discussion on the handling of whitespace in
> the diffs... Would patch's -l (lowercase L) flag be helpful? Or
> would it hinder by hiding some other issue?
>
> Here's the relevant section from the man page:
> -l or --ignore-whitespace
> Match patterns loosely, in case tabs or spaces have been
> munged in your files. Any sequence of one or more blanks in the patch
> file matches any sequence in the original file, and
> sequences of blanks at the ends of lines are ignored. Normal
> characters must
> still match exactly. Each line of the context must still
> match a line in the original file.
>
> Ricky
> aka Cron Stardust
More information about the SLDev
mailing list