click-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adrian A. (JIRA)" <>
Subject [jira] Commented: (CLK-671) Upgrade to Checkstyle 5.1
Date Tue, 25 May 2010 17:12:25 GMT


Adrian A. commented on CLK-671:

>> what exact checks do you have in mind to make use of, besides the actual used ones?
>1. Annotations:

>2. Empty Block:
It's there - line 156, but not active: Click is using many empty blocks right now.

>3. Headers:

>4. Coding:
>       - (too many to enumerate them here. 
IMHO also too many to add and try them now :). In fact they look so many that it would be
an issue on itself to use, configure and tune them.

> 5. Duplicate Code:
I would have liked this greatly but it's buggy: it shows the Apache header as duplicate code
- so it's not smart at all. If I see right it's simply using a regular expression for this
- not something like the IntelliJ inspections.

>6. More from Javadoc:

7. I added a few more that made sense for me. See a diff of the checkstyle file for this.
8. There are a few others commented, because they need more test/tune before enforcing them
upon the source code.

> This might be a separate issue ....

Interesting concept (thought myself about it while using checkstyle), but please move it to
another issue/wiki/discussion because I consider this issue: "upgrade to checkstyle 5.1" as
closed :).

> Upgrade to Checkstyle 5.1
> -------------------------
>                 Key: CLK-671
>                 URL:
>             Project: Click
>          Issue Type: Improvement
>            Reporter: George Stan
>            Assignee: Adrian A.
> Upgrade to Checkstyle 5.1. 
> It has quite a few fixes and also support for Java 5. Since Click is using Java 5 now
this version would be more useful.
> Checkstyle 5.1 (but 5.0 too) also contains many checks not used by Click - /build/checkstyle-checks.xml
 is using just a few of them.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message