PMD Results

The following document contains the results of PMD 7.17.0.

Violations By Priority

Priority 3

jaxen - org/jaxen/VariableContext.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getVariableValue': the method is declared in an interface type 110
jaxen - org/jaxen/dom/DocumentNavigator.java
Rule Violation Line
EmptyControlStatement Empty if statement 395–404
EmptyCatchBlock Avoid empty catch blocks 487–488
EmptyCatchBlock Avoid empty catch blocks 526–527
EmptyCatchBlock Avoid empty catch blocks 548–549
EmptyCatchBlock Avoid empty catch blocks 588–589
jaxen - org/jaxen/dom4j/DocumentNavigator.java
Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 240–242
jaxen - org/jaxen/expr/DefaultStep.java
Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 154–158
jaxen - org/jaxen/expr/FilterExpr.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'asBoolean': the method is declared in an interface type 78
UnnecessaryModifier Unnecessary modifier 'public' on method 'getExpr': the method is declared in an interface type 83
jaxen - org/jaxen/expr/FunctionCallExpr.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getPrefix': the method is declared in an interface type 69
UnnecessaryModifier Unnecessary modifier 'public' on method 'getFunctionName': the method is declared in an interface type 76
UnnecessaryModifier Unnecessary modifier 'public' on method 'addParameter': the method is declared in an interface type 83
UnnecessaryModifier Unnecessary modifier 'public' on method 'getParameters': the method is declared in an interface type 91
jaxen - org/jaxen/expr/LiteralExpr.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLiteral': the method is declared in an interface type 69
jaxen - org/jaxen/expr/NameStep.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getPrefix': the method is declared in an interface type 68
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLocalName': the method is declared in an interface type 75
jaxen - org/jaxen/expr/NumberExpr.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getNumber': the method is declared in an interface type 67
jaxen - org/jaxen/expr/ProcessingInstructionNodeStep.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getName': the method is declared in an interface type 60
jaxen - org/jaxen/expr/Step.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getAxis': the method is declared in an interface type 102
jaxen - org/jaxen/expr/VariableReferenceExpr.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getPrefix': the method is declared in an interface type 67
UnnecessaryModifier Unnecessary modifier 'public' on method 'getVariableName': the method is declared in an interface type 74
jaxen - org/jaxen/javabean/DocumentNavigator.java
Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 225–228
EmptyCatchBlock Avoid empty catch blocks 229–232
jaxen - org/jaxen/jdom/DocumentNavigator.java
Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 256–257
CollapsibleIfStatements This if statement could be combined with its parent 278–279
CollapsibleIfStatements This if statement could be combined with its parent 309–312
jaxen - org/jaxen/pattern/LocationPathPattern.java
Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 113–116
jaxen - org/jaxen/pattern/UnionPattern.java
Rule Violation Line
UnusedNullCheckInEquals Invoke equals() on the object you've already ensured is not null 159
jaxen - org/jaxen/saxpath/Operator.java
Rule Violation Line
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'NO_OP': the field is declared in an interface type 63
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'EQUALS': the field is declared in an interface type 70
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'NOT_EQUALS': the field is declared in an interface type 75
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'LESS_THAN': the field is declared in an interface type 81
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'LESS_THAN_EQUALS': the field is declared in an interface type 86
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'GREATER_THAN': the field is declared in an interface type 91
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'GREATER_THAN_EQUALS': the field is declared in an interface type 96
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'ADD': the field is declared in an interface type 103
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'SUBTRACT': the field is declared in an interface type 108
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'MULTIPLY': the field is declared in an interface type 115
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'MOD': the field is declared in an interface type 121
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'DIV': the field is declared in an interface type 127
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'NEGATIVE': the field is declared in an interface type 134
jaxen - org/jaxen/saxpath/helpers/XPathReaderFactory.java
Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 95–98
jaxen - org/jaxen/xom/DocumentNavigator.java
Rule Violation Line
UselessOverridingMethod Overriding method merely calls super 279
UselessOverridingMethod Overriding method merely calls super 283
EmptyControlStatement Empty if statement 303–311

Priority 4

jaxen - org/jaxen/BaseXPath.java
Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen': 'XPathSyntaxException' is already in scope because it is declared in the same package 119
jaxen - org/jaxen/JaxenHandler.java
Rule Violation Line
UselessParentheses Useless parentheses around `peekFrame().size() > 0`. 604
jaxen - org/jaxen/QualifiedName.java
Rule Violation Line
UselessParentheses Useless parentheses around `localName.hashCode() ^ namespaceURI.hashCode()`. 84
UselessParentheses Useless parentheses around `namespaceURI.equals(other.namespaceURI) && othe...`. 95–96
jaxen - org/jaxen/dom/DocumentNavigator.java
Rule Violation Line
UselessParentheses Useless parentheses around `object instanceof NamespaceNode`. 615
UselessParentheses Useless parentheses around `node != null`. 922
jaxen - org/jaxen/dom/NamespaceNode.java
Rule Violation Line
UselessParentheses Useless parentheses around `equals(parent, ns.getParentNode()) && equals(na...`. 522–524
UselessParentheses Useless parentheses around `o == null ? 0 : o.hashCode()`. 540
UselessParentheses Useless parentheses around `(a == null && b == null) || (a != null && a.equ...`. 554–555
jaxen - org/jaxen/dom4j/DocumentNavigator.java
Rule Violation Line
UselessParentheses Useless parentheses around `obj instanceof Text || obj instanceof CDATA`. 126–128
jaxen - org/jaxen/expr/DefaultNameStep.java
Rule Violation Line
UselessParentheses Useless parentheses around `!matchesAnyName && iterableAxis.supportsNamedAc...`. 161
UselessParentheses Useless parentheses around `uri != null && uri.length() > 0`. 383
UselessParentheses Useless parentheses around `uri2.length() == 0`. 398
UselessParentheses Useless parentheses around `uri1.length() == 0`. 401
jaxen - org/jaxen/expr/DefaultTruthExpr.java
Rule Violation Line
UselessParentheses Useless parentheses around `lhs instanceof List && rhs instanceof List`. 72–74
UselessParentheses Useless parentheses around `lhs instanceof List || rhs instanceof List`. 80–82
UselessParentheses Useless parentheses around `obj instanceof List`. 87
UselessParentheses Useless parentheses around `obj instanceof Boolean`. 92
UselessParentheses Useless parentheses around `set == null || set.isEmpty()`. 97
UselessParentheses Useless parentheses around `lhs instanceof Boolean || rhs instanceof Boolean`. 103–105
UselessParentheses Useless parentheses around `lhs instanceof Boolean && rhs instanceof Boolean`. 111–113
UselessParentheses Useless parentheses around `lhs instanceof Number || rhs instanceof Number`. 119–121
jaxen - org/jaxen/expr/iter/IterableAttributeAxis.java
Rule Violation Line
UselessParentheses Useless parentheses around `support.getNavigator() instanceof NamedAccessNa...`. 103
jaxen - org/jaxen/expr/iter/IterableChildAxis.java
Rule Violation Line
UselessParentheses Useless parentheses around `support.getNavigator() instanceof NamedAccessNa...`. 105
jaxen - org/jaxen/function/BooleanFunction.java
Rule Violation Line
UselessParentheses Useless parentheses around `((String) obj).length() > 0 ? Boolean.TRUE : Bo...`. 187–189
jaxen - org/jaxen/function/ContainsFunction.java
Rule Violation Line
UselessParentheses Useless parentheses around `(str.indexOf(match) >= 0) ? Boolean.TRUE : Bool...`. 131–134
jaxen - org/jaxen/function/NotFunction.java
Rule Violation Line
UselessParentheses Useless parentheses around `BooleanFunction.evaluate(obj, nav).booleanValue...`. 116–119
UselessParentheses Useless parentheses around `BooleanFunction.evaluate(obj, nav).booleanValue()`. 116
jaxen - org/jaxen/function/StartsWithFunction.java
Rule Violation Line
UselessParentheses Useless parentheses around `str.startsWith(match) ? Boolean.TRUE : Boolean....`. 130–133
jaxen - org/jaxen/function/SubstringFunction.java
Rule Violation Line
UselessParentheses Useless parentheses around `StringLengthFunction.evaluate(args.get(0), nav)`. 180
jaxen - org/jaxen/function/ext/EndsWithFunction.java
Rule Violation Line
UselessParentheses Useless parentheses around `str.endsWith(match) ? Boolean.TRUE : Boolean.FALSE`. 89–92
jaxen - org/jaxen/javabean/DocumentNavigator.java
Rule Violation Line
UselessParentheses Useless parentheses around `obj instanceof Element`. 87
UselessParentheses Useless parentheses around `obj instanceof String`. 97
jaxen - org/jaxen/jdom/DocumentNavigator.java
Rule Violation Line
UselessParentheses Useless parentheses around `obj instanceof Text || obj instanceof CDATA`. 120–122
UselessParentheses Useless parentheses around `namespaceURI == null ? Namespace.NO_NAMESPACE :...`. 371–372
jaxen - org/jaxen/jdom/XPathNamespace.java
Rule Violation Line
UselessParentheses Useless parentheses around `"[xmlns:" + jdomNamespace.getPrefix() + "=\"" +...`. 108–110
jaxen - org/jaxen/pattern/NameTest.java
Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ELEMENT_NODE' is already in scope because it is inherited by an enclosing type 77
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ATTRIBUTE_NODE' is already in scope because it is inherited by an enclosing type 82
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ATTRIBUTE_NODE' is already in scope because it is inherited by an enclosing type 115
jaxen - org/jaxen/pattern/NamespaceTest.java
Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ELEMENT_NODE' is already in scope because it is inherited by an enclosing type 83
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ATTRIBUTE_NODE' is already in scope because it is inherited by an enclosing type 88
jaxen - org/jaxen/pattern/TextNodeTest.java
Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'TEXT_NODE' is already in scope because it is inherited by an enclosing type 79
jaxen - org/jaxen/saxpath/XPathHandler.java
Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 76
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 80
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 84
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 88
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 92
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 96
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 100
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 104
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 115
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 119
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 125
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 129
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 135
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 139
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 145
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 149
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 158
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 162
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 166
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 170
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 174
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 178
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 182
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 190
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 194
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 202
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 206
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 215
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 219
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 228
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 232
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 241
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 245
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 254
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 258
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 268
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 272
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 280
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 286
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 292
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 298
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 306
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 314
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 318
jaxen - org/jaxen/saxpath/base/Token.java
Rule Violation Line
UselessParentheses Useless parentheses around `"[ (" + tokenType + ") (" + getTokenText() + ")"`. 115
jaxen - org/jaxen/saxpath/base/TokenTypes.java
Rule Violation Line
UselessParentheses Useless parentheses around `"Unrecognized token type: " + tokenType`. 184
jaxen - org/jaxen/saxpath/base/Verifier.java
Rule Violation Line
UselessParentheses Useless parentheses around `isXMLLetter(c) || isXMLDigit(c) || c == '.' || ...`. 67–69
UselessParentheses Useless parentheses around `isXMLLetter(c) || c == '_'`. 84
jaxen - org/jaxen/saxpath/base/XPathLexer.java
Rule Violation Line
UselessParentheses Useless parentheses around `i - 1`. 891
UselessParentheses Useless parentheses around `i - 1`. 896
jaxen - org/jaxen/util/PrecedingSiblingAxisIterator.java
Rule Violation Line
UselessParentheses Useless parentheses around `this.nextObj != null`. 137
jaxen - org/jaxen/xom/DocumentNavigator.java
Rule Violation Line
UselessParentheses Useless parentheses around `isAttribute(o) ? ((Attribute) o).getLocalName()...`. 126
UselessParentheses Useless parentheses around `isAttribute(o) ? ((Attribute) o).getNamespaceUR...`. 130
UselessParentheses Useless parentheses around `isAttribute(o) ? ((Attribute) o).getQualifiedNa...`. 134
UselessParentheses Useless parentheses around `isAttribute(o) ? ((Attribute) o).getValue() : null`. 138
UselessParentheses Useless parentheses around `isComment(o) ? ((Comment) o).getValue() : null`. 144
UselessParentheses Useless parentheses around `isElement(o) ? ((Element) o).getLocalName() : null`. 148
UselessParentheses Useless parentheses around `isElement(o) ? ((Element) o).getNamespaceURI() ...`. 152
UselessParentheses Useless parentheses around `isElement(o) ? ((Element) o).getQualifiedName()...`. 156
UselessParentheses Useless parentheses around `o instanceof Node ? ((Node) o).getValue() : null`. 160
UselessParentheses Useless parentheses around `o instanceof Text ? ((Text) o).getValue() : null`. 190
UselessParentheses Useless parentheses around `parent != null ? new SingleObjectIterator(paren...`. 270
UselessParentheses Useless parentheses around `o instanceof Node ? ((Node) o).getParent() : null`. 274
UselessParentheses Useless parentheses around `o instanceof ProcessingInstruction ? ((Processi...`. 290
UselessParentheses Useless parentheses around `o instanceof ProcessingInstruction ? ((Processi...`. 294
UselessParentheses Useless parentheses around `"[xmlns:" + prefix + "=\"" + uri + "\", element...`. 365–367

Files

jaxen - org/jaxen/BaseXPath.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen': 'XPathSyntaxException' is already in scope because it is declared in the same package 4 119

jaxen - org/jaxen/JaxenHandler.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `peekFrame().size() > 0`. 4 604

jaxen - org/jaxen/QualifiedName.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `localName.hashCode() ^ namespaceURI.hashCode()`. 4 84
UselessParentheses Useless parentheses around `namespaceURI.equals(other.namespaceURI) && othe...`. 4 95–96

jaxen - org/jaxen/VariableContext.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getVariableValue': the method is declared in an interface type 3 110

jaxen - org/jaxen/dom/DocumentNavigator.java

Rule Violation Priority Line
EmptyControlStatement Empty if statement 3 395–404
EmptyCatchBlock Avoid empty catch blocks 3 487–488
EmptyCatchBlock Avoid empty catch blocks 3 526–527
EmptyCatchBlock Avoid empty catch blocks 3 548–549
EmptyCatchBlock Avoid empty catch blocks 3 588–589
UselessParentheses Useless parentheses around `object instanceof NamespaceNode`. 4 615
UselessParentheses Useless parentheses around `node != null`. 4 922

jaxen - org/jaxen/dom/NamespaceNode.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `equals(parent, ns.getParentNode()) && equals(na...`. 4 522–524
UselessParentheses Useless parentheses around `o == null ? 0 : o.hashCode()`. 4 540
UselessParentheses Useless parentheses around `(a == null && b == null) || (a != null && a.equ...`. 4 554–555

jaxen - org/jaxen/dom4j/DocumentNavigator.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `obj instanceof Text || obj instanceof CDATA`. 4 126–128
CollapsibleIfStatements This if statement could be combined with its parent 3 240–242

jaxen - org/jaxen/expr/DefaultNameStep.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `!matchesAnyName && iterableAxis.supportsNamedAc...`. 4 161
UselessParentheses Useless parentheses around `uri != null && uri.length() > 0`. 4 383
UselessParentheses Useless parentheses around `uri2.length() == 0`. 4 398
UselessParentheses Useless parentheses around `uri1.length() == 0`. 4 401

jaxen - org/jaxen/expr/DefaultStep.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 154–158

jaxen - org/jaxen/expr/DefaultTruthExpr.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `lhs instanceof List && rhs instanceof List`. 4 72–74
UselessParentheses Useless parentheses around `lhs instanceof List || rhs instanceof List`. 4 80–82
UselessParentheses Useless parentheses around `obj instanceof List`. 4 87
UselessParentheses Useless parentheses around `obj instanceof Boolean`. 4 92
UselessParentheses Useless parentheses around `set == null || set.isEmpty()`. 4 97
UselessParentheses Useless parentheses around `lhs instanceof Boolean || rhs instanceof Boolean`. 4 103–105
UselessParentheses Useless parentheses around `lhs instanceof Boolean && rhs instanceof Boolean`. 4 111–113
UselessParentheses Useless parentheses around `lhs instanceof Number || rhs instanceof Number`. 4 119–121

jaxen - org/jaxen/expr/FilterExpr.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'asBoolean': the method is declared in an interface type 3 78
UnnecessaryModifier Unnecessary modifier 'public' on method 'getExpr': the method is declared in an interface type 3 83

jaxen - org/jaxen/expr/FunctionCallExpr.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getPrefix': the method is declared in an interface type 3 69
UnnecessaryModifier Unnecessary modifier 'public' on method 'getFunctionName': the method is declared in an interface type 3 76
UnnecessaryModifier Unnecessary modifier 'public' on method 'addParameter': the method is declared in an interface type 3 83
UnnecessaryModifier Unnecessary modifier 'public' on method 'getParameters': the method is declared in an interface type 3 91

jaxen - org/jaxen/expr/LiteralExpr.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLiteral': the method is declared in an interface type 3 69

jaxen - org/jaxen/expr/NameStep.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getPrefix': the method is declared in an interface type 3 68
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLocalName': the method is declared in an interface type 3 75

jaxen - org/jaxen/expr/NumberExpr.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getNumber': the method is declared in an interface type 3 67

jaxen - org/jaxen/expr/ProcessingInstructionNodeStep.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getName': the method is declared in an interface type 3 60

jaxen - org/jaxen/expr/Step.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getAxis': the method is declared in an interface type 3 102

jaxen - org/jaxen/expr/VariableReferenceExpr.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getPrefix': the method is declared in an interface type 3 67
UnnecessaryModifier Unnecessary modifier 'public' on method 'getVariableName': the method is declared in an interface type 3 74

jaxen - org/jaxen/expr/iter/IterableAttributeAxis.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `support.getNavigator() instanceof NamedAccessNa...`. 4 103

jaxen - org/jaxen/expr/iter/IterableChildAxis.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `support.getNavigator() instanceof NamedAccessNa...`. 4 105

jaxen - org/jaxen/function/BooleanFunction.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `((String) obj).length() > 0 ? Boolean.TRUE : Bo...`. 4 187–189

jaxen - org/jaxen/function/ContainsFunction.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `(str.indexOf(match) >= 0) ? Boolean.TRUE : Bool...`. 4 131–134

jaxen - org/jaxen/function/NotFunction.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `BooleanFunction.evaluate(obj, nav).booleanValue...`. 4 116–119
UselessParentheses Useless parentheses around `BooleanFunction.evaluate(obj, nav).booleanValue()`. 4 116

jaxen - org/jaxen/function/StartsWithFunction.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `str.startsWith(match) ? Boolean.TRUE : Boolean....`. 4 130–133

jaxen - org/jaxen/function/SubstringFunction.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `StringLengthFunction.evaluate(args.get(0), nav)`. 4 180

jaxen - org/jaxen/function/ext/EndsWithFunction.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `str.endsWith(match) ? Boolean.TRUE : Boolean.FALSE`. 4 89–92

jaxen - org/jaxen/javabean/DocumentNavigator.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `obj instanceof Element`. 4 87
UselessParentheses Useless parentheses around `obj instanceof String`. 4 97
EmptyCatchBlock Avoid empty catch blocks 3 225–228
EmptyCatchBlock Avoid empty catch blocks 3 229–232

jaxen - org/jaxen/jdom/DocumentNavigator.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `obj instanceof Text || obj instanceof CDATA`. 4 120–122
CollapsibleIfStatements This if statement could be combined with its parent 3 256–257
CollapsibleIfStatements This if statement could be combined with its parent 3 278–279
CollapsibleIfStatements This if statement could be combined with its parent 3 309–312
UselessParentheses Useless parentheses around `namespaceURI == null ? Namespace.NO_NAMESPACE :...`. 4 371–372

jaxen - org/jaxen/jdom/XPathNamespace.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `"[xmlns:" + jdomNamespace.getPrefix() + "=\"" +...`. 4 108–110

jaxen - org/jaxen/pattern/LocationPathPattern.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 113–116

jaxen - org/jaxen/pattern/NameTest.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ELEMENT_NODE' is already in scope because it is inherited by an enclosing type 4 77
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ATTRIBUTE_NODE' is already in scope because it is inherited by an enclosing type 4 82
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ATTRIBUTE_NODE' is already in scope because it is inherited by an enclosing type 4 115

jaxen - org/jaxen/pattern/NamespaceTest.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ELEMENT_NODE' is already in scope because it is inherited by an enclosing type 4 83
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'ATTRIBUTE_NODE' is already in scope because it is inherited by an enclosing type 4 88

jaxen - org/jaxen/pattern/TextNodeTest.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pattern': 'TEXT_NODE' is already in scope because it is inherited by an enclosing type 4 79

jaxen - org/jaxen/pattern/UnionPattern.java

Rule Violation Priority Line
UnusedNullCheckInEquals Invoke equals() on the object you've already ensured is not null 3 159

jaxen - org/jaxen/saxpath/Operator.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'NO_OP': the field is declared in an interface type 3 63
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'EQUALS': the field is declared in an interface type 3 70
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'NOT_EQUALS': the field is declared in an interface type 3 75
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'LESS_THAN': the field is declared in an interface type 3 81
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'LESS_THAN_EQUALS': the field is declared in an interface type 3 86
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'GREATER_THAN': the field is declared in an interface type 3 91
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'GREATER_THAN_EQUALS': the field is declared in an interface type 3 96
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'ADD': the field is declared in an interface type 3 103
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'SUBTRACT': the field is declared in an interface type 3 108
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'MULTIPLY': the field is declared in an interface type 3 115
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'MOD': the field is declared in an interface type 3 121
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'DIV': the field is declared in an interface type 3 127
UnnecessaryModifier Unnecessary modifiers 'static final' on field 'NEGATIVE': the field is declared in an interface type 3 134

jaxen - org/jaxen/saxpath/XPathHandler.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 76
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 80
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 84
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 88
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 92
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 96
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 100
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 104
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 115
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 119
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 125
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 129
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 135
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 139
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 145
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 149
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 158
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 162
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 166
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 170
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 174
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 178
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 182
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 190
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 194
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 202
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 206
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 215
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 219
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 228
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 232
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 241
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 245
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 254
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 258
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 268
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 272
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 280
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 286
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 292
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 298
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 306
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 314
UnnecessaryFullyQualifiedName Unnecessary qualifier 'org.jaxen.saxpath': 'SAXPathException' is already in scope because it is declared in the same package 4 318

jaxen - org/jaxen/saxpath/base/Token.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `"[ (" + tokenType + ") (" + getTokenText() + ")"`. 4 115

jaxen - org/jaxen/saxpath/base/TokenTypes.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `"Unrecognized token type: " + tokenType`. 4 184

jaxen - org/jaxen/saxpath/base/Verifier.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `isXMLLetter(c) || isXMLDigit(c) || c == '.' || ...`. 4 67–69
UselessParentheses Useless parentheses around `isXMLLetter(c) || c == '_'`. 4 84

jaxen - org/jaxen/saxpath/base/XPathLexer.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `i - 1`. 4 891
UselessParentheses Useless parentheses around `i - 1`. 4 896

jaxen - org/jaxen/saxpath/helpers/XPathReaderFactory.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 95–98

jaxen - org/jaxen/util/PrecedingSiblingAxisIterator.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `this.nextObj != null`. 4 137

jaxen - org/jaxen/xom/DocumentNavigator.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `isAttribute(o) ? ((Attribute) o).getLocalName()...`. 4 126
UselessParentheses Useless parentheses around `isAttribute(o) ? ((Attribute) o).getNamespaceUR...`. 4 130
UselessParentheses Useless parentheses around `isAttribute(o) ? ((Attribute) o).getQualifiedNa...`. 4 134
UselessParentheses Useless parentheses around `isAttribute(o) ? ((Attribute) o).getValue() : null`. 4 138
UselessParentheses Useless parentheses around `isComment(o) ? ((Comment) o).getValue() : null`. 4 144
UselessParentheses Useless parentheses around `isElement(o) ? ((Element) o).getLocalName() : null`. 4 148
UselessParentheses Useless parentheses around `isElement(o) ? ((Element) o).getNamespaceURI() ...`. 4 152
UselessParentheses Useless parentheses around `isElement(o) ? ((Element) o).getQualifiedName()...`. 4 156
UselessParentheses Useless parentheses around `o instanceof Node ? ((Node) o).getValue() : null`. 4 160
UselessParentheses Useless parentheses around `o instanceof Text ? ((Text) o).getValue() : null`. 4 190
UselessParentheses Useless parentheses around `parent != null ? new SingleObjectIterator(paren...`. 4 270
UselessParentheses Useless parentheses around `o instanceof Node ? ((Node) o).getParent() : null`. 4 274
UselessOverridingMethod Overriding method merely calls super 3 279
UselessOverridingMethod Overriding method merely calls super 3 283
UselessParentheses Useless parentheses around `o instanceof ProcessingInstruction ? ((Processi...`. 4 290
UselessParentheses Useless parentheses around `o instanceof ProcessingInstruction ? ((Processi...`. 4 294
EmptyControlStatement Empty if statement 3 303–311
UselessParentheses Useless parentheses around `"[xmlns:" + prefix + "=\"" + uri + "\", element...`. 4 365–367