On 12/12/06, Aaron Bentley <<a href="mailto:aaron.bentley@utoronto.ca">aaron.bentley@utoronto.ca</a>> wrote:<div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> Aaron Bentley has voted +1. Status is now: Approved Comment: This has quite a lot of spurious whitespace changes.  Please try to avoid that in the future.  I'll fix them as part of the merge. For details, see: <a href="http://bundlebuggy.aaronbentley.com/request/%3C418c22640611292054t7aed8ca7p4fac2db426677ea2%40mail.gmail.com%3E">http://bundlebuggy.aaronbentley.com/request/%3C418c22640611292054t7aed8ca7p4fac2db426677ea2%40mail.gmail.com%3E </a> </blockquote></div>thanks, for the vote and the merge. sorry about the whitespace (my editor removes it automatically), I would have reverted that, but I remembered something on the mailinglist about removing all the trailing whitespace, so I removed them. I tried to see if you fixed this, but it looks like you forgot. ----some previous post---- <blockquote style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;" class="gmail_quote">I think it is fine. And I think we are happy to get the code cleaned up. If you want some more things to look for, we also need to remove trailing whitespace from lines. Though that may be a more controversial change, since it messes up annotations. John =:-> </blockquote> Some notes: 1.) Can we add something to the branch to specify that for certain or all files we should   a.) automatially remove trailing whitespace or   b.) refuse to commit it.   So that we don't get into this mess. This might have been descussed before. It could be implemented as a plugin. 2.) Annotate should be able to cope with this. mabye add an option to ignore whitespace changes.      I often get into a situation where I try to find the true author of a line (to blame/praise them:),      but sometimes the line might have been tuched by several people in the past.      I thought the bisect plugin could help with this, but I was never able to obtain it,      I must probably try again. -- I code therefore I am.