Mailing-List: contact general-help@xmlgraphics.apache.org; run by ezmlm
Precedence: bulk
Reply-To: general@xmlgraphics.apache.org
Received-SPF: pass (athena.apache.org: domain of vhennebert@gmail.com
 designates 74.125.82.49 as permitted sender)
Message-ID: <51372370.1010900@gmail.com>
Date: Wed, 06 Mar 2013 12:07:28 +0100
From: Vincent Hennebert <vhennebert@gmail.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64;
 rv:17.0) Gecko/20130221 Thunderbird/17.0.3
MIME-Version: 1.0
To: general@xmlgraphics.apache.org
Subject: Re: Checkstyle, Epilogue
References: <512E7CE9.7070707@gmail.com>
In-Reply-To: <512E7CE9.7070707@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

Ok, so it seems that consensus has been reached. I’ll update the
Checkstyle config file progressively as I implement the new rules. I’ll
also update the documentation on the website as suggested by Glenn.

Thanks,
Vincent

On 02/27/2013 10:38 PM, Vincent Hennebert wrote:
> Hi All,
>
> An extremely long time ago I proposed on this list [1] a new set of
> Checkstyle rules that we would all agree to follow, so that we can
> enforce a no warning policy. Glenn expressed disagreements with some of
> the rules.
>
> [1] http://markmail.org/thread/wnevlowocmlcm6xa
>
> A very long time ago I had a Skype discussion with Glenn in order to try
> and find a common ground; We agreed that, pending some action from my
> part, we would settle on a compromise.
>
> A long time ago I undertook that action and submitted my results to
> Glenn, who was ok to proceed with the compromise we had found.
>
> So, today, I’d like to propose this compromise to the rest of the
> community and ask if everybody is ok with it. Mainly, we keep the rules
> about whitespace that I suggested last year, but we introduce an
> exception regarding line length: files originally authored by Glenn are
> allowed to disable that rule.
>
> The differences with the Checkstyle file currently in FOP are the
> following (see [2] for rule descriptions):
> • remove SuppressWithNearbyCommentFilter
> • remove EqualsHashCode
> • remove NoWhiteSpaceAfter for ARRAY_INIT
> • add LineLength
> • add MethodParamPad
> • add ParenPad
> • WhiteSpaceAfter: add TYPECAST
> • WhiteSpaceAround: remove DIV and STAR
>
> [2] http://checkstyle.sourceforge.net/availablechecks.html
>
> I would also like to add ExplicitInitialization eventually, once I’ve
> figured out an automatic way of removing the existing ones.
>
> Some source files (e.g. o.a.f.complexscripts.bidi.BidiClass) were
> automatically generated and are not meant to be human-readable,
> therefore I will exclude them from the checks.
>
> I’ll apply lazy consensus and, if nobody objects within 3 working days,
> I will start to progressively apply the new rules and enable them in
> Checkstyle. Then we can modify Gump to automatically run Checkstyle for
> us after every commit.
>
>
> Thanks,
> Vincent
>

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@xmlgraphics.apache.org
For additional commands, e-mail: general-help@xmlgraphics.apache.org