PMD Results

The following document contains the results of PMD 3.9.

Files

org/apache/james/jspf/core/Configurable.java

ViolationLine
Found 'DU'-anomaly for variable 'params' (lines '36'-'36').36

org/apache/james/jspf/core/Configuration.java

ViolationLine
Avoid modifiers which are implied by the context27
Found 'DU'-anomaly for variable 'arg0' (lines '27'-'27').27
Avoid modifiers which are implied by the context32

org/apache/james/jspf/core/DNSService.java

ViolationLine
Avoid modifiers which are implied by the context33 - 35
Each class should declare at least one constructor33 - 35
Avoid modifiers which are implied by the context38
Avoid modifiers which are implied by the context39
Avoid modifiers which are implied by the context40
Avoid modifiers which are implied by the context41
Avoid modifiers which are implied by the context42
Avoid modifiers which are implied by the context43
Avoid modifiers which are implied by the context54
Found 'DU'-anomaly for variable 'hostname' (lines '54'-'54').54
Found 'DU'-anomaly for variable 'recordType' (lines '54'-'54').54
Avoid modifiers which are implied by the context61
Avoid modifiers which are implied by the context69
Found 'DU'-anomaly for variable 'timeOut' (lines '69'-'69').69
Avoid modifiers which are implied by the context74
Avoid modifiers which are implied by the context80
Found 'DU'-anomaly for variable 'recordLimit' (lines '80'-'80').80

org/apache/james/jspf/core/IPAddr.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method96
Found 'DU'-anomaly for variable 'startMask' (lines '131'-'153').131 - 153
Found 'DU'-anomaly for variable 'startMask' (lines '137'-'153').137 - 153
Found 'DD'-anomaly for variable 'data' (lines '162'-'164').162 - 164
A method should have only one exit point, and that should be the last statement in the method167
A method should have only one exit point, and that should be the last statement in the method298
Found 'DU'-anomaly for variable 'addressData' (lines '310'-'316').310 - 316
Found 'DD'-anomaly for variable 'convertAddress' (lines '311'-'313').311 - 313
Found 'DD'-anomaly for variable 'convertAddress' (lines '313'-'313').313
Found 'DU'-anomaly for variable 'maskData' (lines '327'-'334').327 - 334
Found 'DU'-anomaly for variable 'addressData' (lines '327'-'334').327 - 334
Found 'DD'-anomaly for variable 'maskedAddress' (lines '328'-'331').328 - 331
Found 'DD'-anomaly for variable 'maskedAddress' (lines '331'-'331').331
Found 'DU'-anomaly for variable 'addressData' (lines '343'-'352').343 - 352
Found 'DD'-anomaly for variable 'reverseIP' (lines '344'-'348').344 - 348
Found 'DD'-anomaly for variable 'reverseIP' (lines '348'-'349').348 - 349
Found 'DD'-anomaly for variable 'reverseIP' (lines '349'-'348').349 - 348
A method should have only one exit point, and that should be the last statement in the method387
A method should have only one exit point, and that should be the last statement in the method433
Avoid empty catch blocks434 - 436
A method should have only one exit point, and that should be the last statement in the method467

org/apache/james/jspf/core/Logger.java

ViolationLine
Found 'DU'-anomaly for variable 'message' (lines '34'-'34').34
Found 'DU'-anomaly for variable 'throwable' (lines '44'-'44').44
Found 'DU'-anomaly for variable 'message' (lines '44'-'44').44
Found 'DU'-anomaly for variable 'message' (lines '59'-'59').59
Found 'DU'-anomaly for variable 'throwable' (lines '69'-'69').69
Found 'DU'-anomaly for variable 'message' (lines '69'-'69').69
Found 'DU'-anomaly for variable 'message' (lines '84'-'84').84
Found 'DU'-anomaly for variable 'throwable' (lines '94'-'94').94
Found 'DU'-anomaly for variable 'message' (lines '94'-'94').94
Found 'DU'-anomaly for variable 'message' (lines '109'-'109').109
Found 'DU'-anomaly for variable 'throwable' (lines '119'-'119').119
Found 'DU'-anomaly for variable 'message' (lines '119'-'119').119
Found 'DU'-anomaly for variable 'message' (lines '134'-'134').134
Found 'DU'-anomaly for variable 'throwable' (lines '144'-'144').144
Found 'DU'-anomaly for variable 'message' (lines '144'-'144').144
Found 'DU'-anomaly for variable 'name' (lines '162'-'162').162

org/apache/james/jspf/core/Mechanism.java

ViolationLine
Avoid modifiers which are implied by the context46 - 47
Found 'DU'-anomaly for variable 'spfData' (lines '46'-'47').46 - 47

org/apache/james/jspf/core/Modifier.java

ViolationLine
Avoid modifiers which are implied by the context34

org/apache/james/jspf/core/SPF1Constants.java

ViolationLine
Avoid modifiers which are implied by the context31
Avoid modifiers which are implied by the context36
Avoid modifiers which are implied by the context41
Avoid modifiers which are implied by the context46
Avoid modifiers which are implied by the context51

org/apache/james/jspf/core/SPF1Data.java

ViolationLine
Perhaps 'readableIP' could be replaced by a local variable.52
Perhaps 'currentDepth' could be replaced by a local variable.56
Found 'DU'-anomaly for variable 'helo' (lines '113'-'135').113 - 135
A method should have only one exit point, and that should be the last statement in the method210
Avoid empty catch blocks211 - 212

org/apache/james/jspf/core/SPF1Record.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.37
Perhaps 'directives' could be replaced by a local variable.44
Perhaps 'modifiers' could be replaced by a local variable.46

org/apache/james/jspf/core/SPFChecker.java

ViolationLine
Avoid modifiers which are implied by the context46 - 48
Found 'DU'-anomaly for variable 'spfData' (lines '46'-'48').46 - 48

org/apache/james/jspf/core/SPFRecordParser.java

ViolationLine
Avoid modifiers which are implied by the context42 - 43
Found 'DU'-anomaly for variable 'spfRecord' (lines '42'-'43').42 - 43

org/apache/james/jspf/impl/DNSServiceXBillImpl.java

ViolationLine
Avoid empty catch blocks89 - 91
Found 'DU'-anomaly for variable 'hostname' (lines '114'-'190').114 - 190
Found 'DD'-anomaly for variable 'recordCount' (lines '118'-'178').118 - 178
Found 'DU'-anomaly for variable 'recordCount' (lines '118'-'190').118 - 190
A method should have only one exit point, and that should be the last statement in the method127
Found 'DU'-anomaly for variable 'records' (lines '145'-'190').145 - 190
Found 'DD'-anomaly for variable 'records' (lines '145'-'187').145 - 187
A method should have only one exit point, and that should be the last statement in the method174
Assigning an Object to null is a code smell. Consider refactoring.180
Found 'DD'-anomaly for variable 'records' (lines '180'-'187').180 - 187
Assigning an Object to null is a code smell. Consider refactoring.187

org/apache/james/jspf/impl/SPFQuery.java

ViolationLine
Each class should declare at least one constructor40 - 207
Found 'DD'-anomaly for variable 'ip' (lines '82'-'101').82 - 101
Found 'DD'-anomaly for variable 'sender' (lines '83'-'102').83 - 102
Found 'DD'-anomaly for variable 'helo' (lines '84'-'103').84 - 103
Found 'DD'-anomaly for variable 'defaultExplanation' (lines '85'-'104').85 - 104
Found 'DD'-anomaly for variable 'useBestGuess' (lines '86'-'105').86 - 105
Found 'DD'-anomaly for variable 'useTrustedForwarder' (lines '87'-'106').87 - 106
Found 'DU'-anomaly for variable 'defaultExplanation' (lines '104'-'142').104 - 142
Found 'DU'-anomaly for variable 'useBestGuess' (lines '105'-'142').105 - 142
Found 'DU'-anomaly for variable 'useTrustedForwarder' (lines '106'-'142').106 - 142
A method should have only one exit point, and that should be the last statement in the method189
A method should have only one exit point, and that should be the last statement in the method191
A method should have only one exit point, and that should be the last statement in the method193
A method should have only one exit point, and that should be the last statement in the method195
A method should have only one exit point, and that should be the last statement in the method197
A method should have only one exit point, and that should be the last statement in the method199
A method should have only one exit point, and that should be the last statement in the method201

org/apache/james/jspf/macro/MacroData.java

ViolationLine
Avoid modifiers which are implied by the context35
Avoid modifiers which are implied by the context42
Avoid modifiers which are implied by the context49
Avoid modifiers which are implied by the context56
Avoid modifiers which are implied by the context63
Avoid modifiers which are implied by the context70
Avoid modifiers which are implied by the context77
Avoid modifiers which are implied by the context84
Avoid modifiers which are implied by the context91
Avoid modifiers which are implied by the context98
Avoid modifiers which are implied by the context105

org/apache/james/jspf/macro/MacroExpand.java

ViolationLine
Perhaps 'cellPattern' could be replaced by a local variable.56
A method should have only one exit point, and that should be the last statement in the method86
Found 'DU'-anomaly for variable 'macroData' (lines '101'-'114').101 - 114
Found 'DU'-anomaly for variable 'res' (lines '134'-'157').134 - 157
Found 'UR'-anomaly for variable 'pos' (lines '168'-'168').168
Found 'UR'-anomaly for variable 'inputMatcher' (lines '168'-'168').168
Found 'DU'-anomaly for variable 'isExplanation' (lines '168'-'202').168 - 202
Found 'DU'-anomaly for variable 'macroData' (lines '168'-'202').168 - 202
Found 'DU'-anomaly for variable 'macroData' (lines '213'-'284').213 - 284
Found 'DD'-anomaly for variable 'variable' (lines '215'-'233').215 - 233
Found 'DD'-anomaly for variable 'variable' (lines '215'-'235').215 - 235
Found 'DD'-anomaly for variable 'domainNumber' (lines '216'-'248').216 - 248
Found 'DD'-anomaly for variable 'isReversed' (lines '217'-'258').217 - 258
Found 'DD'-anomaly for variable 'delimeters' (lines '218'-'265').218 - 265
Found 'DD'-anomaly for variable 'isReversed' (lines '258'-'258').258
Found 'DD'-anomaly for variable 'delimeters' (lines '265'-'265').265
Found 'DD'-anomaly for variable 'data' (lines '269'-'271').269 - 271
Found 'UR'-anomaly for variable 'variable' (lines '296'-'296').296
Found 'DU'-anomaly for variable 'macroData' (lines '296'-'378').296 - 378
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'302').298 - 302
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'304').298 - 304
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'306').298 - 306
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'308').298 - 308
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'310').298 - 310
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'312').298 - 312
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'314').298 - 314
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'316').298 - 316
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'318').298 - 318
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'350').298 - 350
Found 'DD'-anomaly for variable 'rValue' (lines '298'-'352').298 - 352
Found 'DD'-anomaly for variable 'rValue' (lines '320'-'335').320 - 335
Found 'DU'-anomaly for variable 'ip6' (lines '324'-'378').324 - 378
Avoid empty catch blocks341 - 343
Avoid empty catch blocks343 - 345
Found 'DD'-anomaly for variable 'rValue' (lines '354'-'360').354 - 360
Found 'DU'-anomaly for variable 'delimeters' (lines '391'-'408').391 - 408
Found 'DD'-anomaly for variable 'data' (lines '464'-'472').464 - 472
Avoid empty catch blocks473 - 475

org/apache/james/jspf/parser/DefaultSPF1Parser.java

ViolationLine
Found 'DU'-anomaly for variable 'first' (lines '235'-'248').235 - 248
Found 'DU'-anomaly for variable 'first' (lines '239'-'248').239 - 248
Found 'UR'-anomaly for variable 'terms' (lines '253'-'253').253
Found 'UR'-anomaly for variable 'modifierString' (lines '253'-'253').253
Found 'UR'-anomaly for variable 'i' (lines '253'-'253').253
A method should have only one exit point, and that should be the last statement in the method339

org/apache/james/jspf/parser/DefaultTermDefinition.java

ViolationLine
Found 'DU'-anomaly for variable 'pString' (lines '52'-'65').52 - 65
Found 'DU'-anomaly for variable 'i' (lines '53'-'65').53 - 65
Found 'DD'-anomaly for variable 'c' (lines '54'-'61').54 - 61
Found 'DU'-anomaly for variable 'p2' (lines '57'-'65').57 - 65
Found 'DD'-anomaly for variable 'c' (lines '61'-'61').61
Found 'DU'-anomaly for variable 'i' (lines '62'-'65').62 - 65

org/apache/james/jspf/parser/DefaultTermsFactory.java

ViolationLine
Perhaps 'termFile' could be replaced by a local variable.42
Found 'DD'-anomaly for variable 'knownMechanisms' (lines '78'-'82').78 - 82
Found 'DD'-anomaly for variable 'knownMechanisms' (lines '82'-'82').82
Found 'DD'-anomaly for variable 'knownModifiers' (lines '87'-'91').87 - 91
Found 'DD'-anomaly for variable 'knownModifiers' (lines '91'-'91').91
Found 'DU'-anomaly for variable 'subres' (lines '133'-'156').133 - 156

org/apache/james/jspf/parser/TermDefinition.java

ViolationLine
Avoid modifiers which are implied by the context35
Avoid modifiers which are implied by the context42
Avoid modifiers which are implied by the context50

org/apache/james/jspf/parser/TermsFactory.java

ViolationLine
Avoid modifiers which are implied by the context42 - 43
Found 'DU'-anomaly for variable 'klass' (lines '42'-'43').42 - 43
Found 'DU'-anomaly for variable 'subres' (lines '42'-'43').42 - 43
Avoid modifiers which are implied by the context50
Avoid modifiers which are implied by the context57

org/apache/james/jspf/policies/ChainPolicy.java

ViolationLine
Found 'DU'-anomaly for variable 'currentDomain' (lines '52'-'67').52 - 67

org/apache/james/jspf/policies/InitialChecksPolicy.java

ViolationLine
Each class should declare at least one constructor33 - 50

org/apache/james/jspf/policies/local/BestGuessPolicy.java

ViolationLine
Each class should declare at least one constructor30 - 45
Found 'DU'-anomaly for variable 'currentDomain' (lines '35'-'44').35 - 44
A method should have only one exit point, and that should be the last statement in the method40

org/apache/james/jspf/policies/local/DefaultExplanationPolicy.java

ViolationLine
Found 'DU'-anomaly for variable 'currentDomain' (lines '63'-'98').63 - 98
Found 'DU'-anomaly for variable 'currentDomain' (lines '63'-'98').63 - 98
Found 'DU'-anomaly for variable 'currentDomain' (lines '63'-'98').63 - 98
A method should have only one exit point, and that should be the last statement in the method64
These nested if statements could be combined70 - 85
Found 'DD'-anomaly for variable 'explanation' (lines '73'-'78').73 - 78
Found 'DD'-anomaly for variable 'explanation' (lines '73'-'78').73 - 78
Found 'DD'-anomaly for variable 'explanation' (lines '73'-'78').73 - 78
Found 'DD'-anomaly for variable 'explanation' (lines '75'-'78').75 - 78
Found 'DD'-anomaly for variable 'explanation' (lines '75'-'78').75 - 78
Found 'DD'-anomaly for variable 'explanation' (lines '75'-'78').75 - 78

org/apache/james/jspf/policies/local/FallbackPolicy.java

ViolationLine
Found 'DU'-anomaly for variable 'currentDomain' (lines '118'-'124').118 - 124
A method should have only one exit point, and that should be the last statement in the method120
Found 'DD'-anomaly for variable 'entry' (lines '135'-'138').135 - 138
A method should have only one exit point, and that should be the last statement in the method142
Found 'DU'-anomaly for variable 'host' (lines '157'-'169').157 - 169
A method should have only one exit point, and that should be the last statement in the method165

org/apache/james/jspf/policies/local/TrustedForwarderPolicy.java

ViolationLine
Found 'DU'-anomaly for variable 'currentDomain' (lines '52'-'75').52 - 75
Found 'DU'-anomaly for variable 'currentDomain' (lines '52'-'75').52 - 75
A method should have only one exit point, and that should be the last statement in the method53
Found 'DU'-anomaly for variable 'trusted' (lines '58'-'75').58 - 75
Found 'DU'-anomaly for variable 'trusted' (lines '58'-'75').58 - 75
Avoid empty catch blocks70 - 72

org/apache/james/jspf/policies/NeutralIfNotMatchPolicy.java

ViolationLine
Each class should declare at least one constructor34 - 59
Found 'DU'-anomaly for variable 'currentDomain' (lines '39'-'58').39 - 58
Found 'DU'-anomaly for variable 'currentDomain' (lines '39'-'58').39 - 58
Found 'DU'-anomaly for variable 'currentDomain' (lines '39'-'58').39 - 58
A method should have only one exit point, and that should be the last statement in the method40

org/apache/james/jspf/policies/NoSPFRecordFoundPolicy.java

ViolationLine
Each class should declare at least one constructor31 - 42

org/apache/james/jspf/policies/ParseRecordPolicy.java

ViolationLine
Found 'DU'-anomaly for variable 'currentDomain' (lines '42'-'50').42 - 50
A method should have only one exit point, and that should be the last statement in the method43
A method should have only one exit point, and that should be the last statement in the method46

org/apache/james/jspf/policies/Policy.java

ViolationLine
Avoid modifiers which are implied by the context44 - 46
Found 'DU'-anomaly for variable 'currentDomain' (lines '44'-'46').44 - 46

org/apache/james/jspf/policies/PolicyPostFilter.java

ViolationLine
Avoid modifiers which are implied by the context45 - 47
Found 'DU'-anomaly for variable 'record' (lines '45'-'47').45 - 47
Found 'DU'-anomaly for variable 'currentDomain' (lines '45'-'47').45 - 47

org/apache/james/jspf/policies/SPFRetriever.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method41
A method should have only one exit point, and that should be the last statement in the method76

org/apache/james/jspf/policies/SPFStrictCheckerRetriever.java

ViolationLine
Found 'DD'-anomaly for variable 'spfR1' (lines '40'-'48').40 - 48
Found 'DD'-anomaly for variable 'spfR2' (lines '41'-'49').41 - 49
A method should have only one exit point, and that should be the last statement in the method52
A method should have only one exit point, and that should be the last statement in the method54
A method should have only one exit point, and that should be the last statement in the method59

org/apache/james/jspf/SPF.java

ViolationLine
Use explicit scoping instead of the default package private level65
Use explicit scoping instead of the default package private level69
Use explicit scoping instead of the default package private level71
Found 'DD'-anomaly for variable 'spfData' (lines '133'-'139').133 - 139
Found 'DD'-anomaly for variable 'result' (lines '134'-'142').134 - 142
Found 'DD'-anomaly for variable 'explanation' (lines '135'-'143').135 - 143
Found 'DD'-anomaly for variable 'result' (lines '142'-'145').142 - 145
Found 'DU'-anomaly for variable 'spfData' (lines '185'-'193').185 - 193

org/apache/james/jspf/SPF1Utils.java

ViolationLine
Each class should declare at least one constructor32 - 89
A method should have only one exit point, and that should be the last statement in the method61
A method should have only one exit point, and that should be the last statement in the method63
A method should have only one exit point, and that should be the last statement in the method65
A method should have only one exit point, and that should be the last statement in the method67
A method should have only one exit point, and that should be the last statement in the method83

org/apache/james/jspf/SPFResult.java

ViolationLine
Found 'DU'-anomaly for variable 'spfData' (lines '84'-'126').84 - 126

org/apache/james/jspf/terms/AllMechanism.java

ViolationLine
Each class should declare at least one constructor33 - 59
Found 'DU'-anomaly for variable 'spfData' (lines '40'-'42').40 - 42
Found 'DU'-anomaly for variable 'params' (lines '47'-'50').47 - 50

org/apache/james/jspf/terms/AMechanism.java

ViolationLine
Each class should declare at least one constructor40 - 258
Found 'DU'-anomaly for variable 'checkAddress' (lines '71'-'109').71 - 109
A method should have only one exit point, and that should be the last statement in the method78
A method should have only one exit point, and that should be the last statement in the method82
Found 'DU'-anomaly for variable 'checkAddress' (lines '85'-'109').85 - 109
A method should have only one exit point, and that should be the last statement in the method92
A method should have only one exit point, and that should be the last statement in the method96
Found 'DU'-anomaly for variable 'cidr' (lines '144'-'161').144 - 161
Found 'DU'-anomaly for variable 'checkAddress' (lines '144'-'161').144 - 161
A method should have only one exit point, and that should be the last statement in the method156
Found 'DU'-anomaly for variable 'dns' (lines '206'-'221').206 - 221
Found 'DU'-anomaly for variable 'dns' (lines '236'-'249').236 - 249

org/apache/james/jspf/terms/ExistsMechanism.java

ViolationLine
Each class should declare at least one constructor36 - 88
Found 'DU'-anomaly for variable 'aRecords' (lines '60'-'71').60 - 71
A method should have only one exit point, and that should be the last statement in the method62
A method should have only one exit point, and that should be the last statement in the method66

org/apache/james/jspf/terms/ExpModifier.java

ViolationLine
Each class should declare at least one constructor39 - 162
Found 'DU'-anomaly for variable 'spfData' (lines '63'-'101').63 - 101
Found 'DU'-anomaly for variable 'exp' (lines '64'-'101').64 - 101
Found 'DD'-anomaly for variable 'exp' (lines '64'-'85').64 - 85
A method should have only one exit point, and that should be the last statement in the method69
A method should have only one exit point, and that should be the last statement in the method74
A method should have only one exit point, and that should be the last statement in the method79
Found 'DU'-anomaly for variable 'exp' (lines '85'-'101').85 - 101
A method should have only one exit point, and that should be the last statement in the method88
A method should have only one exit point, and that should be the last statement in the method98
Avoid unnecessary return statements100
A method should have only one exit point, and that should be the last statement in the method121

org/apache/james/jspf/terms/GenericMechanism.java

ViolationLine
Each class should declare at least one constructor37 - 110
Assigning an Object to null is a code smell. Consider refactoring.86

org/apache/james/jspf/terms/GenericModifier.java

ViolationLine
Each class should declare at least one constructor36 - 90
Found 'DU'-anomaly for variable 'spfData' (lines '54'-'55').54 - 55

org/apache/james/jspf/terms/IncludeMechanism.java

ViolationLine
Each class should declare at least one constructor44 - 163
Found 'DD'-anomaly for variable 'host' (lines '72'-'78').72 - 78
Found 'DU'-anomaly for variable 'prevRes' (lines '80'-'117').80 - 117
Found 'DU'-anomaly for variable 'prevHost' (lines '81'-'117').81 - 117
A method should have only one exit point, and that should be the last statement in the method104

org/apache/james/jspf/terms/IP4Mechanism.java

ViolationLine
Each class should declare at least one constructor33 - 117
A method should have only one exit point, and that should be the last statement in the method54
Found 'UR'-anomaly for variable 'ipString' (lines '64'-'64').64
Found 'UR'-anomaly for variable 'maskLengthString' (lines '64'-'64').64
Found 'DD'-anomaly for variable 'maskLength' (lines '72'-'75').72 - 75
A method should have only one exit point, and that should be the last statement in the method112

org/apache/james/jspf/terms/IP6Mechanism.java

ViolationLine
Each class should declare at least one constructor29 - 63
A method should have only one exit point, and that should be the last statement in the method57

org/apache/james/jspf/terms/MXMechanism.java

ViolationLine
Each class should declare at least one constructor38 - 126
Found 'DU'-anomaly for variable 'checkAddress' (lines '66'-'79').66 - 79
A method should have only one exit point, and that should be the last statement in the method71
A method should have only one exit point, and that should be the last statement in the method74
Found 'DU'-anomaly for variable 'type' (lines '87'-'117').87 - 117
Found 'DU'-anomaly for variable 'mxR' (lines '90'-'117').90 - 117
A method should have only one exit point, and that should be the last statement in the method94
Assigning an Object to null is a code smell. Consider refactoring.113

org/apache/james/jspf/terms/PTRMechanism.java

ViolationLine
Each class should declare at least one constructor38 - 125
Found 'DU'-anomaly for variable 'validatedHosts' (lines '54'-'115').54 - 115
Found 'DU'-anomaly for variable 'host' (lines '60'-'115').60 - 115
A method should have only one exit point, and that should be the last statement in the method68
Found 'DD'-anomaly for variable 'aList' (lines '79'-'84').79 - 84
Found 'DD'-anomaly for variable 'aList' (lines '79'-'88').79 - 88
A method should have only one exit point, and that should be the last statement in the method106

org/apache/james/jspf/terms/RedirectModifier.java

ViolationLine
Each class should declare at least one constructor38 - 115
Found 'DD'-anomaly for variable 'host' (lines '67'-'73').67 - 73

org/apache/james/jspf/terms/UnknownModifier.java

ViolationLine
Each class should declare at least one constructor34 - 70
Found 'DU'-anomaly for variable 'spfData' (lines '47'-'49').47 - 49
Avoid unnecessary return statements48

org/apache/james/jspf/util/Inet6Util.java

ViolationLine
Each class should declare at least one constructor32 - 428
Found 'DU'-anomaly for variable 'tokenizer' (lines '42'-'155').42 - 155
Found 'DD'-anomaly for variable 'token' (lines '44'-'48').44 - 48
Found 'DU'-anomaly for variable 'token' (lines '44'-'155').44 - 155
Found 'DD'-anomaly for variable 'tempInt' (lines '45'-'49').45 - 49
Found 'DU'-anomaly for variable 'tempInt' (lines '45'-'155').45 - 155
Found 'DD'-anomaly for variable 'byteAddress' (lines '46'-'50').46 - 50
Found 'DD'-anomaly for variable 'byteAddress' (lines '50'-'50').50
A method should have only one exit point, and that should be the last statement in the method53
Found 'DD'-anomaly for variable 'prevToken' (lines '66'-'76').66 - 76
Found 'DD'-anomaly for variable 'doubleColonIndex' (lines '67'-'81').67 - 81
Found 'DD'-anomaly for variable 'doubleColonIndex' (lines '81'-'81').81
Found 'DD'-anomaly for variable 'doubleColonIndex' (lines '81'-'92').81 - 92
Found 'DU'-anomaly for variable 'hexStringsLength' (lines '102'-'155').102 - 155
Found 'DD'-anomaly for variable 'ipByteArray' (lines '118'-'127').118 - 127
Found 'DD'-anomaly for variable 'ipByteArray' (lines '127'-'127').127
Found 'DD'-anomaly for variable 'ipV4' (lines '133'-'136').133 - 136
Found 'DD'-anomaly for variable 'ipV4' (lines '133'-'142').133 - 142
Found 'DD'-anomaly for variable 'ipV4' (lines '136'-'142').136 - 142
Found 'DD'-anomaly for variable 'ipv4ByteArray' (lines '146'-'148').146 - 148
Found 'DD'-anomaly for variable 'ipv4ByteArray' (lines '148'-'148').148
A method should have only one exit point, and that should be the last statement in the method150
Found 'DD'-anomaly for variable 'ipByteArray' (lines '158'-'163').158 - 163
Found 'DD'-anomaly for variable 'ipByteArray' (lines '163'-'164').163 - 164
Found 'DD'-anomaly for variable 'ipByteArray' (lines '164'-'170').164 - 170
Found 'DD'-anomaly for variable 'ipByteArray' (lines '164'-'176').164 - 176
Found 'DD'-anomaly for variable 'ipByteArray' (lines '164'-'182').164 - 182
Found 'DD'-anomaly for variable 'ipByteArray' (lines '164'-'187').164 - 187
Use explicit scoping instead of the default package private level190 - 231
A method should have only one exit point, and that should be the last statement in the method194
A method should have only one exit point, and that should be the last statement in the method196
A method should have only one exit point, and that should be the last statement in the method198
A method should have only one exit point, and that should be the last statement in the method200
A method should have only one exit point, and that should be the last statement in the method202
A method should have only one exit point, and that should be the last statement in the method204
A method should have only one exit point, and that should be the last statement in the method206
A method should have only one exit point, and that should be the last statement in the method208
A method should have only one exit point, and that should be the last statement in the method210
A method should have only one exit point, and that should be the last statement in the method212
A method should have only one exit point, and that should be the last statement in the method218
A method should have only one exit point, and that should be the last statement in the method220
A method should have only one exit point, and that should be the last statement in the method222
A method should have only one exit point, and that should be the last statement in the method224
A method should have only one exit point, and that should be the last statement in the method226
A method should have only one exit point, and that should be the last statement in the method228
Found 'DU'-anomaly for variable 'doubleColon' (lines '235'-'363').235 - 363
Found 'DU'-anomaly for variable 'numberOfColons' (lines '236'-'363').236 - 363
Found 'DD'-anomaly for variable 'numberOfColons' (lines '236'-'296').236 - 296
Found 'DU'-anomaly for variable 'numberOfPeriods' (lines '237'-'363').237 - 363
Found 'DD'-anomaly for variable 'numberOfPeriods' (lines '237'-'273').237 - 273
Found 'DU'-anomaly for variable 'numberOfPercent' (lines '238'-'363').238 - 363
Found 'DD'-anomaly for variable 'numberOfPercent' (lines '238'-'311').238 - 311
Found 'DU'-anomaly for variable 'word' (lines '239'-'363').239 - 363
Found 'DD'-anomaly for variable 'word' (lines '239'-'306').239 - 306
Found 'DU'-anomaly for variable 'c' (lines '240'-'363').240 - 363
Found 'DU'-anomaly for variable 'prevChar' (lines '241'-'363').241 - 363
Found 'DD'-anomaly for variable 'prevChar' (lines '241'-'248').241 - 248
Found 'DU'-anomaly for variable 'offset' (lines '242'-'363').242 - 363
Found 'DD'-anomaly for variable 'offset' (lines '242'-'258').242 - 258
A method should have only one exit point, and that should be the last statement in the method245
Found 'DU'-anomaly for variable 'prevChar' (lines '248'-'363').248 - 363
Found 'DD'-anomaly for variable 'prevChar' (lines '248'-'248').248
A method should have only one exit point, and that should be the last statement in the method255
A method should have only one exit point, and that should be the last statement in the method257
Found 'DU'-anomaly for variable 'offset' (lines '258'-'363').258 - 363
A method should have only one exit point, and that should be the last statement in the method260
A method should have only one exit point, and that should be the last statement in the method266
A method should have only one exit point, and that should be the last statement in the method268
A method should have only one exit point, and that should be the last statement in the method275
A method should have only one exit point, and that should be the last statement in the method277
A method should have only one exit point, and that should be the last statement in the method279
A method should have only one exit point, and that should be the last statement in the method284
Found 'DU'-anomaly for variable 'word' (lines '285'-'363').285 - 363
A method should have only one exit point, and that should be the last statement in the method293
A method should have only one exit point, and that should be the last statement in the method298
A method should have only one exit point, and that should be the last statement in the method300
A method should have only one exit point, and that should be the last statement in the method303
Found 'DU'-anomaly for variable 'doubleColon' (lines '304'-'363').304 - 363
Found 'DU'-anomaly for variable 'word' (lines '306'-'363').306 - 363
A method should have only one exit point, and that should be the last statement in the method310
Found 'DU'-anomaly for variable 'numberOfPercent' (lines '311'-'363').311 - 363
A method should have only one exit point, and that should be the last statement in the method317
A method should have only one exit point, and that should be the last statement in the method325
A method should have only one exit point, and that should be the last statement in the method332
A method should have only one exit point, and that should be the last statement in the method334
A method should have only one exit point, and that should be the last statement in the method343
A method should have only one exit point, and that should be the last statement in the method348
These nested if statements could be combined355 - 358
A method should have only one exit point, and that should be the last statement in the method357
A method should have only one exit point, and that should be the last statement in the method368
A method should have only one exit point, and that should be the last statement in the method372
A method should have only one exit point, and that should be the last statement in the method375
Use explicit scoping instead of the default package private level379 - 383
Found 'DU'-anomaly for variable 'periods' (lines '393'-'426').393 - 426
Found 'DD'-anomaly for variable 'periods' (lines '393'-'404').393 - 404
Found 'DU'-anomaly for variable 'i' (lines '394'-'426').394 - 426
Found 'DD'-anomaly for variable 'i' (lines '394'-'401').394 - 401
A method should have only one exit point, and that should be the last statement in the method398
Found 'DD'-anomaly for variable 'c' (lines '399'-'402').399 - 402
Found 'DU'-anomaly for variable 'c' (lines '399'-'426').399 - 426
Found 'DU'-anomaly for variable 'word' (lines '400'-'426').400 - 426
A method should have only one exit point, and that should be the last statement in the method406
A method should have only one exit point, and that should be the last statement in the method408
A method should have only one exit point, and that should be the last statement in the method410
Found 'DU'-anomaly for variable 'word' (lines '411'-'426').411 - 426
A method should have only one exit point, and that should be the last statement in the method413
A method should have only one exit point, and that should be the last statement in the method416
A method should have only one exit point, and that should be the last statement in the method422
A method should have only one exit point, and that should be the last statement in the method424

org/apache/james/jspf/util/SPFTermsRegexps.java

ViolationLine
Avoid modifiers which are implied by the context28
Avoid modifiers which are implied by the context30
Avoid modifiers which are implied by the context32
Avoid modifiers which are implied by the context34
Avoid modifiers which are implied by the context36
Avoid modifiers which are implied by the context38
Avoid modifiers which are implied by the context40 - 41
Avoid modifiers which are implied by the context43
Avoid modifiers which are implied by the context48 - 49
Avoid modifiers which are implied by the context55
Avoid modifiers which are implied by the context57
Avoid modifiers which are implied by the context64 - 69
Avoid modifiers which are implied by the context74 - 75
Avoid modifiers which are implied by the context80 - 81
Avoid modifiers which are implied by the context86 - 87

org/apache/james/jspf/wiring/DNSServiceEnabled.java

ViolationLine
Found 'DU'-anomaly for variable 'service' (lines '35'-'35').35

org/apache/james/jspf/wiring/LogEnabled.java

ViolationLine
Found 'DU'-anomaly for variable 'logger' (lines '35'-'35').35

org/apache/james/jspf/wiring/MacroExpandEnabled.java

ViolationLine
Found 'DU'-anomaly for variable 'macroExpand' (lines '35'-'35').35

org/apache/james/jspf/wiring/SPFCheckEnabled.java

ViolationLine
Found 'DU'-anomaly for variable 'checker' (lines '35'-'35').35

org/apache/james/jspf/wiring/WiringService.java

ViolationLine
Avoid modifiers which are implied by the context28 - 34
Found 'DU'-anomaly for variable 'component' (lines '42'-'42').42

org/apache/james/jspf/wiring/WiringServiceTable.java

ViolationLine
Each class should declare at least one constructor28 - 52
Found 'UR'-anomaly for variable 'enablingClass' (lines '30'-'30').30
Found 'DU'-anomaly for variable 'component' (lines '30'-'50').30 - 50