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

pmd: fix #1104 IdempotentOperation false positive

Browse Source
This commit is contained in:
Andreas Dangel 2013-08-03 17:53:14 +02:00
parent dc64e2df77
commit a417f0003f
3 changed files with 30 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
pmd/etc/changelog.txt
View File

@ -1,6 +1,7 @@
????? ??, 2013 - 5.0.5:
Fixed bug 991: AvoidSynchronizedAtMethodLevel for static methods
Fixed bug 1104: IdempotentOperation false positive
Fixed bug 1111: False positive: Useless parentheses
Fixed bug 1114: CPD - Tokenizer not initialized with requested properties
Fixed bug 1118: ClassCastException in pmd.lang.ecmascript.ast.ASTElementGet

17
pmd/src/main/java/net/sourceforge/pmd/lang/java/rule/design/IdempotentOperationsRule.java
View File

@ -3,6 +3,8 @@
*/
package net.sourceforge.pmd.lang.java.rule.design;
import java.util.List;
import net.sourceforge.pmd.lang.ast.Node;
import net.sourceforge.pmd.lang.java.ast.ASTAssignmentOperator;
import net.sourceforge.pmd.lang.java.ast.ASTExpression;
@ -59,6 +61,21 @@ public class IdempotentOperationsRule extends AbstractJavaRule {
return super.visit(node, data);
}
List<ASTPrimarySuffix> lhsSuffixes = lhs.jjtGetParent().jjtGetParent().findDescendantsOfType(ASTPrimarySuffix.class);
List<ASTPrimarySuffix> rhsSuffixes = rhs.jjtGetParent().jjtGetParent().findDescendantsOfType(ASTPrimarySuffix.class);
if (lhsSuffixes.size() != rhsSuffixes.size()) {
return super.visit(node, data);
}
for (int i = 0; i < lhsSuffixes.size(); i++) {
ASTPrimarySuffix l = lhsSuffixes.get(i);
ASTPrimarySuffix r = rhsSuffixes.get(i);
if (!l.hasImageEqualTo(r.getImage())) {
return super.visit(node, data);
}
}
addViolation(data, node);
return super.visit(node, data);
}

12
pmd/src/test/resources/net/sourceforge/pmd/lang/java/rule/design/xml/IdempotentOperations.xml
View File

@ -61,4 +61,16 @@ public class Foo {
}
]]></code>
</test-code>
<test-code>
<description>#1104 IdempotentOperation false positive</description>
<expected-problems>0</expected-problems>
<code><![CDATA[
public class Foo {
void bar() {
SignDelayer.this.sign = SignDelayer.this.arg.sign();
MMultiplexer.this.lastEvent = MMultiplexer.this.firstEvent;
}
}
]]></code>
</test-code>
</test-data>