forked from phoedos/pmd
Fixups from review (#4128)
This commit is contained in:
@ -204,7 +204,7 @@ public class UnnecessaryFullyQualifiedNameRule extends AbstractJavaRule {
|
||||
String importStr = firstMatch.getImportedName() + (firstMatch.isImportOnDemand() ? ".*" : "");
|
||||
String type = firstMatch.isStatic() ? "static " : "";
|
||||
|
||||
addViolation(data, node, new Object[]{node.getImage(), importStr, type});
|
||||
asCtx(data).addViolation(node, node.getImage(), importStr, type);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
@ -623,7 +623,7 @@ public class UnnecessaryFullyQualifiedName {
|
||||
</test-code>
|
||||
|
||||
<test-code>
|
||||
<description>False positive when same package inner class is referenced (not enum)</description>
|
||||
<description>False positive when same package inner class is referenced (not enum) #4085</description>
|
||||
<expected-problems>0</expected-problems>
|
||||
<code><![CDATA[
|
||||
package net.sourceforge.pmd.lang.java.rule.codestyle.unnecessaryfullyqualifiedname;
|
||||
@ -642,7 +642,7 @@ public class OuterTestClass {
|
||||
</test-code>
|
||||
|
||||
<test-code>
|
||||
<description>Should report fully-qualified name usage of a class in itself.</description>
|
||||
<description>Should report fully-qualified name usage of a class in itself #4085</description>
|
||||
<expected-problems>1</expected-problems>
|
||||
<expected-linenumbers>4</expected-linenumbers>
|
||||
<code><![CDATA[
|
||||
|
Reference in New Issue
Block a user