ARM interrupts fixes

Sam Leffler sam at freebsd.org
Thu Sep 4 16:43:04 UTC 2008


Rafal Jaworowski wrote:
> Stanislav Sedov wrote:
>> The patch looks goot to me. But what about splitting the whitespace
>> fixes and the meaningful ones when committing to the tree to not
>> pollute the history?
> 
> I can do it, but was under the impression that white space fix-only commits
> were rather frowned upon, isn't it the case any longer?

Senseless churn is frowned upon.  Ya know, the gratuitous knob-polishing
some folks like to do that unintentionally introduce subtle bugs because
the result isn't tested (and often not found until long after).

If you're working in an area and it warrants cleanup then one hopes
you're also testing.  That's fine but as stas requested please commit
the changes separately.

	Sam


More information about the freebsd-arm mailing list