On Mon, Oct 26, 2009 at 3:44 PM, Mark Hindess <mark.hind...@googlemail.com>wrote:
> Modern programmers do refactoring and testing. I'm more than happy with > this. I want our code to be readable and using idiomatic style goes a > long way to achieving that. Working is important but being maintainable > is important too. > > +1 for making such changes but I'd prefer to see them as distinct > patches not part of broader changes so it is easier to verify that they > really are syntactic refactorings. > Sounds good. If I find myself cleaning up code en route to fixing it, I'll try to make a habit of submitting the non-functional change as an intermediate step.