Sevntu Checkstyle release 1.13.0

Braking backward compatibility:

VariableDeclarationUsageDistanceCheck was moved to Checkstyle library . Author: Roman Ivanov

AbbreviationAsWordInNameCheck is transfered to Checkstyle library . Author: Roman Ivanov

 

 

New:

New Check RequiredParameterForAnnotationCheck. Fixes #321 . Author: Andrew Uljanenko

New Check UselessSuperCtorCallCheck implemeted. Fixes #186 . Author: Alexey

New Check EmptyPublicCtorInClassCheck implemented. fixes #291 . Author: Alexey

New Check NameConventionForJunit4TestClassesCheck implemented. Fixes #80 . Author: Alexey

New Check UselessSingleCatchCheck implemented. issue #267 fixed . Author: Alexey

New Check: AvoidConditionInversion . #112 . Author: alexkravin

New Check: NestedSwitchCheck - Checking if there is no switch block inside other switch block.  Issue #249. Author: Damian Szczepanik

 

Bugs:

fix for NPE in ConfusingConditionCheck . Author: melodicwing

Fixed false positive in check ForbidReturnInFinallyBlockChech. Fixes #269 . Author: Andrew Uljanenko

Diamond operator - fixed violations on anonymous classes . Issue #270 . Author: alexkravin

 

 

Miscellaneous:

Refactored prefixes, changed indentation of switch-case, issue #298 . Author: alexkravin

Moved test-input folder from src/test/java to src/test/resources . Author: alexkravin

no java6 support since 6.2 release . Author: Roman Ivanov

version 1.13.0 bump, java6, fixes for errors of m2e . Author: Roman Ivanov

Updated maximum line length to 100 symbols . Author: Daniil Yaroslavtsev

Small code changes and extended UT's. Fixes #317 . Author: Andrew Uljanenko

New test case added. Fixes #275 . Author: Andrew Uljanenko

New test case added. Fixes #274 . Author: Andrew Uljanenko

Extended javadoc, UTs. Fixed #264 . Author: Andrew Uljanenko

 

Code coverage improvement . Author: Damian Szczepanik

Short-circuit logic is used in boolean expressions, issue #169 . Author: Michal Kordas

Code coverage improvements for ReturnCountExtendedCheck . Author: Damian Szczepanik

Improved AvoidHidingCauseExceptionCheck code coverage . Author: Damian Szczepanik

Improved UselessSingleCatchCheck code coverage . Author: Damian Szczepanik

Improved AvoidDefaultSerializableInInnerClasses code coverage to 100% . Author: Damian Szczepanik

Fixes several Sonar violations . Author: Damian Szczepanik

Remove unused input for AvoidHidingCauseExceptionCheck . Author: Damian Szczepanik

Increase AvoidNotShortCircuitOperatorsForBooleanCheck code coverage Pull #324 . Author: Damian Szczepanik

Moved static duplicated methods into Utils class . Author: Damian Szczepanik

Sonar violation: Tabulation characters should not be used . Author: Damian Szczepanik

improved InterfaceTypeParameterNameCheck code coverage . Author: Damian Szczepanik

Improved UnnecessaryParenthesesExtendedCheck test coverage . Author: Damian Szczepanik

Fixed Sonar critical and major violations: - "static final" arrays should be "private" - "for" loop incrementers should modify the variable being tested in the loop's stop condition - Declarations should use Java collection interfaces such as "List" rather than specific implementation classes such as "LinkedList" . Author: Damian Szczepanik

Run cobentura with mvn check or mavn install and fail if code coverage criteria are not met. For new files we expect to have 100% code coverage, for older one that do not pass this rule we added some rules . Author: Damian Szczepanik

Added Utils class that has one method for reporting unsupported AST token . Author: Damian Szczepanik

Removed duplicate check for the empty list: first condition isEmpty() but then for-each which also skips iteration if list is empty . Author: Damian Szczepanik

Increase OverridableMethodInConstructorCheck code coverage: - Added more test cases (examples) - Replaced to Serializable/Clonable stirngs to Cloneable.class.getSimpleName() - Increase coverage by using true and false values of checkCloneMethod and checkReadObjectMethod . Author: Damian Szczepanik

MapIterationInForEachLoopCheck - Converted strings to .class.getName() - moved initialization to beginTree method . Author: Damian Szczepanik

Improved test coverity of EitherLogOrThrowCheck by: - adding new test with inner class - invoking setters method by setting attributes in test scenario . Author: Damian Szczepanik

Improved test coverity in ConfusingConditionCheck by setting default parameters in test case . Author: Damian Szczepanik

Improved coverage of AvoidDefaultSerializableInInnerClasses by importing full qualified class name Serializable (use java.io.Serializable) . Author: Damian Szczepanik

Improved test coverity of class ChildBlockLengthCheckCheck by adding test for tested class . Author: Damian Szczepanik

Improved CustomDeclarationOrderCheck - Added tests to increase coverage - Fixed rule in test method - Fixed formatting (indents) . Author: Damian Szczepanik

SevNTU Checkstyle version 1.6

We finished new version of our extension for Checkstyle - release 1.6.
I updated wiki pages to allow other developers easily join the project.
Read our publications, in Russian language, latest is for 1.5.3 release, for 1.6 will be soon.

Release 1.6.0 (21/October/2012):

LogicConditionNeedOptimizationCheck -  This check prevents the placement of local variables and fields after calling methods in '&&' and '||' conditions. Done by Ilja Dubinin.
Example:
"if (isVisible() && active)" == should be refactored to ==> "if (active && isVisible())"
Reason: swap of conditions expressions will be beneficial, but some time you can find logic that in isVisible() change 'active' variable value and this optimization will not be correct - but this is sign of bad  design!

ForbidCCommentsInMetods -  Check prevents usage of C-style (/* ... */) comments inside method body. If you have class declaration inside method body with JavaDoc you will get error too, that is sad side effect for now, but I did not met such cases for now, so I thunk it will be incentive to move class out. Done by Ilja Dubinin.
Reason: C-style of comments are problem for future investigation, it should be treated as DEBUG/temporal comments, but should be cleared finally.

InterfaceTypeParameterNameCheck -  Checks that interface type parameter(for template) names conform to a format specified by the format property. Default format is ^[A-Z]$. Done by Dmitry Gridyushko.
Reason: to avoid template types to be like <INPUT>, <CUSTOM>, <LISTENER> and force developer to name them by 1 letter <I>, <C>, <L>. to clearly indicate nature of variable in code down the file.

All details are here: http://sevntu-checkstyle.github.com/sevntu.checkstyle/
We will do bug fixing in 1.6.1 and 1.6.2 as always, please test and report bug ...... .