hiifong/pmd
1
0
Fork 0
forked from phoedos/pmd
Code Pull Requests Activity

Correct -benchmark reporting of Rule visits via the RuleChain

Browse Source 
git-svn-id: https://pmd.svn.sourceforge.net/svnroot/pmd/branches/pmd/4.2.x@6912 51baf565-9d33-0410-a72c-fc3788e3496d
This commit is contained in:
Ryan Gustafson
2009-04-18 08:21:27 +00:00
parent ac439685fc
commit c168ecc6e5
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
pmd/etc/changelog.txt
View File

@ -4,6 +4,7 @@ Fixed bug 2590258 - NPE with nicerhtml output
Fixed bug 2317099 - False + in SimplifyCondition Fixed bug 2317099 - False + in SimplifyCondition
Fixed bug 2606609 - False "UnusedImports" positive in package-info.java Fixed bug 2606609 - False "UnusedImports" positive in package-info.java
Fixed bug 2645268 - ClassCastException in UselessOperationOnImmutable.getDeclaration Fixed bug 2645268 - ClassCastException in UselessOperationOnImmutable.getDeclaration
Correct -benchmark reporting of Rule visits via the RuleChain
Android ruleset: CallSuperLast rule now also checks for finish() redefinitions Android ruleset: CallSuperLast rule now also checks for finish() redefinitions

2
pmd/src/net/sourceforge/pmd/AbstractRuleChainVisitor.java
View File

@ -59,9 +59,9 @@ public abstract class AbstractRuleChainVisitor implements RuleChainVisitor {
continue; continue;
} }
// For each rule, allow it to visit the nodes it desires // For each rule, allow it to visit the nodes it desires
int visits = 0;
start = System.nanoTime(); start = System.nanoTime();
for (Rule rule: ruleSetRules.get(ruleSet)) { for (Rule rule: ruleSetRules.get(ruleSet)) {
int visits = 0;
final List<String> nodeNames = rule.getRuleChainVisits(); final List<String> nodeNames = rule.getRuleChainVisits();
for (int j = 0; j < nodeNames.size(); j++) { for (int j = 0; j < nodeNames.size(); j++) {
List<SimpleNode> nodes = nodeNameToNodes.get(nodeNames.get(j)); List<SimpleNode> nodes = nodeNameToNodes.get(nodeNames.get(j));