On 10/09/2012 09:53 PM, David M. Lloyd wrote:
On 10/09/2012 11:55 AM, Jeff Mesnil wrote:
> If we agree with this default order, I can add a checkstyle rule[1] for this.
We tried this before but it didn't work out because checkstyle gets
confused about imports which are only used in Javadoc, and because
regardless of checkstyle rules everyone cannot agree on an order, and
anyway changes to existing code should NEVER EVER EVER EVER include
import "cleanups", "reorgs", or anything of the sort (no matter how
bad
you think it is) along with "real" changes because it just obscures what
has changed and causes trouble.
I agree with you and my point was to avoid these cosmetic changes as much as possible.
Then the onus is on me to not rely on the IDE to build the package imports and check
manually in case I added by error unwanted
packages.
--
Jeff Mesnil
JBoss, a division of Red Hat
http://jmesnil.net/