You talk about trailing whitespace as if getting rid of it were a worthy goal in itself. It is not. Humans don't care about trailing whitespace, we don't see it unless you misconfigure your editor. Its purely for the benefit of automated merge tools. If we disallow patch hunks that change trailing whitespace only then the needs of merge tools are satisfied and there is no further need for any policy beyond that.
On Saturday, November 24, 2012 11:55:24 AM UTC, P Purkayastha wrote: > C) We care about trailing whitespace. No new patches should introduce > trailing whitespace in modified lines. But patch hunks that ONLY change > trailing whitespace are disallowed. It is of course encouraged to get > rid of trailing whitespace in the code surrounding the lines your patch > touches. > > > -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To post to this group, send email to sage-devel@googlegroups.com. To unsubscribe from this group, send email to sage-devel+unsubscr...@googlegroups.com. Visit this group at http://groups.google.com/group/sage-devel?hl=en.
