Mailing-List: contact dev-help@maven.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Maven Developers List" <dev@maven.apache.org>
Received-SPF: pass (nike.apache.org: domain of dennisl.apache@gmail.com
 designates 209.85.192.170 as permitted sender)
MIME-Version: 1.0
Sender: dennisl.apache@gmail.com
In-Reply-To: 
 <CAK2FUK0Aa_4dOTWBrV_J=sMW1oyboSRsJFMMTtdY=Q56DFQaTA@mail.gmail.com>
References: <20141010163527.6CF552388993@eris.apache.org>
	<2374233.Qz7M4zpVy7@herve-desktop>
	<CAPoyBqSG=BgvjFHP0c5Jm5YdX685ZYT6W13r6mtc+rvsoumKdw@mail.gmail.com>
	<1421860.PuR5VvIDjY@herve-desktop>
	<CAK2FUK0Aa_4dOTWBrV_J=sMW1oyboSRsJFMMTtdY=Q56DFQaTA@mail.gmail.com>
Date: Sun, 26 Oct 2014 10:17:13 +0100
Message-ID: 
 <CA+BhkyMK=WW-sZqZMD_15cGAY+Kx33PBq96xzPfvrCMAPy9k-Q@mail.gmail.com>
Subject: Re: errors reported by Checkstyle
From: Dennis Lundberg <dennisl@apache.org>
To: Maven Developers List <dev@maven.apache.org>
Content-Type: multipart/alternative; boundary=047d7b15aaaf317a3405064fe157

--047d7b15aaaf317a3405064fe157
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hi,

My personal preference is to have all annotations first, i.e. before the
visibility modifier.

I checked the JLS, but couldn't find any recommendation. The examples in
chapter 9.7 use the order that Checkstyle suggests.

--
Dennis Lundberg
Den 25 okt 2014 20:38 skrev "Andreas Gudian" <andreas.gudian@gmail.com>:

> I've started switching on maven-parent 25 in surefire now and I see
> literary hundreds of checkstyle errors (I ignore the warnings for now).
> Most of the stuff is fine with me and I'm on fixing them for a couple of
> days already.
>
> But there's one thing that struck me and that I'm a bit reluctant to chan=
ge
> - and I'd like your opinion on it.
>
> We have for example this method:
>    private @Nonnull List<String> getExcluded() ...
>
> Checkstyle now tells me that the @Nonnull should preceed the private
> modifier:
>
>    @Nonnull private List<String> getExcluded() ...
>
> Especially as Java 8 introduced the notion of annotated type declarations
> (as in @Nonnull String myNeverNullVariable; ), I think that this particul=
ar
> @Nonnull annotation should be right in front of the return type and not
> before the visibility modifier.
>
> The message is thrown by the ModifierOrder check (which does other good
> things) and that one can't be configured to behave differently.
>
> So now I'd have to specifically ignore that warning on each method for
> which we qualify the return type with @Nonnull / @Nullable.
>
> What's you pick on this? Is the form suggested by checkstyle really to be
> preferred? I'd open an issue for them if you also think that it might nee=
d
> some tweaking.
>
> Thanks,
> Andreas
>
>
>
> 2014-10-14 8:34 GMT+02:00 Herv=C3=A9 BOUTEMY <herve.boutemy@free.fr>:
>
> > Le mardi 14 octobre 2014 09:28:21 Olivier Lamy a =C3=A9crit :
> > > On 14 October 2014 05:01, Herv=C3=A9 BOUTEMY <herve.boutemy@free.fr> =
wrote:
> > > > 2. if we configured Checkstyle to report an error, this means check
> > should
> > > > fail: if you find that it should not fail, please help improve
> > Checkstyle
> > > > configuration by setting severity to warning only
> > >
> > > /me not a checkstyle expert configuration :-)
> > this makes me remember there is a feature improvement for m-checkstyle-=
p
> > 2.13
> > that I should explain: I was not a Checkstyle configuration master and
> felt
> > like you
> > Than I consistently reported Checkstyle rule name and category in every
> > message and report, and a link to Checkstyle documentation for each rul=
e
> > [1]
> >
> > this gives a good intro to Checkstyle configuration and help a lot when
> > needing
> > to @SuppressWarnings( "checkstyle:name of rule, in lowercase")
> >
> > > /me asking himself if having such hard checkstyle requirement help to
> > > improve user experience.
> > from my perspective, previous feature really improved m-checkstyle-p
> > experience
> > and I know I should submit patches to Checkstyle itself, because I thin=
k
> > this
> > could help their users too when using bare-Checkstyle
> >
> > Regards,
> >
> > Herv=C3=A9
> >
> >
> > [1]
> > http://maven.apache.org/plugins/maven-checkstyle-plugin/checkstyle.html
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> > For additional commands, e-mail: dev-help@maven.apache.org
> >
> >
>

--047d7b15aaaf317a3405064fe157--