Yeah agreed. I think that having a standard way of fixing the import order list just makes
for easy reading. And sadly, in my case I know exactly where I have to manually put in
imports when my IDE fails to recognise something like java.util.List.
On the point earlier about checkstyles not being able to look at the import order I seem
to remember the checkstyle tool on Tattletale being able to look out for that - because
IntelliJ got the orders wrong. I don't know if Jesper is listening/watching but
perhaps he could shed some light on this?
----- Original Message -----
From: "Brian Stansberry" <brian.stansberry(a)redhat.com>
To: jboss-as7-dev(a)lists.jboss.org
Sent: Wednesday, October 10, 2012 11:25:13 PM
Subject: Re: [jboss-as7-dev] order of import statements
+1.
And I know a year or so ago a number of IDEA users tweaked their configs
to follow the eclipse standard Jeff listed, so my vote is this becomes
the standard.
On 10/10/12 9:46 AM, Jason Greene wrote:
I do agree that we should avoid lots of reordering, but IMO one thing
that would help (to Jeff's point) is if we did have an official standard for it.
On Oct 10, 2012, at 5:06 AM, Carlo de Wolf <cdewolf(a)redhat.com> wrote:
> On 10/09/2012 09:53 PM, David M. Lloyd wrote:
>> If
>> a file is very offensively formatted, I could see considering a
>> specific, dedicated commit whose sole purpose and content is to clean up
>> the whitespace (i.e. absolutely no functional changes), but other than
>> that... no.
>
> Even then I would reconsider it, because it makes for a hell should
> cherry-picking or history analysis be needed.
>
> Carlo
> _______________________________________________
> jboss-as7-dev mailing list
> jboss-as7-dev(a)lists.jboss.org
>
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
_______________________________________________
jboss-as7-dev mailing list
jboss-as7-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
--
Brian Stansberry
Principal Software Engineer
JBoss by Red Hat
_______________________________________________
jboss-as7-dev mailing list
jboss-as7-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev