PMD Results

The following document contains the results of PMD 3.9.

Files

org/apache/james/Constants.java

ViolationLine
Each class should declare at least one constructor30 - 85

org/apache/james/James.java

ViolationLine
Each class should declare at least one constructor89 - 885
Perhaps 'context' could be replaced by a local variable.107
Perhaps 'conf' could be replaced by a local variable.112
Perhaps 'mailetLogger' could be replaced by a local variable.117
Perhaps 'usersStore' could be replaced by a local variable.127
Found 'UR'-anomaly for variable 'conf' (lines '215'-'265').215 - 265
Found 'DD'-anomaly for variable 'hostName' (lines '252'-'254').252 - 254
Found 'DD'-anomaly for variable 'hostName' (lines '254'-'256').254 - 256
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.343
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.344
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.345
Found 'DD'-anomaly for variable 'userInbox' (lines '523'-'525').523 - 525
A method should have only one exit point, and that should be the last statement in the method528
Found 'DD'-anomaly for variable 'localCount' (lines '570'-'572').570 - 572
Found 'DU'-anomaly for variable 'args' (lines '589'-'594').589 - 594
Found 'DD'-anomaly for variable 'dnsServer' (lines '606'-'608').606 - 608
Found 'DU'-anomaly for variable 'bouncer' (lines '671'-'690').671 - 690
Found 'DU'-anomaly for variable 'message' (lines '671'-'690').671 - 690
A method should have only one exit point, and that should be the last statement in the method730
Found 'DD'-anomaly for variable 'dnsServer' (lines '853'-'855').853 - 855

org/apache/james/JamesMBean.java

ViolationLine
Found 'DU'-anomaly for variable 'password' (lines '39'-'39').39
Found 'DU'-anomaly for variable 'userName' (lines '39'-'39').39

org/apache/james/Main.java

ViolationLine
Each class should declare at least one constructor29 - 44
Found 'DU'-anomaly for variable 'args' (lines '37'-'43').37 - 43

org/apache/james/context/AvalonContextUtilities.java

ViolationLine
Found 'DD'-anomaly for variable 'baseDirectory' (lines '76'-'80').76 - 80

org/apache/james/core/AbstractJamesService.java

ViolationLine
Each class should declare at least one constructor54 - 541
Perhaps 'm_disposed' could be replaced by a local variable.161
Found 'DD'-anomaly for variable 'confSocketType' (lines '197'-'199').197 - 199
Found 'DD'-anomaly for variable 'hostName' (lines '245'-'247').245 - 247
Found 'DD'-anomaly for variable 'hostName' (lines '247'-'249').247 - 249
A method should have only one exit point, and that should be the last statement in the method387
Assigning an Object to null is a code smell. Consider refactoring.424
Assigning an Object to null is a code smell. Consider refactoring.426
Assigning an Object to null is a code smell. Consider refactoring.427
Found 'DD'-anomaly for variable 'theWatchdogFactory' (lines '444'-'445').444 - 445
A method should have only one exit point, and that should be the last statement in the method527

org/apache/james/core/AvalonMailStore.java

ViolationLine
Each class should declare at least one constructor49 - 398
Found 'DU'-anomaly for variable 'className' (lines '147'-'184').147 - 184
Found 'DU'-anomaly for variable 'infoEnabled' (lines '148'-'184').148 - 184
Found 'DU'-anomaly for variable 'types' (lines '151'-'184').151 - 184
Found 'DU'-anomaly for variable 'protocol' (lines '154'-'184').154 - 184
Found 'DD'-anomaly for variable 'protocol' (lines '154'-'154').154
Found 'DU'-anomaly for variable 'defConf' (lines '157'-'184').157 - 184
Found 'DD'-anomaly for variable 'defConf' (lines '157'-'157').157
Found 'UR'-anomaly for variable 'error' (lines '204'-'204').204
Found 'UR'-anomaly for variable 'destination' (lines '204'-'204').204
Found 'DD'-anomaly for variable 'repConf' (lines '205'-'207').205 - 207
Found 'DD'-anomaly for variable 'destination' (lines '212'-'215').212 - 215
Found 'DD'-anomaly for variable 'protocol' (lines '213'-'221').213 - 221
Found 'DU'-anomaly for variable 'protocol' (lines '221'-'317').221 - 317
Found 'DD'-anomaly for variable 'logBuffer' (lines '232'-'235').232 - 235
Found 'DU'-anomaly for variable 'logBuffer' (lines '232'-'317').232 - 317
Found 'DD'-anomaly for variable 'logBuffer' (lines '232'-'249').232 - 249
Found 'DD'-anomaly for variable 'logBuffer' (lines '232'-'295').232 - 295
A method should have only one exit point, and that should be the last statement in the method243
Found 'DD'-anomaly for variable 'comp' (lines '342'-'344').342 - 344
Found 'DU'-anomaly for variable 'toConfig' (lines '360'-'389').360 - 389
Found 'DU'-anomaly for variable 'component' (lines '397'-'397').397

org/apache/james/core/AvalonUsersStore.java

ViolationLine
Each class should declare at least one constructor45 - 173
Found 'UR'-anomaly for variable 'error' (lines '97'-'97').97
Found 'DU'-anomaly for variable 'theClassLoader' (lines '104'-'144').104 - 144

org/apache/james/core/LocalUsersRepository.java

ViolationLine
Each class should declare at least one constructor32 - 101

org/apache/james/core/MailHeaders.java

ViolationLine
Overriding method merely calls super134 - 137

org/apache/james/core/MailImpl.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.127
Found 'UR'-anomaly for variable 'returnPathHeader' (lines '234'-'234').234
Found 'DD'-anomaly for variable 'mailAddress' (lines '235'-'244').235 - 244
Found 'DD'-anomaly for variable 'returnPathHeader' (lines '237'-'239').237 - 239
Found 'DU'-anomaly for variable 'returnPathHeader' (lines '237'-'252').237 - 252
A method should have only one exit point, and that should be the last statement in the method271
Avoid empty catch blocks272 - 274
Found 'DU'-anomaly for variable 'out' (lines '460'-'466').460 - 466
Assigning an Object to null is a code smell. Consider refactoring.485
Assigning an Object to null is a code smell. Consider refactoring.537

org/apache/james/core/MailetConfigImpl.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you57 - 58

org/apache/james/core/MatcherConfigImpl.java

ViolationLine
Each class should declare at least one constructor29 - 103

org/apache/james/core/MimeMessageCopyOnWriteProxy.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.79
Assigning an Object to null is a code smell. Consider refactoring.168

org/apache/james/core/MimeMessageInputStreamSource.java

ViolationLine
Use explicit scoping instead of the default package private level51
Use explicit scoping instead of the default package private level56
It is a good practice to call super() in a constructor69 - 119
Avoid assignments in operands78
Assigning an Object to null is a code smell. Consider refactoring.91
Avoid empty catch blocks92 - 94
Assigning an Object to null is a code smell. Consider refactoring.98
Avoid empty catch blocks107 - 109
Avoid empty catch blocks115 - 117
Avoid empty catch blocks158 - 160
Assigning an Object to null is a code smell. Consider refactoring.161

org/apache/james/core/MimeMessageSource.java

ViolationLine
Each class should declare at least one constructor32 - 82
Found 'DD'-anomaly for variable 'in' (lines '61'-'63').61 - 63
Found 'DD'-anomaly for variable 'read' (lines '64'-'66').64 - 66
Avoid assignments in operands66
Found 'DU'-anomaly for variable 'read' (lines '66'-'80').66 - 80
Avoid empty catch blocks74 - 77

org/apache/james/core/MimeMessageUtil.java

ViolationLine
Each class should declare at least one constructor43 - 304
Found 'DD'-anomaly for variable 'testMessage' (lines '58'-'61').58 - 61
Found 'DD'-anomaly for variable 'bos' (lines '102'-'127').102 - 127
Found 'DD'-anomaly for variable 'bis' (lines '103'-'126').103 - 126
Found 'DD'-anomaly for variable 'bis' (lines '126'-'144').126 - 144
Found 'DD'-anomaly for variable 'bos' (lines '127'-'145').127 - 145
Found 'DD'-anomaly for variable 'read' (lines '166'-'167').166 - 167
Avoid assignments in operands167
Found 'DU'-anomaly for variable 'read' (lines '167'-'171').167 - 171
Each class should declare at least one constructor222 - 240
Use explicit scoping instead of the default package private level223
Found 'DU'-anomaly for variable 'arg0' (lines '225'-'227').225 - 227
Found 'DU'-anomaly for variable 'arg1' (lines '233'-'235').233 - 235
Found 'DU'-anomaly for variable 'arg0' (lines '233'-'235').233 - 235
Found 'DD'-anomaly for variable 'size' (lines '250'-'254').250 - 254
Found 'DD'-anomaly for variable 'size' (lines '250'-'257').250 - 257

org/apache/james/core/MimeMessageWrapper.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.82
Assigning an Object to null is a code smell. Consider refactoring.201
Assigning an Object to null is a code smell. Consider refactoring.211
Found 'DD'-anomaly for variable 'in' (lines '303'-'305').303 - 305
Found 'DU'-anomaly for variable 'in' (lines '305'-'332').305 - 332
A method should have only one exit point, and that should be the last statement in the method307
A method should have only one exit point, and that should be the last statement in the method310
Avoid empty while statements323 - 325
A method should have only one exit point, and that should be the last statement in the method326
A method should have only one exit point, and that should be the last statement in the method340
Found 'DU'-anomaly for variable 'newHeaders' (lines '511'-'518').511 - 518
A method should have only one exit point, and that should be the last statement in the method514
A method should have only one exit point, and that should be the last statement in the method540

org/apache/james/dnsserver/DNSServer.java

ViolationLine
Each class should declare at least one constructor61 - 538
Perhaps 'mxComparator' could be replaced by a local variable.97
A method should have only one exit point, and that should be the last statement in the method205
Found 'DD'-anomaly for variable 'mxAnswers' (lines '208'-'210').208 - 210
Found 'DD'-anomaly for variable 'mxAnswers' (lines '210'-'210').210
Found 'DD'-anomaly for variable 'servers' (lines '235'-'237').235 - 237
Found 'DU'-anomaly for variable 'querysent' (lines '301'-'358').301 - 358
Found 'DU'-anomaly for variable 'type' (lines '301'-'358').301 - 358
Found 'DD'-anomaly for variable 'name' (lines '302'-'304').302 - 304
Found 'DU'-anomaly for variable 'name' (lines '304'-'358').304 - 358
A method should have only one exit point, and that should be the last statement in the method308
A method should have only one exit point, and that should be the last statement in the method319
A method should have only one exit point, and that should be the last statement in the method322
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 method341
A method should have only one exit point, and that should be the last statement in the method348
A method should have only one exit point, and that should be the last statement in the method353
Found 'DD'-anomaly for variable 'answerCount' (lines '362'-'365').362 - 365
Found 'DU'-anomaly for variable 'n' (lines '362'-'380').362 - 380
Found 'DD'-anomaly for variable 'answers' (lines '370'-'376').370 - 376
Found 'DD'-anomaly for variable 'answers' (lines '376'-'376').376
Each class should declare at least one constructor399 - 406
Found 'DU'-anomaly for variable 'domainName' (lines '425'-'483').425 - 483
Found 'DU'-anomaly for variable 'domainName' (lines '425'-'483').425 - 483
Found 'DU'-anomaly for variable 'domainName' (lines '425'-'483').425 - 483
Found 'DU'-anomaly for variable 'domainName' (lines '425'-'483').425 - 483
Found 'DU'-anomaly for variable 'domainName' (lines '425'-'483').425 - 483
Found 'DU'-anomaly for variable 'domainName' (lines '425'-'483').425 - 483
Found 'DU'-anomaly for variable 'domainName' (lines '425'-'483').425 - 483

org/apache/james/dnsserver/DNSServerMBean.java

ViolationLine
Avoid modifiers which are implied by the context32

org/apache/james/fetchmail/Account.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.199
Found 'DD'-anomaly for variable 'messageIDs' (lines '280'-'281').280 - 281
Avoid assignments in operands281 - 282
Found 'DU'-anomaly for variable 'messageIDs' (lines '281'-'288').281 - 288
A method should have only one exit point, and that should be the last statement in the method285

org/apache/james/fetchmail/FetchMail.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method115
A method should have only one exit point, and that should be the last statement in the method117
Avoid unnecessary constructors - the compiler will generate these for you404 - 407
Found 'UR'-anomaly for variable 'accountsChild' (lines '417'-'417').417
Found 'DU'-anomaly for variable 'arg0' (lines '488'-'571').488 - 571
Found 'UR'-anomaly for variable 'errorBuffer' (lines '585'-'585').585
Found 'DU'-anomaly for variable 'errorBuffer' (lines '593'-'603').593 - 603
A method should have only one exit point, and that should be the last statement in the method682
Found 'DU'-anomaly for variable 'oldAccounts' (lines '756'-'781').756 - 781
A method should have only one exit point, and that should be the last statement in the method792
Found 'DU'-anomaly for variable 'parameters' (lines '824'-'857').824 - 857
Found 'DU'-anomaly for variable 'oldAccounts' (lines '824'-'857').824 - 857
A method should have only one exit point, and that should be the last statement in the method876
Found 'DD'-anomaly for variable 'session' (lines '905'-'906').905 - 906
Avoid assignments in operands906
Found 'DU'-anomaly for variable 'session' (lines '906'-'912').906 - 912
A method should have only one exit point, and that should be the last statement in the method909
Found 'DU'-anomaly for variable 'properties' (lines '962'-'982').962 - 982

org/apache/james/fetchmail/FetchScheduler.java

ViolationLine
Each class should declare at least one constructor46 - 153

org/apache/james/fetchmail/FetchSchedulerMBean.java

ViolationLine
Avoid modifiers which are implied by the context35

org/apache/james/fetchmail/FolderProcessor.java

ViolationLine
Found 'DD'-anomaly for variable 'messagesProcessed' (lines '68'-'98').68 - 98
Found 'DD'-anomaly for variable 'messageCount' (lines '69'-'87').69 - 87
Found 'DD'-anomaly for variable 'messagesProcessed' (lines '98'-'98').98
Avoid empty catch blocks126 - 129
Avoid unnecessary return statements153
Found 'DD'-anomaly for variable 'openFlag' (lines '192'-'195').192 - 195
Found 'DD'-anomaly for variable 'isSeen' (lines '217'-'219').217 - 219
Found 'DD'-anomaly for variable 'isSeen' (lines '217'-'221').217 - 221
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.231 - 232
Found 'DD'-anomaly for variable 'markSeenPermanent' (lines '268'-'269').268 - 269
Avoid assignments in operands269
Found 'DU'-anomaly for variable 'markSeenPermanent' (lines '269'-'275').269 - 275
A method should have only one exit point, and that should be the last statement in the method272

org/apache/james/fetchmail/MessageProcessor.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method306
A method should have only one exit point, and that should be the last statement in the method324
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 method349
A method should have only one exit point, and that should be the last statement in the method355
A method should have only one exit point, and that should be the last statement in the method362
A method should have only one exit point, and that should be the last statement in the method369
Found 'DD'-anomaly for variable 'mail' (lines '380'-'383').380 - 383
Found 'DU'-anomaly for variable 'mail' (lines '383'-'408').383 - 408
A method should have only one exit point, and that should be the last statement in the method388
A method should have only one exit point, and that should be the last statement in the method393
Avoid unnecessary return statements431
Avoid unnecessary return statements454
Avoid unnecessary return statements481
Avoid unnecessary return statements505
Avoid unnecessary return statements529
Avoid unnecessary return statements551
Found 'DD'-anomaly for variable 'messageOut' (lines '567'-'569').567 - 569
Found 'DD'-anomaly for variable 'messageOut' (lines '567'-'571').567 - 571
Found 'DD'-anomaly for variable 'from' (lines '680'-'684').680 - 684
Found 'DD'-anomaly for variable 'internetAddress' (lines '681'-'703').681 - 703
Found 'DD'-anomaly for variable 'internetAddress' (lines '681'-'714').681 - 714
Found 'DD'-anomaly for variable 'from' (lines '684'-'688').684 - 688
Found 'DD'-anomaly for variable 'headers' (lines '750'-'753').750 - 753
Found 'DU'-anomaly for variable 'headerTokens' (lines '762'-'817').762 - 817
Avoid assignments in operands783 - 784
Found 'DU'-anomaly for variable 'ex' (lines '842'-'858').842 - 858
Found 'DU'-anomaly for variable 'ex' (lines '865'-'883').865 - 883
Found 'DD'-anomaly for variable 'count' (lines '936'-'941').936 - 941
Found 'DD'-anomaly for variable 'count' (lines '941'-'941').941
A method should have only one exit point, and that should be the last statement in the method997
Found 'DD'-anomaly for variable 'start' (lines '1,010'-'1,020').1010 - 1020
Found 'DD'-anomaly for variable 'end' (lines '1,011'-'1,021').1011 - 1021
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,012'-'1,016').1012 - 1016
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,016'-'1,029').1016 - 1029
Found 'DU'-anomaly for variable 'hasBracket' (lines '1,016'-'1,065').1016 - 1065
Found 'DD'-anomaly for variable 'end' (lines '1,021'-'1,035').1021 - 1035
Found 'DD'-anomaly for variable 'end' (lines '1,021'-'1,038').1021 - 1038
Found 'DU'-anomaly for variable 'end' (lines '1,021'-'1,065').1021 - 1065
Found 'DU'-anomaly for variable 'nextSearchAt' (lines '1,022'-'1,065').1022 - 1065
Found 'DD'-anomaly for variable 'nextSearchAt' (lines '1,022'-'1,008').1022 - 1008
Found 'DU'-anomaly for variable 'hasBracket' (lines '1,029'-'1,065').1029 - 1065
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,029'-'1,012').1029 - 1012
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,029'-'1,029').1029
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,029'-'1,016').1029 - 1016
Found 'DD'-anomaly for variable 'usableAddress' (lines '1,032'-'1,032').1032
A method should have only one exit point, and that should be the last statement in the method1055
A method should have only one exit point, and that should be the last statement in the method1084
A method should have only one exit point, and that should be the last statement in the method1098
A method should have only one exit point, and that should be the last statement in the method1115
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.1200
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.1202
Avoid unnecessary temporaries when converting primitives to Strings1338
Found 'DU'-anomaly for variable 'mail' (lines '1,365'-'1,377').1365 - 1377
Avoid assignments in operands1414
Found 'DU'-anomaly for variable 'remoteDomain' (lines '1,414'-'1,420').1414 - 1420
A method should have only one exit point, and that should be the last statement in the method1417
Found 'DD'-anomaly for variable 'address' (lines '1,459'-'1,466').1459 - 1466
Found 'DD'-anomaly for variable 'address' (lines '1,459'-'1,472').1459 - 1472
Found 'DD'-anomaly for variable 'validatedAddress' (lines '1,460'-'1,474').1460 - 1474
Found 'DD'-anomaly for variable 'ipAddressEnd' (lines '1,462'-'1,464').1462 - 1464
Avoid assignments in operands1505
Found 'DU'-anomaly for variable 'remoteAddress' (lines '1,505'-'1,511').1505 - 1511
A method should have only one exit point, and that should be the last statement in the method1508
Avoid assignments in operands1530
Found 'DU'-anomaly for variable 'remoteHostName' (lines '1,530'-'1,536').1530 - 1536
A method should have only one exit point, and that should be the last statement in the method1533
Found 'DD'-anomaly for variable 'isMaxMessageSizeExceeded' (lines '1,598'-'1,599').1598 - 1599
Avoid assignments in operands1599 - 1600
Found 'DU'-anomaly for variable 'isMaxMessageSizeExceeded' (lines '1,599'-'1,606').1599 - 1606
A method should have only one exit point, and that should be the last statement in the method1603
A method should have only one exit point, and that should be the last statement in the method1623
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.1624
Found 'DD'-anomaly for variable 'isInvalid' (lines '1,651'-'1,652').1651 - 1652
Avoid assignments in operands1652
Found 'DU'-anomaly for variable 'isInvalid' (lines '1,652'-'1,658').1652 - 1658
A method should have only one exit point, and that should be the last statement in the method1655
Found 'DD'-anomaly for variable 'isInvalid' (lines '1,667'-'1,674').1667 - 1674

org/apache/james/fetchmail/ParsedConfiguration.java

ViolationLine
Found 'UR'-anomaly for variable 'token' (lines '672'-'672').672
Found 'DD'-anomaly for variable 'token' (lines '677'-'682').677 - 682
Found 'DU'-anomaly for variable 'token' (lines '677'-'692').677 - 692
Found 'DD'-anomaly for variable 'defaultDomainName' (lines '808'-'809').808 - 809
Avoid assignments in operands809
Found 'DU'-anomaly for variable 'defaultDomainName' (lines '809'-'815').809 - 815
A method should have only one exit point, and that should be the last statement in the method812
Found 'DD'-anomaly for variable 'hostName' (lines '864'-'872').864 - 872
Found 'DD'-anomaly for variable 'hostName' (lines '872'-'876').872 - 876

org/apache/james/fetchmail/ProcessorAbstract.java

ViolationLine
Found 'DD'-anomaly for variable 'value' (lines '518'-'519').518 - 519
Avoid assignments in operands519
Found 'DU'-anomaly for variable 'value' (lines '519'-'525').519 - 525
A method should have only one exit point, and that should be the last statement in the method522

org/apache/james/fetchmail/ReaderInputStream.java

ViolationLine
It is a good practice to call super() in a constructor33 - 35

org/apache/james/fetchmail/StoreProcessor.java

ViolationLine
Found 'DD'-anomaly for variable 'store' (lines '53'-'69').53 - 69
Found 'DD'-anomaly for variable 'folder' (lines '54'-'80').54 - 80

org/apache/james/mailrepository/AvalonMailRepository.java

ViolationLine
Each class should declare at least one constructor62 - 442
Found 'UR'-anomaly for variable 'exceptionString' (lines '91'-'91').91
Found 'DU'-anomaly for variable 'exceptionString' (lines '98'-'108').98 - 108
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 method235
Found 'DD'-anomaly for variable 'wasLocked' (lines '251'-'253').251 - 253
Found 'DD'-anomaly for variable 'saveStream' (lines '264'-'295').264 - 295
Found 'DD'-anomaly for variable 'out' (lines '299'-'301').299 - 301
Found 'DD'-anomaly for variable 'mc' (lines '346'-'348').346 - 348
Found 'DU'-anomaly for variable 'mc' (lines '348'-'372').348 - 372
A method should have only one exit point, and that should be the last statement in the method362
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '402'-'402').402
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '412'-'419').412 - 419

org/apache/james/mailrepository/AvalonSpoolRepository.java

ViolationLine
Each class should declare at least one constructor39 - 183
Found 'DU'-anomaly for variable 'delay' (lines '78'-'118').78 - 118
Found 'DU'-anomaly for variable 'delay' (lines '78'-'118').78 - 118
Found 'DU'-anomaly for variable 'delay' (lines '78'-'118').78 - 118
Found 'DU'-anomaly for variable 'filter' (lines '133'-'181').133 - 181

org/apache/james/mailrepository/JDBCMailRepository.java

ViolationLine
Each class should declare at least one constructor91 - 1068
Found 'DD'-anomaly for variable 'logBuffer' (lines '184'-'186').184 - 186
Found 'DU'-anomaly for variable 'logBuffer' (lines '184'-'196').184 - 196
Found 'UR'-anomaly for variable 'message' (lines '201'-'201').201
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '201'-'201').201
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '229'-'303').229 - 303
Found 'DD'-anomaly for variable 'logBuffer' (lines '316'-'347').316 - 347
Found 'DD'-anomaly for variable 'logBuffer' (lines '316'-'381').316 - 381
Found 'DU'-anomaly for variable 'logBuffer' (lines '316'-'396').316 - 396
Found 'DD'-anomaly for variable 'logBuffer' (lines '316'-'347').316 - 347
Found 'DD'-anomaly for variable 'logBuffer' (lines '316'-'381').316 - 381
Found 'DU'-anomaly for variable 'logBuffer' (lines '316'-'396').316 - 396
Found 'DD'-anomaly for variable 'createStatement' (lines '332'-'376').332 - 376
Found 'DD'-anomaly for variable 'createStatement' (lines '332'-'376').332 - 376
Found 'DD'-anomaly for variable 'sqlFile' (lines '337'-'339').337 - 339
Found 'DD'-anomaly for variable 'sqlFile' (lines '337'-'339').337 - 339
Assigning an Object to null is a code smell. Consider refactoring.340
Found 'UR'-anomaly for variable 'logBuffer' (lines '406'-'406').406
Found 'DD'-anomaly for variable 'hasUpdateMessageAttributesSQL' (lines '408'-'425').408 - 425
Found 'DD'-anomaly for variable 'hasRetrieveMessageAttributesSQL' (lines '409'-'433').409 - 433
Found 'DU'-anomaly for variable 'logBuffer' (lines '413'-'484').413 - 484
A method should have only one exit point, and that should be the last statement in the method506
A method should have only one exit point, and that should be the last statement in the method532
Found 'DD'-anomaly for variable 'conn' (lines '543'-'555').543 - 555
Found 'DD'-anomaly for variable 'wasLocked' (lines '544'-'548').544 - 548
Found 'DD'-anomaly for variable 'checkMessageExists' (lines '562'-'566').562 - 566
Found 'DD'-anomaly for variable 'rsExists' (lines '563'-'570').563 - 570
Found 'DD'-anomaly for variable 'exists' (lines '564'-'571').564 - 571
Found 'DD'-anomaly for variable 'updateMessage' (lines '579'-'582').579 - 582
Assigning an Object to null is a code smell. Consider refactoring.608
Found 'DU'-anomaly for variable 'updateMessage' (lines '608'-'775').608 - 775
Found 'DD'-anomaly for variable 'updateMessageAttr' (lines '616'-'618').616 - 618
Found 'DD'-anomaly for variable 'saveBody' (lines '660'-'668').660 - 668
Found 'DD'-anomaly for variable 'saveBody' (lines '660'-'670').660 - 670
Found 'DD'-anomaly for variable 'conn' (lines '788'-'792').788 - 792
Found 'DD'-anomaly for variable 'retrieveMessage' (lines '789'-'797').789 - 797
Found 'DD'-anomaly for variable 'rsMessage' (lines '790'-'801').790 - 801
A method should have only one exit point, and that should be the last statement in the method815
Found 'DD'-anomaly for variable 'retrieveMessageAttr' (lines '818'-'825').818 - 825
Found 'DU'-anomaly for variable 'retrieveMessageAttr' (lines '818'-'926').818 - 926
Found 'DD'-anomaly for variable 'attributes' (lines '819'-'846').819 - 846
Found 'DD'-anomaly for variable 'rsMessageAttr' (lines '823'-'830').823 - 830
Found 'DD'-anomaly for variable 'serialized_attr' (lines '834'-'838').834 - 838
Found 'DD'-anomaly for variable 'serialized_attr' (lines '834'-'840').834 - 840
Found 'DD'-anomaly for variable 'conn' (lines '958'-'961').958 - 961
Found 'DD'-anomaly for variable 'removeMessage' (lines '959'-'962').959 - 962
Found 'DD'-anomaly for variable 'conn' (lines '988'-'992').988 - 992
Found 'DD'-anomaly for variable 'listMessages' (lines '989'-'993').989 - 993
Found 'DD'-anomaly for variable 'rsListMessages' (lines '990'-'996').990 - 996
A method should have only one exit point, and that should be the last statement in the method1027
Found 'DD'-anomaly for variable 'result' (lines '1,042'-'1,044').1042 - 1044
Found 'DD'-anomaly for variable 'result' (lines '1,042'-'1,047').1042 - 1047
Found 'DD'-anomaly for variable 'result' (lines '1,044'-'1,047').1044 - 1047

org/apache/james/mailrepository/JDBCSpoolRepository.java

ViolationLine
Each class should declare at least one constructor91 - 327
Found 'DU'-anomaly for variable 'delay' (lines '142'-'179').142 - 179
Found 'DU'-anomaly for variable 'delay' (lines '142'-'179').142 - 179
Found 'DU'-anomaly for variable 'delay' (lines '142'-'179').142 - 179
Found 'DU'-anomaly for variable 'filter' (lines '191'-'230').191 - 230
Found 'DD'-anomaly for variable 'next' (lines '195'-'196').195 - 196
Avoid assignments in operands196
Found 'DD'-anomaly for variable 'next' (lines '196'-'195').196 - 195
Found 'DU'-anomaly for variable 'next' (lines '196'-'230').196 - 230
Found 'DU'-anomaly for variable 'filter' (lines '248'-'262').248 - 262
A method should have only one exit point, and that should be the last statement in the method257
Found 'DU'-anomaly for variable 'filter' (lines '267'-'309').267 - 309
Found 'DD'-anomaly for variable 'conn' (lines '272'-'276').272 - 276
Found 'DD'-anomaly for variable 'listMessages' (lines '273'-'277').273 - 277
Found 'DD'-anomaly for variable 'rsListMessages' (lines '274'-'285').274 - 285

org/apache/james/mailrepository/MBoxMailRepository.java

ViolationLine
Each class should declare at least one constructor114 - 868
Use explicit scoping instead of the default package private level119
Use explicit scoping instead of the default package private level120
Use explicit scoping instead of the default package private level121
Use explicit scoping instead of the default package private level122
Use explicit scoping instead of the default package private level123
Use explicit scoping instead of the default package private level124
Use explicit scoping instead of the default package private level125
Avoid modifiers which are implied by the context151
Avoid modifiers which are implied by the context152
Found 'DU'-anomaly for variable 'bodyText' (lines '152'-'152').152
Found 'DU'-anomaly for variable 'messageStart' (lines '152'-'152').152
Found 'DU'-anomaly for variable 'messageSeparator' (lines '152'-'152').152
Found 'DD'-anomaly for variable 'mimeMessage' (lines '176'-'182').176 - 182
Found 'DU'-anomaly for variable 'messAct' (lines '243'-'348').243 - 348
Found 'DU'-anomaly for variable 'sepMatchPattern' (lines '256'-'348').256 - 348
Found 'DU'-anomaly for variable 'sepMatch' (lines '257'-'348').257 - 348
Found 'DU'-anomaly for variable 'inMessage' (lines '260'-'348').260 - 348
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '262'-'287').262 - 287
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '262'-'348').262 - 348
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '262'-'315').262 - 315
Found 'DD'-anomaly for variable 'foundSep' (lines '263'-'269').263 - 269
Found 'DU'-anomaly for variable 'foundSep' (lines '263'-'348').263 - 348
Found 'DD'-anomaly for variable 'foundSep' (lines '263'-'298').263 - 298
Found 'DU'-anomaly for variable 'prevMessageStart' (lines '265'-'348').265 - 348
Found 'DD'-anomaly for variable 'line' (lines '267'-'268').267 - 268
Avoid assignments in operands268
Found 'DU'-anomaly for variable 'line' (lines '268'-'348').268 - 348
Found 'DD'-anomaly for variable 'endResult' (lines '275'-'275').275
Found 'DU'-anomaly for variable 'endResult' (lines '275'-'348').275 - 348
A method should have only one exit point, and that should be the last statement in the method278
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '280'-'287').280 - 287
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '280'-'348').280 - 348
Found 'DU'-anomaly for variable 'prevMessageStart' (lines '281'-'348').281 - 348
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '287'-'348').287 - 348
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '287'-'287').287
Found 'DU'-anomaly for variable 'line' (lines '295'-'348').295 - 348
Avoid assignments in operands296
Found 'DU'-anomaly for variable 'c' (lines '296'-'348').296 - 348
A method should have only one exit point, and that should be the last statement in the method306
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '308'-'315').308 - 315
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '308'-'348').308 - 348
Found 'DU'-anomaly for variable 'prevMessageStart' (lines '309'-'348').309 - 348
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '315'-'348').315 - 348
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '315'-'315').315
Found 'DU'-anomaly for variable 'inMessage' (lines '316'-'348').316 - 348
Found 'DU'-anomaly for variable 'line' (lines '321'-'348').321 - 348
A method should have only one exit point, and that should be the last statement in the method330
Found 'DD'-anomaly for variable 'foundMessage' (lines '358'-'361').358 - 361
Assigning an Object to null is a code smell. Consider refactoring.367
Found 'DU'-anomaly for variable 'foundMessage' (lines '379'-'447').379 - 447
Found 'DU'-anomaly for variable 'foundMessage' (lines '379'-'447').379 - 447
Found 'DU'-anomaly for variable 'foundMessage' (lines '379'-'447').379 - 447
A method should have only one exit point, and that should be the last statement in the method392
Found 'DD'-anomaly for variable 'ins' (lines '407'-'409').407 - 409
Found 'DD'-anomaly for variable 'ins' (lines '407'-'409').407 - 409
Found 'DD'-anomaly for variable 'ins' (lines '407'-'409').407 - 409
Found 'DU'-anomaly for variable 'ins' (lines '409'-'447').409 - 447
Found 'DU'-anomaly for variable 'ins' (lines '409'-'447').409 - 447
Found 'DU'-anomaly for variable 'ins' (lines '409'-'447').409 - 447
Found 'DU'-anomaly for variable 'op' (lines '413'-'447').413 - 447
Found 'DU'-anomaly for variable 'op' (lines '413'-'447').413 - 447
Found 'DU'-anomaly for variable 'op' (lines '413'-'447').413 - 447
These nested if statements could be combined434 - 442
Found 'DD'-anomaly for variable 'ins' (lines '456'-'458').456 - 458
Found 'DD'-anomaly for variable 'ins' (lines '456'-'458').456 - 458
Found 'DD'-anomaly for variable 'ins' (lines '456'-'458').456 - 458
Assigning an Object to null is a code smell. Consider refactoring.510
Found 'DD'-anomaly for variable 'fromHeader' (lines '512'-'518').512 - 518
Found 'DD'-anomaly for variable 'fromHeader' (lines '512'-'520').512 - 520
Found 'DD'-anomaly for variable 'message' (lines '513'-'515').513 - 515
Found 'DD'-anomaly for variable 'saveFile' (lines '529'-'531').529 - 531
Found 'DU'-anomaly for variable 'res' (lines '578'-'600').578 - 600
Found 'DD'-anomaly for variable 'res' (lines '578'-'585').578 - 585
A method should have only one exit point, and that should be the last statement in the method583
Found 'DU'-anomaly for variable 'sleepCount' (lines '620'-'648').620 - 648
Found 'DU'-anomaly for variable 'mails' (lines '674'-'752').674 - 752
Found 'DU'-anomaly for variable 'mails' (lines '674'-'752').674 - 752
Found 'DU'-anomaly for variable 'mails' (lines '674'-'752').674 - 752
Found 'DU'-anomaly for variable 'outputFile' (lines '691'-'752').691 - 752
Found 'DU'-anomaly for variable 'outputFile' (lines '691'-'752').691 - 752
Found 'DU'-anomaly for variable 'outputFile' (lines '691'-'752').691 - 752
Found 'DU'-anomaly for variable 'key' (lines '758'-'771').758 - 771
Found 'DU'-anomaly for variable 'key' (lines '778'-'780').778 - 780
Found 'DU'-anomaly for variable 'key' (lines '787'-'789').787 - 789
Found 'DU'-anomaly for variable 'componentManager' (lines '795'-'797').795 - 797
Found 'UR'-anomaly for variable 'exceptionString' (lines '804'-'804').804
Assigning an Object to null is a code smell. Consider refactoring.806
Found 'DU'-anomaly for variable 'exceptionString' (lines '822'-'828').822 - 828
Found 'DU'-anomaly for variable 'args' (lines '839'-'867').839 - 867
Suspicious decimal characters following octal escape in string literal842
Avoid empty while statements844 - 857

org/apache/james/mailrepository/MailStoreSpoolRepository.java

ViolationLine
Each class should declare at least one constructor51 - 178

org/apache/james/mailrepository/MessageInputStream.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.70
Found 'DD'-anomaly for variable 'bodyOut' (lines '119'-'124').119 - 124
Found 'DD'-anomaly for variable 'bodyOut' (lines '119'-'127').119 - 127
Assigning an Object to null is a code smell. Consider refactoring.157
Assigning an Object to null is a code smell. Consider refactoring.158
Assigning an Object to null is a code smell. Consider refactoring.205

org/apache/james/mailrepository/MimeMessageAvalonSource.java

ViolationLine
Use explicit scoping instead of the default package private level35
Use explicit scoping instead of the default package private level40
Use explicit scoping instead of the default package private level46
Perhaps 'size' could be replaced by a local variable.48
It is a good practice to call super() in a constructor50 - 54

org/apache/james/mailrepository/MimeMessageJDBCSource.java

ViolationLine
Use explicit scoping instead of the default package private level48
Use explicit scoping instead of the default package private level49
Use explicit scoping instead of the default package private level50
Perhaps 'size' could be replaced by a local variable.52
Use explicit scoping instead of the default package private level57
Use explicit scoping instead of the default package private level62
Found 'DU'-anomaly for variable 'logString' (lines '69'-'72').69 - 72
It is a good practice to call super() in a constructor79 - 96
Found 'DD'-anomaly for variable 'conn' (lines '120'-'124').120 - 124
Found 'DD'-anomaly for variable 'retrieveMessageStream' (lines '121'-'133').121 - 133
Found 'DD'-anomaly for variable 'rsRetrieveMessageStream' (lines '122'-'136').122 - 136
Found 'DD'-anomaly for variable 'headers' (lines '126'-'145').126 - 145
Found 'DD'-anomaly for variable 'headers' (lines '126'-'147').126 - 147
Found 'DD'-anomaly for variable 'start' (lines '128'-'130').128 - 130
Found 'DU'-anomaly for variable 'start' (lines '128'-'171').128 - 171
Found 'DU'-anomaly for variable 'start' (lines '130'-'171').130 - 171
Found 'DD'-anomaly for variable 'in' (lines '154'-'157').154 - 157
Avoid empty catch blocks159 - 162
A method should have only one exit point, and that should be the last statement in the method177
A method should have only one exit point, and that should be the last statement in the method181
Found 'DD'-anomaly for variable 'conn' (lines '183'-'187').183 - 187
Found 'DD'-anomaly for variable 'retrieveMessageSize' (lines '184'-'189').184 - 189
Found 'DD'-anomaly for variable 'rsRetrieveMessageSize' (lines '185'-'192').185 - 192
Found 'DD'-anomaly for variable 'in' (lines '200'-'206').200 - 206
Found 'DD'-anomaly for variable 'len' (lines '207'-'209').207 - 209
Avoid assignments in operands209
Found 'DU'-anomaly for variable 'len' (lines '209'-'235').209 - 235
Avoid empty catch blocks214 - 217
Avoid empty catch blocks222 - 224
A method should have only one exit point, and that should be the last statement in the method245 - 246
Found 'DD'-anomaly for variable 'result' (lines '257'-'259').257 - 259
Found 'DD'-anomaly for variable 'result' (lines '257'-'262').257 - 262
Found 'DD'-anomaly for variable 'result' (lines '259'-'262').259 - 262

org/apache/james/mailrepository/filepair/AbstractFileRepository.java

ViolationLine
Each class should declare at least one constructor49 - 373
Found 'DU'-anomaly for variable 'configuration' (lines '91'-'99').91 - 99
Found 'UR'-anomaly for variable 'directory' (lines '152'-'152').152
Found 'DD'-anomaly for variable 'directory' (lines '167'-'176').167 - 176
Found 'DD'-anomaly for variable 'directory' (lines '171'-'176').171 - 176
Found 'DD'-anomaly for variable 'child' (lines '197'-'201').197 - 201
Found 'DD'-anomaly for variable 'buffer' (lines '336'-'341').336 - 341
Found 'DU'-anomaly for variable 'j' (lines '338'-'351').338 - 351
Found 'DD'-anomaly for variable 'buffer' (lines '341'-'342').341 - 342
Found 'DD'-anomaly for variable 'buffer' (lines '342'-'341').342 - 341
Found 'DD'-anomaly for variable 'filename' (lines '359'-'361').359 - 361
Found 'DD'-anomaly for variable 'bytes' (lines '363'-'367').363 - 367
Found 'DU'-anomaly for variable 'j' (lines '365'-'372').365 - 372
Found 'DD'-anomaly for variable 'bytes' (lines '367'-'367').367

org/apache/james/mailrepository/filepair/File_Persistent_Object_Repository.java

ViolationLine
Each class should declare at least one constructor34 - 142

org/apache/james/mailrepository/filepair/File_Persistent_Stream_Repository.java

ViolationLine
Each class should declare at least one constructor35 - 87
Found 'UR'-anomaly for variable 'message' (lines '47'-'47').47
Found 'UR'-anomaly for variable 'message' (lines '64'-'64').64
A method should have only one exit point, and that should be the last statement in the method81

org/apache/james/mailrepository/filepair/NumberedRepositoryFileFilter.java

ViolationLine
Found 'DU'-anomaly for variable 'file' (lines '39'-'59').39 - 59
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 method49

org/apache/james/mailrepository/filepair/RepositoryManager.java

ViolationLine
Each class should declare at least one constructor44 - 222
Found 'DU'-anomaly for variable 'types' (lines '88'-'108').88 - 108
Found 'DU'-anomaly for variable 'modelIterator' (lines '89'-'108').89 - 108
Found 'DU'-anomaly for variable 'protocol' (lines '94'-'108').94 - 108
Found 'DD'-anomaly for variable 'protocol' (lines '94'-'94').94
Found 'DU'-anomaly for variable 'type' (lines '98'-'108').98 - 108
Found 'DD'-anomaly for variable 'type' (lines '98'-'98').98
Found 'DU'-anomaly for variable 'component' (lines '110'-'112').110 - 112
A method should have only one exit point, and that should be the last statement in the method117
Found 'UR'-anomaly for variable 'message' (lines '122'-'122').122
Found 'UR'-anomaly for variable 'error' (lines '122'-'122').122
Found 'DD'-anomaly for variable 'repConf' (lines '125'-'128').125 - 128
Found 'DD'-anomaly for variable 'destination' (lines '136'-'139').136 - 139
A method should have only one exit point, and that should be the last statement in the method163
Found 'DU'-anomaly for variable 'message' (lines '167'-'216').167 - 216

org/apache/james/nntpserver/NNTPException.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.44

org/apache/james/nntpserver/NNTPHandler.java

ViolationLine
Each class should declare at least one constructor67 - 1748
Use explicit scoping instead of the default package private level302
Perhaps 'theWatchdogTarget' could be replaced by a local variable.312
Use explicit scoping instead of the default package private level319 - 321
Use explicit scoping instead of the default package private level328 - 330
Use explicit scoping instead of the default package private level338 - 340
Use explicit scoping instead of the default package private level345 - 366
Avoid empty catch blocks353 - 355
Assigning an Object to null is a code smell. Consider refactoring.356
Assigning an Object to null is a code smell. Consider refactoring.363
Avoid empty catch blocks432
Assigning an Object to null is a code smell. Consider refactoring.448
Assigning an Object to null is a code smell. Consider refactoring.459
Assigning an Object to null is a code smell. Consider refactoring.462
Assigning an Object to null is a code smell. Consider refactoring.466
Assigning an Object to null is a code smell. Consider refactoring.468
Assigning an Object to null is a code smell. Consider refactoring.470
Assigning an Object to null is a code smell. Consider refactoring.471
Assigning an Object to null is a code smell. Consider refactoring.479
Assigning an Object to null is a code smell. Consider refactoring.483
Assigning an Object to null is a code smell. Consider refactoring.487
Assigning an Object to null is a code smell. Consider refactoring.491
Assigning an Object to null is a code smell. Consider refactoring.492
Assigning an Object to null is a code smell. Consider refactoring.496
A method should have only one exit point, and that should be the last statement in the method513
Found 'DD'-anomaly for variable 'argument' (lines '520'-'523').520 - 523
Found 'DU'-anomaly for variable 'argument' (lines '520'-'593').520 - 593
Found 'DU'-anomaly for variable 'argument' (lines '523'-'593').523 - 593
Found 'DD'-anomaly for variable 'returnValue' (lines '528'-'564').528 - 564
A method should have only one exit point, and that should be the last statement in the method532
Found 'DU'-anomaly for variable 'command' (lines '601'-'612').601 - 612
Found 'DU'-anomaly for variable 'argument' (lines '601'-'612').601 - 612
Found 'DD'-anomaly for variable 'command' (lines '622'-'627').622 - 627
Found 'DD'-anomaly for variable 'value' (lines '623'-'628').623 - 628
Found 'DU'-anomaly for variable 'value' (lines '623'-'679').623 - 679
Found 'DU'-anomaly for variable 'value' (lines '628'-'679').628 - 679
A method should have only one exit point, and that should be the last statement in the method633
Assigning an Object to null is a code smell. Consider refactoring.643
Assigning an Object to null is a code smell. Consider refactoring.644
A method should have only one exit point, and that should be the last statement in the method647
A method should have only one exit point, and that should be the last statement in the method655
Assigning an Object to null is a code smell. Consider refactoring.659
Assigning an Object to null is a code smell. Consider refactoring.660
A method should have only one exit point, and that should be the last statement in the method663
Assigning an Object to null is a code smell. Consider refactoring.672
Assigning an Object to null is a code smell. Consider refactoring.673
Found 'DD'-anomaly for variable 'wildmat' (lines '691'-'696').691 - 696
Found 'DU'-anomaly for variable 'wildmat' (lines '691'-'730').691 - 730
Found 'DU'-anomaly for variable 'wildmat' (lines '696'-'730').696 - 730
A method should have only one exit point, and that should be the last statement in the method701
A method should have only one exit point, and that should be the last statement in the method706
Found 'DD'-anomaly for variable 'theDate' (lines '709'-'711').709 - 711
Found 'DU'-anomaly for variable 'theDate' (lines '711'-'730').711 - 730
Found 'DD'-anomaly for variable 'theDate' (lines '749'-'751').749 - 751
Found 'DU'-anomaly for variable 'theDate' (lines '751'-'773').751 - 773
Found 'DU'-anomaly for variable 'argument' (lines '780'-'783').780 - 783
Found 'DU'-anomaly for variable 'argument' (lines '791'-'793').791 - 793
Found 'DU'-anomaly for variable 'argument' (lines '800'-'804').800 - 804
Found 'DU'-anomaly for variable 'argument' (lines '811'-'813').811 - 813
Found 'DD'-anomaly for variable 'wildmat' (lines '822'-'829').822 - 829
Found 'DU'-anomaly for variable 'wildmat' (lines '822'-'875').822 - 875
Found 'DD'-anomaly for variable 'isListNewsgroups' (lines '823'-'837').823 - 837
Found 'DD'-anomaly for variable 'isListNewsgroups' (lines '823'-'839').823 - 839
Found 'DU'-anomaly for variable 'isListNewsgroups' (lines '823'-'875').823 - 875
Found 'DD'-anomaly for variable 'extension' (lines '825'-'830').825 - 830
Found 'DD'-anomaly for variable 'extension' (lines '825'-'832').825 - 832
Found 'DU'-anomaly for variable 'wildmat' (lines '829'-'875').829 - 875
Found 'DD'-anomaly for variable 'extension' (lines '830'-'832').830 - 832
Found 'DU'-anomaly for variable 'isListNewsgroups' (lines '837'-'875').837 - 875
Found 'DU'-anomaly for variable 'isListNewsgroups' (lines '839'-'875').839 - 875
A method should have only one exit point, and that should be the last statement in the method842
A method should have only one exit point, and that should be the last statement in the method845
A method should have only one exit point, and that should be the last statement in the method849
A method should have only one exit point, and that should be the last statement in the method853
A method should have only one exit point, and that should be the last statement in the method857
Found 'DD'-anomaly for variable 'article' (lines '930'-'932').930 - 932
Found 'DU'-anomaly for variable 'article' (lines '930'-'980').930 - 980
Found 'DD'-anomaly for variable 'article' (lines '930'-'954').930 - 954
Found 'DD'-anomaly for variable 'article' (lines '930'-'959').930 - 959
A method should have only one exit point, and that should be the last statement in the method935
Found 'DU'-anomaly for variable 'newArticleNumber' (lines '944'-'980').944 - 980
Found 'DD'-anomaly for variable 'newArticleNumber' (lines '944'-'958').944 - 958
A method should have only one exit point, and that should be the last statement in the method947
A method should have only one exit point, and that should be the last statement in the method952
Found 'DD'-anomaly for variable 'article' (lines '993'-'995').993 - 995
Found 'DU'-anomaly for variable 'article' (lines '993'-'1,048').993 - 1048
Found 'DD'-anomaly for variable 'article' (lines '993'-'1,017').993 - 1017
Found 'DD'-anomaly for variable 'article' (lines '993'-'1,022').993 - 1022
A method should have only one exit point, and that should be the last statement in the method998
Found 'DU'-anomaly for variable 'newArticleNumber' (lines '1,007'-'1,048').1007 - 1048
Found 'DD'-anomaly for variable 'newArticleNumber' (lines '1,007'-'1,021').1007 - 1021
A method should have only one exit point, and that should be the last statement in the method1010
A method should have only one exit point, and that should be the last statement in the method1015
Found 'DD'-anomaly for variable 'article' (lines '1,061'-'1,063').1061 - 1063
Found 'DU'-anomaly for variable 'article' (lines '1,061'-'1,116').1061 - 1116
Found 'DD'-anomaly for variable 'article' (lines '1,061'-'1,085').1061 - 1085
Found 'DD'-anomaly for variable 'article' (lines '1,061'-'1,090').1061 - 1090
A method should have only one exit point, and that should be the last statement in the method1066
Found 'DU'-anomaly for variable 'newArticleNumber' (lines '1,075'-'1,116').1075 - 1116
Found 'DD'-anomaly for variable 'newArticleNumber' (lines '1,075'-'1,089').1075 - 1089
A method should have only one exit point, and that should be the last statement in the method1078
A method should have only one exit point, and that should be the last statement in the method1083
Found 'DD'-anomaly for variable 'article' (lines '1,129'-'1,131').1129 - 1131
Found 'DU'-anomaly for variable 'article' (lines '1,129'-'1,185').1129 - 1185
Found 'DD'-anomaly for variable 'article' (lines '1,129'-'1,153').1129 - 1153
Found 'DD'-anomaly for variable 'article' (lines '1,129'-'1,158').1129 - 1158
A method should have only one exit point, and that should be the last statement in the method1134
Found 'DU'-anomaly for variable 'newArticleNumber' (lines '1,143'-'1,185').1143 - 1185
Found 'DD'-anomaly for variable 'newArticleNumber' (lines '1,143'-'1,157').1143 - 1157
A method should have only one exit point, and that should be the last statement in the method1146
A method should have only one exit point, and that should be the last statement in the method1151
Found 'DU'-anomaly for variable 'argument' (lines '1,312'-'1,316').1312 - 1316
Found 'DU'-anomaly for variable 'argument' (lines '1,323'-'1,326').1323 - 1326
A method should have only one exit point, and that should be the last statement in the method1339
Found 'DU'-anomaly for variable 'argument' (lines '1,390'-'1,393').1390 - 1393
A method should have only one exit point, and that should be the last statement in the method1415
Found 'DD'-anomaly for variable 'range' (lines '1,418'-'1,421').1418 - 1421
Found 'DU'-anomaly for variable 'range' (lines '1,418'-'1,453').1418 - 1453
Found 'DU'-anomaly for variable 'range' (lines '1,421'-'1,453').1421 - 1453
A method should have only one exit point, and that should be the last statement in the method1426
Found 'DU'-anomaly for variable 'range' (lines '1,471'-'1,497').1471 - 1497
A method should have only one exit point, and that should be the last statement in the method1475
Found 'DD'-anomaly for variable 'msgIn' (lines '1,504'-'1,506').1504 - 1506
Found 'DD'-anomaly for variable 'messageIn' (lines '1,534'-'1,536').1534 - 1536
Avoid empty catch blocks1542 - 1544
Assigning an Object to null is a code smell. Consider refactoring.1545
Found 'DU'-anomaly for variable 'messageIn' (lines '1,545'-'1,548').1545 - 1548
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '1,558'-'1,558').1558
Found 'DU'-anomaly for variable 'd' (lines '1,569'-'1,592').1569 - 1592
Found 'DD'-anomaly for variable 'dt' (lines '1,576'-'1,578').1576 - 1578
A method should have only one exit point, and that should be the last statement in the method1608 - 1609
Found 'DD'-anomaly for variable 'start' (lines '1,616'-'1,620').1616 - 1620
Found 'DD'-anomaly for variable 'start' (lines '1,616'-'1,623').1616 - 1623
Found 'DD'-anomaly for variable 'end' (lines '1,617'-'1,621').1617 - 1621
Found 'DD'-anomaly for variable 'end' (lines '1,617'-'1,625').1617 - 1625
Found 'DD'-anomaly for variable 'end' (lines '1,617'-'1,627').1617 - 1627
Found 'DU'-anomaly for variable 'command' (lines '1,647'-'1,657').1647 - 1657
A method should have only one exit point, and that should be the last statement in the method1650
Found 'DD'-anomaly for variable 'allowed' (lines '1,653'-'1,654').1653 - 1654
A method should have only one exit point, and that should be the last statement in the method1667
A method should have only one exit point, and that should be the last statement in the method1669
A method should have only one exit point, and that should be the last statement in the method1688
Found 'DU'-anomaly for variable 'responseString' (lines '1,702'-'1,706').1702 - 1706
Use explicit scoping instead of the default package private level1715 - 1719
Use explicit scoping instead of the default package private level1727 - 1730
Each class should declare at least one constructor1737 - 1747

org/apache/james/nntpserver/NNTPServer.java

ViolationLine
Each class should declare at least one constructor45 - 244
Perhaps 'theHandlerFactory' could be replaced by a local variable.70
Each class should declare at least one constructor184 - 207
Found 'DU'-anomaly for variable 'object' (lines '204'-'206').204 - 206
Avoid unnecessary return statements205
Each class should declare at least one constructor212 - 243

org/apache/james/nntpserver/NNTPServerMBean.java

ViolationLine
Avoid modifiers which are implied by the context36
Avoid modifiers which are implied by the context45
Avoid modifiers which are implied by the context54
Avoid modifiers which are implied by the context63

org/apache/james/nntpserver/repository/ArticleIDRepository.java

ViolationLine
Perhaps 'counter' could be replaced by a local variable.61
Use explicit scoping instead of the default package private level71 - 85
Use explicit scoping instead of the default package private level92 - 105
Found 'DD'-anomaly for variable 'fout' (lines '96'-'98').96 - 98
Use explicit scoping instead of the default package private level115 - 123
Use explicit scoping instead of the default package private level148 - 150
Use explicit scoping instead of the default package private level162 - 188
Found 'DU'-anomaly for variable 'repo' (lines '162'-'188').162 - 188
A method should have only one exit point, and that should be the last statement in the method165
Found 'DD'-anomaly for variable 'fin' (lines '167'-'170').167 - 170
Found 'DU'-anomaly for variable 'number' (lines '181'-'188').181 - 188
Found 'DD'-anomaly for variable 'number' (lines '181'-'181').181

org/apache/james/nntpserver/repository/NNTPArticle.java

ViolationLine
Found 'DU'-anomaly for variable 'wrt' (lines '55'-'55').55
Found 'DU'-anomaly for variable 'wrt' (lines '62'-'62').62
Found 'DU'-anomaly for variable 'wrt' (lines '69'-'69').69
Found 'DU'-anomaly for variable 'wrt' (lines '76'-'76').76
Found 'DU'-anomaly for variable 'headerName' (lines '84'-'84').84

org/apache/james/nntpserver/repository/NNTPArticleImpl.java

ViolationLine
Found 'DD'-anomaly for variable 'fin' (lines '78'-'80').78 - 80
Assigning an Object to null is a code smell. Consider refactoring.84
Found 'DU'-anomaly for variable 'out' (lines '95'-'115').95 - 115
Found 'DD'-anomaly for variable 'fileStream' (lines '96'-'98').96 - 98
Found 'DD'-anomaly for variable 'read' (lines '100'-'101').100 - 101
Avoid assignments in operands101
Found 'DU'-anomaly for variable 'read' (lines '101'-'115').101 - 115
Avoid empty catch blocks110 - 112
Found 'DU'-anomaly for variable 'out' (lines '120'-'142').120 - 142
Found 'DD'-anomaly for variable 'fileStream' (lines '121'-'123').121 - 123
Avoid empty catch blocks137 - 139
Found 'DU'-anomaly for variable 'out' (lines '147'-'168').147 - 168
Found 'DD'-anomaly for variable 'fileStream' (lines '148'-'150').148 - 150
Found 'DU'-anomaly for variable 'headers' (lines '151'-'168').151 - 168
Found 'DD'-anomaly for variable 'read' (lines '153'-'154').153 - 154
Avoid assignments in operands154
Found 'DU'-anomaly for variable 'read' (lines '154'-'168').154 - 168
Avoid empty catch blocks163 - 165
Found 'DD'-anomaly for variable 'fileStream' (lines '174'-'176').174 - 176
Found 'DD'-anomaly for variable 'lines' (lines '189'-'191').189 - 191
Found 'DD'-anomaly for variable 'lines' (lines '191'-'191').191
Avoid empty catch blocks215 - 217

org/apache/james/nntpserver/repository/NNTPGroup.java

ViolationLine
Found 'DU'-anomaly for variable 'number' (lines '82'-'82').82
Found 'DU'-anomaly for variable 'dt' (lines '93'-'93').93
Found 'DU'-anomaly for variable 'newsStream' (lines '126'-'126').126

org/apache/james/nntpserver/repository/NNTPGroupImpl.java

ViolationLine
Perhaps 'articleRangeInfoCollected' could be replaced by a local variable.70
It is a good practice to call super() in a constructor77 - 79
Assigning an Object to null is a code smell. Consider refactoring.157
Found 'UR'-anomaly for variable 'artNum' (lines '216'-'230').216 - 230
Found 'DD'-anomaly for variable 'articleFile' (lines '218'-'230').218 - 230
Found 'DD'-anomaly for variable 'fout' (lines '238'-'240').238 - 240
Avoid empty catch blocks248 - 251

org/apache/james/nntpserver/repository/NNTPRepository.java

ViolationLine
Found 'DU'-anomaly for variable 'groupName' (lines '38'-'38').38
Found 'DU'-anomaly for variable 'id' (lines '47'-'47').47
Found 'DU'-anomaly for variable 'in' (lines '55'-'55').55
Found 'DU'-anomaly for variable 'wildmat' (lines '64'-'64').64
Found 'DU'-anomaly for variable 'dt' (lines '73'-'73').73
Found 'DU'-anomaly for variable 'dt' (lines '82'-'82').82

org/apache/james/nntpserver/repository/NNTPRepositoryImpl.java

ViolationLine
Each class should declare at least one constructor52 - 485
Perhaps 'overviewFormat' could be replaced by a local variable.124 - 131
Perhaps 'repositoryGroups' could be replaced by a local variable.138
Found 'UR'-anomaly for variable 'errorBuffer' (lines '196'-'196').196
Found 'DD'-anomaly for variable 'articleIDPath' (lines '199'-'204').199 - 204
Found 'DU'-anomaly for variable 'errorBuffer' (lines '227'-'265').227 - 265
A method should have only one exit point, and that should be the last statement in the method282
Found 'DD'-anomaly for variable 'groupToReturn' (lines '285'-'287').285 - 287
Assigning an Object to null is a code smell. Consider refactoring.297
A method should have only one exit point, and that should be the last statement in the method309
Found 'DD'-anomaly for variable 'fout' (lines '326'-'328').326 - 328
Found 'DD'-anomaly for variable 'bytesRead' (lines '330'-'331').330 - 331
Avoid assignments in operands331
Found 'DU'-anomaly for variable 'bytesRead' (lines '331'-'352').331 - 352
Assigning an Object to null is a code smell. Consider refactoring.336
Avoid empty catch blocks347 - 349
Each class should declare at least one constructor354 - 362
Found 'DU'-anomaly for variable 'dir' (lines '355'-'361').355 - 361
Found 'DU'-anomaly for variable 'giter' (lines '408'-'439').408 - 439
Found 'DU'-anomaly for variable 'giter' (lines '408'-'439').408 - 439
Found 'DU'-anomaly for variable 'giter' (lines '408'-'439').408 - 439
Found 'DU'-anomaly for variable 'giter' (lines '408'-'439').408 - 439
Assigning an Object to null is a code smell. Consider refactoring.426
Found 'UR'-anomaly for variable 'errorString' (lines '453'-'453').453
Found 'DD'-anomaly for variable 'className' (lines '455'-'459').455 - 459
Avoid empty catch blocks460 - 462

org/apache/james/nntpserver/repository/NNTPSpooler.java

ViolationLine
Each class should declare at least one constructor49 - 345
Found 'UR'-anomaly for variable 'errorBuffer' (lines '98'-'98').98
Found 'DU'-anomaly for variable 'errorBuffer' (lines '106'-'129').106 - 129
Use explicit scoping instead of the default package private level136 - 140
Found 'DU'-anomaly for variable 'repo' (lines '136'-'140').136 - 140
Use explicit scoping instead of the default package private level147 - 151
Found 'DU'-anomaly for variable 'articleIDRepo' (lines '147'-'151').147 - 151
Use explicit scoping instead of the default package private level159 - 161
It is a good practice to call super() in a constructor191 - 194
Use explicit scoping instead of the default package private level201 - 203
Use explicit scoping instead of the default package private level210 - 212
Assigning an Object to null is a code smell. Consider refactoring.237
Assigning an Object to null is a code smell. Consider refactoring.239
Found 'DD'-anomaly for variable 'list' (lines '239'-'222').239 - 222
Found 'DU'-anomaly for variable 'list' (lines '239'-'250').239 - 250
Found 'DD'-anomaly for variable 'lineCount' (lines '276'-'285').276 - 285
Found 'DU'-anomaly for variable 'lineCount' (lines '276'-'343').276 - 343
Found 'DD'-anomaly for variable 'lines' (lines '280'-'282').280 - 282
Found 'DD'-anomaly for variable 'lines' (lines '282'-'282').282
Assigning an Object to null is a code smell. Consider refactoring.293

org/apache/james/pop3server/POP3Handler.java

ViolationLine
Each class should declare at least one constructor60 - 1154
Perhaps 'theWatchdogTarget' could be replaced by a local variable.161
Use explicit scoping instead of the default package private level168 - 170
Use explicit scoping instead of the default package private level177 - 179
Use explicit scoping instead of the default package private level187 - 189
Use explicit scoping instead of the default package private level194 - 216
Avoid empty catch blocks202 - 204
Assigning an Object to null is a code smell. Consider refactoring.205
Assigning an Object to null is a code smell. Consider refactoring.212
Found 'DD'-anomaly for variable 'remoteHost' (lines '224'-'235').224 - 235
Found 'DD'-anomaly for variable 'remoteIP' (lines '225'-'234').225 - 234
Assigning an Object to null is a code smell. Consider refactoring.317
Assigning an Object to null is a code smell. Consider refactoring.325
Avoid empty catch blocks327 - 329
Assigning an Object to null is a code smell. Consider refactoring.330
Avoid empty catch blocks337 - 339
Assigning an Object to null is a code smell. Consider refactoring.340
Avoid empty catch blocks347 - 349
Assigning an Object to null is a code smell. Consider refactoring.350
Avoid empty catch blocks357 - 359
Assigning an Object to null is a code smell. Consider refactoring.360
Assigning an Object to null is a code smell. Consider refactoring.364
Assigning an Object to null is a code smell. Consider refactoring.368
Assigning an Object to null is a code smell. Consider refactoring.369
Assigning an Object to null is a code smell. Consider refactoring.372
Assigning an Object to null is a code smell. Consider refactoring.377
Assigning an Object to null is a code smell. Consider refactoring.381
Use explicit scoping instead of the default package private level422 - 432
A method should have only one exit point, and that should be the last statement in the method448
Found 'DU'-anomaly for variable 'returnValue' (lines '450'-'503').450 - 503
Found 'DD'-anomaly for variable 'returnValue' (lines '450'-'497').450 - 497
A method should have only one exit point, and that should be the last statement in the method455
Found 'DD'-anomaly for variable 'argument' (lines '467'-'469').467 - 469
Found 'DD'-anomaly for variable 'argument1' (lines '471'-'473').471 - 473
Found 'DU'-anomaly for variable 'command' (lines '513'-'523').513 - 523
Found 'DU'-anomaly for variable 'argument1' (lines '513'-'523').513 - 523
Found 'DD'-anomaly for variable 'responseString' (lines '514'-'518').514 - 518
Found 'DD'-anomaly for variable 'responseString' (lines '514'-'520').514 - 520
Found 'DU'-anomaly for variable 'command' (lines '533'-'557').533 - 557
Found 'DU'-anomaly for variable 'argument1' (lines '533'-'557').533 - 557
Found 'DD'-anomaly for variable 'responseString' (lines '534'-'543').534 - 543
Found 'DD'-anomaly for variable 'responseString' (lines '534'-'549').534 - 549
Found 'DD'-anomaly for variable 'responseString' (lines '534'-'554').534 - 554
Found 'DU'-anomaly for variable 'command' (lines '568'-'598').568 - 598
Found 'DU'-anomaly for variable 'argument' (lines '568'-'598').568 - 598
Found 'DU'-anomaly for variable 'argument1' (lines '568'-'598').568 - 598
Found 'DD'-anomaly for variable 'responseString' (lines '569'-'588').569 - 588
Found 'DD'-anomaly for variable 'responseString' (lines '569'-'595').569 - 595
Found 'DD'-anomaly for variable 'count' (lines '572'-'578').572 - 578
Found 'DD'-anomaly for variable 'count' (lines '578'-'578').578
Found 'DU'-anomaly for variable 'command' (lines '610'-'704').610 - 704
Found 'DU'-anomaly for variable 'argument1' (lines '610'-'704').610 - 704
Found 'DU'-anomaly for variable 'argument' (lines '610'-'704').610 - 704
Found 'DD'-anomaly for variable 'responseString' (lines '611'-'631').611 - 631
Found 'DD'-anomaly for variable 'responseString' (lines '611'-'665').611 - 665
Found 'DD'-anomaly for variable 'responseString' (lines '611'-'674').611 - 674
Found 'DD'-anomaly for variable 'responseString' (lines '611'-'701').611 - 701
Found 'DD'-anomaly for variable 'count' (lines '615'-'621').615 - 621
Found 'DD'-anomaly for variable 'count' (lines '621'-'621').621
Found 'DD'-anomaly for variable 'count' (lines '633'-'634').633 - 634
Found 'DU'-anomaly for variable 'count' (lines '633'-'704').633 - 704
Found 'DU'-anomaly for variable 'count' (lines '634'-'704').634 - 704
Found 'DD'-anomaly for variable 'count' (lines '634'-'634').634
Found 'DD'-anomaly for variable 'num' (lines '653'-'655').653 - 655
Found 'DU'-anomaly for variable 'command' (lines '714'-'782').714 - 782
Found 'DU'-anomaly for variable 'argument1' (lines '714'-'782').714 - 782
Found 'DU'-anomaly for variable 'argument' (lines '714'-'782').714 - 782
Found 'DD'-anomaly for variable 'responseString' (lines '715'-'718').715 - 718
Found 'DD'-anomaly for variable 'responseString' (lines '715'-'747').715 - 747
Found 'DD'-anomaly for variable 'responseString' (lines '715'-'756').715 - 756
Found 'DU'-anomaly for variable 'responseString' (lines '715'-'782').715 - 782
Found 'DD'-anomaly for variable 'count' (lines '720'-'721').720 - 721
Found 'DU'-anomaly for variable 'count' (lines '720'-'782').720 - 782
Found 'DU'-anomaly for variable 'count' (lines '721'-'782').721 - 782
Found 'DD'-anomaly for variable 'count' (lines '721'-'721').721
Found 'DD'-anomaly for variable 'num' (lines '735'-'737').735 - 737
Found 'DU'-anomaly for variable 'command' (lines '792'-'801').792 - 801
Found 'DU'-anomaly for variable 'argument' (lines '792'-'801').792 - 801
Found 'DU'-anomaly for variable 'argument1' (lines '792'-'801').792 - 801
Found 'DD'-anomaly for variable 'responseString' (lines '793'-'796').793 - 796
Found 'DD'-anomaly for variable 'responseString' (lines '793'-'798').793 - 798
Found 'DU'-anomaly for variable 'command' (lines '812'-'852').812 - 852
Found 'DU'-anomaly for variable 'argument1' (lines '812'-'852').812 - 852
Found 'DU'-anomaly for variable 'argument' (lines '812'-'852').812 - 852
Found 'DD'-anomaly for variable 'responseString' (lines '813'-'819').813 - 819
Found 'DD'-anomaly for variable 'responseString' (lines '813'-'849').813 - 849
Found 'DD'-anomaly for variable 'num' (lines '815'-'817').815 - 817
Found 'DU'-anomaly for variable 'num' (lines '817'-'852').817 - 852
Found 'DU'-anomaly for variable 'command' (lines '862'-'871').862 - 871
Found 'DU'-anomaly for variable 'argument' (lines '862'-'871').862 - 871
Found 'DU'-anomaly for variable 'argument1' (lines '862'-'871').862 - 871
Found 'DD'-anomaly for variable 'responseString' (lines '863'-'865').863 - 865
Found 'DD'-anomaly for variable 'responseString' (lines '863'-'868').863 - 868
Found 'DU'-anomaly for variable 'command' (lines '882'-'942').882 - 942
Found 'DU'-anomaly for variable 'argument1' (lines '882'-'942').882 - 942
Found 'DU'-anomaly for variable 'argument' (lines '882'-'942').882 - 942
Found 'DD'-anomaly for variable 'responseString' (lines '883'-'889').883 - 889
Found 'DD'-anomaly for variable 'responseString' (lines '883'-'939').883 - 939
Found 'DD'-anomaly for variable 'num' (lines '885'-'887').885 - 887
Found 'DU'-anomaly for variable 'num' (lines '887'-'942').887 - 942
Found 'DU'-anomaly for variable 'command' (lines '956'-'1,022').956 - 1022
Found 'DU'-anomaly for variable 'argument' (lines '956'-'1,022').956 - 1022
Found 'DU'-anomaly for variable 'argument1' (lines '956'-'1,022').956 - 1022
Found 'DD'-anomaly for variable 'responseString' (lines '957'-'965').957 - 965
Found 'DD'-anomaly for variable 'responseString' (lines '957'-'1,019').957 - 1019
Found 'DD'-anomaly for variable 'num' (lines '959'-'962').959 - 962
Found 'DD'-anomaly for variable 'lines' (lines '960'-'963').960 - 963
Found 'DU'-anomaly for variable 'num' (lines '962'-'1,022').962 - 1022
Found 'DU'-anomaly for variable 'lines' (lines '963'-'1,022').963 - 1022
Found 'DU'-anomaly for variable 'out' (lines '1,034'-'1,054').1034 - 1054
Found 'DU'-anomaly for variable 'lines' (lines '1,034'-'1,054').1034 - 1054
Avoid assignments in operands1042
Found 'DU'-anomaly for variable 'line' (lines '1,042'-'1,054').1042 - 1054
Found 'DU'-anomaly for variable 'command' (lines '1,064'-'1,085').1064 - 1085
Found 'DU'-anomaly for variable 'argument' (lines '1,064'-'1,085').1064 - 1085
Found 'DU'-anomaly for variable 'argument1' (lines '1,064'-'1,085').1064 - 1085
Found 'DD'-anomaly for variable 'responseString' (lines '1,065'-'1,067').1065 - 1067
Found 'DD'-anomaly for variable 'responseString' (lines '1,065'-'1,078').1065 - 1078
Found 'DU'-anomaly for variable 'command' (lines '1,095'-'1,097').1095 - 1097
Found 'DU'-anomaly for variable 'argument' (lines '1,095'-'1,097').1095 - 1097
Found 'DU'-anomaly for variable 'argument1' (lines '1,095'-'1,097').1095 - 1097
Found 'DU'-anomaly for variable 'responseString' (lines '1,107'-'1,111').1107 - 1111
Use explicit scoping instead of the default package private level1120 - 1124
Use explicit scoping instead of the default package private level1132 - 1135
Each class should declare at least one constructor1142 - 1152

org/apache/james/pop3server/POP3Server.java

ViolationLine
Each class should declare at least one constructor48 - 242
Perhaps 'theHandlerFactory' could be replaced by a local variable.75
Each class should declare at least one constructor183 - 206
Found 'DU'-anomaly for variable 'object' (lines '203'-'205').203 - 205
Avoid unnecessary return statements204
Each class should declare at least one constructor211 - 241

org/apache/james/pop3server/POP3ServerMBean.java

ViolationLine
Avoid modifiers which are implied by the context36
Avoid modifiers which are implied by the context45
Avoid modifiers which are implied by the context54
Avoid modifiers which are implied by the context63

org/apache/james/remotemanager/RemoteManager.java

ViolationLine
Each class should declare at least one constructor52 - 283
Perhaps 'theHandlerFactory' could be replaced by a local variable.88
Each class should declare at least one constructor209 - 232
Found 'DU'-anomaly for variable 'object' (lines '229'-'231').229 - 231
Avoid unnecessary return statements230
Each class should declare at least one constructor237 - 282

org/apache/james/remotemanager/RemoteManagerHandler.java

ViolationLine
Each class should declare at least one constructor55 - 1084
Perhaps 'theWatchdogTarget' could be replaced by a local variable.183
Use explicit scoping instead of the default package private level190 - 195
Use explicit scoping instead of the default package private level202 - 204
Use explicit scoping instead of the default package private level212 - 214
Use explicit scoping instead of the default package private level219 - 240
Avoid empty catch blocks227 - 229
Assigning an Object to null is a code smell. Consider refactoring.230
Assigning an Object to null is a code smell. Consider refactoring.237
Found 'DU'-anomaly for variable 'remoteIP' (lines '249'-'340').249 - 340
Found 'DU'-anomaly for variable 'remoteHost' (lines '250'-'340').250 - 340
Found 'DD'-anomaly for variable 'password' (lines '272'-'281').272 - 281
Avoid empty catch blocks310 - 312
Assigning an Object to null is a code smell. Consider refactoring.350
Assigning an Object to null is a code smell. Consider refactoring.353
Assigning an Object to null is a code smell. Consider refactoring.354
Assigning an Object to null is a code smell. Consider refactoring.365
Assigning an Object to null is a code smell. Consider refactoring.369
Assigning an Object to null is a code smell. Consider refactoring.376
A method should have only one exit point, and that should be the last statement in the method390
Found 'DD'-anomaly for variable 'argument' (lines '393'-'396').393 - 396
Found 'DU'-anomaly for variable 'argument' (lines '393'-'438').393 - 438
Found 'DU'-anomaly for variable 'argument' (lines '396'-'438').396 - 438
A method should have only one exit point, and that should be the last statement in the method403
A method should have only one exit point, and that should be the last statement in the method405
A method should have only one exit point, and that should be the last statement in the method407
A method should have only one exit point, and that should be the last statement in the method409
A method should have only one exit point, and that should be the last statement in the method411
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 method415
A method should have only one exit point, and that should be the last statement in the method417
A method should have only one exit point, and that should be the last statement in the method419
A method should have only one exit point, and that should be the last statement in the method421
A method should have only one exit point, and that should be the last statement in the method423
A method should have only one exit point, and that should be the last statement in the method425
A method should have only one exit point, and that should be the last statement in the method427
A method should have only one exit point, and that should be the last statement in the method429
A method should have only one exit point, and that should be the last statement in the method431
A method should have only one exit point, and that should be the last statement in the method433
A method should have only one exit point, and that should be the last statement in the method435
Found 'DD'-anomaly for variable 'breakIndex' (lines '470'-'471').470 - 471
Avoid assignments in operands471 - 473
Found 'DU'-anomaly for variable 'breakIndex' (lines '471'-'511').471 - 511
A method should have only one exit point, and that should be the last statement in the method475
A method should have only one exit point, and that should be the last statement in the method481
Found 'DD'-anomaly for variable 'success' (lines '484'-'494').484 - 494
Found 'DD'-anomaly for variable 'breakIndex' (lines '521'-'522').521 - 522
Avoid assignments in operands522 - 524
Found 'DU'-anomaly for variable 'breakIndex' (lines '522'-'557').522 - 557
A method should have only one exit point, and that should be the last statement in the method526
A method should have only one exit point, and that should be the last statement in the method533
A method should have only one exit point, and that should be the last statement in the method538
A method should have only one exit point, and that should be the last statement in the method569
Found 'DU'-anomaly for variable 'argument' (lines '612'-'619').612 - 619
Found 'DU'-anomaly for variable 'argument' (lines '627'-'630').627 - 630
A method should have only one exit point, and that should be the last statement in the method642
Found 'DU'-anomaly for variable 'argument' (lines '671'-'691').671 - 691
Found 'DD'-anomaly for variable 'breakIndex' (lines '700'-'701').700 - 701
Avoid assignments in operands701 - 703
Found 'DU'-anomaly for variable 'breakIndex' (lines '701'-'750').701 - 750
A method should have only one exit point, and that should be the last statement in the method705
A method should have only one exit point, and that should be the last statement in the method711
A method should have only one exit point, and that should be the last statement in the method717
A method should have only one exit point, and that should be the last statement in the method721
Found 'DU'-anomaly for variable 'user' (lines '724'-'750').724 - 750
A method should have only one exit point, and that should be the last statement in the method728
Found 'DD'-anomaly for variable 'breakIndex' (lines '759'-'760').759 - 760
Avoid assignments in operands760 - 762
Found 'DU'-anomaly for variable 'breakIndex' (lines '760'-'811').760 - 811
A method should have only one exit point, and that should be the last statement in the method764
A method should have only one exit point, and that should be the last statement in the method770
A method should have only one exit point, and that should be the last statement in the method776
A method should have only one exit point, and that should be the last statement in the method779
Found 'DU'-anomaly for variable 'user' (lines '781'-'811').781 - 811
Found 'DU'-anomaly for variable 'forwardAddr' (lines '785'-'811').785 - 811
A method should have only one exit point, and that should be the last statement in the method789
A method should have only one exit point, and that should be the last statement in the method822
A method should have only one exit point, and that should be the last statement in the method829
A method should have only one exit point, and that should be the last statement in the method832
A method should have only one exit point, and that should be the last statement in the method838
A method should have only one exit point, and that should be the last statement in the method843
A method should have only one exit point, and that should be the last statement in the method852
A method should have only one exit point, and that should be the last statement in the method868
A method should have only one exit point, and that should be the last statement in the method875
A method should have only one exit point, and that should be the last statement in the method878
A method should have only one exit point, and that should be the last statement in the method883
A method should have only one exit point, and that should be the last statement in the method888
A method should have only one exit point, and that should be the last statement in the method897
A method should have only one exit point, and that should be the last statement in the method913
A method should have only one exit point, and that should be the last statement in the method946
A method should have only one exit point, and that should be the last statement in the method979
Found 'DU'-anomaly for variable 'argument' (lines '1,003'-'1,006').1003 - 1006
Found 'DU'-anomaly for variable 'argument' (lines '1,014'-'1,018').1014 - 1018
Found 'DU'-anomaly for variable 'responseString' (lines '1,039'-'1,043').1039 - 1043
Use explicit scoping instead of the default package private level1052 - 1056
Use explicit scoping instead of the default package private level1064 - 1067
Each class should declare at least one constructor1074 - 1083

org/apache/james/remotemanager/RemoteManagerMBean.java

ViolationLine
Avoid modifiers which are implied by the context36
Avoid modifiers which are implied by the context45
Avoid modifiers which are implied by the context54
Avoid modifiers which are implied by the context63

org/apache/james/security/DigestUtil.java

ViolationLine
Found 'DU'-anomaly for variable 'alg' (lines '50'-'85').50 - 85
Found 'DD'-anomaly for variable 'alg' (lines '50'-'66').50 - 66
Found 'DU'-anomaly for variable 'file' (lines '51'-'85').51 - 85
Found 'DD'-anomaly for variable 'file' (lines '51'-'69').51 - 69
A method should have only one exit point, and that should be the last statement in the method55
A method should have only one exit point, and that should be the last statement in the method63
Found 'DU'-anomaly for variable 'alg' (lines '66'-'85').66 - 85
Found 'DD'-anomaly for variable 'alg' (lines '66'-'66').66
Found 'DU'-anomaly for variable 'file' (lines '69'-'85').69 - 85
Found 'DD'-anomaly for variable 'file' (lines '69'-'69').69
A method should have only one exit point, and that should be the last statement in the method75
Found 'DU'-anomaly for variable 'b' (lines '105'-'136').105 - 136
Found 'DU'-anomaly for variable 'count' (lines '106'-'136').106 - 136
Found 'DD'-anomaly for variable 'read' (lines '107'-'114').107 - 114
Found 'DU'-anomaly for variable 'read' (lines '107'-'136').107 - 136
Found 'DD'-anomaly for variable 'fis' (lines '108'-'112').108 - 112
Found 'DD'-anomaly for variable 'fos' (lines '109'-'124').109 - 124
Avoid empty catch blocks134

org/apache/james/security/InitJCE.java

ViolationLine
Each class should declare at least one constructor40 - 68

org/apache/james/security/KeyHolder.java

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

org/apache/james/security/KeyStoreHolder.java

ViolationLine
Found 'DU'-anomaly for variable 'certs' (lines '117'-'154').117 - 154
Found 'DD'-anomaly for variable 'i' (lines '125'-'125').125
Found 'DU'-anomaly for variable 'i' (lines '125'-'154').125 - 154
A method should have only one exit point, and that should be the last statement in the method196

org/apache/james/security/SMIMEAttributeNames.java

ViolationLine
Avoid modifiers which are implied by the context33
Avoid modifiers which are implied by the context40
Avoid modifiers which are implied by the context45
Avoid modifiers which are implied by the context50

org/apache/james/services/DNSServer.java

ViolationLine
Found 'DU'-anomaly for variable 'hostname' (lines '46'-'46').46
Found 'DU'-anomaly for variable 'domainName' (lines '60'-'60').60

org/apache/james/services/JamesConnectionManager.java

ViolationLine
Found 'DU'-anomaly for variable 'threadPool' (lines '60'-'65').60 - 65
Found 'DU'-anomaly for variable 'maxOpenConnections' (lines '60'-'65').60 - 65
Found 'DU'-anomaly for variable 'handlerFactory' (lines '60'-'65').60 - 65
Found 'DU'-anomaly for variable 'socket' (lines '60'-'65').60 - 65
Found 'DU'-anomaly for variable 'name' (lines '60'-'65').60 - 65
Found 'DU'-anomaly for variable 'maxOpenConnections' (lines '77'-'81').77 - 81
Found 'DU'-anomaly for variable 'handlerFactory' (lines '77'-'81').77 - 81
Found 'DU'-anomaly for variable 'socket' (lines '77'-'81').77 - 81
Found 'DU'-anomaly for variable 'name' (lines '77'-'81').77 - 81
Found 'DU'-anomaly for variable 'threadPool' (lines '94'-'98').94 - 98
Found 'DU'-anomaly for variable 'handlerFactory' (lines '94'-'98').94 - 98
Found 'DU'-anomaly for variable 'socket' (lines '94'-'98').94 - 98
Found 'DU'-anomaly for variable 'name' (lines '94'-'98').94 - 98
Found 'DU'-anomaly for variable 'handlerFactory' (lines '110'-'113').110 - 113
Found 'DU'-anomaly for variable 'socket' (lines '110'-'113').110 - 113
Found 'DU'-anomaly for variable 'name' (lines '110'-'113').110 - 113

org/apache/james/services/JamesUser.java

ViolationLine
Found 'DU'-anomaly for variable 'pass' (lines '39'-'39').39
Found 'DU'-anomaly for variable 'forward' (lines '47'-'47').47
Found 'DU'-anomaly for variable 'address' (lines '60'-'60').60
Found 'DU'-anomaly for variable 'alias' (lines '71'-'71').71
Found 'DU'-anomaly for variable 'address' (lines '83'-'83').83

org/apache/james/services/MailRepository.java

ViolationLine
Found 'DU'-anomaly for variable 'mc' (lines '53'-'53').53
Found 'DU'-anomaly for variable 'key' (lines '70'-'70').70
Found 'DU'-anomaly for variable 'mail' (lines '77'-'77').77
Found 'DU'-anomaly for variable 'mails' (lines '85'-'85').85
Found 'DU'-anomaly for variable 'key' (lines '92'-'92').92
Found 'DU'-anomaly for variable 'key' (lines '101'-'101').101
Found 'DU'-anomaly for variable 'key' (lines '110'-'110').110

org/apache/james/services/MailServer.java

ViolationLine
Found 'DU'-anomaly for variable 'sender' (lines '64'-'65').64 - 65
Found 'DU'-anomaly for variable 'recipients' (lines '64'-'65').64 - 65
Found 'DU'-anomaly for variable 'msg' (lines '64'-'65').64 - 65
Found 'DU'-anomaly for variable 'sender' (lines '78'-'79').78 - 79
Found 'DU'-anomaly for variable 'recipients' (lines '78'-'79').78 - 79
Found 'DU'-anomaly for variable 'msg' (lines '78'-'79').78 - 79
Found 'DU'-anomaly for variable 'mail' (lines '86'-'87').86 - 87
Found 'DU'-anomaly for variable 'message' (lines '96'-'97').96 - 97
Found 'DU'-anomaly for variable 'userName' (lines '106'-'106').106
Found 'DU'-anomaly for variable 'password' (lines '126'-'126').126
Found 'DU'-anomaly for variable 'userName' (lines '126'-'126').126
Found 'DU'-anomaly for variable 'serverName' (lines '134'-'134').134

org/apache/james/services/MailetLoader.java

ViolationLine
Avoid modifiers which are implied by the context43 - 44
Found 'DU'-anomaly for variable 'mailetName' (lines '43'-'44').43 - 44
Found 'DU'-anomaly for variable 'configuration' (lines '43'-'44').43 - 44

org/apache/james/services/MatcherLoader.java

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

org/apache/james/services/SpoolRepository.java

ViolationLine
Avoid modifiers which are implied by the context43 - 64
Found 'DU'-anomaly for variable 'key' (lines '55'-'55').55
Found 'DU'-anomaly for variable 'lastUpdated' (lines '55'-'55').55
Found 'DU'-anomaly for variable 'state' (lines '55'-'55').55
Found 'DU'-anomaly for variable 'errorMessage' (lines '55'-'55').55
Found 'DU'-anomaly for variable 'delay' (lines '90'-'90').90
Found 'DU'-anomaly for variable 'filter' (lines '102'-'102').102

org/apache/james/services/User.java

ViolationLine
Found 'DU'-anomaly for variable 'pass' (lines '44'-'44').44
Found 'DU'-anomaly for variable 'newPass' (lines '53'-'53').53

org/apache/james/services/UsersRepository.java

ViolationLine
Found 'DU'-anomaly for variable 'user' (lines '49'-'49').49
Found 'DU'-anomaly for variable 'attributes' (lines '58'-'58').58
Found 'DU'-anomaly for variable 'name' (lines '58'-'58').58
Found 'DU'-anomaly for variable 'password' (lines '69'-'69').69
Found 'DU'-anomaly for variable 'username' (lines '69'-'69').69
Found 'DU'-anomaly for variable 'name' (lines '80'-'80').80
Found 'DU'-anomaly for variable 'name' (lines '91'-'91').91
Found 'DU'-anomaly for variable 'name' (lines '100'-'100').100
Found 'DU'-anomaly for variable 'user' (lines '108'-'108').108
Found 'DU'-anomaly for variable 'name' (lines '115'-'115').115
Found 'DU'-anomaly for variable 'name' (lines '123'-'123').123
Found 'DU'-anomaly for variable 'name' (lines '132'-'132').132
Found 'DU'-anomaly for variable 'password' (lines '145'-'145').145
Found 'DU'-anomaly for variable 'name' (lines '145'-'145').145

org/apache/james/services/UsersStore.java

ViolationLine
Found 'DU'-anomaly for variable 'name' (lines '46'-'46').46

org/apache/james/smtpserver/AddHeaderHandler.java

ViolationLine
Each class should declare at least one constructor28 - 30

org/apache/james/smtpserver/AuthCmdHandler.java

ViolationLine
Each class should declare at least one constructor33 - 274
Found 'DU'-anomaly for variable 'argument' (lines '73'-'101').73 - 101
Found 'DD'-anomaly for variable 'responseString' (lines '75'-'77').75 - 77
Found 'DD'-anomaly for variable 'responseString' (lines '75'-'81').75 - 81
Found 'DU'-anomaly for variable 'responseString' (lines '75'-'101').75 - 101
Found 'DD'-anomaly for variable 'initialResponse' (lines '84'-'86').84 - 86
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 method95
Found 'DD'-anomaly for variable 'responseString' (lines '119'-'121').119 - 121
Found 'DD'-anomaly for variable 'userpass' (lines '119'-'123').119 - 123
Found 'DD'-anomaly for variable 'user' (lines '119'-'148').119 - 148
Found 'DD'-anomaly for variable 'pass' (lines '119'-'150').119 - 150
Found 'DD'-anomaly for variable 'userpass' (lines '119'-'125').119 - 125
Found 'DD'-anomaly for variable 'responseString' (lines '119'-'181').119 - 181
Found 'DD'-anomaly for variable 'responseString' (lines '119'-'185').119 - 185
Found 'DD'-anomaly for variable 'responseString' (lines '119'-'189').119 - 189
Found 'DD'-anomaly for variable 'pass' (lines '150'-'168').150 - 168
Assigning an Object to null is a code smell. Consider refactoring.172
Found 'DU'-anomaly for variable 'authTokenizer' (lines '172'-'194').172 - 194
Avoid empty catch blocks175 - 178
Avoid unnecessary return statements193
Found 'DD'-anomaly for variable 'responseString' (lines '204'-'206').204 - 206
Found 'DD'-anomaly for variable 'user' (lines '204'-'208').204 - 208
Found 'DD'-anomaly for variable 'pass' (lines '204'-'223').204 - 223
Found 'DD'-anomaly for variable 'user' (lines '204'-'210').204 - 210
Found 'DD'-anomaly for variable 'responseString' (lines '204'-'221').204 - 221
Assigning an Object to null is a code smell. Consider refactoring.218
Assigning an Object to null is a code smell. Consider refactoring.230
Avoid unnecessary return statements249
Found 'DU'-anomaly for variable 'initialResponse' (lines '259'-'271').259 - 271
Found 'DU'-anomaly for variable 'authType' (lines '259'-'271').259 - 271
Avoid unnecessary return statements270

org/apache/james/smtpserver/CommandHandler.java

ViolationLine
Found 'DU'-anomaly for variable 'session' (lines '32'-'32').32

org/apache/james/smtpserver/ConnectHandler.java

ViolationLine
Found 'DU'-anomaly for variable 'session' (lines '32'-'32').32

org/apache/james/smtpserver/DNSRBLHandler.java

ViolationLine
Each class should declare at least one constructor32 - 179
Found 'DU'-anomaly for variable 'rblserverCollection' (lines '48'-'79').48 - 79
Found 'DU'-anomaly for variable 'ipAddress' (lines '102'-'177').102 - 177
A method should have only one exit point, and that should be the last statement in the method110
Found 'DU'-anomaly for variable 'reversedOctets' (lines '119'-'177').119 - 177
Found 'DU'-anomaly for variable 'addr' (lines '124'-'177').124 - 177
A method should have only one exit point, and that should be the last statement in the method140
A method should have only one exit point, and that should be the last statement in the method167

org/apache/james/smtpserver/DataCmdHandler.java

ViolationLine
Each class should declare at least one constructor51 - 317
Found 'DD'-anomaly for variable 'responseString' (lines '94'-'96').94 - 96
Found 'DD'-anomaly for variable 'responseString' (lines '94'-'100').94 - 100
Found 'DD'-anomaly for variable 'responseString' (lines '94'-'103').94 - 103
Found 'DD'-anomaly for variable 'responseString' (lines '94'-'106').94 - 106
Found 'DD'-anomaly for variable 'msgIn' (lines '108'-'110').108 - 110
Found 'DD'-anomaly for variable 'headers' (lines '131'-'132').131 - 132
Assigning an Object to null is a code smell. Consider refactoring.134
Found 'DU'-anomaly for variable 'headers' (lines '134'-'179').134 - 179
Avoid empty catch blocks172 - 174
Assigning an Object to null is a code smell. Consider refactoring.175
Assigning an Object to null is a code smell. Consider refactoring.240
Found 'DU'-anomaly for variable 'headerLineBuffer' (lines '240'-'248').240 - 248
Found 'DD'-anomaly for variable 'headersIn' (lines '260'-'264').260 - 264
Found 'DD'-anomaly for variable 'mail' (lines '261'-'266').261 - 266
Found 'DD'-anomaly for variable 'recipientCollection' (lines '262'-'265').262 - 265
Assigning an Object to null is a code smell. Consider refactoring.304
Found 'DU'-anomaly for variable 'recipientCollection' (lines '304'-'315').304 - 315
Avoid empty catch blocks308 - 310
Assigning an Object to null is a code smell. Consider refactoring.312
Found 'DU'-anomaly for variable 'headersIn' (lines '312'-'315').312 - 315

org/apache/james/smtpserver/EhloCmdHandler.java

ViolationLine
Each class should declare at least one constructor34 - 156
Found 'DD'-anomaly for variable 'responseString' (lines '81'-'99').81 - 99
Found 'DD'-anomaly for variable 'responseString' (lines '81'-'107').81 - 107
Found 'DU'-anomaly for variable 'responseString' (lines '81'-'154').81 - 154
Found 'DU'-anomaly for variable 'responseBuffer' (lines '82'-'154').82 - 154
Found 'DD'-anomaly for variable 'badEhlo' (lines '83'-'98').83 - 98
Found 'DU'-anomaly for variable 'badEhlo' (lines '83'-'154').83 - 154
These nested if statements could be combined91 - 103
Found 'DU'-anomaly for variable 'badEhlo' (lines '98'-'154').98 - 154

org/apache/james/smtpserver/ExpnCmdHandler.java

ViolationLine
Each class should declare at least one constructor27 - 46

org/apache/james/smtpserver/HeloCmdHandler.java

ViolationLine
Each class should declare at least one constructor33 - 126
Found 'DU'-anomaly for variable 'session' (lines '80'-'125').80 - 125
Found 'DD'-anomaly for variable 'responseString' (lines '81'-'98').81 - 98
Found 'DD'-anomaly for variable 'responseString' (lines '81'-'107').81 - 107
Found 'DD'-anomaly for variable 'responseString' (lines '81'-'122').81 - 122
Found 'DU'-anomaly for variable 'responseString' (lines '81'-'125').81 - 125
Found 'DD'-anomaly for variable 'badHelo' (lines '82'-'97').82 - 97
Found 'DU'-anomaly for variable 'badHelo' (lines '82'-'125').82 - 125
These nested if statements could be combined91 - 103
Found 'DU'-anomaly for variable 'badHelo' (lines '97'-'125').97 - 125

org/apache/james/smtpserver/HelpCmdHandler.java

ViolationLine
Each class should declare at least one constructor27 - 45

org/apache/james/smtpserver/MailCmdHandler.java

ViolationLine
Each class should declare at least one constructor40 - 294
Found 'DD'-anomaly for variable 'responseString' (lines '97'-'108').97 - 108
Found 'DD'-anomaly for variable 'responseString' (lines '97'-'111').97 - 111
Found 'DD'-anomaly for variable 'responseString' (lines '97'-'115').97 - 115
Found 'DU'-anomaly for variable 'responseString' (lines '97'-'236').97 - 236
Found 'DD'-anomaly for variable 'responseString' (lines '97'-'161').97 - 161
Found 'DD'-anomaly for variable 'responseString' (lines '97'-'187').97 - 187
Found 'DD'-anomaly for variable 'responseString' (lines '97'-'220').97 - 220
Found 'DD'-anomaly for variable 'responseString' (lines '97'-'232').97 - 232
Found 'DU'-anomaly for variable 'responseBuffer' (lines '98'-'236').98 - 236
Found 'DD'-anomaly for variable 'sender' (lines '99'-'104').99 - 104
Found 'DU'-anomaly for variable 'sender' (lines '99'-'236').99 - 236
Found 'DU'-anomaly for variable 'badSenderDomain' (lines '100'-'236').100 - 236
Found 'DD'-anomaly for variable 'badSenderDomain' (lines '100'-'215').100 - 215
Found 'DU'-anomaly for variable 'sender' (lines '104'-'236').104 - 236
Found 'DD'-anomaly for variable 'mailOptionName' (lines '133'-'136').133 - 136
Found 'DD'-anomaly for variable 'mailOptionValue' (lines '134'-'137').134 - 137
Found 'DU'-anomaly for variable 'mailOptionValue' (lines '134'-'236').134 - 236
Found 'DD'-anomaly for variable 'mailOptionValue' (lines '134'-'134').134
Found 'DU'-anomaly for variable 'mailOptionValue' (lines '137'-'236').137 - 236
Found 'DD'-anomaly for variable 'mailOptionValue' (lines '137'-'134').137 - 134
A method should have only one exit point, and that should be the last statement in the method144
A method should have only one exit point, and that should be the last statement in the method171
Found 'DD'-anomaly for variable 'senderAddress' (lines '173'-'185').173 - 185
Avoid empty if statements176 - 178
Found 'DU'-anomaly for variable 'senderAddress' (lines '185'-'236').185 - 236
These nested if statements could be combined208 - 224
Found 'DU'-anomaly for variable 'tempSender' (lines '246'-'291').246 - 291
Found 'DD'-anomaly for variable 'size' (lines '247'-'249').247 - 249
Found 'DU'-anomaly for variable 'size' (lines '249'-'291').249 - 291
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 method284

org/apache/james/smtpserver/MessageHandler.java

ViolationLine
Found 'DU'-anomaly for variable 'session' (lines '32'-'32').32

org/apache/james/smtpserver/NoopCmdHandler.java

ViolationLine
Each class should declare at least one constructor27 - 46

org/apache/james/smtpserver/QuitCmdHandler.java

ViolationLine
Each class should declare at least one constructor27 - 69
Found 'DD'-anomaly for variable 'responseString' (lines '55'-'60').55 - 60
Found 'DD'-anomaly for variable 'responseString' (lines '55'-'62').55 - 62

org/apache/james/smtpserver/RcptCmdHandler.java

ViolationLine
Each class should declare at least one constructor36 - 337
Found 'DD'-anomaly for variable 'responseString' (lines '87'-'99').87 - 99
Found 'DD'-anomaly for variable 'responseString' (lines '87'-'103').87 - 103
Found 'DD'-anomaly for variable 'responseString' (lines '87'-'122').87 - 122
Found 'DD'-anomaly for variable 'responseString' (lines '87'-'149').87 - 149
Found 'DU'-anomaly for variable 'responseBuffer' (lines '88'-'304').88 - 304
Found 'DU'-anomaly for variable 'maxRcptReached' (lines '89'-'304').89 - 304
Found 'DU'-anomaly for variable 'useTarpit' (lines '90'-'304').90 - 304
Found 'DD'-anomaly for variable 'recipient' (lines '92'-'95').92 - 95
Found 'DU'-anomaly for variable 'recipient' (lines '92'-'304').92 - 304
Found 'DU'-anomaly for variable 'recipient' (lines '95'-'304').95 - 304
Found 'DU'-anomaly for variable 'rcptColl' (lines '108'-'304').108 - 304
Found 'DD'-anomaly for variable 'rcptOptionString' (lines '114'-'116').114 - 116
Found 'DU'-anomaly for variable 'rcptOptionString' (lines '114'-'304').114 - 304
Found 'DU'-anomaly for variable 'rcptOptionString' (lines '116'-'304').116 - 304
A method should have only one exit point, and that should be the last statement in the method132
Found 'DD'-anomaly for variable 'recipientAddress' (lines '134'-'142').134 - 142
Found 'DU'-anomaly for variable 'recipientAddress' (lines '142'-'304').142 - 304
A method should have only one exit point, and that should be the last statement in the method160
A method should have only one exit point, and that should be the last statement in the method169
A method should have only one exit point, and that should be the last statement in the method184
A method should have only one exit point, and that should be the last statement in the method206
Assigning an Object to null is a code smell. Consider refactoring.249
Found 'DU'-anomaly for variable 'recipient' (lines '307'-'318').307 - 318
Found 'DU'-anomaly for variable 'startCount' (lines '322'-'331').322 - 331
A method should have only one exit point, and that should be the last statement in the method327

org/apache/james/smtpserver/RsetCmdHandler.java

ViolationLine
Each class should declare at least one constructor27 - 69
Found 'DD'-anomaly for variable 'responseString' (lines '51'-'60').51 - 60
Found 'DD'-anomaly for variable 'responseString' (lines '51'-'62').51 - 62

org/apache/james/smtpserver/SMTPHandler.java

ViolationLine
Each class should declare at least one constructor57 - 791
Use explicit scoping instead of the default package private level88
Use explicit scoping instead of the default package private level93
Use explicit scoping instead of the default package private level98
Perhaps 'theWatchdogTarget' could be replaced by a local variable.204
Use explicit scoping instead of the default package private level216 - 218
Use explicit scoping instead of the default package private level225 - 227
Use explicit scoping instead of the default package private level235 - 237
Use explicit scoping instead of the default package private level242 - 260
Avoid empty catch blocks250 - 252
Found 'UR'-anomaly for variable 'exceptionString' (lines '265'-'265').265
Assigning an Object to null is a code smell. Consider refactoring.367
Assigning an Object to null is a code smell. Consider refactoring.368
Assigning an Object to null is a code smell. Consider refactoring.426
Assigning an Object to null is a code smell. Consider refactoring.491
Assigning an Object to null is a code smell. Consider refactoring.492
Assigning an Object to null is a code smell. Consider refactoring.493
Assigning an Object to null is a code smell. Consider refactoring.494
Assigning an Object to null is a code smell. Consider refactoring.495
Assigning an Object to null is a code smell. Consider refactoring.496
Assigning an Object to null is a code smell. Consider refactoring.497
Assigning an Object to null is a code smell. Consider refactoring.501
Assigning an Object to null is a code smell. Consider refactoring.514
Assigning an Object to null is a code smell. Consider refactoring.518
Found 'DU'-anomaly for variable 'responseString' (lines '532'-'536').532 - 536
Use explicit scoping instead of the default package private level545 - 549
Use explicit scoping instead of the default package private level557 - 560
Each class should declare at least one constructor568 - 577

org/apache/james/smtpserver/SMTPHandlerChain.java

ViolationLine
Each class should declare at least one constructor44 - 253
Perhaps 'unknownHandler' could be replaced by a local variable.50
Found 'UR'-anomaly for variable 'className' (lines '64'-'64').64
Found 'DD'-anomaly for variable 'commandName' (lines '124'-'125').124 - 125
Found 'DD'-anomaly for variable 'found' (lines '174'-'180').174 - 180
Use explicit scoping instead of the default package private level220 - 233
A method should have only one exit point, and that should be the last statement in the method222
Use explicit scoping instead of the default package private level240 - 242
Use explicit scoping instead of the default package private level249 - 251

org/apache/james/smtpserver/SMTPHandlerConfigurationData.java

ViolationLine
Found 'DU'-anomaly for variable 'remoteIP' (lines '59'-'59').59
Found 'DU'-anomaly for variable 'remoteIP' (lines '68'-'68').68

org/apache/james/smtpserver/SMTPServer.java

ViolationLine
Each class should declare at least one constructor55 - 445
Use explicit scoping instead of the default package private level62
Use explicit scoping instead of the default package private level67
Perhaps 'theHandlerFactory' could be replaced by a local variable.129
Each class should declare at least one constructor334 - 357
Found 'DU'-anomaly for variable 'object' (lines '354'-'356').354 - 356
Avoid unnecessary return statements355
Each class should declare at least one constructor362 - 444
Found 'DU'-anomaly for variable 'remoteIP' (lines '389'-'395').389 - 395
Found 'DD'-anomaly for variable 'relayingAllowed' (lines '390'-'392').390 - 392
Found 'DU'-anomaly for variable 'remoteIP' (lines '400'-'407').400 - 407
A method should have only one exit point, and that should be the last statement in the method401
Found 'DD'-anomaly for variable 'authRequired' (lines '402'-'404').402 - 404

org/apache/james/smtpserver/SMTPServerMBean.java

ViolationLine
Avoid modifiers which are implied by the context35
Avoid modifiers which are implied by the context44
Avoid modifiers which are implied by the context53
Avoid modifiers which are implied by the context62

org/apache/james/smtpserver/SMTPSession.java

ViolationLine
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
Found 'DU'-anomaly for variable 'respString' (lines '48'-'48').48
Found 'DU'-anomaly for variable 'mail' (lines '107'-'107').107
Found 'DU'-anomaly for variable 'blocklisted' (lines '167'-'167').167
Found 'DU'-anomaly for variable 'user' (lines '209'-'209').209

org/apache/james/smtpserver/SendMailHandler.java

ViolationLine
Each class should declare at least one constructor38 - 119
Found 'DD'-anomaly for variable 'responseString' (lines '60'-'94').60 - 94
Found 'DD'-anomaly for variable 'responseString' (lines '60'-'107').60 - 107
Found 'DD'-anomaly for variable 'recipientString' (lines '64'-'66').64 - 66
Found 'DU'-anomaly for variable 'recipientString' (lines '64'-'117').64 - 117
Found 'DU'-anomaly for variable 'recipientString' (lines '66'-'117').66 - 117

org/apache/james/smtpserver/SetMimeHeaderHandler.java

ViolationLine
Each class should declare at least one constructor32 - 97
Found 'DU'-anomaly for variable 'message' (lines '82'-'93').82 - 93

org/apache/james/smtpserver/SizeLimitedInputStream.java

ViolationLine
It is a good practice to call super() in a constructor50 - 53

org/apache/james/smtpserver/UnknownCmdHandler.java

ViolationLine
Each class should declare at least one constructor27 - 56

org/apache/james/smtpserver/VrfyCmdHandler.java

ViolationLine
Each class should declare at least one constructor27 - 44

org/apache/james/transport/JamesMailetLoader.java

ViolationLine
Each class should declare at least one constructor32 - 76
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '43'-'43').43
Found 'DU'-anomaly for variable 'configuration' (lines '43'-'75').43 - 75
Avoid empty catch blocks56 - 58
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '60'-'75').60 - 75

org/apache/james/transport/JamesMatcherLoader.java

ViolationLine
Each class should declare at least one constructor33 - 82
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '44'-'44').44
Found 'DD'-anomaly for variable 'condition' (lines '46'-'49').46 - 49
Found 'DU'-anomaly for variable 'condition' (lines '46'-'81').46 - 81
Found 'DU'-anomaly for variable 'condition' (lines '49'-'81').49 - 81
Avoid empty catch blocks62 - 64
Found 'DD'-anomaly for variable 'exceptionBuffer' (lines '66'-'76').66 - 76
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '76'-'81').76 - 81

org/apache/james/transport/JamesSpoolManager.java

ViolationLine
Each class should declare at least one constructor55 - 491
Found 'DD'-anomaly for variable 'mailet' (lines '170'-'203').170 - 203
Found 'DD'-anomaly for variable 'matcher' (lines '171'-'173').171 - 173
Found 'DD'-anomaly for variable 'key' (lines '297'-'300').297 - 300
Assigning an Object to null is a code smell. Consider refactoring.336
Found 'DD'-anomaly for variable 'mail' (lines '336'-'299').336 - 299
Found 'DU'-anomaly for variable 'mail' (lines '336'-'365').336 - 365
Found 'UR'-anomaly for variable 'exceptionMessage' (lines '373'-'373').373
Found 'DU'-anomaly for variable 'mail' (lines '373'-'451').373 - 451
A method should have only one exit point, and that should be the last statement in the method378
Found 'DD'-anomaly for variable 'logMessageBuffer' (lines '395'-'397').395 - 397
Found 'DD'-anomaly for variable 'logMessageBuffer' (lines '395'-'407').395 - 407
Found 'DU'-anomaly for variable 'logMessageBuffer' (lines '395'-'451').395 - 451
Avoid empty catch blocks475

org/apache/james/transport/LinearProcessor.java

ViolationLine
Each class should declare at least one constructor82 - 548
Found 'DU'-anomaly for variable 'debugEnabled' (lines '145'-'153').145 - 153
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '204'-'241').204 - 241
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '204'-'241').204 - 241
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '204'-'241').204 - 241
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '204'-'241').204 - 241
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '204'-'241').204 - 241
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '204'-'241').204 - 241
Found 'DU'-anomaly for variable 'originalState' (lines '298'-'471').298 - 471
Found 'DU'-anomaly for variable 'originalMail' (lines '303'-'471').303 - 471
Assigning an Object to null is a code smell. Consider refactoring.306
Found 'DD'-anomaly for variable 'mail' (lines '306'-'325').306 - 325
Found 'DD'-anomaly for variable 'i' (lines '307'-'328').307 - 328
Assigning an Object to null is a code smell. Consider refactoring.325
Found 'DD'-anomaly for variable 'mail' (lines '325'-'331').325 - 331
Found 'DD'-anomaly for variable 'recipients' (lines '344'-'357').344 - 357
Found 'DD'-anomaly for variable 'logMessageBuffer' (lines '346'-'348').346 - 348
Found 'DD'-anomaly for variable 'recipients' (lines '360'-'376').360 - 376
Found 'DD'-anomaly for variable 'recipients' (lines '360'-'378').360 - 378
Assigning an Object to null is a code smell. Consider refactoring.450
Assigning an Object to null is a code smell. Consider refactoring.462
Found 'DD'-anomaly for variable 'mail' (lines '462'-'325').462 - 325
Found 'DU'-anomaly for variable 'mail' (lines '462'-'471').462 - 471
Assigning an Object to null is a code smell. Consider refactoring.540

org/apache/james/transport/Loader.java

ViolationLine
Each class should declare at least one constructor41 - 111
Found 'DU'-anomaly for variable 'arg0' (lines '109'-'109').109

org/apache/james/transport/mailets/AbstractAddFooter.java

ViolationLine
Each class should declare at least one constructor38 - 191
A method should have only one exit point, and that should be the last statement in the method134
A method should have only one exit point, and that should be the last statement in the method138
A method should have only one exit point, and that should be the last statement in the method150
A method should have only one exit point, and that should be the last statement in the method167

org/apache/james/transport/mailets/AbstractNotify.java

ViolationLine
Each class should declare at least one constructor72 - 271
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.82
Found 'DD'-anomaly for variable 'addresses' (lines '142'-'143').142 - 143
Found 'UR'-anomaly for variable 'addressString' (lines '210'-'210').210
A method should have only one exit point, and that should be the last statement in the method214
A method should have only one exit point, and that should be the last statement in the method220

org/apache/james/transport/mailets/AbstractRedirect.java

ViolationLine
Each class should declare at least one constructor133 - 1707
Each class should declare at least one constructor154 - 179
Avoid empty catch blocks177
Each class should declare at least one constructor186 - 196
Perhaps 'rfc822DateFormat' could be replaced by a local variable.228
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.261
Found 'DU'-anomaly for variable 'originalMail' (lines '271'-'273').271 - 273
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.283
Found 'DU'-anomaly for variable 'originalMail' (lines '293'-'295').293 - 295
Found 'DU'-anomaly for variable 'originalMail' (lines '323'-'325').323 - 325
Found 'DU'-anomaly for variable 'originalMail' (lines '352'-'354').352 - 354
Found 'DU'-anomaly for variable 'originalMail' (lines '375'-'377').375 - 377
Found 'UR'-anomaly for variable 'addressList' (lines '395'-'395').395
Found 'DU'-anomaly for variable 'newRecipients' (lines '396'-'424').396 - 424
A method should have only one exit point, and that should be the last statement in the method401
Found 'DU'-anomaly for variable 'originalMail' (lines '433'-'443').433 - 443
Assigning an Object to null is a code smell. Consider refactoring.437
Found 'DU'-anomaly for variable 'originalMail' (lines '450'-'457').450 - 457
Found 'DU'-anomaly for variable 'newMail' (lines '450'-'457').450 - 457
Found 'UR'-anomaly for variable 'addressList' (lines '475'-'475').475
Found 'DU'-anomaly for variable 'iaarray' (lines '476'-'502').476 - 502
Found 'DD'-anomaly for variable 'iaarray' (lines '476'-'485').476 - 485
A method should have only one exit point, and that should be the last statement in the method481
Found 'DU'-anomaly for variable 'originalMail' (lines '513'-'534').513 - 534
Assigning an Object to null is a code smell. Consider refactoring.520
Found 'DU'-anomaly for variable 'originalMail' (lines '541'-'548').541 - 548
Found 'DU'-anomaly for variable 'newMail' (lines '541'-'548').541 - 548
Found 'UR'-anomaly for variable 'addressString' (lines '563'-'563').563
A method should have only one exit point, and that should be the last statement in the method570
A method should have only one exit point, and that should be the last statement in the method574
Found 'DU'-anomaly for variable 'originalMail' (lines '592'-'602').592 - 602
Assigning an Object to null is a code smell. Consider refactoring.596
Found 'DU'-anomaly for variable 'originalMail' (lines '610'-'625').610 - 625
Found 'DU'-anomaly for variable 'newMail' (lines '610'-'625').610 - 625
Found 'DD'-anomaly for variable 'iart' (lines '612'-'614').612 - 614
Found 'DD'-anomaly for variable 'iart' (lines '614'-'615').614 - 615
Found 'UR'-anomaly for variable 'addressString' (lines '640'-'640').640
A method should have only one exit point, and that should be the last statement in the method646
A method should have only one exit point, and that should be the last statement in the method650
Found 'DU'-anomaly for variable 'originalMail' (lines '671'-'682').671 - 682
Assigning an Object to null is a code smell. Consider refactoring.675
Assigning an Object to null is a code smell. Consider refactoring.678
Found 'DU'-anomaly for variable 'originalMail' (lines '690'-'700').690 - 700
Found 'DU'-anomaly for variable 'newMail' (lines '690'-'700').690 - 700
Assigning an Object to null is a code smell. Consider refactoring.693
Found 'UR'-anomaly for variable 'addressString' (lines '714'-'714').714
A method should have only one exit point, and that should be the last statement in the method720
A method should have only one exit point, and that should be the last statement in the method724
Found 'DU'-anomaly for variable 'originalMail' (lines '742'-'750').742 - 750
These nested if statements could be combined745 - 747
Assigning an Object to null is a code smell. Consider refactoring.746
Found 'DU'-anomaly for variable 'originalMail' (lines '757'-'765').757 - 765
Found 'DU'-anomaly for variable 'newMail' (lines '757'-'765').757 - 765
Found 'DU'-anomaly for variable 'originalMail' (lines '785'-'787').785 - 787
Found 'DU'-anomaly for variable 'originalMail' (lines '807'-'809').807 - 809
Found 'DU'-anomaly for variable 'newMail' (lines '816'-'842').816 - 842
Found 'DD'-anomaly for variable 'subject' (lines '829'-'833').829 - 833
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.853
Found 'DU'-anomaly for variable 'originalMail' (lines '863'-'865').863 - 865
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.877
Found 'DU'-anomaly for variable 'originalMail' (lines '887'-'889').887 - 889
Found 'DU'-anomaly for variable 'newMail' (lines '895'-'905').895 - 905
Found 'DU'-anomaly for variable 'originalMail' (lines '895'-'905').895 - 905
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.917
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.919
Found 'UR'-anomaly for variable 'logBuffer' (lines '973'-'973').973
Found 'UR'-anomaly for variable 'logBuffer' (lines '973'-'973').973
Found 'DD'-anomaly for variable 'keepMessageId' (lines '975'-'1,028').975 - 1028
Found 'DD'-anomaly for variable 'keepMessageId' (lines '975'-'1,028').975 - 1028
Found 'DU'-anomaly for variable 'logBuffer' (lines '1,061'-'1,079').1061 - 1079
Found 'DU'-anomaly for variable 'logBuffer' (lines '1,061'-'1,079').1061 - 1079
Found 'DD'-anomaly for variable 'count' (lines '1,097'-'1,100').1097 - 1100
Found 'DD'-anomaly for variable 'index' (lines '1,098'-'1,099').1098 - 1099
Avoid assignments in operands1099
Found 'DD'-anomaly for variable 'count' (lines '1,100'-'1,100').1100
Found 'DD'-anomaly for variable 'param' (lines '1,126'-'1,127').1126 - 1127
A method should have only one exit point, and that should be the last statement in the method1129
A method should have only one exit point, and that should be the last statement in the method1132
A method should have only one exit point, and that should be the last statement in the method1135
A method should have only one exit point, and that should be the last statement in the method1138
A method should have only one exit point, and that should be the last statement in the method1141
A method should have only one exit point, and that should be the last statement in the method1144
A method should have only one exit point, and that should be the last statement in the method1154
Found 'DU'-anomaly for variable 'out' (lines '1,216'-'1,316').1216 - 1316
Found 'DU'-anomaly for variable 'head' (lines '1,217'-'1,316').1217 - 1316
Found 'DD'-anomaly for variable 'all' (lines '1,218'-'1,230').1218 - 1230
Found 'DU'-anomaly for variable 'all' (lines '1,218'-'1,316').1218 - 1316
Found 'DU'-anomaly for variable 'newMail' (lines '1,321'-'1,329').1321 - 1329
A method should have only one exit point, and that should be the last statement in the method1343
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,349'-'1,352').1349 - 1352
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,352'-'1,355').1352 - 1355
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,355'-'1,358').1355 - 1358
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,355'-'1,361').1355 - 1361
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,355'-'1,364').1355 - 1364
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,358'-'1,361').1358 - 1361
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,358'-'1,364').1358 - 1364
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,358'-'1,367').1358 - 1367
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,358'-'1,370').1358 - 1370
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,358'-'1,373').1358 - 1373
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,358'-'1,376').1358 - 1376
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,358'-'1,379').1358 - 1379
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,361'-'1,364').1361 - 1364
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,361'-'1,367').1361 - 1367
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,361'-'1,370').1361 - 1370
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,361'-'1,373').1361 - 1373
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,361'-'1,376').1361 - 1376
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,361'-'1,379').1361 - 1379
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,364'-'1,367').1364 - 1367
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,364'-'1,370').1364 - 1370
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,364'-'1,373').1364 - 1373
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,364'-'1,376').1364 - 1376
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,364'-'1,379').1364 - 1379
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,367'-'1,370').1367 - 1370
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,367'-'1,373').1367 - 1373
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,367'-'1,376').1367 - 1376
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,367'-'1,379').1367 - 1379
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,370'-'1,373').1370 - 1373
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,370'-'1,376').1370 - 1376
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,370'-'1,379').1370 - 1379
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,373'-'1,376').1373 - 1376
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,373'-'1,379').1373 - 1379
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,376'-'1,379').1376 - 1379
Found 'DD'-anomaly for variable 'allowed' (lines '1,385'-'1,391').1385 - 1391
Found 'DD'-anomaly for variable 'allowedSpecial' (lines '1,387'-'1,388').1387 - 1388
A method should have only one exit point, and that should be the last statement in the method1422
Found 'UR'-anomaly for variable 'bad' (lines '1,430'-'1,430').1430
Found 'DU'-anomaly for variable 'allowed' (lines '1,436'-'1,455').1436 - 1455
A method should have only one exit point, and that should be the last statement in the method1529
A method should have only one exit point, and that should be the last statement in the method1531
A method should have only one exit point, and that should be the last statement in the method1534
A method should have only one exit point, and that should be the last statement in the method1536
A method should have only one exit point, and that should be the last statement in the method1538
A method should have only one exit point, and that should be the last statement in the method1539
Found 'DU'-anomaly for variable 'mail' (lines '1,558'-'1,610').1558 - 1610
Found 'DD'-anomaly for variable 'parsedAddressCount' (lines '1,571'-'1,578').1571 - 1578
Found 'DD'-anomaly for variable 'parsedAddressCount' (lines '1,578'-'1,578').1578
Found 'DU'-anomaly for variable 'mail' (lines '1,627'-'1,705').1627 - 1705

org/apache/james/transport/mailets/AbstractVirtualUserTable.java

ViolationLine
Each class should declare at least one constructor48 - 316
Found 'DU'-anomaly for variable 'recipientsMap' (lines '198'-'198').198
Found 'DU'-anomaly for variable 'address' (lines '207'-'224').207 - 224
Found 'DU'-anomaly for variable 'mail' (lines '207'-'224').207 - 224
Found 'DU'-anomaly for variable 'code' (lines '211'-'224').211 - 224
Found 'DU'-anomaly for variable 'mail' (lines '237'-'267').237 - 267
Found 'DD'-anomaly for variable 'result' (lines '238'-'258').238 - 258
Found 'DD'-anomaly for variable 'count' (lines '295'-'298').295 - 298
Found 'DD'-anomaly for variable 'index' (lines '296'-'297').296 - 297
Avoid assignments in operands297
Found 'DD'-anomaly for variable 'count' (lines '298'-'298').298

org/apache/james/transport/mailets/AddFooter.java

ViolationLine
Each class should declare at least one constructor29 - 86
Use explicit scoping instead of the default package private level34

org/apache/james/transport/mailets/AddHabeasWarrantMark.java

ViolationLine
Each class should declare at least one constructor75 - 117

org/apache/james/transport/mailets/AddHeader.java

ViolationLine
Each class should declare at least one constructor28 - 39

org/apache/james/transport/mailets/AvalonListserv.java

ViolationLine
Each class should declare at least one constructor68 - 217
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.106
Avoid empty catch blocks107 - 109
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.111
Avoid empty catch blocks112 - 114
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.116
Avoid empty catch blocks117 - 119
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.123
Avoid empty catch blocks124 - 126

org/apache/james/transport/mailets/AvalonListservManager.java

ViolationLine
Each class should declare at least one constructor39 - 96

org/apache/james/transport/mailets/BayesianAnalysis.java

ViolationLine
Each class should declare at least one constructor112 - 498
Perhaps 'theJDBCUtil' could be replaced by a local variable.117 - 121
Perhaps 'rfc822DateFormat' could be replaced by a local variable.142
Found 'DU'-anomaly for variable 'message' (lines '273'-'334').273 - 334
These nested if statements could be combined277 - 280
A method should have only one exit point, and that should be the last statement in the method279
Found 'DU'-anomaly for variable 'baos' (lines '289'-'334').289 - 334
Found 'DU'-anomaly for variable 'probabilityString' (lines '305'-'334').305 - 334
Found 'DU'-anomaly for variable 'senderString' (lines '309'-'334').309 - 334
Found 'DU'-anomaly for variable 'senderString' (lines '311'-'334').311 - 334
A method should have only one exit point, and that should be the last statement in the method362
Found 'DU'-anomaly for variable 'i' (lines '368'-'376').368 - 376
Avoid empty catch blocks387
Found 'DD'-anomaly for variable 'rcptAddr' (lines '401'-'402').401 - 402

org/apache/james/transport/mailets/BayesianAnalysisFeeder.java

ViolationLine
Each class should declare at least one constructor108 - 324
Perhaps 'theJDBCUtil' could be replaced by a local variable.113 - 117
Found 'DU'-anomaly for variable 'dbUpdated' (lines '226'-'310').226 - 310
Found 'DD'-anomaly for variable 'dbUpdated' (lines '226'-'260').226 - 260
Found 'DU'-anomaly for variable 'baos' (lines '230'-'310').230 - 310
Found 'DU'-anomaly for variable 'conn' (lines '232'-'310').232 - 310
Found 'DD'-anomaly for variable 'conn' (lines '232'-'254').232 - 254
Found 'DU'-anomaly for variable 'dbUpdated' (lines '305'-'310').305 - 310
Avoid empty catch blocks307
Avoid empty catch blocks319

org/apache/james/transport/mailets/Bounce.java

ViolationLine
Each class should declare at least one constructor102 - 189
Found 'DD'-anomaly for variable 'apparentlyTo' (lines '149'-'150').149 - 150
Found 'DU'-anomaly for variable 'originalMail' (lines '157'-'159').157 - 159

org/apache/james/transport/mailets/ClamAVScan.java

ViolationLine
Each class should declare at least one constructor180 - 951
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.268
Found 'UR'-anomaly for variable 'logText' (lines '503'-'512').503 - 512
Found 'DD'-anomaly for variable 'address' (lines '505'-'518').505 - 518
Found 'DU'-anomaly for variable 'address' (lines '505'-'531').505 - 531
Found 'DU'-anomaly for variable 'usedAddresses' (lines '507'-'531').507 - 531
A method should have only one exit point, and that should be the last statement in the method575
Found 'DD'-anomaly for variable 'reader' (lines '590'-'598').590 - 598
Found 'DD'-anomaly for variable 'writer' (lines '591'-'599').591 - 599
Found 'DD'-anomaly for variable 'streamSocket' (lines '592'-'609').592 - 609
Found 'DD'-anomaly for variable 'bos' (lines '593'-'610').593 - 610
Found 'DD'-anomaly for variable 'answer' (lines '618'-'622').618 - 622
Found 'DU'-anomaly for variable 'answer' (lines '618'-'711').618 - 711
Found 'DD'-anomaly for variable 'virusFound' (lines '619'-'628').619 - 628
Found 'DD'-anomaly for variable 'logMessage' (lines '620'-'629').620 - 629
Found 'DU'-anomaly for variable 'logMessage' (lines '620'-'711').620 - 711
Found 'DD'-anomaly for variable 'virusFound' (lines '628'-'628').628
Avoid empty catch blocks688
Avoid empty catch blocks693
Avoid empty catch blocks698
Avoid empty catch blocks703
Avoid empty catch blocks708
Found 'UR'-anomaly for variable 'bad' (lines '719'-'719').719
Found 'DU'-anomaly for variable 'allowed' (lines '725'-'744').725 - 744
A method should have only one exit point, and that should be the last statement in the method751
Found 'DD'-anomaly for variable 'socket' (lines '783'-'791').783 - 791
Found 'DU'-anomaly for variable 'ping' (lines '785'-'844').785 - 844
Found 'DD'-anomaly for variable 'pongReceived' (lines '819'-'827').819 - 827
Found 'DD'-anomaly for variable 'pongReceived' (lines '827'-'827').827
Found 'DD'-anomaly for variable 'port' (lines '853'-'856').853 - 856
Avoid empty catch blocks857 - 859
Found 'DD'-anomaly for variable 'sender' (lines '914'-'915').914 - 915
Found 'DD'-anomaly for variable 'rcpts' (lines '923'-'924').923 - 924

org/apache/james/transport/mailets/CommandListservFooter.java

ViolationLine
It is a good practice to call super() in a constructor68 - 76

org/apache/james/transport/mailets/CommandListservManager.java

ViolationLine
Each class should declare at least one constructor101 - 439
Found 'DU'-anomaly for variable 'xmlFile' (lines '222'-'237').222 - 237
Found 'DU'-anomaly for variable 'listName' (lines '225'-'237').225 - 237
Found 'DD'-anomaly for variable 'xmlResources' (lines '227'-'229').227 - 229
Found 'DU'-anomaly for variable 'configuration' (lines '375'-'396').375 - 396
Found 'DU'-anomaly for variable 'className' (lines '375'-'396').375 - 396
Found 'DU'-anomaly for variable 'theClassLoader' (lines '379'-'396').379 - 396
Found 'DD'-anomaly for variable 'listServCommand' (lines '383'-'385').383 - 385
Found 'DU'-anomaly for variable 'name' (lines '422'-'438').422 - 438
A method should have only one exit point, and that should be the last statement in the method431

org/apache/james/transport/mailets/CommandListservProcessor.java

ViolationLine
Each class should declare at least one constructor72 - 577
Found 'DU'-anomaly for variable 'listservAddr' (lines '198'-'233').198 - 233
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 method207
Found 'DU'-anomaly for variable 'message' (lines '244'-'262').244 - 262
Found 'DD'-anomaly for variable 'subj' (lines '257'-'259').257 - 259
A method should have only one exit point, and that should be the last statement in the method303
A method should have only one exit point, and that should be the last statement in the method322
A method should have only one exit point, and that should be the last statement in the method342
Found 'DD'-anomaly for variable 'value' (lines '375'-'377').375 - 377
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.377
Avoid empty catch blocks378 - 380
Found 'DD'-anomaly for variable 'value' (lines '391'-'393').391 - 393
Avoid empty catch blocks394 - 396
Found 'UR'-anomaly for variable 'repName' (lines '411'-'411').411
Found 'DU'-anomaly for variable 'name' (lines '524'-'540').524 - 540
A method should have only one exit point, and that should be the last statement in the method533
A method should have only one exit point, and that should be the last statement in the method568

org/apache/james/transport/mailets/DSNBounce.java

ViolationLine
Each class should declare at least one constructor89 - 675
Found 'DD'-anomaly for variable 'machine' (lines '258'-'261').258 - 261
Found 'DD'-anomaly for variable 'machine' (lines '261'-'263').261 - 263
Found 'DU'-anomaly for variable 'machine' (lines '263'-'287').263 - 287
Found 'DD'-anomaly for variable 'nameType' (lines '300'-'318').300 - 318
Found 'DD'-anomaly for variable 'diagnosticType' (lines '373'-'385').373 - 385
Found 'DD'-anomaly for variable 'diagnosticType' (lines '373'-'387').373 - 387
Found 'DU'-anomaly for variable 'sb' (lines '457'-'544').457 - 544
A method should have only one exit point, and that should be the last statement in the method461
A method should have only one exit point, and that should be the last statement in the method465
A method should have only one exit point, and that should be the last statement in the method470
A method should have only one exit point, and that should be the last statement in the method473
Found 'DU'-anomaly for variable 'sb' (lines '477'-'544').477 - 544
A method should have only one exit point, and that should be the last statement in the method481
Found 'DD'-anomaly for variable 'smtpCode' (lines '484'-'486').484 - 486
Avoid empty catch blocks487 - 488
A method should have only one exit point, and that should be the last statement in the method493
A method should have only one exit point, and that should be the last statement in the method495
A method should have only one exit point, and that should be the last statement in the method497
A method should have only one exit point, and that should be the last statement in the method499
A method should have only one exit point, and that should be the last statement in the method501
A method should have only one exit point, and that should be the last statement in the method503
A method should have only one exit point, and that should be the last statement in the method505
A method should have only one exit point, and that should be the last statement in the method507
A method should have only one exit point, and that should be the last statement in the method509
A method should have only one exit point, and that should be the last statement in the method512
A method should have only one exit point, and that should be the last statement in the method514
A method should have only one exit point, and that should be the last statement in the method516
A method should have only one exit point, and that should be the last statement in the method518
A method should have only one exit point, and that should be the last statement in the method520
A method should have only one exit point, and that should be the last statement in the method526
A method should have only one exit point, and that should be the last statement in the method527
A method should have only one exit point, and that should be the last statement in the method532
A method should have only one exit point, and that should be the last statement in the method535
A method should have only one exit point, and that should be the last statement in the method538
A method should have only one exit point, and that should be the last statement in the method553
A method should have only one exit point, and that should be the last statement in the method565
Found 'DD'-anomaly for variable 'count' (lines '593'-'596').593 - 596
Found 'DD'-anomaly for variable 'index' (lines '594'-'595').594 - 595
Avoid assignments in operands595
Found 'DD'-anomaly for variable 'count' (lines '596'-'596').596
Found 'DD'-anomaly for variable 'apparentlyTo' (lines '659'-'660').659 - 660
Found 'DU'-anomaly for variable 'originalMail' (lines '667'-'669').667 - 669

org/apache/james/transport/mailets/Forward.java

ViolationLine
Each class should declare at least one constructor59 - 202
Found 'UR'-anomaly for variable 'addressList' (lines '111'-'111').111

org/apache/james/transport/mailets/FromRepository.java

ViolationLine
Each class should declare at least one constructor46 - 149
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.76
Avoid empty catch blocks77 - 79
Found 'DU'-anomaly for variable 'processed' (lines '104'-'139').104 - 139

org/apache/james/transport/mailets/GenericListserv.java

ViolationLine
Each class should declare at least one constructor41 - 240
A method should have only one exit point, and that should be the last statement in the method173
A method should have only one exit point, and that should be the last statement in the method180
Found 'DD'-anomaly for variable 'subj' (lines '216'-'218').216 - 218

org/apache/james/transport/mailets/GenericListservManager.java

ViolationLine
Each class should declare at least one constructor32 - 90
Found 'DU'-anomaly for variable 'address' (lines '38'-'38').38
Found 'DU'-anomaly for variable 'address' (lines '44'-'44').44
Found 'DU'-anomaly for variable 'address' (lines '51'-'51').51

org/apache/james/transport/mailets/ICommandListservManager.java

ViolationLine
Avoid modifiers which are implied by the context48
Avoid modifiers which are implied by the context55
Found 'DU'-anomaly for variable 'displayFormat' (lines '55'-'55').55
Avoid modifiers which are implied by the context61
Avoid modifiers which are implied by the context67
Avoid modifiers which are implied by the context74
Found 'DU'-anomaly for variable 'name' (lines '74'-'74').74
Avoid modifiers which are implied by the context80
Avoid modifiers which are implied by the context87
Found 'DU'-anomaly for variable 'mailAddress' (lines '87'-'87').87
Avoid modifiers which are implied by the context93
Avoid modifiers which are implied by the context101
Found 'DU'-anomaly for variable 'errorMessage' (lines '101'-'101').101
Found 'DU'-anomaly for variable 'mail' (lines '101'-'101').101
Found 'DU'-anomaly for variable 'subject' (lines '101'-'101').101
Avoid modifiers which are implied by the context106
Avoid modifiers which are implied by the context112
Avoid modifiers which are implied by the context120
Found 'DU'-anomaly for variable 'names' (lines '120'-'120').120

org/apache/james/transport/mailets/JDBCAlias.java

ViolationLine
Each class should declare at least one constructor56 - 200
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '72'-'72').72
Found 'DD'-anomaly for variable 'conn' (lines '80'-'94').80 - 94
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '101'-'129').101 - 129
Found 'DD'-anomaly for variable 'conn' (lines '134'-'142').134 - 142
Found 'DD'-anomaly for variable 'mappingStmt' (lines '135'-'143').135 - 143
Found 'DD'-anomaly for variable 'mappingRS' (lines '136'-'150').136 - 150
Found 'DU'-anomaly for variable 'mappingRS' (lines '136'-'189').136 - 189
Assigning an Object to null is a code smell. Consider refactoring.176
Found 'DD'-anomaly for variable 'mappingRS' (lines '176'-'150').176 - 150
Found 'DU'-anomaly for variable 'mappingRS' (lines '176'-'189').176 - 189

org/apache/james/transport/mailets/JDBCListserv.java

ViolationLine
Each class should declare at least one constructor59 - 332
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '92'-'92').92
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.113
Avoid empty catch blocks114 - 116
Found 'DD'-anomaly for variable 'conn' (lines '119'-'128').119 - 128
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '135'-'181').135 - 181
Found 'DD'-anomaly for variable 'conn' (lines '240'-'246').240 - 246
Found 'DD'-anomaly for variable 'stmt' (lines '241'-'248').241 - 248
Found 'DD'-anomaly for variable 'rs' (lines '242'-'250').242 - 250
Assigning an Object to null is a code smell. Consider refactoring.273
Found 'DD'-anomaly for variable 'rs' (lines '273'-'283').273 - 283
Assigning an Object to null is a code smell. Consider refactoring.277
Found 'DD'-anomaly for variable 'stmt' (lines '277'-'281').277 - 281
Assigning an Object to null is a code smell. Consider refactoring.298
Assigning an Object to null is a code smell. Consider refactoring.312

org/apache/james/transport/mailets/JDBCVirtualUserTable.java

ViolationLine
Each class should declare at least one constructor91 - 203
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '112'-'112').112
Found 'DD'-anomaly for variable 'conn' (lines '123'-'132').123 - 132
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '139'-'158').139 - 158
Found 'DD'-anomaly for variable 'conn' (lines '167'-'173').167 - 173
Found 'DD'-anomaly for variable 'mappingStmt' (lines '168'-'174').168 - 174
Found 'DD'-anomaly for variable 'mappingRS' (lines '177'-'183').177 - 183

org/apache/james/transport/mailets/LocalDelivery.java

ViolationLine
Each class should declare at least one constructor42 - 133
Found 'DU'-anomaly for variable 'm' (lines '87'-'131').87 - 131
Found 'DU'-anomaly for variable 'm' (lines '87'-'131').87 - 131
Found 'DU'-anomaly for variable 'm' (lines '87'-'131').87 - 131
Found 'DU'-anomaly for variable 'm' (lines '87'-'131').87 - 131
Found 'DU'-anomaly for variable 'm' (lines '87'-'131').87 - 131

org/apache/james/transport/mailets/LogMessage.java

ViolationLine
Each class should declare at least one constructor40 - 122
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.57
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.58
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.59
Avoid empty catch blocks62 - 64
Found 'DD'-anomaly for variable 'read' (lines '83'-'84').83 - 84
Avoid assignments in operands84
Found 'DU'-anomaly for variable 'read' (lines '84'-'99').84 - 99

org/apache/james/transport/mailets/NotifyPostmaster.java

ViolationLine
Each class should declare at least one constructor83 - 164
Found 'DD'-anomaly for variable 'iaarray' (lines '133'-'134').133 - 134
Found 'DD'-anomaly for variable 'iaarray' (lines '134'-'139').134 - 139
A method should have only one exit point, and that should be the last statement in the method155
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.157

org/apache/james/transport/mailets/NotifySender.java

ViolationLine
Each class should declare at least one constructor82 - 163
Found 'DD'-anomaly for variable 'iaarray' (lines '132'-'133').132 - 133
Found 'DD'-anomaly for variable 'iaarray' (lines '133'-'138').133 - 138
A method should have only one exit point, and that should be the last statement in the method154
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.156

org/apache/james/transport/mailets/Null.java

ViolationLine
Each class should declare at least one constructor29 - 49

org/apache/james/transport/mailets/PostmasterAlias.java

ViolationLine
Each class should declare at least one constructor39 - 83
Found 'DU'-anomaly for variable 'recipientsToRemove' (lines '51'-'73').51 - 73
Found 'DU'-anomaly for variable 'mailetContext' (lines '52'-'73').52 - 73
Found 'DD'-anomaly for variable 'postmasterAddressed' (lines '53'-'66').53 - 66
Found 'DD'-anomaly for variable 'postmasterAddressed' (lines '66'-'66').66

org/apache/james/transport/mailets/Redirect.java

ViolationLine
Each class should declare at least one constructor269 - 443
Found 'UR'-anomaly for variable 'addressList' (lines '331'-'331').331
Found 'DU'-anomaly for variable 'newRecipients' (lines '332'-'360').332 - 360
A method should have only one exit point, and that should be the last statement in the method337
Found 'UR'-anomaly for variable 'addressList' (lines '371'-'371').371
Found 'DU'-anomaly for variable 'iaarray' (lines '372'-'398').372 - 398
Found 'DD'-anomaly for variable 'iaarray' (lines '372'-'381').372 - 381
A method should have only one exit point, and that should be the last statement in the method377
Found 'UR'-anomaly for variable 'addressString' (lines '407'-'407').407
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 method417

org/apache/james/transport/mailets/RemoteDelivery.java

ViolationLine
Each class should declare at least one constructor99 - 1402
Each class should declare at least one constructor142 - 199
Use explicit scoping instead of the default package private level147
Found 'DU'-anomaly for variable 'key' (lines '157'-'184').157 - 184
Found 'DU'-anomaly for variable 'lastUpdated' (lines '157'-'184').157 - 184
Found 'DU'-anomaly for variable 'errorMessage' (lines '157'-'184').157 - 184
A method should have only one exit point, and that should be the last statement in the method163
A method should have only one exit point, and that should be the last statement in the method171
A method should have only one exit point, and that should be the last statement in the method178
A method should have only one exit point, and that should be the last statement in the method192
Perhaps 'deliveryThreadCount' could be replaced by a local variable.212
Perhaps 'bindAddress' could be replaced by a local variable.216
Perhaps 'delayFilter' could be replaced by a local variable.225
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.232
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.294
Found 'DU'-anomaly for variable 'gatewayPort' (lines '299'-'370').299 - 370
Avoid empty catch blocks342 - 343
Found 'DU'-anomaly for variable 'sfe' (lines '375'-'410').375 - 410
Avoid assignments in operands389
Found 'UR'-anomaly for variable 'lastError' (lines '423'-'423').423
Found 'DU'-anomaly for variable 'session' (lines '423'-'777').423 - 777
Found 'DU'-anomaly for variable 'message' (lines '428'-'777').428 - 777
Found 'DD'-anomaly for variable 'addr' (lines '432'-'436').432 - 436
Found 'DU'-anomaly for variable 'j' (lines '434'-'777').434 - 777
Found 'DD'-anomaly for variable 'addr' (lines '436'-'436').436
A method should have only one exit point, and that should be the last statement in the method441
Found 'DD'-anomaly for variable 'targetServers' (lines '446'-'452').446 - 452
Found 'DD'-anomaly for variable 'targetServers' (lines '446'-'463').446 - 463
A method should have only one exit point, and that should be the last statement in the method460
Found 'DU'-anomaly for variable 'lastError' (lines '466'-'777').466 - 777
Found 'DD'-anomaly for variable 'transport' (lines '497'-'499').497 - 499
Found 'DU'-anomaly for variable 'smtpTransport' (lines '517'-'777').517 - 777
Assigning an Object to null is a code smell. Consider refactoring.580
Found 'DU'-anomaly for variable 'transport' (lines '580'-'777').580 - 777
A method should have only one exit point, and that should be the last statement in the method594
Found 'DU'-anomaly for variable 'recipients' (lines '666'-'777').666 - 777
Found 'DD'-anomaly for variable 'deleteMessage' (lines '668'-'694').668 - 694
Found 'DD'-anomaly for variable 'deleteMessage' (lines '668'-'703').668 - 703
Found 'DD'-anomaly for variable 'deleteMessage' (lines '668'-'726').668 - 726
Found 'DD'-anomaly for variable 'deleteMessage' (lines '668'-'747').668 - 747
Found 'DD'-anomaly for variable 'deleteMessage' (lines '668'-'749').668 - 749
Found 'DD'-anomaly for variable 'deleteMessage' (lines '694'-'726').694 - 726
Found 'DD'-anomaly for variable 'deleteMessage' (lines '694'-'747').694 - 747
Found 'DD'-anomaly for variable 'deleteMessage' (lines '694'-'749').694 - 749
Avoid assignments in operands699
Found 'DD'-anomaly for variable 'deleteMessage' (lines '703'-'703').703
Found 'DD'-anomaly for variable 'deleteMessage' (lines '703'-'726').703 - 726
Found 'DD'-anomaly for variable 'deleteMessage' (lines '703'-'747').703 - 747
Found 'DD'-anomaly for variable 'deleteMessage' (lines '703'-'749').703 - 749
Found 'DD'-anomaly for variable 'deleteMessage' (lines '726'-'747').726 - 747
Found 'DD'-anomaly for variable 'deleteMessage' (lines '726'-'749').726 - 749
A method should have only one exit point, and that should be the last statement in the method754
A method should have only one exit point, and that should be the last statement in the method767
Found 'DU'-anomaly for variable 'ex' (lines '841'-'912').841 - 912
A method should have only one exit point, and that should be the last statement in the method876
A method should have only one exit point, and that should be the last statement in the method891
Found 'DD'-anomaly for variable 'machine' (lines '917'-'920').917 - 920
Found 'DD'-anomaly for variable 'machine' (lines '920'-'922').920 - 922
Found 'DU'-anomaly for variable 'recipients' (lines '985'-'1,044').985 - 1044
Found 'DU'-anomaly for variable 'name' (lines '1,005'-'1,044').1005 - 1044
Avoid empty catch blocks1078
Found 'DU'-anomaly for variable 'session' (lines '1,124'-'1,175').1124 - 1175
Assigning an Object to null is a code smell. Consider refactoring.1156
Found 'DD'-anomaly for variable 'delays' (lines '1,203'-'1,209').1203 - 1209
Found 'DU'-anomaly for variable 'idx' (lines '1,205'-'1,213').1205 - 1213
Found 'DD'-anomaly for variable 'delays' (lines '1,209'-'1,209').1209
A method should have only one exit point, and that should be the last statement in the method1222
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,334'-'1,401').1334 - 1401
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,334'-'1,401').1334 - 1401
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,334'-'1,401').1334 - 1401
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,334'-'1,401').1334 - 1401
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,334'-'1,401').1334 - 1401
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,334'-'1,401').1334 - 1401
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,334'-'1,401').1334 - 1401
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,334'-'1,401').1334 - 1401
Assigning an Object to null is a code smell. Consider refactoring.1394

org/apache/james/transport/mailets/RemoteDeliverySocketFactory.java

ViolationLine
Each class should declare at least one constructor53 - 135
Use explicit scoping instead of the default package private level58 - 61
Assigning an Object to null is a code smell. Consider refactoring.59

org/apache/james/transport/mailets/RemoveAllMailAttributes.java

ViolationLine
Each class should declare at least one constructor35 - 58

org/apache/james/transport/mailets/RemoveMailAttribute.java

ViolationLine
Each class should declare at least one constructor42 - 89
Found 'DU'-anomaly for variable 'mail' (lines '80'-'86').80 - 86

org/apache/james/transport/mailets/Resend.java

ViolationLine
Each class should declare at least one constructor245 - 288

org/apache/james/transport/mailets/ServerTime.java

ViolationLine
Each class should declare at least one constructor34 - 77

org/apache/james/transport/mailets/SetMailAttribute.java

ViolationLine
Each class should declare at least one constructor44 - 93
Perhaps 'attributesToSet' could be replaced by a local variable.46
Found 'DU'-anomaly for variable 'mail' (lines '82'-'90').82 - 90

org/apache/james/transport/mailets/SetMimeHeader.java

ViolationLine
Each class should declare at least one constructor40 - 95

org/apache/james/transport/mailets/ToMultiRepository.java

ViolationLine
Each class should declare at least one constructor69 - 319
Found 'DD'-anomaly for variable 'message' (lines '119'-'121').119 - 121
Found 'DU'-anomaly for variable 'message' (lines '121'-'188').121 - 188
Found 'DU'-anomaly for variable 'deliveredTo' (lines '135'-'188').135 - 188
These nested if statements could be combined157 - 167
Found 'UR'-anomaly for variable 'errorBuffer' (lines '206'-'206').206
Found 'DU'-anomaly for variable 'errorBuffer' (lines '225'-'234').225 - 234
Found 'DD'-anomaly for variable 'localCount' (lines '242'-'244').242 - 244
Found 'DD'-anomaly for variable 'userInbox' (lines '310'-'313').310 - 313
Assigning an Object to null is a code smell. Consider refactoring.313

org/apache/james/transport/mailets/ToProcessor.java

ViolationLine
Each class should declare at least one constructor38 - 109
Use explicit scoping instead of the default package private level48
Use explicit scoping instead of the default package private level53
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.61

org/apache/james/transport/mailets/ToRepository.java

ViolationLine
Each class should declare at least one constructor39 - 112
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.63
Avoid empty catch blocks64 - 66

org/apache/james/transport/mailets/UseHeaderRecipients.java

ViolationLine
Each class should declare at least one constructor55 - 151
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.68

org/apache/james/transport/mailets/UsersRepositoryAliasingForwarding.java

ViolationLine
Each class should declare at least one constructor59 - 277
Found 'UR'-anomaly for variable 'errorBuffer' (lines '171'-'171').171
Found 'DU'-anomaly for variable 'sender' (lines '171'-'233').171 - 233
Found 'DU'-anomaly for variable 'errorBuffer' (lines '186'-'233').186 - 233
A method should have only one exit point, and that should be the last statement in the method220
Found 'DU'-anomaly for variable 'usersStore' (lines '245'-'275').245 - 275
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.248 - 250
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.251 - 253
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.254 - 256

org/apache/james/transport/mailets/WhiteListManager.java

ViolationLine
Each class should declare at least one constructor92 - 831
Perhaps 'rfc822DateFormat' could be replaced by a local variable.106
Perhaps 'usersStore' could be replaced by a local variable.111
Perhaps 'sqlFile' could be replaced by a local variable.135
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.163
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 method262
Found 'DD'-anomaly for variable 'senderUser' (lines '311'-'314').311 - 314
Found 'DD'-anomaly for variable 'dbUpdated' (lines '319'-'360').319 - 360
Found 'DD'-anomaly for variable 'selectRS' (lines '324'-'346').324 - 346
Found 'DD'-anomaly for variable 'dbUpdated' (lines '369'-'360').369 - 360
Found 'DU'-anomaly for variable 'dbUpdated' (lines '382'-'388').382 - 388
Avoid empty catch blocks385
Found 'DD'-anomaly for variable 'senderUser' (lines '395'-'398').395 - 398
Found 'DD'-anomaly for variable 'conn' (lines '400'-'412').400 - 412
Found 'DD'-anomaly for variable 'selectStmt' (lines '401'-'413').401 - 413
Found 'DD'-anomaly for variable 'selectRS' (lines '402'-'416').402 - 416
Found 'DD'-anomaly for variable 'senderUser' (lines '444'-'447').444 - 447
Found 'DD'-anomaly for variable 'dbUpdated' (lines '452'-'513').452 - 513
Found 'DD'-anomaly for variable 'selectRS' (lines '470'-'498').470 - 498
Found 'DD'-anomaly for variable 'dbUpdated' (lines '513'-'513').513
Found 'DU'-anomaly for variable 'dbUpdated' (lines '554'-'560').554 - 560
Avoid empty catch blocks557
Found 'DD'-anomaly for variable 'senderUser' (lines '567'-'570').567 - 570
Found 'DD'-anomaly for variable 'dbUpdated' (lines '575'-'636').575 - 636
Found 'DD'-anomaly for variable 'selectRS' (lines '593'-'621').593 - 621
Found 'DD'-anomaly for variable 'dbUpdated' (lines '636'-'636').636
Found 'DU'-anomaly for variable 'dbUpdated' (lines '677'-'683').677 - 683
Avoid empty catch blocks680
Found 'DD'-anomaly for variable 'rcptAddr' (lines '696'-'697').696 - 697
Found 'DD'-anomaly for variable 'username' (lines '748'-'752').748 - 752
Found 'DU'-anomaly for variable 'dbMetaData' (lines '781'-'795').781 - 795
Found 'DD'-anomaly for variable 'dbUpdated' (lines '785'-'787').785 - 787
Found 'DU'-anomaly for variable 'dbUpdated' (lines '792'-'795').792 - 795
Found 'DU'-anomaly for variable 'createSqlStringName' (lines '797'-'829').797 - 829
A method should have only one exit point, and that should be the last statement in the method804
Found 'DD'-anomaly for variable 'createStatement' (lines '807'-'810').807 - 810
Found 'DD'-anomaly for variable 'logBuffer' (lines '814'-'815').814 - 815

org/apache/james/transport/mailets/XMLVirtualUserTable.java

ViolationLine
Each class should declare at least one constructor58 - 142
A method should have only one exit point, and that should be the last statement in the method119
A method should have only one exit point, and that should be the last statement in the method126
A method should have only one exit point, and that should be the last statement in the method133

org/apache/james/transport/mailets/debug/Counter.java

ViolationLine
Each class should declare at least one constructor30 - 56
Use explicit scoping instead of the default package private level35

org/apache/james/transport/mailets/debug/DumpSystemErr.java

ViolationLine
Each class should declare at least one constructor33 - 59

org/apache/james/transport/mailets/debug/ExceptionThrowingMailet.java

ViolationLine
Each class should declare at least one constructor32 - 53
Found 'DU'-anomaly for variable 'mail' (lines '41'-'43').41 - 43

org/apache/james/transport/mailets/debug/Identity.java

ViolationLine
Each class should declare at least one constructor29 - 48
Found 'DU'-anomaly for variable 'mail' (lines '36'-'38').36 - 38

org/apache/james/transport/mailets/listservcommands/BaseCommand.java

ViolationLine
Each class should declare at least one constructor58 - 225

org/apache/james/transport/mailets/listservcommands/ErrorCommand.java

ViolationLine
Each class should declare at least one constructor53 - 105

org/apache/james/transport/mailets/listservcommands/IListServCommand.java

ViolationLine
Avoid modifiers which are implied by the context70
Avoid modifiers which are implied by the context77
Found 'DU'-anomaly for variable 'configuration' (lines '77'-'77').77
Found 'DU'-anomaly for variable 'commandListservManager' (lines '77'-'77').77
Avoid modifiers which are implied by the context84
Found 'DU'-anomaly for variable 'mail' (lines '84'-'84').84

org/apache/james/transport/mailets/listservcommands/Info.java

ViolationLine
Each class should declare at least one constructor57 - 120

org/apache/james/transport/mailets/listservcommands/MailDataSource.java

ViolationLine
Found 'DD'-anomaly for variable 'inputStream' (lines '110'-'111').110 - 111
Found 'DD'-anomaly for variable 'outputStream' (lines '110'-'112').110 - 112
Avoid assignments in operands117
Found 'DU'-anomaly for variable 'len' (lines '117'-'123').117 - 123

org/apache/james/transport/mailets/listservcommands/Owner.java

ViolationLine
Each class should declare at least one constructor39 - 65

org/apache/james/transport/mailets/listservcommands/Subscribe.java

ViolationLine
Each class should declare at least one constructor58 - 108
A method should have only one exit point, and that should be the last statement in the method104

org/apache/james/transport/mailets/listservcommands/SubscribeConfirm.java

ViolationLine
Each class should declare at least one constructor59 - 121
A method should have only one exit point, and that should be the last statement in the method117

org/apache/james/transport/mailets/listservcommands/UnSubscribe.java

ViolationLine
Each class should declare at least one constructor58 - 105
A method should have only one exit point, and that should be the last statement in the method101

org/apache/james/transport/mailets/listservcommands/UnSubscribeConfirm.java

ViolationLine
Each class should declare at least one constructor58 - 107
A method should have only one exit point, and that should be the last statement in the method103

org/apache/james/transport/mailets/smime/SMIMEAbstractSign.java

ViolationLine
Each class should declare at least one constructor100 - 713
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.154
These nested if statements could be combined223 - 225
These nested if statements could be combined230 - 232
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.280
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.304
A method should have only one exit point, and that should be the last statement in the method515
A method should have only one exit point, and that should be the last statement in the method521
A method should have only one exit point, and that should be the last statement in the method528
A method should have only one exit point, and that should be the last statement in the method533
A method should have only one exit point, and that should be the last statement in the method537
A method should have only one exit point, and that should be the last statement in the method546
Found 'DU'-anomaly for variable 'mail' (lines '559'-'559').559
Found 'UR'-anomaly for variable 'bad' (lines '565'-'565').565
Found 'DU'-anomaly for variable 'allowed' (lines '571'-'590').571 - 590
A method should have only one exit point, and that should be the last statement in the method597
A method should have only one exit point, and that should be the last statement in the method622
Found 'DD'-anomaly for variable 'mailAddress' (lines '629'-'631').629 - 631
A method should have only one exit point, and that should be the last statement in the method637
Found 'DD'-anomaly for variable 'replacedExplanationText' (lines '677'-'679').677 - 679
Found 'DU'-anomaly for variable 'pattern' (lines '694'-'711').694 - 711
Avoid assignments in operands699
Found 'DU'-anomaly for variable 'index' (lines '699'-'711').699 - 711
A method should have only one exit point, and that should be the last statement in the method707

org/apache/james/transport/mailets/smime/SMIMECheckSignature.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you110 - 113
Found 'DU'-anomaly for variable 'type' (lines '128'-'142').128 - 142
Found 'DD'-anomaly for variable 'strippedMessage' (lines '153'-'167').153 - 167
Found 'DD'-anomaly for variable 'strippedMessage' (lines '153'-'207').153 - 207
Found 'DD'-anomaly for variable 'signers' (lines '155'-'166').155 - 166
Found 'DD'-anomaly for variable 'signers' (lines '155'-'176').155 - 176
Assigning an Object to null is a code smell. Consider refactoring.163
Found 'DD'-anomaly for variable 'signers' (lines '166'-'176').166 - 176
Found 'DD'-anomaly for variable 'strippedMessage' (lines '167'-'207').167 - 207
Assigning an Object to null is a code smell. Consider refactoring.176
Found 'DD'-anomaly for variable 'signers' (lines '176'-'179').176 - 179
Assigning an Object to null is a code smell. Consider refactoring.179
Found 'DD'-anomaly for variable 'signers' (lines '179'-'182').179 - 182
Assigning an Object to null is a code smell. Consider refactoring.182
Found 'DD'-anomaly for variable 'signers' (lines '182'-'186').182 - 186
Assigning an Object to null is a code smell. Consider refactoring.186
Assigning an Object to null is a code smell. Consider refactoring.207

org/apache/james/transport/mailets/smime/SMIMEDecrypt.java

ViolationLine
Each class should declare at least one constructor69 - 161
Found 'DD'-anomaly for variable 'strippedMessage' (lines '107'-'120').107 - 120
Found 'DD'-anomaly for variable 'strippedMessage' (lines '120'-'120').120

org/apache/james/transport/mailets/smime/SMIMESign.java

ViolationLine
Each class should declare at least one constructor56 - 200
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.130
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.137
Found 'DU'-anomaly for variable 'mail' (lines '155'-'198').155 - 198
A method should have only one exit point, and that should be the last statement in the method161

org/apache/james/transport/matchers/AbstractNetworkMatcher.java

ViolationLine
Each class should declare at least one constructor47 - 85
Found 'DD'-anomaly for variable 'networks' (lines '69'-'72').69 - 72

org/apache/james/transport/matchers/AbstractQuotaMatcher.java

ViolationLine
Each class should declare at least one constructor39 - 150
Found 'DD'-anomaly for variable 'matching' (lines '50'-'52').50 - 52
Found 'DU'-anomaly for variable 'user' (lines '73'-'82').73 - 82
A method should have only one exit point, and that should be the last statement in the method77
Found 'DU'-anomaly for variable 'address' (lines '114'-'114').114
Found 'DU'-anomaly for variable 'mail' (lines '114'-'114').114
Found 'DU'-anomaly for variable 'address' (lines '122'-'122').122
Found 'DU'-anomaly for variable 'mail' (lines '122'-'122').122

org/apache/james/transport/matchers/AbstractStorageQuota.java

ViolationLine
Each class should declare at least one constructor49 - 135
Found 'DU'-anomaly for variable '_' (lines '96'-'112').96 - 112
Found 'DD'-anomaly for variable 'username' (lines '126'-'130').126 - 130

org/apache/james/transport/matchers/All.java

ViolationLine
Each class should declare at least one constructor30 - 34

org/apache/james/transport/matchers/AttachmentFileNameIs.java

ViolationLine
Each class should declare at least one constructor55 - 260
Each class should declare at least one constructor69 - 75
A method should have only one exit point, and that should be the last statement in the method131
Found 'UR'-anomaly for variable 'anException' (lines '147'-'147').147
A method should have only one exit point, and that should be the last statement in the method156
Found 'DU'-anomaly for variable 'content' (lines '162'-'205').162 - 205
A method should have only one exit point, and that should be the last statement in the method165
A method should have only one exit point, and that should be the last statement in the method176
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 method194
Found 'DU'-anomaly for variable 'fileName' (lines '210'-'226').210 - 226
A method should have only one exit point, and that should be the last statement in the method222
Found 'DU'-anomaly for variable 'zis' (lines '232'-'252').232 - 252
A method should have only one exit point, and that should be the last statement in the method245

org/apache/james/transport/matchers/CommandForListserv.java

ViolationLine
Each class should declare at least one constructor33 - 50
These nested if statements could be combined43 - 46
A method should have only one exit point, and that should be the last statement in the method45

org/apache/james/transport/matchers/CommandListservMatcher.java

ViolationLine
Each class should declare at least one constructor42 - 64
These nested if statements could be combined58 - 60
A method should have only one exit point, and that should be the last statement in the method59

org/apache/james/transport/matchers/CompareNumericHeaderValue.java

ViolationLine
Each class should declare at least one constructor51 - 169
Found 'UR'-anomaly for variable 'comparisonOperatorString' (lines '64'-'64').64
Found 'UR'-anomaly for variable 'headerValueString' (lines '64'-'64').64
Found 'UR'-anomaly for variable 'headerArray' (lines '119'-'119').119
Found 'DU'-anomaly for variable 'comparison' (lines '130'-'168').130 - 168
A method should have only one exit point, and that should be the last statement in the method134
A method should have only one exit point, and that should be the last statement in the method139
A method should have only one exit point, and that should be the last statement in the method144
A method should have only one exit point, and that should be the last statement in the method149
A method should have only one exit point, and that should be the last statement in the method154

org/apache/james/transport/matchers/FetchedFrom.java

ViolationLine
Each class should declare at least one constructor37 - 47
A method should have only one exit point, and that should be the last statement in the method43

org/apache/james/transport/matchers/FileRegexMatcher.java

ViolationLine
Each class should declare at least one constructor32 - 68
Found 'DD'-anomaly for variable 'patternSource' (lines '34'-'36').34 - 36
Found 'DD'-anomaly for variable 'lines' (lines '37'-'38').37 - 38
Found 'DD'-anomaly for variable 'lines' (lines '38'-'38').38
Avoid empty catch blocks62 - 64

org/apache/james/transport/matchers/GenericRegexMatcher.java

ViolationLine
Each class should declare at least one constructor46 - 83
Found 'DU'-anomaly for variable 'message' (lines '61'-'82').61 - 82
Found 'DU'-anomaly for variable 'matcher' (lines '62'-'82').62 - 82
Found 'DU'-anomaly for variable 'pattern' (lines '69'-'82').69 - 82
Found 'DD'-anomaly for variable 'pattern' (lines '69'-'69').69
A method should have only one exit point, and that should be the last statement in the method76

org/apache/james/transport/matchers/HasAttachment.java

ViolationLine
Each class should declare at least one constructor36 - 89
Found 'UR'-anomaly for variable 'anException' (lines '42'-'42').42
Found 'DU'-anomaly for variable 'anException' (lines '44'-'88').44 - 88
Found 'DD'-anomaly for variable 'anException' (lines '44'-'69').44 - 69
Found 'DD'-anomaly for variable 'anException' (lines '44'-'79').44 - 79
A method should have only one exit point, and that should be the last statement in the method55
A method should have only one exit point, and that should be the last statement in the method66
Found 'DU'-anomaly for variable 'anException' (lines '69'-'88').69 - 88
Found 'DD'-anomaly for variable 'anException' (lines '69'-'69').69
Found 'DD'-anomaly for variable 'anException' (lines '69'-'79').69 - 79
A method should have only one exit point, and that should be the last statement in the method75

org/apache/james/transport/matchers/HasHabeasWarrantMark.java

ViolationLine
Each class should declare at least one constructor67 - 124
Found 'DU'-anomaly for variable 'message' (lines '84'-'109').84 - 109
A method should have only one exit point, and that should be the last statement in the method99
A method should have only one exit point, and that should be the last statement in the method104

org/apache/james/transport/matchers/HasHeader.java

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

org/apache/james/transport/matchers/HasMailAttribute.java

ViolationLine
Each class should declare at least one constructor40 - 73
A method should have only one exit point, and that should be the last statement in the method68

org/apache/james/transport/matchers/HasMailAttributeWithValue.java

ViolationLine
Each class should declare at least one constructor66 - 159
A method should have only one exit point, and that should be the last statement in the method92

org/apache/james/transport/matchers/HasMailAttributeWithValueRegex.java

ViolationLine
Each class should declare at least one constructor49 - 100
Perhaps 'matcher' could be replaced by a local variable.53
Found 'UR'-anomaly for variable 'pattern_string' (lines '65'-'65').65
A method should have only one exit point, and that should be the last statement in the method95

org/apache/james/transport/matchers/HostIs.java

ViolationLine
Each class should declare at least one constructor33 - 48

org/apache/james/transport/matchers/HostIsLocal.java

ViolationLine
Each class should declare at least one constructor29 - 34

org/apache/james/transport/matchers/InSpammerBlacklist.java

ViolationLine
Each class should declare at least one constructor46 - 78
Use explicit scoping instead of the default package private level47
A method should have only one exit point, and that should be the last statement in the method72

org/apache/james/transport/matchers/IsInWhiteList.java

ViolationLine
Each class should declare at least one constructor58 - 271
Perhaps 'usersStore' could be replaced by a local variable.65
Perhaps 'sqlFile' could be replaced by a local variable.89
Found 'DD'-anomaly for variable 'repositoryPath' (lines '115'-'118').115 - 118
A method should have only one exit point, and that should be the last statement in the method161
A method should have only one exit point, and that should be the last statement in the method168
Found 'DU'-anomaly for variable 'conn' (lines '178'-'228').178 - 228
Found 'DU'-anomaly for variable 'selectStmt' (lines '179'-'228').179 - 228
Found 'DD'-anomaly for variable 'selectRS' (lines '180'-'207').180 - 207
Found 'DU'-anomaly for variable 'selectRS' (lines '180'-'228').180 - 228
Found 'DD'-anomaly for variable 'recipientUser' (lines '186'-'194').186 - 194
Found 'DU'-anomaly for variable 'conn' (lines '197'-'228').197 - 228
Found 'DD'-anomaly for variable 'username' (lines '237'-'241').237 - 241

org/apache/james/transport/matchers/IsSingleRecipient.java

ViolationLine
Each class should declare at least one constructor30 - 39
A method should have only one exit point, and that should be the last statement in the method34

org/apache/james/transport/matchers/NESSpamCheck.java

ViolationLine
Each class should declare at least one constructor33 - 100

org/apache/james/transport/matchers/RecipientIs.java

ViolationLine
Each class should declare at least one constructor31 - 46

org/apache/james/transport/matchers/RecipientIsLocal.java

ViolationLine
Each class should declare at least one constructor30 - 38

org/apache/james/transport/matchers/RecipientIsOverFixedQuota.java

ViolationLine
Each class should declare at least one constructor56 - 72
Found 'DU'-anomaly for variable '_' (lines '69'-'71').69 - 71
Found 'DU'-anomaly for variable 'recipient' (lines '69'-'71').69 - 71

org/apache/james/transport/matchers/RecipientIsRegex.java

ViolationLine
Each class should declare at least one constructor47 - 74
Use explicit scoping instead of the default package private level48
Found 'UR'-anomaly for variable 'patternString' (lines '50'-'50').50
A method should have only one exit point, and that should be the last statement in the method69

org/apache/james/transport/matchers/RelayLimit.java

ViolationLine
Each class should declare at least one constructor36 - 58
Use explicit scoping instead of the default package private level37
Found 'DD'-anomaly for variable 'count' (lines '45'-'49').45 - 49
Found 'DD'-anomaly for variable 'count' (lines '49'-'49').49
A method should have only one exit point, and that should be the last statement in the method53

org/apache/james/transport/matchers/RemoteAddrInNetwork.java

ViolationLine
Each class should declare at least one constructor35 - 39
Assigning an Object to null is a code smell. Consider refactoring.37

org/apache/james/transport/matchers/RemoteAddrNotInNetwork.java

ViolationLine
Each class should declare at least one constructor35 - 39
Assigning an Object to null is a code smell. Consider refactoring.37

org/apache/james/transport/matchers/SMTPAuthSuccessful.java

ViolationLine
Each class should declare at least one constructor39 - 54
A method should have only one exit point, and that should be the last statement in the method49

org/apache/james/transport/matchers/SMTPAuthUserIs.java

ViolationLine
Each class should declare at least one constructor40 - 65
A method should have only one exit point, and that should be the last statement in the method60

org/apache/james/transport/matchers/SenderHostIs.java

ViolationLine
Each class should declare at least one constructor41 - 81
A method should have only one exit point, and that should be the last statement in the method73

org/apache/james/transport/matchers/SenderInFakeDomain.java

ViolationLine
Each class should declare at least one constructor33 - 72
A method should have only one exit point, and that should be the last statement in the method37
A method should have only one exit point, and that should be the last statement in the method45
A method should have only one exit point, and that should be the last statement in the method65

org/apache/james/transport/matchers/SenderIs.java

ViolationLine
Each class should declare at least one constructor32 - 51
A method should have only one exit point, and that should be the last statement in the method46

org/apache/james/transport/matchers/SenderIsNull.java

ViolationLine
Each class should declare at least one constructor37 - 46
A method should have only one exit point, and that should be the last statement in the method41

org/apache/james/transport/matchers/SenderIsRegex.java

ViolationLine
Each class should declare at least one constructor56 - 86
Use explicit scoping instead of the default package private level57
Found 'UR'-anomaly for variable 'patternString' (lines '59'-'59').59
A method should have only one exit point, and that should be the last statement in the method77
A method should have only one exit point, and that should be the last statement in the method82

org/apache/james/transport/matchers/SizeGreaterThan.java

ViolationLine
Each class should declare at least one constructor39 - 70
Use explicit scoping instead of the default package private level41
A method should have only one exit point, and that should be the last statement in the method65

org/apache/james/transport/matchers/SubjectIs.java

ViolationLine
Each class should declare at least one constructor33 - 42
A method should have only one exit point, and that should be the last statement in the method38

org/apache/james/transport/matchers/SubjectStartsWith.java

ViolationLine
Each class should declare at least one constructor34 - 44
A method should have only one exit point, and that should be the last statement in the method40

org/apache/james/transport/matchers/UserIs.java

ViolationLine
Each class should declare at least one constructor31 - 45
Use explicit scoping instead of the default package private level32

org/apache/james/transport/matchers/smime/IsSMIMEEncrypted.java

ViolationLine
Each class should declare at least one constructor33 - 50
A method should have only one exit point, and that should be the last statement in the method39
A method should have only one exit point, and that should be the last statement in the method42
A method should have only one exit point, and that should be the last statement in the method46

org/apache/james/transport/matchers/smime/IsSMIMESigned.java

ViolationLine
Each class should declare at least one constructor33 - 53
A method should have only one exit point, and that should be the last statement in the method39
A method should have only one exit point, and that should be the last statement in the method42
A method should have only one exit point, and that should be the last statement in the method50

org/apache/james/transport/matchers/smime/IsX509CertificateSubject.java

ViolationLine
Each class should declare at least one constructor49 - 98
Found 'UR'-anomaly for variable 'condition' (lines '53'-'53').53
Found 'DD'-anomaly for variable 'valid' (lines '74'-'84').74 - 84
Found 'DD'-anomaly for variable 'valid' (lines '84'-'84').84
A method should have only one exit point, and that should be the last statement in the method89
A method should have only one exit point, and that should be the last statement in the method91

org/apache/james/userrepository/AbstractJdbcUsersRepository.java

ViolationLine
Each class should declare at least one constructor76 - 621
Perhaps 'm_createUserTableSql' could be replaced by a local variable.107
Found 'DD'-anomaly for variable 'logBuffer' (lines '126'-'129').126 - 129
Found 'DU'-anomaly for variable 'logBuffer' (lines '126'-'138').126 - 138
Found 'DD'-anomaly for variable 'logBuffer' (lines '159'-'161').159 - 161
Found 'DD'-anomaly for variable 'logBuffer' (lines '159'-'202').159 - 202
Found 'DU'-anomaly for variable 'logBuffer' (lines '159'-'228').159 - 228
Found 'DD'-anomaly for variable 'logBuffer' (lines '242'-'244').242 - 244
Found 'DD'-anomaly for variable 'logBuffer' (lines '242'-'276').242 - 276
Found 'DD'-anomaly for variable 'logBuffer' (lines '242'-'338').242 - 338
Found 'DU'-anomaly for variable 'logBuffer' (lines '242'-'355').242 - 355
Found 'DD'-anomaly for variable 'logBuffer' (lines '242'-'244').242 - 244
Found 'DD'-anomaly for variable 'logBuffer' (lines '242'-'276').242 - 276
Found 'DD'-anomaly for variable 'logBuffer' (lines '242'-'338').242 - 338
Found 'DU'-anomaly for variable 'logBuffer' (lines '242'-'355').242 - 355
Found 'DD'-anomaly for variable 'sqlFile' (lines '266'-'269').266 - 269
Found 'DD'-anomaly for variable 'sqlFile' (lines '266'-'269').266 - 269
Found 'DD'-anomaly for variable 'createStatement' (lines '329'-'331').329 - 331
Found 'DD'-anomaly for variable 'createStatement' (lines '329'-'331').329 - 331
Found 'DD'-anomaly for variable 'getUsersStatement' (lines '391'-'395').391 - 395
Found 'DD'-anomaly for variable 'rsUsers' (lines '392'-'397').392 - 397
Found 'DD'-anomaly for variable 'addUserStatement' (lines '426'-'431').426 - 431
Found 'DD'-anomaly for variable 'removeUserStatement' (lines '458'-'462').458 - 462
Found 'DD'-anomaly for variable 'updateUserStatement' (lines '483'-'487').483 - 487
Found 'DU'-anomaly for variable 'ignoreCase' (lines '511'-'556').511 - 556
A method should have only one exit point, and that should be the last statement in the method516
Found 'DD'-anomaly for variable 'getUsersStatement' (lines '522'-'527').522 - 527
Found 'DD'-anomaly for variable 'rsUsers' (lines '523'-'531').523 - 531
Found 'DD'-anomaly for variable 'user' (lines '534'-'541').534 - 541
Found 'DU'-anomaly for variable 'rsUsers' (lines '570'-'571').570 - 571
Found 'DU'-anomaly for variable 'userInsert' (lines '585'-'587').585 - 587
Found 'DU'-anomaly for variable 'user' (lines '585'-'587').585 - 587
Found 'DU'-anomaly for variable 'user' (lines '601'-'603').601 - 603
Found 'DU'-anomaly for variable 'userUpdate' (lines '601'-'603').601 - 603

org/apache/james/userrepository/AbstractUsersRepository.java

ViolationLine
Each class should declare at least one constructor37 - 296
Found 'DU'-anomaly for variable 'user' (lines '55'-'55').55
Found 'DU'-anomaly for variable 'user' (lines '61'-'61').61
Found 'DU'-anomaly for variable 'user' (lines '66'-'66').66
Found 'DU'-anomaly for variable 'ignoreCase' (lines '99'-'112').99 - 112
Found 'DU'-anomaly for variable 'name' (lines '99'-'112').99 - 112
A method should have only one exit point, and that should be the last statement in the method107
A method should have only one exit point, and that should be the last statement in the method130
Found 'DU'-anomaly for variable 'name' (lines '144'-'153').144 - 153
A method should have only one exit point, and that should be the last statement in the method175
A method should have only one exit point, and that should be the last statement in the method235
Found 'DU'-anomaly for variable 'password' (lines '269'-'276').269 - 276
A method should have only one exit point, and that should be the last statement in the method272

org/apache/james/userrepository/DefaultJamesUser.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.74

org/apache/james/userrepository/DefaultUsersJdbcRepository.java

ViolationLine
Each class should declare at least one constructor40 - 110

org/apache/james/userrepository/JamesUsersJdbcRepository.java

ViolationLine
Each class should declare at least one constructor34 - 177
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '44'-'44').44
Found 'DD'-anomaly for variable 'forwardAddress' (lines '55'-'58').55 - 58
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '61'-'80').61 - 80
Found 'UR'-anomaly for variable 'jamesUser' (lines '122'-'151').122 - 151
Found 'DD'-anomaly for variable 'nameIndex' (lines '125'-'128').125 - 128
Found 'DD'-anomaly for variable 'colOffset' (lines '126'-'129').126 - 129
Found 'DD'-anomaly for variable 'forwardDestination' (lines '157'-'159').157 - 159

org/apache/james/userrepository/ListUsersJdbcRepository.java

ViolationLine
Each class should declare at least one constructor34 - 93
Found 'DU'-anomaly for variable 'user' (lines '76'-'81').76 - 81
Found 'DU'-anomaly for variable 'userUpdate' (lines '76'-'81').76 - 81

org/apache/james/userrepository/UsersFileRepository.java

ViolationLine
Each class should declare at least one constructor52 - 263
Found 'UR'-anomaly for variable 'message' (lines '72'-'72').72
A method should have only one exit point, and that should be the last statement in the method150
Found 'DU'-anomaly for variable 'name' (lines '160'-'170').160 - 170
A method should have only one exit point, and that should be the last statement in the method181
A method should have only one exit point, and that should be the last statement in the method194
Found 'DU'-anomaly for variable 'name' (lines '199'-'208').199 - 208
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 method213
Found 'DU'-anomaly for variable 'name' (lines '231'-'239').231 - 239
A method should have only one exit point, and that should be the last statement in the method235
Found 'DU'-anomaly for variable 'password' (lines '241'-'253').241 - 253
A method should have only one exit point, and that should be the last statement in the method247
Found 'DD'-anomaly for variable 'count' (lines '256'-'258').256 - 258
Found 'DD'-anomaly for variable 'count' (lines '258'-'258').258

org/apache/james/userrepository/UsersLDAPRepository.java

ViolationLine
Each class should declare at least one constructor61 - 743
Perhaps 'baseURL' could be replaced by a local variable.72
Found 'DU'-anomaly for variable 'compMgr' (lines '96'-'98').96 - 98
Found 'DD'-anomaly for variable 'destination' (lines '170'-'185').170 - 185
Found 'DD'-anomaly for variable 'destination' (lines '170'-'213').170 - 213
Found 'DU'-anomaly for variable 'user' (lines '258'-'260').258 - 260
Found 'DU'-anomaly for variable 'name' (lines '262'-'264').262 - 264
Found 'DU'-anomaly for variable 'name' (lines '277'-'279').277 - 279
Found 'DU'-anomaly for variable 'user' (lines '281'-'283').281 - 283
Found 'DU'-anomaly for variable 'attributes' (lines '290'-'348').290 - 348
Found 'DD'-anomaly for variable 'mods' (lines '314'-'315').314 - 315
Found 'DU'-anomaly for variable 'password' (lines '353'-'360').353 - 360
A method should have only one exit point, and that should be the last statement in the method356
Found 'DD'-anomaly for variable 'rootCtx' (lines '369'-'371').369 - 371
Found 'DU'-anomaly for variable 'returnAttrs' (lines '373'-'430').373 - 430
Found 'DD'-anomaly for variable 'mods' (lines '451'-'452').451 - 452
Avoid empty if statements477 - 479
Found 'DD'-anomaly for variable 'rootCtx' (lines '490'-'492').490 - 492
Found 'DD'-anomaly for variable 'mods' (lines '531'-'532').531 - 532
Assigning an Object to null is a code smell. Consider refactoring.566
Found 'DU'-anomaly for variable 'rootCtx' (lines '566'-'568').566 - 568
Found 'DD'-anomaly for variable 'found' (lines '572'-'578').572 - 578
Found 'DU'-anomaly for variable 'testPassword' (lines '601'-'699').601 - 699
Found 'DD'-anomaly for variable 'result' (lines '602'-'668').602 - 668
Found 'DD'-anomaly for variable 'foundFlag' (lines '603'-'638').603 - 638
Found 'DD'-anomaly for variable 'userDN' (lines '604'-'637').604 - 637
Found 'DU'-anomaly for variable 'userDN' (lines '604'-'699').604 - 699
Found 'DD'-anomaly for variable 'rootCtx' (lines '623'-'626').623 - 626
Found 'DU'-anomaly for variable 'userDN' (lines '637'-'699').637 - 699
Found 'DD'-anomaly for variable 'testCtx' (lines '665'-'667').665 - 667
Found 'DD'-anomaly for variable 'result' (lines '668'-'671').668 - 671
Found 'DD'-anomaly for variable 'result' (lines '704'-'709').704 - 709
Found 'DD'-anomaly for variable 'result' (lines '704'-'711').704 - 711
Assigning an Object to null is a code smell. Consider refactoring.731

org/apache/james/util/Base64.java

ViolationLine
Each class should declare at least one constructor35 - 78
A method should have only one exit point, and that should be the last statement in the method47
A method should have only one exit point, and that should be the last statement in the method51

org/apache/james/util/BayesianAnalyzer.java

ViolationLine
Each class should declare at least one constructor145 - 188
Use explicit scoping instead of the default package private level150
Use explicit scoping instead of the default package private level155
A method should have only one exit point, and that should be the last statement in the method168
Avoid unnecessary constructors - the compiler will generate these for you193 - 194
Found 'DU'-anomaly for variable 'target' (lines '369'-'411').369 - 411
Found 'DU'-anomaly for variable 'header' (lines '372'-'411').372 - 411
Avoid assignments in operands375
Found 'DU'-anomaly for variable 'token' (lines '375'-'411').375 - 411
Found 'DD'-anomaly for variable 'endingLine' (lines '376'-'378').376 - 378
Found 'DD'-anomaly for variable 'value' (lines '396'-'399').396 - 399
Found 'DD'-anomaly for variable 'value' (lines '396'-'401').396 - 401
Found 'DU'-anomaly for variable 'header' (lines '408'-'411').408 - 411
Found 'DU'-anomaly for variable 'header' (lines '424'-'458').424 - 458
Avoid assignments in operands427
Found 'DU'-anomaly for variable 'token' (lines '427'-'458').427 - 458
Found 'DD'-anomaly for variable 'endingLine' (lines '428'-'430').428 - 430
Found 'DU'-anomaly for variable 'header' (lines '452'-'458').452 - 458
Found 'DU'-anomaly for variable 'token' (lines '461'-'536').461 - 536
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '464'-'536').464 - 536
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '464'-'494').464 - 494
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '464'-'498').464 - 498
Found 'DU'-anomaly for variable 'tokenCharFound' (lines '465'-'536').465 - 536
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '465'-'493').465 - 493
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '465'-'497').465 - 497
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '465'-'509').465 - 509
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '465'-'521').465 - 521
A method should have only one exit point, and that should be the last statement in the method468
Avoid assignments in operands471
Found 'DU'-anomaly for variable 'i' (lines '471'-'536').471 - 536
A method should have only one exit point, and that should be the last statement in the method482
Found 'DU'-anomaly for variable 'tokenCharFound' (lines '493'-'536').493 - 536
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '493'-'493').493
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '493'-'497').493 - 497
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '493'-'509').493 - 509
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '493'-'521').493 - 521
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '494'-'536').494 - 536
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '494'-'494').494
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '494'-'498').494 - 498
Found 'DU'-anomaly for variable 'tokenCharFound' (lines '497'-'536').497 - 536
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '497'-'493').497 - 493
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '497'-'497').497
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '497'-'509').497 - 509
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '497'-'521').497 - 521
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '498'-'536').498 - 536
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '498'-'494').498 - 494
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '498'-'498').498
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '502'-'536').502 - 536
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '502'-'510').502 - 510
Found 'DU'-anomaly for variable 'tokenCharFound' (lines '509'-'536').509 - 536
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '509'-'493').509 - 493
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '509'-'497').509 - 497
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '509'-'509').509
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '509'-'521').509 - 521
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '510'-'536').510 - 536
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '510'-'494').510 - 494
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '510'-'498').510 - 498
Avoid empty if statements517 - 519
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '522'-'536').522 - 536
A method should have only one exit point, and that should be the last statement in the method532
Found 'DD'-anomaly for variable 'hamFactor' (lines '545'-'563').545 - 563
Found 'DD'-anomaly for variable 'spamFactor' (lines '546'-'570').546 - 570
Found 'DD'-anomaly for variable 'foundInHam' (lines '548'-'555').548 - 555
Found 'DD'-anomaly for variable 'foundInSpam' (lines '549'-'559').549 - 559
Found 'DD'-anomaly for variable 'minThreshold' (lines '551'-'565').551 - 565
Found 'DU'-anomaly for variable 'minThreshold' (lines '551'-'585').551 - 585
Found 'DD'-anomaly for variable 'maxThreshold' (lines '552'-'572').552 - 572
Found 'DU'-anomaly for variable 'maxThreshold' (lines '552'-'585').552 - 585
Found 'DU'-anomaly for variable 'minThreshold' (lines '565'-'585').565 - 585
Found 'DU'-anomaly for variable 'maxThreshold' (lines '572'-'585').572 - 585
A method should have only one exit point, and that should be the last statement in the method578
Found 'DD'-anomaly for variable 'corpusProbability' (lines '612'-'628').612 - 628
Found 'DU'-anomaly for variable 'corpusProbability' (lines '612'-'644').612 - 644
Found 'DD'-anomaly for variable 'corpusProbability' (lines '612'-'612').612
Found 'DD'-anomaly for variable 'isTokenDegeneratedFound' (lines '614'-'626').614 - 626
Found 'DD'-anomaly for variable 'tokenDegenerated' (lines '618'-'621').618 - 621
Found 'DU'-anomaly for variable 'tokenDegenerated' (lines '618'-'644').618 - 644
Found 'DD'-anomaly for variable 'tokenDegenerated' (lines '618'-'618').618
Found 'DD'-anomaly for variable 'isTokenDegeneratedFound' (lines '626'-'626').626
Found 'DD'-anomaly for variable 'corpusProbability' (lines '628'-'628').628
Found 'DU'-anomaly for variable 'corpusProbability' (lines '628'-'644').628 - 644
Found 'DD'-anomaly for variable 'corpusProbability' (lines '628'-'612').628 - 612
Found 'DU'-anomaly for variable 'workCorpus' (lines '699'-'723').699 - 723
Found 'DD'-anomaly for variable 'theDoubleValue' (lines '711'-'715').711 - 715
A method should have only one exit point, and that should be the last statement in the method727
Found 'DU'-anomaly for variable 'c' (lines '730'-'738').730 - 738
A method should have only one exit point, and that should be the last statement in the method734
A method should have only one exit point, and that should be the last statement in the method743

org/apache/james/util/CRLFTerminatedReader.java

ViolationLine
Use explicit scoping instead of the default package private level66
It is a good practice to call super() in a constructor68 - 70
Perhaps 'lineBuffer' could be replaced by a local variable.76
Perhaps 'EOF' could be replaced by a local variable.77 - 80
Perhaps 'tainted' could be replaced by a local variable.82
Found 'UR'-anomaly for variable 'pos' (lines '98'-'98').98
Found 'DU'-anomaly for variable 'cr_just_received' (lines '107'-'150').107 - 150
Found 'DU'-anomaly for variable 'cr_just_received' (lines '118'-'150').118 - 150
Found 'DD'-anomaly for variable 'cr_just_received' (lines '118'-'145').118 - 145
A method should have only one exit point, and that should be the last statement in the method120
Found 'DU'-anomaly for variable 'pos' (lines '131'-'150').131 - 150
A method should have only one exit point, and that should be the last statement in the method135
Found 'DU'-anomaly for variable 'cr_just_received' (lines '145'-'150').145 - 150
Found 'DD'-anomaly for variable 'cbuf' (lines '160'-'163').160 - 163
Found 'DU'-anomaly for variable 'off' (lines '160'-'165').160 - 165
Found 'DU'-anomaly for variable 'cbuf' (lines '160'-'165').160 - 165
Found 'DD'-anomaly for variable 'cbuf' (lines '163'-'163').163
Found 'DU'-anomaly for variable 'cbuf' (lines '163'-'165').163 - 165

org/apache/james/util/CharTerminatedInputStream.java

ViolationLine
Perhaps 'endFound' could be replaced by a local variable.59
It is a good practice to call super() in a constructor70 - 84
A method should have only one exit point, and that should be the last statement in the method96
A method should have only one exit point, and that should be the last statement in the method107
A method should have only one exit point, and that should be the last statement in the method118
A method should have only one exit point, and that should be the last statement in the method133
A method should have only one exit point, and that should be the last statement in the method141

org/apache/james/util/DebugInputStream.java

ViolationLine
Use explicit scoping instead of the default package private level33
It is a good practice to call super() in a constructor40 - 42

org/apache/james/util/DotStuffingInputStream.java

ViolationLine
Found 'DU'-anomaly for variable 'len' (lines '68'-'97').68 - 97
Found 'DU'-anomaly for variable 'off' (lines '68'-'97').68 - 97
A method should have only one exit point, and that should be the last statement in the method75
A method should have only one exit point, and that should be the last statement in the method80
Found 'DU'-anomaly for variable 'b' (lines '82'-'97').82 - 97
Found 'DU'-anomaly for variable 'b' (lines '92'-'97').92 - 97

org/apache/james/util/JDBCBayesianAnalyzer.java

ViolationLine
Found 'DU'-anomaly for variable 'errorString' (lines '59'-'59').59
Perhaps 'sqlFile' could be replaced by a local variable.80
Avoid unnecessary constructors - the compiler will generate these for you148 - 149
It is a good practice to call super() in a constructor148 - 149
Found 'DD'-anomaly for variable 'pstmt' (lines '158'-'162').158 - 162
Found 'DD'-anomaly for variable 'rs' (lines '159'-'163').159 - 163
Found 'DU'-anomaly for variable 'token' (lines '167'-'230').167 - 230
Found 'DD'-anomaly for variable 'token' (lines '167'-'167').167
Avoid empty catch blocks215 - 216
Assigning an Object to null is a code smell. Consider refactoring.218
Found 'DU'-anomaly for variable 'rs' (lines '218'-'230').218 - 230
Avoid empty catch blocks224 - 225
Assigning an Object to null is a code smell. Consider refactoring.227
Found 'DU'-anomaly for variable 'pstmt' (lines '227'-'230').227 - 230
Found 'DD'-anomaly for variable 'init' (lines '262'-'267').262 - 267
Found 'DD'-anomaly for variable 'update' (lines '263'-'268').263 - 268
Avoid empty catch blocks281 - 282
Avoid empty catch blocks287 - 288
Found 'DD'-anomaly for variable 'insert' (lines '295'-'300').295 - 300
Found 'DD'-anomaly for variable 'update' (lines '296'-'303').296 - 303
Avoid empty catch blocks326 - 327
Assigning an Object to null is a code smell. Consider refactoring.329
Found 'DU'-anomaly for variable 'insert' (lines '329'-'341').329 - 341
Avoid empty catch blocks335 - 336
Assigning an Object to null is a code smell. Consider refactoring.338
Found 'DU'-anomaly for variable 'update' (lines '338'-'341').338 - 341
Found 'DU'-anomaly for variable 'dbMetaData' (lines '367'-'385').367 - 385
Found 'DD'-anomaly for variable 'dbUpdated' (lines '371'-'373').371 - 373
Found 'DD'-anomaly for variable 'dbUpdated' (lines '373'-'375').373 - 375
Found 'DD'-anomaly for variable 'dbUpdated' (lines '375'-'377').375 - 377
Found 'DU'-anomaly for variable 'dbUpdated' (lines '382'-'385').382 - 385
Found 'DU'-anomaly for variable 'createSqlStringName' (lines '387'-'419').387 - 419
A method should have only one exit point, and that should be the last statement in the method394
Found 'DD'-anomaly for variable 'createStatement' (lines '397'-'400').397 - 400
Found 'DD'-anomaly for variable 'logBuffer' (lines '404'-'405').404 - 405

org/apache/james/util/JDBCUtil.java

ViolationLine
Each class should declare at least one constructor38 - 197
Found 'DU'-anomaly for variable 'errorString' (lines '46'-'46').46
Avoid empty catch blocks192 - 194

org/apache/james/util/Lock.java

ViolationLine
Each class should declare at least one constructor28 - 113
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 method74
A method should have only one exit point, and that should be the last statement in the method76
A method should have only one exit point, and that should be the last statement in the method95
A method should have only one exit point, and that should be the last statement in the method98

org/apache/james/util/NetMatcher.java

ViolationLine
Found 'DD'-anomaly for variable 'ip' (lines '63'-'67').63 - 67
Found 'DU'-anomaly for variable 'ip' (lines '84'-'94').84 - 94
Found 'DU'-anomaly for variable 's' (lines '114'-'114').114
A method should have only one exit point, and that should be the last statement in the method177 - 183
Avoid empty catch blocks185
Found 'DU'-anomaly for variable 'srcb' (lines '208'-'214').208 - 214
Found 'DD'-anomaly for variable 'octets' (lines '209'-'211').209 - 211
Found 'DD'-anomaly for variable 'octets' (lines '211'-'211').211
Assigning an Object to null is a code smell. Consider refactoring.242
Found 'DU'-anomaly for variable 'ip' (lines '246'-'265').246 - 265
Found 'DD'-anomaly for variable 'addr' (lines '248'-'250').248 - 250
Avoid empty catch blocks251 - 252
Avoid empty catch blocks252 - 253

org/apache/james/util/SchedulerNotifyInputStream.java

ViolationLine
Use explicit scoping instead of the default package private level38
Use explicit scoping instead of the default package private level43
Use explicit scoping instead of the default package private level48
Use explicit scoping instead of the default package private level53
Use explicit scoping instead of the default package private level58
It is a good practice to call super() in a constructor66 - 74

org/apache/james/util/SchedulerNotifyOutputStream.java

ViolationLine
Use explicit scoping instead of the default package private level38
Use explicit scoping instead of the default package private level43
Use explicit scoping instead of the default package private level48
Use explicit scoping instead of the default package private level53
Use explicit scoping instead of the default package private level58
It is a good practice to call super() in a constructor60 - 68

org/apache/james/util/SqlResources.java

ViolationLine
Each class should declare at least one constructor48 - 373
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '87'-'87').87
Found 'UR'-anomaly for variable 'sqlMap' (lines '87'-'222').87 - 222
Found 'DD'-anomaly for variable 'dbProduct' (lines '100'-'102').100 - 102
Assigning an Object to null is a code smell. Consider refactoring.103
Found 'DD'-anomaly for variable 'sectionElement' (lines '122'-'125').122 - 125
Found 'DD'-anomaly for variable 'found' (lines '123'-'128').123 - 128
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '134'-'228').134 - 228
Found 'DD'-anomaly for variable 'sqlString' (lines '196'-'209').196 - 209
Found 'DU'-anomaly for variable 'dbProductName' (lines '247'-'268').247 - 268
Found 'DD'-anomaly for variable 'dbMatchName' (lines '254'-'254').254
Found 'DU'-anomaly for variable 'dbMatchName' (lines '254'-'268').254 - 268
A method should have only one exit point, and that should be the last statement in the method264
Found 'DU'-anomaly for variable 'dbOptionsMap' (lines '279'-'294').279 - 294
Found 'DU'-anomaly for variable 'dbProduct' (lines '279'-'294').279 - 294
Found 'DU'-anomaly for variable 'find_length' (lines '307'-'323').307 - 323
Found 'DU'-anomaly for variable 'replace_length' (lines '308'-'323').308 - 323
Found 'DU'-anomaly for variable 'outputOffset' (lines '312'-'323').312 - 323
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '347'-'347').347
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '352'-'360').352 - 360

org/apache/james/util/XMLResources.java

ViolationLine
Each class should declare at least one constructor77 - 386
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '107'-'107').107
Found 'UR'-anomaly for variable 'resMap' (lines '107'-'226').107 - 226
Found 'DD'-anomaly for variable 'selectTag' (lines '119'-'121').119 - 121
Found 'DU'-anomaly for variable 'selectTag' (lines '121'-'232').121 - 232
Assigning an Object to null is a code smell. Consider refactoring.122
Found 'DD'-anomaly for variable 'sectionElement' (lines '128'-'130').128 - 130
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '138'-'232').138 - 232
Found 'DD'-anomaly for variable 'resString' (lines '200'-'213').200 - 213
Found 'DU'-anomaly for variable 'selectTagName' (lines '249'-'269').249 - 269
Found 'DD'-anomaly for variable 'matchName' (lines '255'-'255').255
Found 'DU'-anomaly for variable 'matchName' (lines '255'-'269').255 - 269
A method should have only one exit point, and that should be the last statement in the method265
Found 'DU'-anomaly for variable 'find_length' (lines '282'-'298').282 - 298
Found 'DU'-anomaly for variable 'replace_length' (lines '283'-'298').283 - 298
Found 'DU'-anomaly for variable 'outputOffset' (lines '287'-'298').287 - 298
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '320'-'320').320
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '325'-'333').325 - 333
Found 'DU'-anomaly for variable 'replaceBuffer' (lines '374'-'385').374 - 385

org/apache/james/util/connection/ServerConnection.java

ViolationLine
Perhaps 'theRunnerFactory' could be replaced by a local variable.83
It is a good practice to call super() in a constructor123 - 133
Assigning an Object to null is a code smell. Consider refactoring.161
Avoid empty catch blocks165 - 171
Assigning an Object to null is a code smell. Consider refactoring.183
Assigning an Object to null is a code smell. Consider refactoring.193
Found 'DD'-anomaly for variable 'runner' (lines '193'-'191').193 - 191
Found 'DU'-anomaly for variable 'runner' (lines '193'-'200').193 - 200
Found 'DU'-anomaly for variable 'ioExceptionCount' (lines '257'-'352').257 - 352
Avoid empty catch blocks260 - 262
Found 'DD'-anomaly for variable 'clientSocket' (lines '274'-'281').274 - 281
Found 'DU'-anomaly for variable 'clientSocket' (lines '281'-'352').281 - 352
Avoid empty catch blocks313 - 315
Avoid empty catch blocks334 - 336
Assigning an Object to null is a code smell. Consider refactoring.348
Avoid unnecessary constructors - the compiler will generate these for you379 - 380
It is a good practice to call super() in a constructor379 - 380
Assigning an Object to null is a code smell. Consider refactoring.395
Found 'DD'-anomaly for variable 'handler' (lines '420'-'424').420 - 424
Found 'DD'-anomaly for variable 'connectionString' (lines '425'-'427').425 - 427
Found 'DU'-anomaly for variable 'connectionString' (lines '425'-'476').425 - 476
Assigning an Object to null is a code smell. Consider refactoring.452
Assigning an Object to null is a code smell. Consider refactoring.457
Assigning an Object to null is a code smell. Consider refactoring.467
Found 'DU'-anomaly for variable 'handler' (lines '467'-'476').467 - 476
A method should have only one exit point, and that should be the last statement in the method485
Each class should declare at least one constructor504 - 527
Found 'DU'-anomaly for variable 'object' (lines '524'-'526').524 - 526
Avoid unnecessary return statements525

org/apache/james/util/connection/SimpleConnectionManager.java

ViolationLine
Each class should declare at least one constructor42 - 259
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '84'-'84').84
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '88'-'107').88 - 107
Found 'DU'-anomaly for variable 'tearDown' (lines '241'-'248').241 - 248

org/apache/james/util/dbcp/JdbcDataSource.java

ViolationLine
Each class should declare at least one constructor81 - 222
Use explicit scoping instead of the default package private level86
Found 'DU'-anomaly for variable 'dburl' (lines '100'-'202').100 - 202
Found 'DU'-anomaly for variable 'dburl' (lines '100'-'202').100 - 202
Found 'DU'-anomaly for variable 'dburl' (lines '100'-'202').100 - 202
Found 'DU'-anomaly for variable 'user' (lines '101'-'202').101 - 202
Found 'DU'-anomaly for variable 'user' (lines '101'-'202').101 - 202
Found 'DU'-anomaly for variable 'user' (lines '101'-'202').101 - 202
Found 'DU'-anomaly for variable 'password' (lines '102'-'202').102 - 202
Found 'DU'-anomaly for variable 'password' (lines '102'-'202').102 - 202
Found 'DU'-anomaly for variable 'password' (lines '102'-'202').102 - 202
This statement may have some unnecessary parentheses111

org/apache/james/util/io/ClassLoaderObjectInputStream.java

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

org/apache/james/util/io/DirectoryFileFilter.java

ViolationLine
Each class should declare at least one constructor41 - 48

org/apache/james/util/io/ExtensionFileFilter.java

ViolationLine
Found 'DU'-anomaly for variable 'file' (lines '62'-'72').62 - 72
Found 'DU'-anomaly for variable 'name' (lines '62'-'72').62 - 72
A method should have only one exit point, and that should be the last statement in the method68

org/apache/james/util/io/IOUtil.java

ViolationLine
Avoid empty catch blocks145 - 147
Avoid empty catch blocks167 - 169
Avoid empty catch blocks188 - 190
Avoid empty catch blocks209 - 211
Found 'DU'-anomaly for variable 'output' (lines '231'-'242').231 - 242
Found 'DD'-anomaly for variable 'n' (lines '237'-'238').237 - 238
Avoid assignments in operands238
Found 'DU'-anomaly for variable 'n' (lines '238'-'242').238 - 242
Found 'DU'-anomaly for variable 'output' (lines '257'-'266').257 - 266
Found 'DD'-anomaly for variable 'n' (lines '261'-'262').261 - 262
Avoid assignments in operands262
Found 'DU'-anomaly for variable 'n' (lines '262'-'266').262 - 266
Found 'DU'-anomaly for variable 'bufferSize' (lines '706'-'712').706 - 712
A method should have only one exit point, and that should be the last statement in the method734
A method should have only one exit point, and that should be the last statement in the method742

org/apache/james/util/io/ResettableFileInputStream.java

ViolationLine
It is a good practice to call super() in a constructor55 - 63

org/apache/james/util/mail/dsn/DSNStatus.java

ViolationLine
Each class should declare at least one constructor27 - 347

org/apache/james/util/mail/handlers/AbstractDataContentHandler.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you42 - 45
Found 'DD'-anomaly for variable 'content' (lines '73'-'76').73 - 76
Avoid empty catch blocks78 - 81
Found 'DU'-anomaly for variable 'aDataSource' (lines '92'-'93').92 - 93
Found 'DU'-anomaly for variable 'aDataSource' (lines '99'-'106').99 - 106
Found 'DD'-anomaly for variable 'content' (lines '102'-'104').102 - 104
Found 'DD'-anomaly for variable 'dataFlavor' (lines '123'-'124').123 - 124
Avoid assignments in operands124
Found 'DU'-anomaly for variable 'dataFlavor' (lines '124'-'130').124 - 130
A method should have only one exit point, and that should be the last statement in the method127

org/apache/james/util/mail/handlers/message_disposition_notification.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you54 - 57
Found 'UR'-anomaly for variable 'encoding' (lines '71'-'71').71
Found 'DD'-anomaly for variable 'reader' (lines '75'-'80').75 - 80
Found 'DD'-anomaly for variable 'content' (lines '77'-'85').77 - 85
Avoid empty catch blocks102 - 105
Found 'UR'-anomaly for variable 'encoding' (lines '114'-'114').114
Found 'DU'-anomaly for variable 'aMimeType' (lines '114'-'134').114 - 134
Found 'DD'-anomaly for variable 'writer' (lines '122'-'125').122 - 125
Found 'DD'-anomaly for variable 'characterSet' (lines '138'-'141').138 - 141
Avoid empty catch blocks143 - 146

org/apache/james/util/mail/handlers/multipart_report.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you42 - 45
Found 'DU'-anomaly for variable 'aMimeType' (lines '60'-'74').60 - 74

org/apache/james/util/mail/mdn/ActionModeAutomatic.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you33 - 36

org/apache/james/util/mail/mdn/ActionModeManual.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you33 - 36

org/apache/james/util/mail/mdn/ModifierError.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/ModifierExpired.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/ModifierFailed.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/ModifierMailboxTerminated.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/ModifierSuperseded.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/ModifierWarning.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/SendingModeAutomatic.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/SendingModeManual.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/TypeDeleted.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/TypeDenied.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/TypeDispatched.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you30 - 33

org/apache/james/util/mail/mdn/TypeDisplayed.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/TypeFailed.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mail/mdn/TypeProcessed.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you31 - 34

org/apache/james/util/mordred/JdbcDataSource.java

ViolationLine
Each class should declare at least one constructor62 - 544
Perhaps 'connCreationsInProgress' could be replaced by a local variable.77
Perhaps 'connErrorMessage' could be replaced by a local variable.79
Perhaps 'connLastCreated' could be replaced by a local variable.81
Perhaps 'connectionCount' could be replaced by a local variable.83
Perhaps 'jdbcDriver' could be replaced by a local variable.85
A method should have only one exit point, and that should be the last statement in the method146
A method should have only one exit point, and that should be the last statement in the method174
Found 'DD'-anomaly for variable 'attempts' (lines '195'-'129').195 - 129
Found 'UR'-anomaly for variable 'exceptionMessage' (lines '223'-'223').223
Assigning an Object to null is a code smell. Consider refactoring.300
Found 'DD'-anomaly for variable 'stmt' (lines '322'-'324').322 - 324
Avoid empty catch blocks331 - 333
A method should have only one exit point, and that should be the last statement in the method345
A method should have only one exit point, and that should be the last statement in the method364
Found 'UR'-anomaly for variable 'entry' (lines '377'-'377').377
Found 'DU'-anomaly for variable 'entry' (lines '469'-'528').469 - 528
Found 'DD'-anomaly for variable 'entry' (lines '469'-'502').469 - 502
A method should have only one exit point, and that should be the last statement in the method473
A method should have only one exit point, and that should be the last statement in the method481
A method should have only one exit point, and that should be the last statement in the method499
A method should have only one exit point, and that should be the last statement in the method512
Avoid empty catch blocks539 - 540

org/apache/james/util/mordred/PoolConnEntry.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method77
Do not use if statements that are always true or always false84
Assigning an Object to null is a code smell. Consider refactoring.114
Found 'DU'-anomaly for variable 'holdability' (lines '347'-'351').347 - 351
Found 'DU'-anomaly for variable 'savepoint' (lines '365'-'369').365 - 369
Found 'DU'-anomaly for variable 'savepoint' (lines '371'-'375').371 - 375
Found 'DU'-anomaly for variable 'savepoint' (lines '377'-'381').377 - 381
Found 'DU'-anomaly for variable 'resulSetType' (lines '383'-'389').383 - 389
Found 'DU'-anomaly for variable 'resultSetHoldability' (lines '383'-'389').383 - 389
Found 'DU'-anomaly for variable 'resultSetConcurrency' (lines '383'-'389').383 - 389
Found 'DU'-anomaly for variable 'resulSetType' (lines '391'-'398').391 - 398
Found 'DU'-anomaly for variable 'resultSetHoldability' (lines '391'-'398').391 - 398
Found 'DU'-anomaly for variable 'resultSetConcurrency' (lines '391'-'398').391 - 398
Found 'DU'-anomaly for variable 'sql' (lines '391'-'398').391 - 398
Found 'DU'-anomaly for variable 'resulSetType' (lines '400'-'407').400 - 407
Found 'DU'-anomaly for variable 'resultSetHoldability' (lines '400'-'407').400 - 407
Found 'DU'-anomaly for variable 'resultSetConcurrency' (lines '400'-'407').400 - 407
Found 'DU'-anomaly for variable 'sql' (lines '400'-'407').400 - 407
Found 'DU'-anomaly for variable 'autoGeneratedKeys' (lines '409'-'414').409 - 414
Found 'DU'-anomaly for variable 'sql' (lines '409'-'414').409 - 414
Found 'DU'-anomaly for variable 'sql' (lines '416'-'421').416 - 421
Found 'DU'-anomaly for variable 'columnIndexes' (lines '416'-'421').416 - 421
Found 'DU'-anomaly for variable 'columnNames' (lines '423'-'428').423 - 428
Found 'DU'-anomaly for variable 'sql' (lines '423'-'428').423 - 428

org/apache/james/util/watchdog/BytesReadResetInputStream.java

ViolationLine
Use explicit scoping instead of the default package private level51
It is a good practice to call super() in a constructor58 - 66

org/apache/james/util/watchdog/BytesWrittenResetOutputStream.java

ViolationLine
Use explicit scoping instead of the default package private level36
Use explicit scoping instead of the default package private level46
Use explicit scoping instead of the default package private level51
It is a good practice to call super() in a constructor53 - 61

org/apache/james/util/watchdog/InaccurateTimeoutWatchdog.java

ViolationLine
It is a good practice to call super() in a constructor85 - 95
Assigning an Object to null is a code smell. Consider refactoring.153
Assigning an Object to null is a code smell. Consider refactoring.175
Assigning an Object to null is a code smell. Consider refactoring.193
Assigning an Object to null is a code smell. Consider refactoring.215
Assigning an Object to null is a code smell. Consider refactoring.219

org/apache/james/util/watchdog/ThreadPerWatchdogFactory.java

ViolationLine
It is a good practice to call super() in a constructor56 - 62

org/apache/james/util/watchdog/WatchdogFactory.java

ViolationLine
Found 'DU'-anomaly for variable 'theTarget' (lines '33'-'33').33

org/apache/mailet/GenericMailet.java

ViolationLine
Each class should declare at least one constructor36 - 212
Found 'DU'-anomaly for variable 'defValue' (lines '73'-'80').73 - 80
A method should have only one exit point, and that should be the last statement in the method76
Found 'DU'-anomaly for variable 'mail' (lines '211'-'211').211

org/apache/mailet/GenericMatcher.java

ViolationLine
Each class should declare at least one constructor36 - 178
Use explicit scoping instead of the default package private level37
Found 'DU'-anomaly for variable 'mail' (lines '177'-'177').177

org/apache/mailet/GenericRecipientMatcher.java

ViolationLine
Each class should declare at least one constructor35 - 64
Found 'DU'-anomaly for variable 'recipient' (lines '63'-'63').63

org/apache/mailet/Mail.java

ViolationLine
Found 'DU'-anomaly for variable 'newName' (lines '55'-'55').55
Found 'DU'-anomaly for variable 'recipients' (lines '74'-'74').74
Found 'DU'-anomaly for variable 'msg' (lines '110'-'110').110
Found 'DU'-anomaly for variable 'message' (lines '116'-'116').116
Found 'DU'-anomaly for variable 'state' (lines '122'-'122').122
Found 'DU'-anomaly for variable 'name' (lines '140'-'140').140
Found 'DU'-anomaly for variable 'name' (lines '166'-'166').166
Found 'DU'-anomaly for variable 'object' (lines '189'-'189').189
Found 'DU'-anomaly for variable 'name' (lines '189'-'189').189
Found 'DU'-anomaly for variable 'lastUpdated' (lines '204'-'204').204

org/apache/mailet/MailAddress.java

ViolationLine
These nested if statements could be combined81 - 86
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 method227
A method should have only one exit point, and that should be the last statement in the method236
A method should have only one exit point, and that should be the last statement in the method239
A method should have only one exit point, and that should be the last statement in the method242
Found 'DU'-anomaly for variable 'address' (lines '260'-'295').260 - 295
Found 'DU'-anomaly for variable 'address' (lines '297'-'348').297 - 348
Found 'DD'-anomaly for variable 'lastCharDot' (lines '300'-'314').300 - 314
Found 'DD'-anomaly for variable 'lastCharDot' (lines '300'-'318').300 - 318
Found 'DD'-anomaly for variable 'lastCharDot' (lines '300'-'341').300 - 341
Found 'DD'-anomaly for variable 'lastCharDot' (lines '314'-'314').314
Found 'DD'-anomaly for variable 'lastCharDot' (lines '314'-'318').314 - 318
Found 'DD'-anomaly for variable 'lastCharDot' (lines '314'-'341').314 - 341
Found 'DD'-anomaly for variable 'lastCharDot' (lines '318'-'314').318 - 314
Found 'DD'-anomaly for variable 'lastCharDot' (lines '318'-'318').318
Found 'DD'-anomaly for variable 'lastCharDot' (lines '318'-'341').318 - 341
Found 'DD'-anomaly for variable 'lastCharDot' (lines '341'-'314').341 - 314
Found 'DD'-anomaly for variable 'lastCharDot' (lines '341'-'318').341 - 318
Found 'DD'-anomaly for variable 'lastCharDot' (lines '341'-'341').341
Found 'DU'-anomaly for variable 'address' (lines '350'-'371').350 - 371
Found 'DU'-anomaly for variable 'address' (lines '436'-'476').436 - 476

org/apache/mailet/Mailet.java

ViolationLine
Found 'DU'-anomaly for variable 'config' (lines '100'-'100').100
Found 'DU'-anomaly for variable 'mail' (lines '120'-'120').120

org/apache/mailet/MailetConfig.java

ViolationLine
Found 'DU'-anomaly for variable 'name' (lines '43'-'43').43

org/apache/mailet/MailetContext.java

ViolationLine
Found 'DU'-anomaly for variable 'message' (lines '50'-'50').50
Found 'DU'-anomaly for variable 'mail' (lines '50'-'50').50
Found 'DU'-anomaly for variable 'bouncer' (lines '60'-'60').60
Found 'DU'-anomaly for variable 'message' (lines '60'-'60').60
Found 'DU'-anomaly for variable 'mail' (lines '60'-'60').60
Found 'DU'-anomaly for variable 'host' (lines '71'-'71').71
Found 'DU'-anomaly for variable 'name' (lines '95'-'95').95
Found 'DU'-anomaly for variable 'serverName' (lines '144'-'144').144
Found 'DU'-anomaly for variable 'userAccount' (lines '152'-'152').152
Found 'DU'-anomaly for variable 'message' (lines '161'-'161').161
Found 'DU'-anomaly for variable 'message' (lines '170'-'170').170
Found 'DU'-anomaly for variable 't' (lines '170'-'170').170
Found 'DU'-anomaly for variable 'name' (lines '179'-'179').179
Found 'DU'-anomaly for variable 'msg' (lines '189'-'190').189 - 190
Found 'DU'-anomaly for variable 'sender' (lines '201'-'202').201 - 202
Found 'DU'-anomaly for variable 'recipients' (lines '201'-'202').201 - 202
Found 'DU'-anomaly for variable 'msg' (lines '201'-'202').201 - 202
Found 'DU'-anomaly for variable 'sender' (lines '215'-'216').215 - 216
Found 'DU'-anomaly for variable 'recipients' (lines '215'-'216').215 - 216
Found 'DU'-anomaly for variable 'msg' (lines '215'-'216').215 - 216
Found 'DU'-anomaly for variable 'state' (lines '215'-'216').215 - 216
Found 'DU'-anomaly for variable 'mail' (lines '226'-'227').226 - 227
Found 'DU'-anomaly for variable 'object' (lines '240'-'240').240
Found 'DU'-anomaly for variable 'name' (lines '240'-'240').240
Found 'DU'-anomaly for variable 'sender' (lines '253'-'254').253 - 254
Found 'DU'-anomaly for variable 'msg' (lines '253'-'254').253 - 254
Found 'DU'-anomaly for variable 'recipient' (lines '253'-'254').253 - 254
Found 'DU'-anomaly for variable 'domainName' (lines '269'-'269').269

org/apache/mailet/Matcher.java

ViolationLine
Found 'DU'-anomaly for variable 'config' (lines '117'-'117').117
Found 'DU'-anomaly for variable 'mail' (lines '138'-'138').138

org/apache/mailet/dates/RFC977DateFormat.java

ViolationLine
Found 'DD'-anomaly for variable 'source' (lines '74'-'75').74 - 75
A method should have only one exit point, and that should be the last statement in the method77
A method should have only one exit point, and that should be the last statement in the method137
A method should have only one exit point, and that should be the last statement in the method140

org/apache/mailet/dates/SimplifiedDateFormat.java

ViolationLine
Avoid modifiers which are implied by the context44
Found 'DU'-anomaly for variable 'd' (lines '44'-'44').44
Avoid modifiers which are implied by the context55
Found 'DU'-anomaly for variable 'source' (lines '55'-'55').55
Avoid modifiers which are implied by the context61
Found 'DU'-anomaly for variable 'zone' (lines '61'-'61').61
Avoid modifiers which are implied by the context67
Avoid modifiers which are implied by the context77
Found 'DU'-anomaly for variable 'lenient' (lines '77'-'77').77
Avoid modifiers which are implied by the context83

org/apache/mailet/dates/SynchronizedDateFormat.java

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