pmd: verify #1047 False Positive in 'for' loops for LocalVariableCouldBeFinal
pmd: fixed #1068 CPD fails on broken symbolic links
Add GuardLogStatement
Based on Heiko Rupp implemetation: http://pilhuhn.blogspot.com/2008/09/pmd-rule-to-check-for-unguarded.html
Enhance GuardLogStatement:
- add property support to specify which statement needs to be checked
- move out of the java logging to be placed in logging-jakarta
Enhance GuardDebugLogging:
- replace xpath implementation by newly created GuardLogStatementRule
- increase performance by 5 (from 0.5s to 0.1s)
Merge changelog
GuardLogStatement: small docs fixes
Redesign GuardLogStatement:
Turns out that the implementation based on the one published by Heiko Rupp
could simply not catch all the case in our test suite. I therefore opted to
backport the previous implementation - a simple Xpath rule, from GuardDebugLogStmt
into a java class.
Performance are not as good as the one based on Heiko's proposal, but it still
twice faster as the simple GuardDebogLogStmt (instead of five times faster). However,
correctness is in this case more important.