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 constructor28 - 83

org/apache/james/context/AvalonContextUtilities.java

ViolationLine
Found 'DD'-anomaly for variable 'baseDirectory' (lines '74'-'78').74 - 78

org/apache/james/core/AbstractJamesService.java

ViolationLine
Each class should declare at least one constructor52 - 539
Perhaps 'm_disposed' could be replaced by a local variable.159
Found 'DD'-anomaly for variable 'confSocketType' (lines '195'-'197').195 - 197
Found 'DD'-anomaly for variable 'hostName' (lines '243'-'245').243 - 245
Found 'DD'-anomaly for variable 'hostName' (lines '245'-'247').245 - 247
A method should have only one exit point, and that should be the last statement in the method385
Assigning an Object to null is a code smell. Consider refactoring.422
Assigning an Object to null is a code smell. Consider refactoring.424
Assigning an Object to null is a code smell. Consider refactoring.425
Found 'DD'-anomaly for variable 'theWatchdogFactory' (lines '442'-'443').442 - 443
A method should have only one exit point, and that should be the last statement in the method525

org/apache/james/core/AvalonMailStore.java

ViolationLine
Each class should declare at least one constructor47 - 396
Found 'DU'-anomaly for variable 'className' (lines '145'-'182').145 - 182
Found 'DU'-anomaly for variable 'infoEnabled' (lines '146'-'182').146 - 182
Found 'DU'-anomaly for variable 'types' (lines '149'-'182').149 - 182
Found 'DU'-anomaly for variable 'protocol' (lines '152'-'182').152 - 182
Found 'DD'-anomaly for variable 'protocol' (lines '152'-'152').152
Found 'DU'-anomaly for variable 'defConf' (lines '155'-'182').155 - 182
Found 'DD'-anomaly for variable 'defConf' (lines '155'-'155').155
Found 'UR'-anomaly for variable 'error' (lines '202'-'202').202
Found 'UR'-anomaly for variable 'destination' (lines '202'-'202').202
Found 'DD'-anomaly for variable 'repConf' (lines '203'-'205').203 - 205
Found 'DD'-anomaly for variable 'destination' (lines '210'-'213').210 - 213
Found 'DD'-anomaly for variable 'protocol' (lines '211'-'219').211 - 219
Found 'DU'-anomaly for variable 'protocol' (lines '219'-'315').219 - 315
Found 'DD'-anomaly for variable 'logBuffer' (lines '230'-'233').230 - 233
Found 'DU'-anomaly for variable 'logBuffer' (lines '230'-'315').230 - 315
Found 'DD'-anomaly for variable 'logBuffer' (lines '230'-'247').230 - 247
Found 'DD'-anomaly for variable 'logBuffer' (lines '230'-'293').230 - 293
A method should have only one exit point, and that should be the last statement in the method241
Found 'DD'-anomaly for variable 'comp' (lines '340'-'342').340 - 342
Found 'DU'-anomaly for variable 'toConfig' (lines '358'-'387').358 - 387
Found 'DU'-anomaly for variable 'component' (lines '395'-'395').395

org/apache/james/core/AvalonUsersStore.java

ViolationLine
Each class should declare at least one constructor43 - 171
Found 'UR'-anomaly for variable 'error' (lines '95'-'95').95
Found 'DU'-anomaly for variable 'theClassLoader' (lines '102'-'142').102 - 142

org/apache/james/core/LocalUsersRepository.java

ViolationLine
Each class should declare at least one constructor30 - 99

org/apache/james/core/MailetConfigImpl.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you55 - 56

org/apache/james/core/MailHeaders.java

ViolationLine
Overriding method merely calls super132 - 135

org/apache/james/core/MailImpl.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.125
Found 'UR'-anomaly for variable 'returnPathHeader' (lines '232'-'232').232
Found 'DD'-anomaly for variable 'mailAddress' (lines '233'-'242').233 - 242
Found 'DD'-anomaly for variable 'returnPathHeader' (lines '235'-'237').235 - 237
Found 'DU'-anomaly for variable 'returnPathHeader' (lines '235'-'250').235 - 250
A method should have only one exit point, and that should be the last statement in the method269
Avoid empty catch blocks270 - 272
Found 'DU'-anomaly for variable 'out' (lines '458'-'464').458 - 464
Assigning an Object to null is a code smell. Consider refactoring.483
Assigning an Object to null is a code smell. Consider refactoring.535

org/apache/james/core/MatcherConfigImpl.java

ViolationLine
Each class should declare at least one constructor27 - 101

org/apache/james/core/MimeMessageCopyOnWriteProxy.java

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

org/apache/james/core/MimeMessageInputStreamSource.java

ViolationLine
Use explicit scoping instead of the default package private level49
Use explicit scoping instead of the default package private level54
It is a good practice to call super() in a constructor67 - 117
Avoid assignments in operands76
Assigning an Object to null is a code smell. Consider refactoring.89
Avoid empty catch blocks90 - 92
Assigning an Object to null is a code smell. Consider refactoring.96
Avoid empty catch blocks105 - 107
Avoid empty catch blocks113 - 115
Avoid empty catch blocks156 - 158
Assigning an Object to null is a code smell. Consider refactoring.159

org/apache/james/core/MimeMessageSource.java

ViolationLine
Each class should declare at least one constructor30 - 80
Found 'DD'-anomaly for variable 'in' (lines '59'-'61').59 - 61
Found 'DD'-anomaly for variable 'read' (lines '62'-'64').62 - 64
Avoid assignments in operands64
Found 'DU'-anomaly for variable 'read' (lines '64'-'78').64 - 78
Avoid empty catch blocks72 - 75

org/apache/james/core/MimeMessageUtil.java

ViolationLine
Each class should declare at least one constructor41 - 302
Found 'DD'-anomaly for variable 'testMessage' (lines '56'-'59').56 - 59
Found 'DD'-anomaly for variable 'bos' (lines '100'-'125').100 - 125
Found 'DD'-anomaly for variable 'bis' (lines '101'-'124').101 - 124
Found 'DD'-anomaly for variable 'bis' (lines '124'-'142').124 - 142
Found 'DD'-anomaly for variable 'bos' (lines '125'-'143').125 - 143
Found 'DD'-anomaly for variable 'read' (lines '164'-'165').164 - 165
Avoid assignments in operands165
Found 'DU'-anomaly for variable 'read' (lines '165'-'169').165 - 169
Each class should declare at least one constructor220 - 238
Use explicit scoping instead of the default package private level221
Found 'DU'-anomaly for variable 'arg0' (lines '223'-'225').223 - 225
Found 'DU'-anomaly for variable 'arg1' (lines '231'-'233').231 - 233
Found 'DU'-anomaly for variable 'arg0' (lines '231'-'233').231 - 233
Found 'DD'-anomaly for variable 'size' (lines '248'-'252').248 - 252
Found 'DD'-anomaly for variable 'size' (lines '248'-'255').248 - 255

org/apache/james/core/MimeMessageWrapper.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.80
Assigning an Object to null is a code smell. Consider refactoring.199
Assigning an Object to null is a code smell. Consider refactoring.209
Found 'DD'-anomaly for variable 'in' (lines '301'-'303').301 - 303
Found 'DU'-anomaly for variable 'in' (lines '303'-'330').303 - 330
A method should have only one exit point, and that should be the last statement in the method305
A method should have only one exit point, and that should be the last statement in the method308
Avoid empty while statements321 - 323
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 method338
Found 'DU'-anomaly for variable 'newHeaders' (lines '509'-'516').509 - 516
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 method538

org/apache/james/dnsserver/DNSServer.java

ViolationLine
Each class should declare at least one constructor59 - 536
Perhaps 'mxComparator' could be replaced by a local variable.95
A method should have only one exit point, and that should be the last statement in the method203
Found 'DD'-anomaly for variable 'mxAnswers' (lines '206'-'208').206 - 208
Found 'DD'-anomaly for variable 'mxAnswers' (lines '208'-'208').208
Found 'DD'-anomaly for variable 'servers' (lines '233'-'235').233 - 235
Found 'DU'-anomaly for variable 'querysent' (lines '299'-'356').299 - 356
Found 'DU'-anomaly for variable 'type' (lines '299'-'356').299 - 356
Found 'DD'-anomaly for variable 'name' (lines '300'-'302').300 - 302
Found 'DU'-anomaly for variable 'name' (lines '302'-'356').302 - 356
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 method317
A method should have only one exit point, and that should be the last statement in the method320
A method should have only one exit point, and that should be the last statement in the method323
A method should have only one exit point, and that should be the last statement in the method339
A method should have only one exit point, and that should be the last statement in the method346
A method should have only one exit point, and that should be the last statement in the method351
Found 'DD'-anomaly for variable 'answerCount' (lines '360'-'363').360 - 363
Found 'DU'-anomaly for variable 'n' (lines '360'-'378').360 - 378
Found 'DD'-anomaly for variable 'answers' (lines '368'-'374').368 - 374
Found 'DD'-anomaly for variable 'answers' (lines '374'-'374').374
Each class should declare at least one constructor397 - 404
Found 'DU'-anomaly for variable 'domainName' (lines '423'-'481').423 - 481
Found 'DU'-anomaly for variable 'domainName' (lines '423'-'481').423 - 481
Found 'DU'-anomaly for variable 'domainName' (lines '423'-'481').423 - 481
Found 'DU'-anomaly for variable 'domainName' (lines '423'-'481').423 - 481
Found 'DU'-anomaly for variable 'domainName' (lines '423'-'481').423 - 481
Found 'DU'-anomaly for variable 'domainName' (lines '423'-'481').423 - 481
Found 'DU'-anomaly for variable 'domainName' (lines '423'-'481').423 - 481

org/apache/james/dnsserver/DNSServerMBean.java

ViolationLine
Avoid modifiers which are implied by the context30

org/apache/james/fetchmail/Account.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.197
Found 'DD'-anomaly for variable 'messageIDs' (lines '278'-'279').278 - 279
Avoid assignments in operands279 - 280
Found 'DU'-anomaly for variable 'messageIDs' (lines '279'-'286').279 - 286
A method should have only one exit point, and that should be the last statement in the method283

org/apache/james/fetchmail/FetchMail.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method113
A method should have only one exit point, and that should be the last statement in the method115
Avoid unnecessary constructors - the compiler will generate these for you402 - 405
Found 'UR'-anomaly for variable 'accountsChild' (lines '415'-'415').415
Found 'DU'-anomaly for variable 'arg0' (lines '486'-'569').486 - 569
Found 'UR'-anomaly for variable 'errorBuffer' (lines '583'-'583').583
Found 'DU'-anomaly for variable 'errorBuffer' (lines '591'-'601').591 - 601
A method should have only one exit point, and that should be the last statement in the method680
Found 'DU'-anomaly for variable 'oldAccounts' (lines '754'-'779').754 - 779
A method should have only one exit point, and that should be the last statement in the method790
Found 'DU'-anomaly for variable 'parameters' (lines '822'-'855').822 - 855
Found 'DU'-anomaly for variable 'oldAccounts' (lines '822'-'855').822 - 855
A method should have only one exit point, and that should be the last statement in the method874
Found 'DD'-anomaly for variable 'session' (lines '903'-'904').903 - 904
Avoid assignments in operands904
Found 'DU'-anomaly for variable 'session' (lines '904'-'910').904 - 910
A method should have only one exit point, and that should be the last statement in the method907
Found 'DU'-anomaly for variable 'properties' (lines '960'-'980').960 - 980

org/apache/james/fetchmail/FetchScheduler.java

ViolationLine
Each class should declare at least one constructor44 - 151

org/apache/james/fetchmail/FetchSchedulerMBean.java

ViolationLine
Avoid modifiers which are implied by the context33

org/apache/james/fetchmail/FolderProcessor.java

ViolationLine
Found 'DD'-anomaly for variable 'messagesProcessed' (lines '66'-'96').66 - 96
Found 'DD'-anomaly for variable 'messageCount' (lines '67'-'85').67 - 85
Found 'DD'-anomaly for variable 'messagesProcessed' (lines '96'-'96').96
Avoid empty catch blocks124 - 127
Avoid unnecessary return statements151
Found 'DD'-anomaly for variable 'openFlag' (lines '190'-'193').190 - 193
Found 'DD'-anomaly for variable 'isSeen' (lines '215'-'217').215 - 217
Found 'DD'-anomaly for variable 'isSeen' (lines '215'-'219').215 - 219
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.229 - 230
Found 'DD'-anomaly for variable 'markSeenPermanent' (lines '266'-'267').266 - 267
Avoid assignments in operands267
Found 'DU'-anomaly for variable 'markSeenPermanent' (lines '267'-'273').267 - 273
A method should have only one exit point, and that should be the last statement in the method270

org/apache/james/fetchmail/MessageProcessor.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method304
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 method341
A method should have only one exit point, and that should be the last statement in the method347
A method should have only one exit point, and that should be the last statement in the method353
A method should have only one exit point, and that should be the last statement in the method360
A method should have only one exit point, and that should be the last statement in the method367
Found 'DD'-anomaly for variable 'mail' (lines '378'-'381').378 - 381
Found 'DU'-anomaly for variable 'mail' (lines '381'-'406').381 - 406
A method should have only one exit point, and that should be the last statement in the method386
A method should have only one exit point, and that should be the last statement in the method391
Avoid unnecessary return statements429
Avoid unnecessary return statements452
Avoid unnecessary return statements479
Avoid unnecessary return statements503
Avoid unnecessary return statements527
Avoid unnecessary return statements549
Found 'DD'-anomaly for variable 'messageOut' (lines '565'-'567').565 - 567
Found 'DD'-anomaly for variable 'messageOut' (lines '565'-'569').565 - 569
Found 'DD'-anomaly for variable 'from' (lines '678'-'682').678 - 682
Found 'DD'-anomaly for variable 'internetAddress' (lines '679'-'701').679 - 701
Found 'DD'-anomaly for variable 'internetAddress' (lines '679'-'712').679 - 712
Found 'DD'-anomaly for variable 'from' (lines '682'-'686').682 - 686
Found 'DD'-anomaly for variable 'headers' (lines '748'-'751').748 - 751
Found 'DU'-anomaly for variable 'headerTokens' (lines '760'-'815').760 - 815
Avoid assignments in operands781 - 782
Found 'DU'-anomaly for variable 'ex' (lines '840'-'856').840 - 856
Found 'DU'-anomaly for variable 'ex' (lines '863'-'881').863 - 881
Found 'DD'-anomaly for variable 'count' (lines '934'-'939').934 - 939
Found 'DD'-anomaly for variable 'count' (lines '939'-'939').939
A method should have only one exit point, and that should be the last statement in the method995
Found 'DD'-anomaly for variable 'start' (lines '1,008'-'1,018').1008 - 1018
Found 'DD'-anomaly for variable 'end' (lines '1,009'-'1,019').1009 - 1019
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,010'-'1,014').1010 - 1014
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,014'-'1,027').1014 - 1027
Found 'DU'-anomaly for variable 'hasBracket' (lines '1,014'-'1,063').1014 - 1063
Found 'DD'-anomaly for variable 'end' (lines '1,019'-'1,033').1019 - 1033
Found 'DD'-anomaly for variable 'end' (lines '1,019'-'1,036').1019 - 1036
Found 'DU'-anomaly for variable 'end' (lines '1,019'-'1,063').1019 - 1063
Found 'DU'-anomaly for variable 'nextSearchAt' (lines '1,020'-'1,063').1020 - 1063
Found 'DD'-anomaly for variable 'nextSearchAt' (lines '1,020'-'1,006').1020 - 1006
Found 'DU'-anomaly for variable 'hasBracket' (lines '1,027'-'1,063').1027 - 1063
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,027'-'1,010').1027 - 1010
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,027'-'1,027').1027
Found 'DD'-anomaly for variable 'hasBracket' (lines '1,027'-'1,014').1027 - 1014
Found 'DD'-anomaly for variable 'usableAddress' (lines '1,030'-'1,030').1030
A method should have only one exit point, and that should be the last statement in the method1053
A method should have only one exit point, and that should be the last statement in the method1082
A method should have only one exit point, and that should be the last statement in the method1096
A method should have only one exit point, and that should be the last statement in the method1113
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.1198
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.1200
Avoid unnecessary temporaries when converting primitives to Strings1336
Found 'DU'-anomaly for variable 'mail' (lines '1,363'-'1,375').1363 - 1375
Avoid assignments in operands1411
Found 'DU'-anomaly for variable 'remoteDomain' (lines '1,411'-'1,417').1411 - 1417
A method should have only one exit point, and that should be the last statement in the method1414
Found 'DD'-anomaly for variable 'address' (lines '1,456'-'1,463').1456 - 1463
Found 'DD'-anomaly for variable 'address' (lines '1,456'-'1,469').1456 - 1469
Found 'DD'-anomaly for variable 'validatedAddress' (lines '1,457'-'1,471').1457 - 1471
Found 'DD'-anomaly for variable 'ipAddressEnd' (lines '1,459'-'1,461').1459 - 1461
Avoid assignments in operands1502
Found 'DU'-anomaly for variable 'remoteAddress' (lines '1,502'-'1,508').1502 - 1508
A method should have only one exit point, and that should be the last statement in the method1505
Avoid assignments in operands1527
Found 'DU'-anomaly for variable 'remoteHostName' (lines '1,527'-'1,533').1527 - 1533
A method should have only one exit point, and that should be the last statement in the method1530
Found 'DD'-anomaly for variable 'isMaxMessageSizeExceeded' (lines '1,595'-'1,596').1595 - 1596
Avoid assignments in operands1596 - 1597
Found 'DU'-anomaly for variable 'isMaxMessageSizeExceeded' (lines '1,596'-'1,603').1596 - 1603
A method should have only one exit point, and that should be the last statement in the method1600
A method should have only one exit point, and that should be the last statement in the method1620
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.1621
Found 'DD'-anomaly for variable 'isInvalid' (lines '1,648'-'1,649').1648 - 1649
Avoid assignments in operands1649
Found 'DU'-anomaly for variable 'isInvalid' (lines '1,649'-'1,655').1649 - 1655
A method should have only one exit point, and that should be the last statement in the method1652
Found 'DD'-anomaly for variable 'isInvalid' (lines '1,664'-'1,671').1664 - 1671

org/apache/james/fetchmail/ParsedConfiguration.java

ViolationLine
Found 'UR'-anomaly for variable 'token' (lines '670'-'670').670
Found 'DD'-anomaly for variable 'token' (lines '675'-'680').675 - 680
Found 'DU'-anomaly for variable 'token' (lines '675'-'690').675 - 690
Found 'DD'-anomaly for variable 'defaultDomainName' (lines '806'-'807').806 - 807
Avoid assignments in operands807
Found 'DU'-anomaly for variable 'defaultDomainName' (lines '807'-'813').807 - 813
A method should have only one exit point, and that should be the last statement in the method810
Found 'DD'-anomaly for variable 'hostName' (lines '862'-'870').862 - 870
Found 'DD'-anomaly for variable 'hostName' (lines '870'-'874').870 - 874

org/apache/james/fetchmail/ProcessorAbstract.java

ViolationLine
Found 'DD'-anomaly for variable 'value' (lines '516'-'517').516 - 517
Avoid assignments in operands517
Found 'DU'-anomaly for variable 'value' (lines '517'-'523').517 - 523
A method should have only one exit point, and that should be the last statement in the method520

org/apache/james/fetchmail/ReaderInputStream.java

ViolationLine
It is a good practice to call super() in a constructor31 - 33

org/apache/james/fetchmail/StoreProcessor.java

ViolationLine
Found 'DD'-anomaly for variable 'store' (lines '51'-'67').51 - 67
Found 'DD'-anomaly for variable 'folder' (lines '52'-'78').52 - 78

org/apache/james/James.java

ViolationLine
Each class should declare at least one constructor87 - 883
Perhaps 'context' could be replaced by a local variable.105
Perhaps 'conf' could be replaced by a local variable.110
Perhaps 'mailetLogger' could be replaced by a local variable.115
Perhaps 'usersStore' could be replaced by a local variable.125
Found 'UR'-anomaly for variable 'conf' (lines '213'-'263').213 - 263
Found 'DD'-anomaly for variable 'hostName' (lines '250'-'252').250 - 252
Found 'DD'-anomaly for variable 'hostName' (lines '252'-'254').252 - 254
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.341
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.342
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.343
Found 'DD'-anomaly for variable 'userInbox' (lines '521'-'523').521 - 523
A method should have only one exit point, and that should be the last statement in the method526
Found 'DD'-anomaly for variable 'localCount' (lines '568'-'570').568 - 570
Found 'DU'-anomaly for variable 'args' (lines '587'-'592').587 - 592
Found 'DD'-anomaly for variable 'dnsServer' (lines '604'-'606').604 - 606
Found 'DU'-anomaly for variable 'bouncer' (lines '669'-'688').669 - 688
Found 'DU'-anomaly for variable 'message' (lines '669'-'688').669 - 688
A method should have only one exit point, and that should be the last statement in the method728
Found 'DD'-anomaly for variable 'dnsServer' (lines '851'-'853').851 - 853

org/apache/james/JamesMBean.java

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

org/apache/james/mailrepository/AvalonMailRepository.java

ViolationLine
Each class should declare at least one constructor60 - 440
Found 'UR'-anomaly for variable 'exceptionString' (lines '89'-'89').89
Found 'DU'-anomaly for variable 'exceptionString' (lines '96'-'106').96 - 106
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 method233
Found 'DD'-anomaly for variable 'wasLocked' (lines '249'-'251').249 - 251
Found 'DD'-anomaly for variable 'saveStream' (lines '262'-'293').262 - 293
Found 'DD'-anomaly for variable 'out' (lines '297'-'299').297 - 299
Found 'DD'-anomaly for variable 'mc' (lines '344'-'346').344 - 346
Found 'DU'-anomaly for variable 'mc' (lines '346'-'370').346 - 370
A method should have only one exit point, and that should be the last statement in the method360
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '400'-'400').400
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '410'-'417').410 - 417

org/apache/james/mailrepository/AvalonSpoolRepository.java

ViolationLine
Each class should declare at least one constructor37 - 181
Found 'DU'-anomaly for variable 'delay' (lines '76'-'116').76 - 116
Found 'DU'-anomaly for variable 'delay' (lines '76'-'116').76 - 116
Found 'DU'-anomaly for variable 'delay' (lines '76'-'116').76 - 116
Found 'DU'-anomaly for variable 'filter' (lines '131'-'179').131 - 179

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

ViolationLine
Each class should declare at least one constructor47 - 371
Found 'DU'-anomaly for variable 'configuration' (lines '89'-'97').89 - 97
Found 'UR'-anomaly for variable 'directory' (lines '150'-'150').150
Found 'DD'-anomaly for variable 'directory' (lines '165'-'174').165 - 174
Found 'DD'-anomaly for variable 'directory' (lines '169'-'174').169 - 174
Found 'DD'-anomaly for variable 'child' (lines '195'-'199').195 - 199
Found 'DD'-anomaly for variable 'buffer' (lines '334'-'339').334 - 339
Found 'DU'-anomaly for variable 'j' (lines '336'-'349').336 - 349
Found 'DD'-anomaly for variable 'buffer' (lines '339'-'340').339 - 340
Found 'DD'-anomaly for variable 'buffer' (lines '340'-'339').340 - 339
Found 'DD'-anomaly for variable 'filename' (lines '357'-'359').357 - 359
Found 'DD'-anomaly for variable 'bytes' (lines '361'-'365').361 - 365
Found 'DU'-anomaly for variable 'j' (lines '363'-'370').363 - 370
Found 'DD'-anomaly for variable 'bytes' (lines '365'-'365').365

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

ViolationLine
Each class should declare at least one constructor32 - 140

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

ViolationLine
Each class should declare at least one constructor33 - 85
Found 'UR'-anomaly for variable 'message' (lines '45'-'45').45
Found 'UR'-anomaly for variable 'message' (lines '62'-'62').62
A method should have only one exit point, and that should be the last statement in the method79

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

ViolationLine
Found 'DU'-anomaly for variable 'file' (lines '37'-'57').37 - 57
A method should have only one exit point, and that should be the last statement in the method41
A method should have only one exit point, and that should be the last statement in the method47

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

ViolationLine
Each class should declare at least one constructor42 - 220
Found 'DU'-anomaly for variable 'types' (lines '86'-'106').86 - 106
Found 'DU'-anomaly for variable 'modelIterator' (lines '87'-'106').87 - 106
Found 'DU'-anomaly for variable 'protocol' (lines '92'-'106').92 - 106
Found 'DD'-anomaly for variable 'protocol' (lines '92'-'92').92
Found 'DU'-anomaly for variable 'type' (lines '96'-'106').96 - 106
Found 'DD'-anomaly for variable 'type' (lines '96'-'96').96
Found 'DU'-anomaly for variable 'component' (lines '108'-'110').108 - 110
A method should have only one exit point, and that should be the last statement in the method115
Found 'UR'-anomaly for variable 'message' (lines '120'-'120').120
Found 'UR'-anomaly for variable 'error' (lines '120'-'120').120
Found 'DD'-anomaly for variable 'repConf' (lines '123'-'126').123 - 126
Found 'DD'-anomaly for variable 'destination' (lines '134'-'137').134 - 137
A method should have only one exit point, and that should be the last statement in the method161
Found 'DU'-anomaly for variable 'message' (lines '165'-'214').165 - 214

org/apache/james/mailrepository/JDBCMailRepository.java

ViolationLine
Each class should declare at least one constructor89 - 1066
Found 'DD'-anomaly for variable 'logBuffer' (lines '182'-'184').182 - 184
Found 'DU'-anomaly for variable 'logBuffer' (lines '182'-'194').182 - 194
Found 'UR'-anomaly for variable 'message' (lines '199'-'199').199
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '199'-'199').199
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '227'-'301').227 - 301
Found 'DD'-anomaly for variable 'logBuffer' (lines '314'-'345').314 - 345
Found 'DD'-anomaly for variable 'logBuffer' (lines '314'-'379').314 - 379
Found 'DU'-anomaly for variable 'logBuffer' (lines '314'-'394').314 - 394
Found 'DD'-anomaly for variable 'logBuffer' (lines '314'-'345').314 - 345
Found 'DD'-anomaly for variable 'logBuffer' (lines '314'-'379').314 - 379
Found 'DU'-anomaly for variable 'logBuffer' (lines '314'-'394').314 - 394
Found 'DD'-anomaly for variable 'createStatement' (lines '330'-'374').330 - 374
Found 'DD'-anomaly for variable 'createStatement' (lines '330'-'374').330 - 374
Found 'DD'-anomaly for variable 'sqlFile' (lines '335'-'337').335 - 337
Found 'DD'-anomaly for variable 'sqlFile' (lines '335'-'337').335 - 337
Assigning an Object to null is a code smell. Consider refactoring.338
Found 'UR'-anomaly for variable 'logBuffer' (lines '404'-'404').404
Found 'DD'-anomaly for variable 'hasUpdateMessageAttributesSQL' (lines '406'-'423').406 - 423
Found 'DD'-anomaly for variable 'hasRetrieveMessageAttributesSQL' (lines '407'-'431').407 - 431
Found 'DU'-anomaly for variable 'logBuffer' (lines '411'-'482').411 - 482
A method should have only one exit point, and that should be the last statement in the method504
A method should have only one exit point, and that should be the last statement in the method530
Found 'DD'-anomaly for variable 'conn' (lines '541'-'553').541 - 553
Found 'DD'-anomaly for variable 'wasLocked' (lines '542'-'546').542 - 546
Found 'DD'-anomaly for variable 'checkMessageExists' (lines '560'-'564').560 - 564
Found 'DD'-anomaly for variable 'rsExists' (lines '561'-'568').561 - 568
Found 'DD'-anomaly for variable 'exists' (lines '562'-'569').562 - 569
Found 'DD'-anomaly for variable 'updateMessage' (lines '577'-'580').577 - 580
Assigning an Object to null is a code smell. Consider refactoring.606
Found 'DU'-anomaly for variable 'updateMessage' (lines '606'-'773').606 - 773
Found 'DD'-anomaly for variable 'updateMessageAttr' (lines '614'-'616').614 - 616
Found 'DD'-anomaly for variable 'saveBody' (lines '658'-'666').658 - 666
Found 'DD'-anomaly for variable 'saveBody' (lines '658'-'668').658 - 668
Found 'DD'-anomaly for variable 'conn' (lines '786'-'790').786 - 790
Found 'DD'-anomaly for variable 'retrieveMessage' (lines '787'-'795').787 - 795
Found 'DD'-anomaly for variable 'rsMessage' (lines '788'-'799').788 - 799
A method should have only one exit point, and that should be the last statement in the method813
Found 'DD'-anomaly for variable 'retrieveMessageAttr' (lines '816'-'823').816 - 823
Found 'DU'-anomaly for variable 'retrieveMessageAttr' (lines '816'-'924').816 - 924
Found 'DD'-anomaly for variable 'attributes' (lines '817'-'844').817 - 844
Found 'DD'-anomaly for variable 'rsMessageAttr' (lines '821'-'828').821 - 828
Found 'DD'-anomaly for variable 'serialized_attr' (lines '832'-'836').832 - 836
Found 'DD'-anomaly for variable 'serialized_attr' (lines '832'-'838').832 - 838
Found 'DD'-anomaly for variable 'conn' (lines '956'-'959').956 - 959
Found 'DD'-anomaly for variable 'removeMessage' (lines '957'-'960').957 - 960
Found 'DD'-anomaly for variable 'conn' (lines '986'-'990').986 - 990
Found 'DD'-anomaly for variable 'listMessages' (lines '987'-'991').987 - 991
Found 'DD'-anomaly for variable 'rsListMessages' (lines '988'-'994').988 - 994
A method should have only one exit point, and that should be the last statement in the method1025
Found 'DD'-anomaly for variable 'result' (lines '1,040'-'1,042').1040 - 1042
Found 'DD'-anomaly for variable 'result' (lines '1,040'-'1,045').1040 - 1045
Found 'DD'-anomaly for variable 'result' (lines '1,042'-'1,045').1042 - 1045

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/MailStoreSpoolRepository.java

ViolationLine
Each class should declare at least one constructor49 - 176

org/apache/james/mailrepository/MBoxMailRepository.java

ViolationLine
Each class should declare at least one constructor113 - 868
Use explicit scoping instead of the default package private level118
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
Avoid modifiers which are implied by the context150
Avoid modifiers which are implied by the context151
Found 'DU'-anomaly for variable 'bodyText' (lines '151'-'151').151
Found 'DU'-anomaly for variable 'messageStart' (lines '151'-'151').151
Found 'DU'-anomaly for variable 'messageSeparator' (lines '151'-'151').151
Found 'DD'-anomaly for variable 'mimeMessage' (lines '175'-'181').175 - 181
Found 'DU'-anomaly for variable 'messAct' (lines '242'-'347').242 - 347
Found 'DU'-anomaly for variable 'sepMatchPattern' (lines '255'-'347').255 - 347
Found 'DU'-anomaly for variable 'sepMatch' (lines '256'-'347').256 - 347
Found 'DU'-anomaly for variable 'inMessage' (lines '259'-'347').259 - 347
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '261'-'286').261 - 286
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '261'-'347').261 - 347
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '261'-'314').261 - 314
Found 'DD'-anomaly for variable 'foundSep' (lines '262'-'268').262 - 268
Found 'DU'-anomaly for variable 'foundSep' (lines '262'-'347').262 - 347
Found 'DD'-anomaly for variable 'foundSep' (lines '262'-'297').262 - 297
Found 'DU'-anomaly for variable 'prevMessageStart' (lines '264'-'347').264 - 347
Found 'DD'-anomaly for variable 'line' (lines '266'-'267').266 - 267
Avoid assignments in operands267
Found 'DU'-anomaly for variable 'line' (lines '267'-'347').267 - 347
Found 'DD'-anomaly for variable 'endResult' (lines '274'-'274').274
Found 'DU'-anomaly for variable 'endResult' (lines '274'-'347').274 - 347
A method should have only one exit point, and that should be the last statement in the method277
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '279'-'286').279 - 286
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '279'-'347').279 - 347
Found 'DU'-anomaly for variable 'prevMessageStart' (lines '280'-'347').280 - 347
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '286'-'347').286 - 347
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '286'-'286').286
Found 'DU'-anomaly for variable 'line' (lines '294'-'347').294 - 347
Avoid assignments in operands295
Found 'DU'-anomaly for variable 'c' (lines '295'-'347').295 - 347
A method should have only one exit point, and that should be the last statement in the method305
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '307'-'314').307 - 314
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '307'-'347').307 - 347
Found 'DU'-anomaly for variable 'prevMessageStart' (lines '308'-'347').308 - 347
Found 'DU'-anomaly for variable 'previousMessageSeparator' (lines '314'-'347').314 - 347
Found 'DD'-anomaly for variable 'previousMessageSeparator' (lines '314'-'314').314
Found 'DU'-anomaly for variable 'inMessage' (lines '315'-'347').315 - 347
Found 'DU'-anomaly for variable 'line' (lines '320'-'347').320 - 347
A method should have only one exit point, and that should be the last statement in the method329
Found 'DD'-anomaly for variable 'foundMessage' (lines '357'-'360').357 - 360
Assigning an Object to null is a code smell. Consider refactoring.366
Found 'DU'-anomaly for variable 'foundMessage' (lines '378'-'446').378 - 446
Found 'DU'-anomaly for variable 'foundMessage' (lines '378'-'446').378 - 446
Found 'DU'-anomaly for variable 'foundMessage' (lines '378'-'446').378 - 446
A method should have only one exit point, and that should be the last statement in the method391
Found 'DD'-anomaly for variable 'ins' (lines '406'-'408').406 - 408
Found 'DD'-anomaly for variable 'ins' (lines '406'-'408').406 - 408
Found 'DD'-anomaly for variable 'ins' (lines '406'-'408').406 - 408
Found 'DU'-anomaly for variable 'ins' (lines '408'-'446').408 - 446
Found 'DU'-anomaly for variable 'ins' (lines '408'-'446').408 - 446
Found 'DU'-anomaly for variable 'ins' (lines '408'-'446').408 - 446
Found 'DU'-anomaly for variable 'op' (lines '412'-'446').412 - 446
Found 'DU'-anomaly for variable 'op' (lines '412'-'446').412 - 446
Found 'DU'-anomaly for variable 'op' (lines '412'-'446').412 - 446
These nested if statements could be combined433 - 441
Found 'DD'-anomaly for variable 'ins' (lines '455'-'457').455 - 457
Found 'DD'-anomaly for variable 'ins' (lines '455'-'457').455 - 457
Found 'DD'-anomaly for variable 'ins' (lines '455'-'457').455 - 457
Assigning an Object to null is a code smell. Consider refactoring.509
Found 'DD'-anomaly for variable 'fromHeader' (lines '511'-'517').511 - 517
Found 'DD'-anomaly for variable 'fromHeader' (lines '511'-'519').511 - 519
Found 'DD'-anomaly for variable 'message' (lines '512'-'514').512 - 514
Found 'DD'-anomaly for variable 'saveFile' (lines '528'-'530').528 - 530
Found 'DU'-anomaly for variable 'res' (lines '577'-'599').577 - 599
Found 'DD'-anomaly for variable 'res' (lines '577'-'584').577 - 584
A method should have only one exit point, and that should be the last statement in the method582
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/MessageInputStream.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.51
Found 'DD'-anomaly for variable 'bodyOut' (lines '100'-'105').100 - 105
Found 'DD'-anomaly for variable 'bodyOut' (lines '100'-'108').100 - 108
Assigning an Object to null is a code smell. Consider refactoring.138
Assigning an Object to null is a code smell. Consider refactoring.139
Assigning an Object to null is a code smell. Consider refactoring.186

org/apache/james/mailrepository/MimeMessageAvalonSource.java

ViolationLine
Use explicit scoping instead of the default package private level33
Use explicit scoping instead of the default package private level38
Use explicit scoping instead of the default package private level44
Perhaps 'size' could be replaced by a local variable.46
It is a good practice to call super() in a constructor48 - 52

org/apache/james/mailrepository/MimeMessageJDBCSource.java

ViolationLine
Use explicit scoping instead of the default package private level46
Use explicit scoping instead of the default package private level47
Use explicit scoping instead of the default package private level48
Perhaps 'size' could be replaced by a local variable.50
Use explicit scoping instead of the default package private level55
Use explicit scoping instead of the default package private level60
Found 'DU'-anomaly for variable 'logString' (lines '67'-'70').67 - 70
It is a good practice to call super() in a constructor77 - 94
Found 'DD'-anomaly for variable 'conn' (lines '118'-'122').118 - 122
Found 'DD'-anomaly for variable 'retrieveMessageStream' (lines '119'-'131').119 - 131
Found 'DD'-anomaly for variable 'rsRetrieveMessageStream' (lines '120'-'134').120 - 134
Found 'DD'-anomaly for variable 'headers' (lines '124'-'143').124 - 143
Found 'DD'-anomaly for variable 'headers' (lines '124'-'145').124 - 145
Found 'DD'-anomaly for variable 'start' (lines '126'-'128').126 - 128
Found 'DU'-anomaly for variable 'start' (lines '126'-'169').126 - 169
Found 'DU'-anomaly for variable 'start' (lines '128'-'169').128 - 169
Found 'DD'-anomaly for variable 'in' (lines '152'-'155').152 - 155
Avoid empty catch blocks157 - 160
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 method179
Found 'DD'-anomaly for variable 'conn' (lines '181'-'185').181 - 185
Found 'DD'-anomaly for variable 'retrieveMessageSize' (lines '182'-'187').182 - 187
Found 'DD'-anomaly for variable 'rsRetrieveMessageSize' (lines '183'-'190').183 - 190
Found 'DD'-anomaly for variable 'in' (lines '198'-'204').198 - 204
Found 'DD'-anomaly for variable 'len' (lines '205'-'207').205 - 207
Avoid assignments in operands207
Found 'DU'-anomaly for variable 'len' (lines '207'-'233').207 - 233
Avoid empty catch blocks212 - 215
Avoid empty catch blocks220 - 222
A method should have only one exit point, and that should be the last statement in the method243 - 244
Found 'DD'-anomaly for variable 'result' (lines '255'-'257').255 - 257
Found 'DD'-anomaly for variable 'result' (lines '255'-'260').255 - 260
Found 'DD'-anomaly for variable 'result' (lines '257'-'260').257 - 260

org/apache/james/Main.java

ViolationLine
Each class should declare at least one constructor27 - 42
Found 'DU'-anomaly for variable 'args' (lines '35'-'41').35 - 41

org/apache/james/nntpserver/NNTPException.java

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

org/apache/james/nntpserver/NNTPHandler.java

ViolationLine
Each class should declare at least one constructor65 - 1746
Use explicit scoping instead of the default package private level300
Perhaps 'theWatchdogTarget' could be replaced by a local variable.310
Use explicit scoping instead of the default package private level317 - 319
Use explicit scoping instead of the default package private level326 - 328
Use explicit scoping instead of the default package private level336 - 338
Use explicit scoping instead of the default package private level343 - 364
Avoid empty catch blocks351 - 353
Assigning an Object to null is a code smell. Consider refactoring.354
Assigning an Object to null is a code smell. Consider refactoring.361
Avoid empty catch blocks430
Assigning an Object to null is a code smell. Consider refactoring.446
Assigning an Object to null is a code smell. Consider refactoring.457
Assigning an Object to null is a code smell. Consider refactoring.460
Assigning an Object to null is a code smell. Consider refactoring.464
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.469
Assigning an Object to null is a code smell. Consider refactoring.477
Assigning an Object to null is a code smell. Consider refactoring.481
Assigning an Object to null is a code smell. Consider refactoring.485
Assigning an Object to null is a code smell. Consider refactoring.489
Assigning an Object to null is a code smell. Consider refactoring.490
Assigning an Object to null is a code smell. Consider refactoring.494
A method should have only one exit point, and that should be the last statement in the method511
Found 'DD'-anomaly for variable 'argument' (lines '518'-'521').518 - 521
Found 'DU'-anomaly for variable 'argument' (lines '518'-'591').518 - 591
Found 'DU'-anomaly for variable 'argument' (lines '521'-'591').521 - 591
Found 'DD'-anomaly for variable 'returnValue' (lines '526'-'562').526 - 562
A method should have only one exit point, and that should be the last statement in the method530
Found 'DU'-anomaly for variable 'command' (lines '599'-'610').599 - 610
Found 'DU'-anomaly for variable 'argument' (lines '599'-'610').599 - 610
Found 'DD'-anomaly for variable 'command' (lines '620'-'625').620 - 625
Found 'DD'-anomaly for variable 'value' (lines '621'-'626').621 - 626
Found 'DU'-anomaly for variable 'value' (lines '621'-'677').621 - 677
Found 'DU'-anomaly for variable 'value' (lines '626'-'677').626 - 677
A method should have only one exit point, and that should be the last statement in the method631
Assigning an Object to null is a code smell. Consider refactoring.641
Assigning an Object to null is a code smell. Consider refactoring.642
A method should have only one exit point, and that should be the last statement in the method645
A method should have only one exit point, and that should be the last statement in the method653
Assigning an Object to null is a code smell. Consider refactoring.657
Assigning an Object to null is a code smell. Consider refactoring.658
A method should have only one exit point, and that should be the last statement in the method661
Assigning an Object to null is a code smell. Consider refactoring.670
Assigning an Object to null is a code smell. Consider refactoring.671
Found 'DD'-anomaly for variable 'wildmat' (lines '689'-'694').689 - 694
Found 'DU'-anomaly for variable 'wildmat' (lines '689'-'728').689 - 728
Found 'DU'-anomaly for variable 'wildmat' (lines '694'-'728').694 - 728
A method should have only one exit point, and that should be the last statement in the method699
A method should have only one exit point, and that should be the last statement in the method704
Found 'DD'-anomaly for variable 'theDate' (lines '707'-'709').707 - 709
Found 'DU'-anomaly for variable 'theDate' (lines '709'-'728').709 - 728
Found 'DD'-anomaly for variable 'theDate' (lines '747'-'749').747 - 749
Found 'DU'-anomaly for variable 'theDate' (lines '749'-'771').749 - 771
Found 'DU'-anomaly for variable 'argument' (lines '778'-'781').778 - 781
Found 'DU'-anomaly for variable 'argument' (lines '789'-'791').789 - 791
Found 'DU'-anomaly for variable 'argument' (lines '798'-'802').798 - 802
Found 'DU'-anomaly for variable 'argument' (lines '809'-'811').809 - 811
Found 'DD'-anomaly for variable 'wildmat' (lines '820'-'827').820 - 827
Found 'DU'-anomaly for variable 'wildmat' (lines '820'-'873').820 - 873
Found 'DD'-anomaly for variable 'isListNewsgroups' (lines '821'-'835').821 - 835
Found 'DD'-anomaly for variable 'isListNewsgroups' (lines '821'-'837').821 - 837
Found 'DU'-anomaly for variable 'isListNewsgroups' (lines '821'-'873').821 - 873
Found 'DD'-anomaly for variable 'extension' (lines '823'-'828').823 - 828
Found 'DD'-anomaly for variable 'extension' (lines '823'-'830').823 - 830
Found 'DU'-anomaly for variable 'wildmat' (lines '827'-'873').827 - 873
Found 'DD'-anomaly for variable 'extension' (lines '828'-'830').828 - 830
Found 'DU'-anomaly for variable 'isListNewsgroups' (lines '835'-'873').835 - 873
Found 'DU'-anomaly for variable 'isListNewsgroups' (lines '837'-'873').837 - 873
A method should have only one exit point, and that should be the last statement in the method840
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 method847
A method should have only one exit point, and that should be the last statement in the method851
A method should have only one exit point, and that should be the last statement in the method855
Found 'DD'-anomaly for variable 'article' (lines '928'-'930').928 - 930
Found 'DU'-anomaly for variable 'article' (lines '928'-'978').928 - 978
Found 'DD'-anomaly for variable 'article' (lines '928'-'952').928 - 952
Found 'DD'-anomaly for variable 'article' (lines '928'-'957').928 - 957
A method should have only one exit point, and that should be the last statement in the method933
Found 'DU'-anomaly for variable 'newArticleNumber' (lines '942'-'978').942 - 978
Found 'DD'-anomaly for variable 'newArticleNumber' (lines '942'-'956').942 - 956
A method should have only one exit point, and that should be the last statement in the method945
A method should have only one exit point, and that should be the last statement in the method950
Found 'DD'-anomaly for variable 'article' (lines '991'-'993').991 - 993
Found 'DU'-anomaly for variable 'article' (lines '991'-'1,046').991 - 1046
Found 'DD'-anomaly for variable 'article' (lines '991'-'1,015').991 - 1015
Found 'DD'-anomaly for variable 'article' (lines '991'-'1,020').991 - 1020
A method should have only one exit point, and that should be the last statement in the method996
Found 'DU'-anomaly for variable 'newArticleNumber' (lines '1,005'-'1,046').1005 - 1046
Found 'DD'-anomaly for variable 'newArticleNumber' (lines '1,005'-'1,019').1005 - 1019
A method should have only one exit point, and that should be the last statement in the method1008
A method should have only one exit point, and that should be the last statement in the method1013
Found 'DD'-anomaly for variable 'article' (lines '1,059'-'1,061').1059 - 1061
Found 'DU'-anomaly for variable 'article' (lines '1,059'-'1,114').1059 - 1114
Found 'DD'-anomaly for variable 'article' (lines '1,059'-'1,083').1059 - 1083
Found 'DD'-anomaly for variable 'article' (lines '1,059'-'1,088').1059 - 1088
A method should have only one exit point, and that should be the last statement in the method1064
Found 'DU'-anomaly for variable 'newArticleNumber' (lines '1,073'-'1,114').1073 - 1114
Found 'DD'-anomaly for variable 'newArticleNumber' (lines '1,073'-'1,087').1073 - 1087
A method should have only one exit point, and that should be the last statement in the method1076
A method should have only one exit point, and that should be the last statement in the method1081
Found 'DD'-anomaly for variable 'article' (lines '1,127'-'1,129').1127 - 1129
Found 'DU'-anomaly for variable 'article' (lines '1,127'-'1,183').1127 - 1183
Found 'DD'-anomaly for variable 'article' (lines '1,127'-'1,151').1127 - 1151
Found 'DD'-anomaly for variable 'article' (lines '1,127'-'1,156').1127 - 1156
A method should have only one exit point, and that should be the last statement in the method1132
Found 'DU'-anomaly for variable 'newArticleNumber' (lines '1,141'-'1,183').1141 - 1183
Found 'DD'-anomaly for variable 'newArticleNumber' (lines '1,141'-'1,155').1141 - 1155
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 method1149
Found 'DU'-anomaly for variable 'argument' (lines '1,310'-'1,314').1310 - 1314
Found 'DU'-anomaly for variable 'argument' (lines '1,321'-'1,324').1321 - 1324
A method should have only one exit point, and that should be the last statement in the method1337
Found 'DU'-anomaly for variable 'argument' (lines '1,388'-'1,391').1388 - 1391
A method should have only one exit point, and that should be the last statement in the method1413
Found 'DD'-anomaly for variable 'range' (lines '1,416'-'1,419').1416 - 1419
Found 'DU'-anomaly for variable 'range' (lines '1,416'-'1,451').1416 - 1451
Found 'DU'-anomaly for variable 'range' (lines '1,419'-'1,451').1419 - 1451
A method should have only one exit point, and that should be the last statement in the method1424
Found 'DU'-anomaly for variable 'range' (lines '1,469'-'1,495').1469 - 1495
A method should have only one exit point, and that should be the last statement in the method1473
Found 'DD'-anomaly for variable 'msgIn' (lines '1,502'-'1,504').1502 - 1504
Found 'DD'-anomaly for variable 'messageIn' (lines '1,532'-'1,534').1532 - 1534
Avoid empty catch blocks1540 - 1542
Assigning an Object to null is a code smell. Consider refactoring.1543
Found 'DU'-anomaly for variable 'messageIn' (lines '1,543'-'1,546').1543 - 1546
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '1,556'-'1,556').1556
Found 'DU'-anomaly for variable 'd' (lines '1,567'-'1,590').1567 - 1590
Found 'DD'-anomaly for variable 'dt' (lines '1,574'-'1,576').1574 - 1576
A method should have only one exit point, and that should be the last statement in the method1606 - 1607
Found 'DD'-anomaly for variable 'start' (lines '1,614'-'1,618').1614 - 1618
Found 'DD'-anomaly for variable 'start' (lines '1,614'-'1,621').1614 - 1621
Found 'DD'-anomaly for variable 'end' (lines '1,615'-'1,619').1615 - 1619
Found 'DD'-anomaly for variable 'end' (lines '1,615'-'1,623').1615 - 1623
Found 'DD'-anomaly for variable 'end' (lines '1,615'-'1,625').1615 - 1625
Found 'DU'-anomaly for variable 'command' (lines '1,645'-'1,655').1645 - 1655
A method should have only one exit point, and that should be the last statement in the method1648
Found 'DD'-anomaly for variable 'allowed' (lines '1,651'-'1,652').1651 - 1652
A method should have only one exit point, and that should be the last statement in the method1665
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 method1686
Found 'DU'-anomaly for variable 'responseString' (lines '1,700'-'1,704').1700 - 1704
Use explicit scoping instead of the default package private level1713 - 1717
Use explicit scoping instead of the default package private level1725 - 1728
Each class should declare at least one constructor1735 - 1745

org/apache/james/nntpserver/NNTPServer.java

ViolationLine
Each class should declare at least one constructor43 - 242
Perhaps 'theHandlerFactory' could be replaced by a local variable.68
Each class should declare at least one constructor182 - 205
Found 'DU'-anomaly for variable 'object' (lines '202'-'204').202 - 204
Avoid unnecessary return statements203
Each class should declare at least one constructor210 - 241

org/apache/james/nntpserver/NNTPServerMBean.java

ViolationLine
Avoid modifiers which are implied by the context34
Avoid modifiers which are implied by the context43
Avoid modifiers which are implied by the context52
Avoid modifiers which are implied by the context61

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

ViolationLine
Perhaps 'counter' could be replaced by a local variable.59
Use explicit scoping instead of the default package private level69 - 83
Use explicit scoping instead of the default package private level90 - 103
Found 'DD'-anomaly for variable 'fout' (lines '94'-'96').94 - 96
Use explicit scoping instead of the default package private level113 - 121
Use explicit scoping instead of the default package private level146 - 148
Use explicit scoping instead of the default package private level160 - 186
Found 'DU'-anomaly for variable 'repo' (lines '160'-'186').160 - 186
A method should have only one exit point, and that should be the last statement in the method163
Found 'DD'-anomaly for variable 'fin' (lines '165'-'168').165 - 168
Found 'DU'-anomaly for variable 'number' (lines '179'-'186').179 - 186
Found 'DD'-anomaly for variable 'number' (lines '179'-'179').179

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

ViolationLine
Found 'DU'-anomaly for variable 'wrt' (lines '53'-'53').53
Found 'DU'-anomaly for variable 'wrt' (lines '60'-'60').60
Found 'DU'-anomaly for variable 'wrt' (lines '67'-'67').67
Found 'DU'-anomaly for variable 'wrt' (lines '74'-'74').74
Found 'DU'-anomaly for variable 'headerName' (lines '82'-'82').82

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

ViolationLine
Found 'DD'-anomaly for variable 'fin' (lines '76'-'78').76 - 78
Assigning an Object to null is a code smell. Consider refactoring.82
Found 'DU'-anomaly for variable 'out' (lines '93'-'113').93 - 113
Found 'DD'-anomaly for variable 'fileStream' (lines '94'-'96').94 - 96
Found 'DD'-anomaly for variable 'read' (lines '98'-'99').98 - 99
Avoid assignments in operands99
Found 'DU'-anomaly for variable 'read' (lines '99'-'113').99 - 113
Avoid empty catch blocks108 - 110
Found 'DU'-anomaly for variable 'out' (lines '118'-'140').118 - 140
Found 'DD'-anomaly for variable 'fileStream' (lines '119'-'121').119 - 121
Avoid empty catch blocks135 - 137
Found 'DU'-anomaly for variable 'out' (lines '145'-'166').145 - 166
Found 'DD'-anomaly for variable 'fileStream' (lines '146'-'148').146 - 148
Found 'DU'-anomaly for variable 'headers' (lines '149'-'166').149 - 166
Found 'DD'-anomaly for variable 'read' (lines '151'-'152').151 - 152
Avoid assignments in operands152
Found 'DU'-anomaly for variable 'read' (lines '152'-'166').152 - 166
Avoid empty catch blocks161 - 163
Found 'DD'-anomaly for variable 'fileStream' (lines '172'-'174').172 - 174
Found 'DD'-anomaly for variable 'lines' (lines '187'-'189').187 - 189
Found 'DD'-anomaly for variable 'lines' (lines '189'-'189').189
Avoid empty catch blocks213 - 215

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

ViolationLine
Found 'DU'-anomaly for variable 'number' (lines '80'-'80').80
Found 'DU'-anomaly for variable 'dt' (lines '91'-'91').91
Found 'DU'-anomaly for variable 'newsStream' (lines '124'-'124').124

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

ViolationLine
Perhaps 'articleRangeInfoCollected' could be replaced by a local variable.68
It is a good practice to call super() in a constructor75 - 77
Assigning an Object to null is a code smell. Consider refactoring.155
Found 'UR'-anomaly for variable 'artNum' (lines '214'-'228').214 - 228
Found 'DD'-anomaly for variable 'articleFile' (lines '216'-'228').216 - 228
Found 'DD'-anomaly for variable 'fout' (lines '236'-'238').236 - 238
Avoid empty catch blocks246 - 249

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

ViolationLine
Found 'DU'-anomaly for variable 'groupName' (lines '36'-'36').36
Found 'DU'-anomaly for variable 'id' (lines '45'-'45').45
Found 'DU'-anomaly for variable 'in' (lines '53'-'53').53
Found 'DU'-anomaly for variable 'wildmat' (lines '62'-'62').62
Found 'DU'-anomaly for variable 'dt' (lines '71'-'71').71
Found 'DU'-anomaly for variable 'dt' (lines '80'-'80').80

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

ViolationLine
Each class should declare at least one constructor50 - 483
Perhaps 'overviewFormat' could be replaced by a local variable.122 - 129
Perhaps 'repositoryGroups' could be replaced by a local variable.136
Found 'UR'-anomaly for variable 'errorBuffer' (lines '194'-'194').194
Found 'DD'-anomaly for variable 'articleIDPath' (lines '197'-'202').197 - 202
Found 'DU'-anomaly for variable 'errorBuffer' (lines '225'-'263').225 - 263
A method should have only one exit point, and that should be the last statement in the method280
Found 'DD'-anomaly for variable 'groupToReturn' (lines '283'-'285').283 - 285
Assigning an Object to null is a code smell. Consider refactoring.295
A method should have only one exit point, and that should be the last statement in the method307
Found 'DD'-anomaly for variable 'fout' (lines '324'-'326').324 - 326
Found 'DD'-anomaly for variable 'bytesRead' (lines '328'-'329').328 - 329
Avoid assignments in operands329
Found 'DU'-anomaly for variable 'bytesRead' (lines '329'-'350').329 - 350
Assigning an Object to null is a code smell. Consider refactoring.334
Avoid empty catch blocks345 - 347
Each class should declare at least one constructor352 - 360
Found 'DU'-anomaly for variable 'dir' (lines '353'-'359').353 - 359
Found 'DU'-anomaly for variable 'giter' (lines '406'-'437').406 - 437
Found 'DU'-anomaly for variable 'giter' (lines '406'-'437').406 - 437
Found 'DU'-anomaly for variable 'giter' (lines '406'-'437').406 - 437
Found 'DU'-anomaly for variable 'giter' (lines '406'-'437').406 - 437
Assigning an Object to null is a code smell. Consider refactoring.424
Found 'UR'-anomaly for variable 'errorString' (lines '451'-'451').451
Found 'DD'-anomaly for variable 'className' (lines '453'-'457').453 - 457
Avoid empty catch blocks458 - 460

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

ViolationLine
Each class should declare at least one constructor47 - 343
Found 'UR'-anomaly for variable 'errorBuffer' (lines '96'-'96').96
Found 'DU'-anomaly for variable 'errorBuffer' (lines '104'-'127').104 - 127
Use explicit scoping instead of the default package private level134 - 138
Found 'DU'-anomaly for variable 'repo' (lines '134'-'138').134 - 138
Use explicit scoping instead of the default package private level145 - 149
Found 'DU'-anomaly for variable 'articleIDRepo' (lines '145'-'149').145 - 149
Use explicit scoping instead of the default package private level157 - 159
It is a good practice to call super() in a constructor189 - 192
Use explicit scoping instead of the default package private level199 - 201
Use explicit scoping instead of the default package private level208 - 210
Assigning an Object to null is a code smell. Consider refactoring.235
Assigning an Object to null is a code smell. Consider refactoring.237
Found 'DD'-anomaly for variable 'list' (lines '237'-'220').237 - 220
Found 'DU'-anomaly for variable 'list' (lines '237'-'248').237 - 248
Found 'DD'-anomaly for variable 'lineCount' (lines '274'-'283').274 - 283
Found 'DU'-anomaly for variable 'lineCount' (lines '274'-'341').274 - 341
Found 'DD'-anomaly for variable 'lines' (lines '278'-'280').278 - 280
Found 'DD'-anomaly for variable 'lines' (lines '280'-'280').280
Assigning an Object to null is a code smell. Consider refactoring.291

org/apache/james/pop3server/POP3Handler.java

ViolationLine
Each class should declare at least one constructor58 - 1152
Perhaps 'theWatchdogTarget' could be replaced by a local variable.159
Use explicit scoping instead of the default package private level166 - 168
Use explicit scoping instead of the default package private level175 - 177
Use explicit scoping instead of the default package private level185 - 187
Use explicit scoping instead of the default package private level192 - 214
Avoid empty catch blocks200 - 202
Assigning an Object to null is a code smell. Consider refactoring.203
Assigning an Object to null is a code smell. Consider refactoring.210
Found 'DD'-anomaly for variable 'remoteHost' (lines '222'-'233').222 - 233
Found 'DD'-anomaly for variable 'remoteIP' (lines '223'-'232').223 - 232
Assigning an Object to null is a code smell. Consider refactoring.315
Assigning an Object to null is a code smell. Consider refactoring.323
Avoid empty catch blocks325 - 327
Assigning an Object to null is a code smell. Consider refactoring.328
Avoid empty catch blocks335 - 337
Assigning an Object to null is a code smell. Consider refactoring.338
Avoid empty catch blocks345 - 347
Assigning an Object to null is a code smell. Consider refactoring.348
Avoid empty catch blocks355 - 357
Assigning an Object to null is a code smell. Consider refactoring.358
Assigning an Object to null is a code smell. Consider refactoring.362
Assigning an Object to null is a code smell. Consider refactoring.366
Assigning an Object to null is a code smell. Consider refactoring.367
Assigning an Object to null is a code smell. Consider refactoring.370
Assigning an Object to null is a code smell. Consider refactoring.375
Assigning an Object to null is a code smell. Consider refactoring.379
Use explicit scoping instead of the default package private level420 - 430
A method should have only one exit point, and that should be the last statement in the method446
Found 'DU'-anomaly for variable 'returnValue' (lines '448'-'501').448 - 501
Found 'DD'-anomaly for variable 'returnValue' (lines '448'-'495').448 - 495
A method should have only one exit point, and that should be the last statement in the method453
Found 'DD'-anomaly for variable 'argument' (lines '465'-'467').465 - 467
Found 'DD'-anomaly for variable 'argument1' (lines '469'-'471').469 - 471
Found 'DU'-anomaly for variable 'command' (lines '511'-'521').511 - 521
Found 'DU'-anomaly for variable 'argument1' (lines '511'-'521').511 - 521
Found 'DD'-anomaly for variable 'responseString' (lines '512'-'516').512 - 516
Found 'DD'-anomaly for variable 'responseString' (lines '512'-'518').512 - 518
Found 'DU'-anomaly for variable 'command' (lines '531'-'555').531 - 555
Found 'DU'-anomaly for variable 'argument1' (lines '531'-'555').531 - 555
Found 'DD'-anomaly for variable 'responseString' (lines '532'-'541').532 - 541
Found 'DD'-anomaly for variable 'responseString' (lines '532'-'547').532 - 547
Found 'DD'-anomaly for variable 'responseString' (lines '532'-'552').532 - 552
Found 'DU'-anomaly for variable 'command' (lines '566'-'596').566 - 596
Found 'DU'-anomaly for variable 'argument' (lines '566'-'596').566 - 596
Found 'DU'-anomaly for variable 'argument1' (lines '566'-'596').566 - 596
Found 'DD'-anomaly for variable 'responseString' (lines '567'-'586').567 - 586
Found 'DD'-anomaly for variable 'responseString' (lines '567'-'593').567 - 593
Found 'DD'-anomaly for variable 'count' (lines '570'-'576').570 - 576
Found 'DD'-anomaly for variable 'count' (lines '576'-'576').576
Found 'DU'-anomaly for variable 'command' (lines '608'-'702').608 - 702
Found 'DU'-anomaly for variable 'argument1' (lines '608'-'702').608 - 702
Found 'DU'-anomaly for variable 'argument' (lines '608'-'702').608 - 702
Found 'DD'-anomaly for variable 'responseString' (lines '609'-'629').609 - 629
Found 'DD'-anomaly for variable 'responseString' (lines '609'-'663').609 - 663
Found 'DD'-anomaly for variable 'responseString' (lines '609'-'672').609 - 672
Found 'DD'-anomaly for variable 'responseString' (lines '609'-'699').609 - 699
Found 'DD'-anomaly for variable 'count' (lines '613'-'619').613 - 619
Found 'DD'-anomaly for variable 'count' (lines '619'-'619').619
Found 'DD'-anomaly for variable 'count' (lines '631'-'632').631 - 632
Found 'DU'-anomaly for variable 'count' (lines '631'-'702').631 - 702
Found 'DU'-anomaly for variable 'count' (lines '632'-'702').632 - 702
Found 'DD'-anomaly for variable 'count' (lines '632'-'632').632
Found 'DD'-anomaly for variable 'num' (lines '651'-'653').651 - 653
Found 'DU'-anomaly for variable 'command' (lines '712'-'780').712 - 780
Found 'DU'-anomaly for variable 'argument1' (lines '712'-'780').712 - 780
Found 'DU'-anomaly for variable 'argument' (lines '712'-'780').712 - 780
Found 'DD'-anomaly for variable 'responseString' (lines '713'-'716').713 - 716
Found 'DD'-anomaly for variable 'responseString' (lines '713'-'745').713 - 745
Found 'DD'-anomaly for variable 'responseString' (lines '713'-'754').713 - 754
Found 'DU'-anomaly for variable 'responseString' (lines '713'-'780').713 - 780
Found 'DD'-anomaly for variable 'count' (lines '718'-'719').718 - 719
Found 'DU'-anomaly for variable 'count' (lines '718'-'780').718 - 780
Found 'DU'-anomaly for variable 'count' (lines '719'-'780').719 - 780
Found 'DD'-anomaly for variable 'count' (lines '719'-'719').719
Found 'DD'-anomaly for variable 'num' (lines '733'-'735').733 - 735
Found 'DU'-anomaly for variable 'command' (lines '790'-'799').790 - 799
Found 'DU'-anomaly for variable 'argument' (lines '790'-'799').790 - 799
Found 'DU'-anomaly for variable 'argument1' (lines '790'-'799').790 - 799
Found 'DD'-anomaly for variable 'responseString' (lines '791'-'794').791 - 794
Found 'DD'-anomaly for variable 'responseString' (lines '791'-'796').791 - 796
Found 'DU'-anomaly for variable 'command' (lines '810'-'850').810 - 850
Found 'DU'-anomaly for variable 'argument1' (lines '810'-'850').810 - 850
Found 'DU'-anomaly for variable 'argument' (lines '810'-'850').810 - 850
Found 'DD'-anomaly for variable 'responseString' (lines '811'-'817').811 - 817
Found 'DD'-anomaly for variable 'responseString' (lines '811'-'847').811 - 847
Found 'DD'-anomaly for variable 'num' (lines '813'-'815').813 - 815
Found 'DU'-anomaly for variable 'num' (lines '815'-'850').815 - 850
Found 'DU'-anomaly for variable 'command' (lines '860'-'869').860 - 869
Found 'DU'-anomaly for variable 'argument' (lines '860'-'869').860 - 869
Found 'DU'-anomaly for variable 'argument1' (lines '860'-'869').860 - 869
Found 'DD'-anomaly for variable 'responseString' (lines '861'-'863').861 - 863
Found 'DD'-anomaly for variable 'responseString' (lines '861'-'866').861 - 866
Found 'DU'-anomaly for variable 'command' (lines '880'-'940').880 - 940
Found 'DU'-anomaly for variable 'argument1' (lines '880'-'940').880 - 940
Found 'DU'-anomaly for variable 'argument' (lines '880'-'940').880 - 940
Found 'DD'-anomaly for variable 'responseString' (lines '881'-'887').881 - 887
Found 'DD'-anomaly for variable 'responseString' (lines '881'-'937').881 - 937
Found 'DD'-anomaly for variable 'num' (lines '883'-'885').883 - 885
Found 'DU'-anomaly for variable 'num' (lines '885'-'940').885 - 940
Found 'DU'-anomaly for variable 'command' (lines '954'-'1,020').954 - 1020
Found 'DU'-anomaly for variable 'argument' (lines '954'-'1,020').954 - 1020
Found 'DU'-anomaly for variable 'argument1' (lines '954'-'1,020').954 - 1020
Found 'DD'-anomaly for variable 'responseString' (lines '955'-'963').955 - 963
Found 'DD'-anomaly for variable 'responseString' (lines '955'-'1,017').955 - 1017
Found 'DD'-anomaly for variable 'num' (lines '957'-'960').957 - 960
Found 'DD'-anomaly for variable 'lines' (lines '958'-'961').958 - 961
Found 'DU'-anomaly for variable 'num' (lines '960'-'1,020').960 - 1020
Found 'DU'-anomaly for variable 'lines' (lines '961'-'1,020').961 - 1020
Found 'DU'-anomaly for variable 'out' (lines '1,032'-'1,052').1032 - 1052
Found 'DU'-anomaly for variable 'lines' (lines '1,032'-'1,052').1032 - 1052
Avoid assignments in operands1040
Found 'DU'-anomaly for variable 'line' (lines '1,040'-'1,052').1040 - 1052
Found 'DU'-anomaly for variable 'command' (lines '1,062'-'1,083').1062 - 1083
Found 'DU'-anomaly for variable 'argument' (lines '1,062'-'1,083').1062 - 1083
Found 'DU'-anomaly for variable 'argument1' (lines '1,062'-'1,083').1062 - 1083
Found 'DD'-anomaly for variable 'responseString' (lines '1,063'-'1,065').1063 - 1065
Found 'DD'-anomaly for variable 'responseString' (lines '1,063'-'1,076').1063 - 1076
Found 'DU'-anomaly for variable 'command' (lines '1,093'-'1,095').1093 - 1095
Found 'DU'-anomaly for variable 'argument' (lines '1,093'-'1,095').1093 - 1095
Found 'DU'-anomaly for variable 'argument1' (lines '1,093'-'1,095').1093 - 1095
Found 'DU'-anomaly for variable 'responseString' (lines '1,105'-'1,109').1105 - 1109
Use explicit scoping instead of the default package private level1118 - 1122
Use explicit scoping instead of the default package private level1130 - 1133
Each class should declare at least one constructor1140 - 1150

org/apache/james/pop3server/POP3Server.java

ViolationLine
Each class should declare at least one constructor46 - 240
Perhaps 'theHandlerFactory' could be replaced by a local variable.73
Each class should declare at least one constructor181 - 204
Found 'DU'-anomaly for variable 'object' (lines '201'-'203').201 - 203
Avoid unnecessary return statements202
Each class should declare at least one constructor209 - 239

org/apache/james/pop3server/POP3ServerMBean.java

ViolationLine
Avoid modifiers which are implied by the context34
Avoid modifiers which are implied by the context43
Avoid modifiers which are implied by the context52
Avoid modifiers which are implied by the context61

org/apache/james/remotemanager/RemoteManager.java

ViolationLine
Each class should declare at least one constructor50 - 281
Perhaps 'theHandlerFactory' could be replaced by a local variable.86
Each class should declare at least one constructor207 - 230
Found 'DU'-anomaly for variable 'object' (lines '227'-'229').227 - 229
Avoid unnecessary return statements228
Each class should declare at least one constructor235 - 280

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 context34
Avoid modifiers which are implied by the context43
Avoid modifiers which are implied by the context52
Avoid modifiers which are implied by the context61

org/apache/james/security/DigestUtil.java

ViolationLine
Found 'DU'-anomaly for variable 'alg' (lines '48'-'83').48 - 83
Found 'DD'-anomaly for variable 'alg' (lines '48'-'64').48 - 64
Found 'DU'-anomaly for variable 'file' (lines '49'-'83').49 - 83
Found 'DD'-anomaly for variable 'file' (lines '49'-'67').49 - 67
A method should have only one exit point, and that should be the last statement in the method53
A method should have only one exit point, and that should be the last statement in the method61
Found 'DU'-anomaly for variable 'alg' (lines '64'-'83').64 - 83
Found 'DD'-anomaly for variable 'alg' (lines '64'-'64').64
Found 'DU'-anomaly for variable 'file' (lines '67'-'83').67 - 83
Found 'DD'-anomaly for variable 'file' (lines '67'-'67').67
A method should have only one exit point, and that should be the last statement in the method73
Found 'DU'-anomaly for variable 'b' (lines '103'-'134').103 - 134
Found 'DU'-anomaly for variable 'count' (lines '104'-'134').104 - 134
Found 'DD'-anomaly for variable 'read' (lines '105'-'112').105 - 112
Found 'DU'-anomaly for variable 'read' (lines '105'-'134').105 - 134
Found 'DD'-anomaly for variable 'fis' (lines '106'-'110').106 - 110
Found 'DD'-anomaly for variable 'fos' (lines '107'-'122').107 - 122
Avoid empty catch blocks132

org/apache/james/security/InitJCE.java

ViolationLine
Each class should declare at least one constructor38 - 66

org/apache/james/security/KeyHolder.java

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

org/apache/james/security/KeyStoreHolder.java

ViolationLine
Found 'DU'-anomaly for variable 'certs' (lines '115'-'152').115 - 152
Found 'DD'-anomaly for variable 'i' (lines '123'-'123').123
Found 'DU'-anomaly for variable 'i' (lines '123'-'152').123 - 152
A method should have only one exit point, and that should be the last statement in the method194

org/apache/james/security/SMIMEAttributeNames.java

ViolationLine
Avoid modifiers which are implied by the context31
Avoid modifiers which are implied by the context38
Avoid modifiers which are implied by the context43
Avoid modifiers which are implied by the context48

org/apache/james/services/DNSServer.java

ViolationLine
Found 'DU'-anomaly for variable 'hostname' (lines '44'-'44').44
Found 'DU'-anomaly for variable 'domainName' (lines '58'-'58').58

org/apache/james/services/JamesConnectionManager.java

ViolationLine
Found 'DU'-anomaly for variable 'threadPool' (lines '58'-'63').58 - 63
Found 'DU'-anomaly for variable 'maxOpenConnections' (lines '58'-'63').58 - 63
Found 'DU'-anomaly for variable 'handlerFactory' (lines '58'-'63').58 - 63
Found 'DU'-anomaly for variable 'socket' (lines '58'-'63').58 - 63
Found 'DU'-anomaly for variable 'name' (lines '58'-'63').58 - 63
Found 'DU'-anomaly for variable 'maxOpenConnections' (lines '75'-'79').75 - 79
Found 'DU'-anomaly for variable 'handlerFactory' (lines '75'-'79').75 - 79
Found 'DU'-anomaly for variable 'socket' (lines '75'-'79').75 - 79
Found 'DU'-anomaly for variable 'name' (lines '75'-'79').75 - 79
Found 'DU'-anomaly for variable 'threadPool' (lines '92'-'96').92 - 96
Found 'DU'-anomaly for variable 'handlerFactory' (lines '92'-'96').92 - 96
Found 'DU'-anomaly for variable 'socket' (lines '92'-'96').92 - 96
Found 'DU'-anomaly for variable 'name' (lines '92'-'96').92 - 96
Found 'DU'-anomaly for variable 'handlerFactory' (lines '108'-'111').108 - 111
Found 'DU'-anomaly for variable 'socket' (lines '108'-'111').108 - 111
Found 'DU'-anomaly for variable 'name' (lines '108'-'111').108 - 111

org/apache/james/services/JamesUser.java

ViolationLine
Found 'DU'-anomaly for variable 'pass' (lines '37'-'37').37
Found 'DU'-anomaly for variable 'forward' (lines '45'-'45').45
Found 'DU'-anomaly for variable 'address' (lines '58'-'58').58
Found 'DU'-anomaly for variable 'alias' (lines '69'-'69').69
Found 'DU'-anomaly for variable 'address' (lines '81'-'81').81

org/apache/james/services/MailetLoader.java

ViolationLine
Avoid modifiers which are implied by the context41 - 42
Found 'DU'-anomaly for variable 'mailetName' (lines '41'-'42').41 - 42
Found 'DU'-anomaly for variable 'configuration' (lines '41'-'42').41 - 42

org/apache/james/services/MailRepository.java

ViolationLine
Found 'DU'-anomaly for variable 'mc' (lines '51'-'51').51
Found 'DU'-anomaly for variable 'key' (lines '68'-'68').68
Found 'DU'-anomaly for variable 'mail' (lines '75'-'75').75
Found 'DU'-anomaly for variable 'mails' (lines '83'-'83').83
Found 'DU'-anomaly for variable 'key' (lines '90'-'90').90
Found 'DU'-anomaly for variable 'key' (lines '99'-'99').99
Found 'DU'-anomaly for variable 'key' (lines '108'-'108').108

org/apache/james/services/MailServer.java

ViolationLine
Found 'DU'-anomaly for variable 'sender' (lines '62'-'63').62 - 63
Found 'DU'-anomaly for variable 'recipients' (lines '62'-'63').62 - 63
Found 'DU'-anomaly for variable 'msg' (lines '62'-'63').62 - 63
Found 'DU'-anomaly for variable 'sender' (lines '76'-'77').76 - 77
Found 'DU'-anomaly for variable 'recipients' (lines '76'-'77').76 - 77
Found 'DU'-anomaly for variable 'msg' (lines '76'-'77').76 - 77
Found 'DU'-anomaly for variable 'mail' (lines '84'-'85').84 - 85
Found 'DU'-anomaly for variable 'message' (lines '94'-'95').94 - 95
Found 'DU'-anomaly for variable 'userName' (lines '104'-'104').104
Found 'DU'-anomaly for variable 'password' (lines '124'-'124').124
Found 'DU'-anomaly for variable 'userName' (lines '124'-'124').124
Found 'DU'-anomaly for variable 'serverName' (lines '132'-'132').132

org/apache/james/services/MatcherLoader.java

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

org/apache/james/services/SpoolRepository.java

ViolationLine
Avoid modifiers which are implied by the context41 - 62
Found 'DU'-anomaly for variable 'key' (lines '53'-'53').53
Found 'DU'-anomaly for variable 'lastUpdated' (lines '53'-'53').53
Found 'DU'-anomaly for variable 'state' (lines '53'-'53').53
Found 'DU'-anomaly for variable 'errorMessage' (lines '53'-'53').53
Found 'DU'-anomaly for variable 'delay' (lines '88'-'88').88
Found 'DU'-anomaly for variable 'filter' (lines '100'-'100').100

org/apache/james/services/User.java

ViolationLine
Found 'DU'-anomaly for variable 'pass' (lines '42'-'42').42
Found 'DU'-anomaly for variable 'newPass' (lines '51'-'51').51

org/apache/james/services/UsersRepository.java

ViolationLine
Found 'DU'-anomaly for variable 'user' (lines '47'-'47').47
Found 'DU'-anomaly for variable 'attributes' (lines '56'-'56').56
Found 'DU'-anomaly for variable 'name' (lines '56'-'56').56
Found 'DU'-anomaly for variable 'password' (lines '67'-'67').67
Found 'DU'-anomaly for variable 'username' (lines '67'-'67').67
Found 'DU'-anomaly for variable 'name' (lines '78'-'78').78
Found 'DU'-anomaly for variable 'name' (lines '89'-'89').89
Found 'DU'-anomaly for variable 'name' (lines '98'-'98').98
Found 'DU'-anomaly for variable 'user' (lines '106'-'106').106
Found 'DU'-anomaly for variable 'name' (lines '113'-'113').113
Found 'DU'-anomaly for variable 'name' (lines '121'-'121').121
Found 'DU'-anomaly for variable 'name' (lines '130'-'130').130
Found 'DU'-anomaly for variable 'password' (lines '143'-'143').143
Found 'DU'-anomaly for variable 'name' (lines '143'-'143').143

org/apache/james/services/UsersStore.java

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

org/apache/james/smtpserver/AddHeaderHandler.java

ViolationLine
Each class should declare at least one constructor26 - 28

org/apache/james/smtpserver/AuthCmdHandler.java

ViolationLine
Each class should declare at least one constructor31 - 272
Found 'DU'-anomaly for variable 'argument' (lines '71'-'99').71 - 99
Found 'DD'-anomaly for variable 'responseString' (lines '73'-'75').73 - 75
Found 'DD'-anomaly for variable 'responseString' (lines '73'-'79').73 - 79
Found 'DU'-anomaly for variable 'responseString' (lines '73'-'99').73 - 99
Found 'DD'-anomaly for variable 'initialResponse' (lines '82'-'84').82 - 84
A method should have only one exit point, and that should be the last statement in the method90
A method should have only one exit point, and that should be the last statement in the method93
Found 'DD'-anomaly for variable 'responseString' (lines '117'-'119').117 - 119
Found 'DD'-anomaly for variable 'userpass' (lines '117'-'121').117 - 121
Found 'DD'-anomaly for variable 'user' (lines '117'-'146').117 - 146
Found 'DD'-anomaly for variable 'pass' (lines '117'-'148').117 - 148
Found 'DD'-anomaly for variable 'userpass' (lines '117'-'123').117 - 123
Found 'DD'-anomaly for variable 'responseString' (lines '117'-'179').117 - 179
Found 'DD'-anomaly for variable 'responseString' (lines '117'-'183').117 - 183
Found 'DD'-anomaly for variable 'responseString' (lines '117'-'187').117 - 187
Found 'DD'-anomaly for variable 'pass' (lines '148'-'166').148 - 166
Assigning an Object to null is a code smell. Consider refactoring.170
Found 'DU'-anomaly for variable 'authTokenizer' (lines '170'-'192').170 - 192
Avoid empty catch blocks173 - 176
Avoid unnecessary return statements191
Found 'DD'-anomaly for variable 'responseString' (lines '202'-'204').202 - 204
Found 'DD'-anomaly for variable 'user' (lines '202'-'206').202 - 206
Found 'DD'-anomaly for variable 'pass' (lines '202'-'221').202 - 221
Found 'DD'-anomaly for variable 'user' (lines '202'-'208').202 - 208
Found 'DD'-anomaly for variable 'responseString' (lines '202'-'219').202 - 219
Assigning an Object to null is a code smell. Consider refactoring.216
Assigning an Object to null is a code smell. Consider refactoring.228
Avoid unnecessary return statements247
Found 'DU'-anomaly for variable 'initialResponse' (lines '257'-'269').257 - 269
Found 'DU'-anomaly for variable 'authType' (lines '257'-'269').257 - 269
Avoid unnecessary return statements268

org/apache/james/smtpserver/CommandHandler.java

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

org/apache/james/smtpserver/ConnectHandler.java

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

org/apache/james/smtpserver/DataCmdHandler.java

ViolationLine
Each class should declare at least one constructor49 - 315
Found 'DD'-anomaly for variable 'responseString' (lines '92'-'94').92 - 94
Found 'DD'-anomaly for variable 'responseString' (lines '92'-'98').92 - 98
Found 'DD'-anomaly for variable 'responseString' (lines '92'-'101').92 - 101
Found 'DD'-anomaly for variable 'responseString' (lines '92'-'104').92 - 104
Found 'DD'-anomaly for variable 'msgIn' (lines '106'-'108').106 - 108
Found 'DD'-anomaly for variable 'headers' (lines '129'-'130').129 - 130
Assigning an Object to null is a code smell. Consider refactoring.132
Found 'DU'-anomaly for variable 'headers' (lines '132'-'177').132 - 177
Avoid empty catch blocks170 - 172
Assigning an Object to null is a code smell. Consider refactoring.173
Assigning an Object to null is a code smell. Consider refactoring.238
Found 'DU'-anomaly for variable 'headerLineBuffer' (lines '238'-'246').238 - 246
Found 'DD'-anomaly for variable 'headersIn' (lines '258'-'262').258 - 262
Found 'DD'-anomaly for variable 'mail' (lines '259'-'264').259 - 264
Found 'DD'-anomaly for variable 'recipientCollection' (lines '260'-'263').260 - 263
Assigning an Object to null is a code smell. Consider refactoring.302
Found 'DU'-anomaly for variable 'recipientCollection' (lines '302'-'313').302 - 313
Avoid empty catch blocks306 - 308
Assigning an Object to null is a code smell. Consider refactoring.310
Found 'DU'-anomaly for variable 'headersIn' (lines '310'-'313').310 - 313

org/apache/james/smtpserver/DNSRBLHandler.java

ViolationLine
Each class should declare at least one constructor30 - 177
Found 'DU'-anomaly for variable 'rblserverCollection' (lines '46'-'77').46 - 77
Found 'DU'-anomaly for variable 'ipAddress' (lines '100'-'175').100 - 175
A method should have only one exit point, and that should be the last statement in the method108
Found 'DU'-anomaly for variable 'reversedOctets' (lines '117'-'175').117 - 175
Found 'DU'-anomaly for variable 'addr' (lines '122'-'175').122 - 175
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 method165

org/apache/james/smtpserver/EhloCmdHandler.java

ViolationLine
Each class should declare at least one constructor32 - 154
Found 'DD'-anomaly for variable 'responseString' (lines '79'-'97').79 - 97
Found 'DD'-anomaly for variable 'responseString' (lines '79'-'105').79 - 105
Found 'DU'-anomaly for variable 'responseString' (lines '79'-'152').79 - 152
Found 'DU'-anomaly for variable 'responseBuffer' (lines '80'-'152').80 - 152
Found 'DD'-anomaly for variable 'badEhlo' (lines '81'-'96').81 - 96
Found 'DU'-anomaly for variable 'badEhlo' (lines '81'-'152').81 - 152
These nested if statements could be combined89 - 101
Found 'DU'-anomaly for variable 'badEhlo' (lines '96'-'152').96 - 152

org/apache/james/smtpserver/ExpnCmdHandler.java

ViolationLine
Each class should declare at least one constructor25 - 44

org/apache/james/smtpserver/HeloCmdHandler.java

ViolationLine
Each class should declare at least one constructor31 - 124
Found 'DU'-anomaly for variable 'session' (lines '78'-'123').78 - 123
Found 'DD'-anomaly for variable 'responseString' (lines '79'-'96').79 - 96
Found 'DD'-anomaly for variable 'responseString' (lines '79'-'105').79 - 105
Found 'DD'-anomaly for variable 'responseString' (lines '79'-'120').79 - 120
Found 'DU'-anomaly for variable 'responseString' (lines '79'-'123').79 - 123
Found 'DD'-anomaly for variable 'badHelo' (lines '80'-'95').80 - 95
Found 'DU'-anomaly for variable 'badHelo' (lines '80'-'123').80 - 123
These nested if statements could be combined89 - 101
Found 'DU'-anomaly for variable 'badHelo' (lines '95'-'123').95 - 123

org/apache/james/smtpserver/HelpCmdHandler.java

ViolationLine
Each class should declare at least one constructor25 - 43

org/apache/james/smtpserver/MailCmdHandler.java

ViolationLine
Each class should declare at least one constructor38 - 292
Found 'DD'-anomaly for variable 'responseString' (lines '95'-'106').95 - 106
Found 'DD'-anomaly for variable 'responseString' (lines '95'-'109').95 - 109
Found 'DD'-anomaly for variable 'responseString' (lines '95'-'113').95 - 113
Found 'DU'-anomaly for variable 'responseString' (lines '95'-'234').95 - 234
Found 'DD'-anomaly for variable 'responseString' (lines '95'-'159').95 - 159
Found 'DD'-anomaly for variable 'responseString' (lines '95'-'185').95 - 185
Found 'DD'-anomaly for variable 'responseString' (lines '95'-'218').95 - 218
Found 'DD'-anomaly for variable 'responseString' (lines '95'-'230').95 - 230
Found 'DU'-anomaly for variable 'responseBuffer' (lines '96'-'234').96 - 234
Found 'DD'-anomaly for variable 'sender' (lines '97'-'102').97 - 102
Found 'DU'-anomaly for variable 'sender' (lines '97'-'234').97 - 234
Found 'DU'-anomaly for variable 'badSenderDomain' (lines '98'-'234').98 - 234
Found 'DD'-anomaly for variable 'badSenderDomain' (lines '98'-'213').98 - 213
Found 'DU'-anomaly for variable 'sender' (lines '102'-'234').102 - 234
Found 'DD'-anomaly for variable 'mailOptionName' (lines '131'-'134').131 - 134
Found 'DD'-anomaly for variable 'mailOptionValue' (lines '132'-'135').132 - 135
Found 'DU'-anomaly for variable 'mailOptionValue' (lines '132'-'234').132 - 234
Found 'DD'-anomaly for variable 'mailOptionValue' (lines '132'-'132').132
Found 'DU'-anomaly for variable 'mailOptionValue' (lines '135'-'234').135 - 234
Found 'DD'-anomaly for variable 'mailOptionValue' (lines '135'-'132').135 - 132
A method should have only one exit point, and that should be the last statement in the method142
A method should have only one exit point, and that should be the last statement in the method169
Found 'DD'-anomaly for variable 'senderAddress' (lines '171'-'183').171 - 183
Avoid empty if statements174 - 176
Found 'DU'-anomaly for variable 'senderAddress' (lines '183'-'234').183 - 234
These nested if statements could be combined206 - 222
Found 'DU'-anomaly for variable 'tempSender' (lines '244'-'289').244 - 289
Found 'DD'-anomaly for variable 'size' (lines '245'-'247').245 - 247
Found 'DU'-anomaly for variable 'size' (lines '247'-'289').247 - 289
A method should have only one exit point, and that should be the last statement in the method253
A method should have only one exit point, and that should be the last statement in the method282

org/apache/james/smtpserver/MessageHandler.java

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

org/apache/james/smtpserver/NoopCmdHandler.java

ViolationLine
Each class should declare at least one constructor25 - 44

org/apache/james/smtpserver/QuitCmdHandler.java

ViolationLine
Each class should declare at least one constructor25 - 67
Found 'DD'-anomaly for variable 'responseString' (lines '53'-'58').53 - 58
Found 'DD'-anomaly for variable 'responseString' (lines '53'-'60').53 - 60

org/apache/james/smtpserver/RcptCmdHandler.java

ViolationLine
Each class should declare at least one constructor34 - 335
Found 'DD'-anomaly for variable 'responseString' (lines '85'-'97').85 - 97
Found 'DD'-anomaly for variable 'responseString' (lines '85'-'101').85 - 101
Found 'DD'-anomaly for variable 'responseString' (lines '85'-'120').85 - 120
Found 'DD'-anomaly for variable 'responseString' (lines '85'-'147').85 - 147
Found 'DU'-anomaly for variable 'responseBuffer' (lines '86'-'302').86 - 302
Found 'DU'-anomaly for variable 'maxRcptReached' (lines '87'-'302').87 - 302
Found 'DU'-anomaly for variable 'useTarpit' (lines '88'-'302').88 - 302
Found 'DD'-anomaly for variable 'recipient' (lines '90'-'93').90 - 93
Found 'DU'-anomaly for variable 'recipient' (lines '90'-'302').90 - 302
Found 'DU'-anomaly for variable 'recipient' (lines '93'-'302').93 - 302
Found 'DU'-anomaly for variable 'rcptColl' (lines '106'-'302').106 - 302
Found 'DD'-anomaly for variable 'rcptOptionString' (lines '112'-'114').112 - 114
Found 'DU'-anomaly for variable 'rcptOptionString' (lines '112'-'302').112 - 302
Found 'DU'-anomaly for variable 'rcptOptionString' (lines '114'-'302').114 - 302
A method should have only one exit point, and that should be the last statement in the method130
Found 'DD'-anomaly for variable 'recipientAddress' (lines '132'-'140').132 - 140
Found 'DU'-anomaly for variable 'recipientAddress' (lines '140'-'302').140 - 302
A method should have only one exit point, and that should be the last statement in the method158
A method should have only one exit point, and that should be the last statement in the method167
A method should have only one exit point, and that should be the last statement in the method182
A method should have only one exit point, and that should be the last statement in the method204
Assigning an Object to null is a code smell. Consider refactoring.247
Found 'DU'-anomaly for variable 'recipient' (lines '305'-'316').305 - 316
Found 'DU'-anomaly for variable 'startCount' (lines '320'-'329').320 - 329
A method should have only one exit point, and that should be the last statement in the method325

org/apache/james/smtpserver/RsetCmdHandler.java

ViolationLine
Each class should declare at least one constructor25 - 67
Found 'DD'-anomaly for variable 'responseString' (lines '49'-'58').49 - 58
Found 'DD'-anomaly for variable 'responseString' (lines '49'-'60').49 - 60

org/apache/james/smtpserver/SendMailHandler.java

ViolationLine
Each class should declare at least one constructor36 - 117
Found 'DD'-anomaly for variable 'responseString' (lines '58'-'92').58 - 92
Found 'DD'-anomaly for variable 'responseString' (lines '58'-'105').58 - 105
Found 'DD'-anomaly for variable 'recipientString' (lines '62'-'64').62 - 64
Found 'DU'-anomaly for variable 'recipientString' (lines '62'-'115').62 - 115
Found 'DU'-anomaly for variable 'recipientString' (lines '64'-'115').64 - 115

org/apache/james/smtpserver/SetMimeHeaderHandler.java

ViolationLine
Each class should declare at least one constructor30 - 95
Found 'DU'-anomaly for variable 'message' (lines '80'-'91').80 - 91

org/apache/james/smtpserver/SizeLimitedInputStream.java

ViolationLine
It is a good practice to call super() in a constructor48 - 51

org/apache/james/smtpserver/SMTPHandler.java

ViolationLine
Each class should declare at least one constructor55 - 789
Use explicit scoping instead of the default package private level86
Use explicit scoping instead of the default package private level91
Use explicit scoping instead of the default package private level96
Perhaps 'theWatchdogTarget' could be replaced by a local variable.202
Use explicit scoping instead of the default package private level214 - 216
Use explicit scoping instead of the default package private level223 - 225
Use explicit scoping instead of the default package private level233 - 235
Use explicit scoping instead of the default package private level240 - 258
Avoid empty catch blocks248 - 250
Found 'UR'-anomaly for variable 'exceptionString' (lines '263'-'263').263
Assigning an Object to null is a code smell. Consider refactoring.365
Assigning an Object to null is a code smell. Consider refactoring.366
Assigning an Object to null is a code smell. Consider refactoring.424
Assigning an Object to null is a code smell. Consider refactoring.489
Assigning an Object to null is a code smell. Consider refactoring.490
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.499
Assigning an Object to null is a code smell. Consider refactoring.512
Assigning an Object to null is a code smell. Consider refactoring.516
Found 'DU'-anomaly for variable 'responseString' (lines '530'-'534').530 - 534
Use explicit scoping instead of the default package private level543 - 547
Use explicit scoping instead of the default package private level555 - 558
Each class should declare at least one constructor566 - 575

org/apache/james/smtpserver/SMTPHandlerChain.java

ViolationLine
Each class should declare at least one constructor42 - 251
Perhaps 'unknownHandler' could be replaced by a local variable.48
Found 'UR'-anomaly for variable 'className' (lines '62'-'62').62
Found 'DD'-anomaly for variable 'commandName' (lines '122'-'123').122 - 123
Found 'DD'-anomaly for variable 'found' (lines '172'-'178').172 - 178
Use explicit scoping instead of the default package private level218 - 231
A method should have only one exit point, and that should be the last statement in the method220
Use explicit scoping instead of the default package private level238 - 240
Use explicit scoping instead of the default package private level247 - 249

org/apache/james/smtpserver/SMTPHandlerConfigurationData.java

ViolationLine
Found 'DU'-anomaly for variable 'remoteIP' (lines '57'-'57').57
Found 'DU'-anomaly for variable 'remoteIP' (lines '66'-'66').66

org/apache/james/smtpserver/SMTPServer.java

ViolationLine
Each class should declare at least one constructor53 - 443
Use explicit scoping instead of the default package private level60
Use explicit scoping instead of the default package private level65
Perhaps 'theHandlerFactory' could be replaced by a local variable.127
Each class should declare at least one constructor332 - 355
Found 'DU'-anomaly for variable 'object' (lines '352'-'354').352 - 354
Avoid unnecessary return statements353
Each class should declare at least one constructor360 - 442
Found 'DU'-anomaly for variable 'remoteIP' (lines '387'-'393').387 - 393
Found 'DD'-anomaly for variable 'relayingAllowed' (lines '388'-'390').388 - 390
Found 'DU'-anomaly for variable 'remoteIP' (lines '398'-'405').398 - 405
A method should have only one exit point, and that should be the last statement in the method399
Found 'DD'-anomaly for variable 'authRequired' (lines '400'-'402').400 - 402

org/apache/james/smtpserver/SMTPServerMBean.java

ViolationLine
Avoid modifiers which are implied by the context33
Avoid modifiers which are implied by the context42
Avoid modifiers which are implied by the context51
Avoid modifiers which are implied by the context60

org/apache/james/smtpserver/SMTPSession.java

ViolationLine
Avoid modifiers which are implied by the context36
Avoid modifiers which are implied by the context37
Avoid modifiers which are implied by the context38
Avoid modifiers which are implied by the context39
Found 'DU'-anomaly for variable 'respString' (lines '46'-'46').46
Found 'DU'-anomaly for variable 'mail' (lines '105'-'105').105
Found 'DU'-anomaly for variable 'blocklisted' (lines '165'-'165').165
Found 'DU'-anomaly for variable 'user' (lines '207'-'207').207

org/apache/james/smtpserver/UnknownCmdHandler.java

ViolationLine
Each class should declare at least one constructor25 - 54

org/apache/james/smtpserver/VrfyCmdHandler.java

ViolationLine
Each class should declare at least one constructor25 - 42

org/apache/james/transport/JamesMailetLoader.java

ViolationLine
Each class should declare at least one constructor30 - 74
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '41'-'41').41
Found 'DU'-anomaly for variable 'configuration' (lines '41'-'73').41 - 73
Avoid empty catch blocks54 - 56
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '58'-'73').58 - 73

org/apache/james/transport/JamesMatcherLoader.java

ViolationLine
Each class should declare at least one constructor31 - 80
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '42'-'42').42
Found 'DD'-anomaly for variable 'condition' (lines '44'-'47').44 - 47
Found 'DU'-anomaly for variable 'condition' (lines '44'-'79').44 - 79
Found 'DU'-anomaly for variable 'condition' (lines '47'-'79').47 - 79
Avoid empty catch blocks60 - 62
Found 'DD'-anomaly for variable 'exceptionBuffer' (lines '64'-'74').64 - 74
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '74'-'79').74 - 79

org/apache/james/transport/JamesSpoolManager.java

ViolationLine
Each class should declare at least one constructor53 - 489
Found 'DD'-anomaly for variable 'mailet' (lines '168'-'201').168 - 201
Found 'DD'-anomaly for variable 'matcher' (lines '169'-'171').169 - 171
Found 'DD'-anomaly for variable 'key' (lines '295'-'298').295 - 298
Assigning an Object to null is a code smell. Consider refactoring.334
Found 'DD'-anomaly for variable 'mail' (lines '334'-'297').334 - 297
Found 'DU'-anomaly for variable 'mail' (lines '334'-'363').334 - 363
Found 'UR'-anomaly for variable 'exceptionMessage' (lines '371'-'371').371
Found 'DU'-anomaly for variable 'mail' (lines '371'-'449').371 - 449
A method should have only one exit point, and that should be the last statement in the method376
Found 'DD'-anomaly for variable 'logMessageBuffer' (lines '393'-'395').393 - 395
Found 'DD'-anomaly for variable 'logMessageBuffer' (lines '393'-'405').393 - 405
Found 'DU'-anomaly for variable 'logMessageBuffer' (lines '393'-'449').393 - 449
Avoid empty catch blocks473

org/apache/james/transport/LinearProcessor.java

ViolationLine
Each class should declare at least one constructor80 - 546
Found 'DU'-anomaly for variable 'debugEnabled' (lines '143'-'151').143 - 151
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '202'-'239').202 - 239
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '202'-'239').202 - 239
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '202'-'239').202 - 239
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '202'-'239').202 - 239
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '202'-'239').202 - 239
Found 'DU'-anomaly for variable 'terminatingMatcher' (lines '202'-'239').202 - 239
Found 'DU'-anomaly for variable 'originalState' (lines '296'-'469').296 - 469
Found 'DU'-anomaly for variable 'originalMail' (lines '301'-'469').301 - 469
Assigning an Object to null is a code smell. Consider refactoring.304
Found 'DD'-anomaly for variable 'mail' (lines '304'-'323').304 - 323
Found 'DD'-anomaly for variable 'i' (lines '305'-'326').305 - 326
Assigning an Object to null is a code smell. Consider refactoring.323
Found 'DD'-anomaly for variable 'mail' (lines '323'-'329').323 - 329
Found 'DD'-anomaly for variable 'recipients' (lines '342'-'355').342 - 355
Found 'DD'-anomaly for variable 'logMessageBuffer' (lines '344'-'346').344 - 346
Found 'DD'-anomaly for variable 'recipients' (lines '358'-'374').358 - 374
Found 'DD'-anomaly for variable 'recipients' (lines '358'-'376').358 - 376
Assigning an Object to null is a code smell. Consider refactoring.448
Assigning an Object to null is a code smell. Consider refactoring.460
Found 'DD'-anomaly for variable 'mail' (lines '460'-'323').460 - 323
Found 'DU'-anomaly for variable 'mail' (lines '460'-'469').460 - 469
Assigning an Object to null is a code smell. Consider refactoring.538

org/apache/james/transport/Loader.java

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

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

ViolationLine
Each class should declare at least one constructor36 - 189
A method should have only one exit point, and that should be the last statement in the method132
A method should have only one exit point, and that should be the last statement in the method136
A method should have only one exit point, and that should be the last statement in the method148
A method should have only one exit point, and that should be the last statement in the method165

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

ViolationLine
Each class should declare at least one constructor70 - 269
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.80
Found 'DD'-anomaly for variable 'addresses' (lines '140'-'141').140 - 141
Found 'UR'-anomaly for variable 'addressString' (lines '208'-'208').208
A method should have only one exit point, and that should be the last statement in the method212
A method should have only one exit point, and that should be the last statement in the method218

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

ViolationLine
Each class should declare at least one constructor131 - 1705
Each class should declare at least one constructor152 - 177
Avoid empty catch blocks175
Each class should declare at least one constructor184 - 194
Perhaps 'rfc822DateFormat' could be replaced by a local variable.226
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.259
Found 'DU'-anomaly for variable 'originalMail' (lines '269'-'271').269 - 271
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.281
Found 'DU'-anomaly for variable 'originalMail' (lines '291'-'293').291 - 293
Found 'DU'-anomaly for variable 'originalMail' (lines '321'-'323').321 - 323
Found 'DU'-anomaly for variable 'originalMail' (lines '350'-'352').350 - 352
Found 'DU'-anomaly for variable 'originalMail' (lines '373'-'375').373 - 375
Found 'UR'-anomaly for variable 'addressList' (lines '393'-'393').393
Found 'DU'-anomaly for variable 'newRecipients' (lines '394'-'422').394 - 422
A method should have only one exit point, and that should be the last statement in the method399
Found 'DU'-anomaly for variable 'originalMail' (lines '431'-'441').431 - 441
Assigning an Object to null is a code smell. Consider refactoring.435
Found 'DU'-anomaly for variable 'originalMail' (lines '448'-'455').448 - 455
Found 'DU'-anomaly for variable 'newMail' (lines '448'-'455').448 - 455
Found 'UR'-anomaly for variable 'addressList' (lines '473'-'473').473
Found 'DU'-anomaly for variable 'iaarray' (lines '474'-'500').474 - 500
Found 'DD'-anomaly for variable 'iaarray' (lines '474'-'483').474 - 483
A method should have only one exit point, and that should be the last statement in the method479
Found 'DU'-anomaly for variable 'originalMail' (lines '511'-'532').511 - 532
Assigning an Object to null is a code smell. Consider refactoring.518
Found 'DU'-anomaly for variable 'originalMail' (lines '539'-'546').539 - 546
Found 'DU'-anomaly for variable 'newMail' (lines '539'-'546').539 - 546
Found 'UR'-anomaly for variable 'addressString' (lines '561'-'561').561
A method should have only one exit point, and that should be the last statement in the method568
A method should have only one exit point, and that should be the last statement in the method572
Found 'DU'-anomaly for variable 'originalMail' (lines '590'-'600').590 - 600
Assigning an Object to null is a code smell. Consider refactoring.594
Found 'DU'-anomaly for variable 'originalMail' (lines '608'-'623').608 - 623
Found 'DU'-anomaly for variable 'newMail' (lines '608'-'623').608 - 623
Found 'DD'-anomaly for variable 'iart' (lines '610'-'612').610 - 612
Found 'DD'-anomaly for variable 'iart' (lines '612'-'613').612 - 613
Found 'UR'-anomaly for variable 'addressString' (lines '638'-'638').638
A method should have only one exit point, and that should be the last statement in the method644
A method should have only one exit point, and that should be the last statement in the method648
Found 'DU'-anomaly for variable 'originalMail' (lines '669'-'680').669 - 680
Assigning an Object to null is a code smell. Consider refactoring.673
Assigning an Object to null is a code smell. Consider refactoring.676
Found 'DU'-anomaly for variable 'originalMail' (lines '688'-'698').688 - 698
Found 'DU'-anomaly for variable 'newMail' (lines '688'-'698').688 - 698
Assigning an Object to null is a code smell. Consider refactoring.691
Found 'UR'-anomaly for variable 'addressString' (lines '712'-'712').712
A method should have only one exit point, and that should be the last statement in the method718
A method should have only one exit point, and that should be the last statement in the method722
Found 'DU'-anomaly for variable 'originalMail' (lines '740'-'748').740 - 748
These nested if statements could be combined743 - 745
Assigning an Object to null is a code smell. Consider refactoring.744
Found 'DU'-anomaly for variable 'originalMail' (lines '755'-'763').755 - 763
Found 'DU'-anomaly for variable 'newMail' (lines '755'-'763').755 - 763
Found 'DU'-anomaly for variable 'originalMail' (lines '783'-'785').783 - 785
Found 'DU'-anomaly for variable 'originalMail' (lines '805'-'807').805 - 807
Found 'DU'-anomaly for variable 'newMail' (lines '814'-'840').814 - 840
Found 'DD'-anomaly for variable 'subject' (lines '827'-'831').827 - 831
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.851
Found 'DU'-anomaly for variable 'originalMail' (lines '861'-'863').861 - 863
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.875
Found 'DU'-anomaly for variable 'originalMail' (lines '885'-'887').885 - 887
Found 'DU'-anomaly for variable 'newMail' (lines '893'-'903').893 - 903
Found 'DU'-anomaly for variable 'originalMail' (lines '893'-'903').893 - 903
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.915
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.917
Found 'UR'-anomaly for variable 'logBuffer' (lines '971'-'971').971
Found 'UR'-anomaly for variable 'logBuffer' (lines '971'-'971').971
Found 'DD'-anomaly for variable 'keepMessageId' (lines '973'-'1,026').973 - 1026
Found 'DD'-anomaly for variable 'keepMessageId' (lines '973'-'1,026').973 - 1026
Found 'DU'-anomaly for variable 'logBuffer' (lines '1,059'-'1,077').1059 - 1077
Found 'DU'-anomaly for variable 'logBuffer' (lines '1,059'-'1,077').1059 - 1077
Found 'DD'-anomaly for variable 'count' (lines '1,095'-'1,098').1095 - 1098
Found 'DD'-anomaly for variable 'index' (lines '1,096'-'1,097').1096 - 1097
Avoid assignments in operands1097
Found 'DD'-anomaly for variable 'count' (lines '1,098'-'1,098').1098
Found 'DD'-anomaly for variable 'param' (lines '1,124'-'1,125').1124 - 1125
A method should have only one exit point, and that should be the last statement in the method1127
A method should have only one exit point, and that should be the last statement in the method1130
A method should have only one exit point, and that should be the last statement in the method1133
A method should have only one exit point, and that should be the last statement in the method1136
A method should have only one exit point, and that should be the last statement in the method1139
A method should have only one exit point, and that should be the last statement in the method1142
A method should have only one exit point, and that should be the last statement in the method1152
Found 'DU'-anomaly for variable 'out' (lines '1,214'-'1,314').1214 - 1314
Found 'DU'-anomaly for variable 'head' (lines '1,215'-'1,314').1215 - 1314
Found 'DD'-anomaly for variable 'all' (lines '1,216'-'1,228').1216 - 1228
Found 'DU'-anomaly for variable 'all' (lines '1,216'-'1,314').1216 - 1314
Found 'DU'-anomaly for variable 'newMail' (lines '1,319'-'1,327').1319 - 1327
A method should have only one exit point, and that should be the last statement in the method1341
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,347'-'1,350').1347 - 1350
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,350'-'1,353').1350 - 1353
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,353'-'1,356').1353 - 1356
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,353'-'1,359').1353 - 1359
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,353'-'1,362').1353 - 1362
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,356'-'1,359').1356 - 1359
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,356'-'1,362').1356 - 1362
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,356'-'1,365').1356 - 1365
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,356'-'1,368').1356 - 1368
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,356'-'1,371').1356 - 1371
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,356'-'1,374').1356 - 1374
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,356'-'1,377').1356 - 1377
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,359'-'1,362').1359 - 1362
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,359'-'1,365').1359 - 1365
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,359'-'1,368').1359 - 1368
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,359'-'1,371').1359 - 1371
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,359'-'1,374').1359 - 1374
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,359'-'1,377').1359 - 1377
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,362'-'1,365').1362 - 1365
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,362'-'1,368').1362 - 1368
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,362'-'1,371').1362 - 1371
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,362'-'1,374').1362 - 1374
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,362'-'1,377').1362 - 1377
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,365'-'1,368').1365 - 1368
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,365'-'1,371').1365 - 1371
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,365'-'1,374').1365 - 1374
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,365'-'1,377').1365 - 1377
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,368'-'1,371').1368 - 1371
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,368'-'1,374').1368 - 1374
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,368'-'1,377').1368 - 1377
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,371'-'1,374').1371 - 1374
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,371'-'1,377').1371 - 1377
Found 'DD'-anomaly for variable 'specialAddress' (lines '1,374'-'1,377').1374 - 1377
Found 'DD'-anomaly for variable 'allowed' (lines '1,383'-'1,389').1383 - 1389
Found 'DD'-anomaly for variable 'allowedSpecial' (lines '1,385'-'1,386').1385 - 1386
A method should have only one exit point, and that should be the last statement in the method1420
Found 'UR'-anomaly for variable 'bad' (lines '1,428'-'1,428').1428
Found 'DU'-anomaly for variable 'allowed' (lines '1,434'-'1,453').1434 - 1453
A method should have only one exit point, and that should be the last statement in the method1527
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 method1532
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 method1537
Found 'DU'-anomaly for variable 'mail' (lines '1,556'-'1,608').1556 - 1608
Found 'DD'-anomaly for variable 'parsedAddressCount' (lines '1,569'-'1,576').1569 - 1576
Found 'DD'-anomaly for variable 'parsedAddressCount' (lines '1,576'-'1,576').1576
Found 'DU'-anomaly for variable 'mail' (lines '1,625'-'1,703').1625 - 1703

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

ViolationLine
Each class should declare at least one constructor46 - 314
Found 'DU'-anomaly for variable 'recipientsMap' (lines '196'-'196').196
Found 'DU'-anomaly for variable 'address' (lines '205'-'222').205 - 222
Found 'DU'-anomaly for variable 'mail' (lines '205'-'222').205 - 222
Found 'DU'-anomaly for variable 'code' (lines '209'-'222').209 - 222
Found 'DU'-anomaly for variable 'mail' (lines '235'-'265').235 - 265
Found 'DD'-anomaly for variable 'result' (lines '236'-'256').236 - 256
Found 'DD'-anomaly for variable 'count' (lines '293'-'296').293 - 296
Found 'DD'-anomaly for variable 'index' (lines '294'-'295').294 - 295
Avoid assignments in operands295
Found 'DD'-anomaly for variable 'count' (lines '296'-'296').296

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

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

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

ViolationLine
Each class should declare at least one constructor73 - 115

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

ViolationLine
Each class should declare at least one constructor26 - 37

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

ViolationLine
Each class should declare at least one constructor66 - 215
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.104
Avoid empty catch blocks105 - 107
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.109
Avoid empty catch blocks110 - 112
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.114
Avoid empty catch blocks115 - 117
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.121
Avoid empty catch blocks122 - 124

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

ViolationLine
Each class should declare at least one constructor37 - 94

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

ViolationLine
Each class should declare at least one constructor110 - 496
Perhaps 'theJDBCUtil' could be replaced by a local variable.115 - 119
Perhaps 'rfc822DateFormat' could be replaced by a local variable.140
Found 'DU'-anomaly for variable 'message' (lines '271'-'332').271 - 332
These nested if statements could be combined275 - 278
A method should have only one exit point, and that should be the last statement in the method277
Found 'DU'-anomaly for variable 'baos' (lines '287'-'332').287 - 332
Found 'DU'-anomaly for variable 'probabilityString' (lines '303'-'332').303 - 332
Found 'DU'-anomaly for variable 'senderString' (lines '307'-'332').307 - 332
Found 'DU'-anomaly for variable 'senderString' (lines '309'-'332').309 - 332
A method should have only one exit point, and that should be the last statement in the method360
Found 'DU'-anomaly for variable 'i' (lines '366'-'374').366 - 374
Avoid empty catch blocks385
Found 'DD'-anomaly for variable 'rcptAddr' (lines '399'-'400').399 - 400

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

ViolationLine
Each class should declare at least one constructor106 - 322
Perhaps 'theJDBCUtil' could be replaced by a local variable.111 - 115
Found 'DU'-anomaly for variable 'dbUpdated' (lines '224'-'308').224 - 308
Found 'DD'-anomaly for variable 'dbUpdated' (lines '224'-'258').224 - 258
Found 'DU'-anomaly for variable 'baos' (lines '228'-'308').228 - 308
Found 'DU'-anomaly for variable 'conn' (lines '230'-'308').230 - 308
Found 'DD'-anomaly for variable 'conn' (lines '230'-'252').230 - 252
Found 'DU'-anomaly for variable 'dbUpdated' (lines '303'-'308').303 - 308
Avoid empty catch blocks305
Avoid empty catch blocks317

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

ViolationLine
Each class should declare at least one constructor100 - 187
Found 'DD'-anomaly for variable 'apparentlyTo' (lines '147'-'148').147 - 148
Found 'DU'-anomaly for variable 'originalMail' (lines '155'-'157').155 - 157

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

ViolationLine
Each class should declare at least one constructor178 - 949
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.266
Found 'UR'-anomaly for variable 'logText' (lines '501'-'510').501 - 510
Found 'DD'-anomaly for variable 'address' (lines '503'-'516').503 - 516
Found 'DU'-anomaly for variable 'address' (lines '503'-'529').503 - 529
Found 'DU'-anomaly for variable 'usedAddresses' (lines '505'-'529').505 - 529
A method should have only one exit point, and that should be the last statement in the method573
Found 'DD'-anomaly for variable 'reader' (lines '588'-'596').588 - 596
Found 'DD'-anomaly for variable 'writer' (lines '589'-'597').589 - 597
Found 'DD'-anomaly for variable 'streamSocket' (lines '590'-'607').590 - 607
Found 'DD'-anomaly for variable 'bos' (lines '591'-'608').591 - 608
Found 'DD'-anomaly for variable 'answer' (lines '616'-'620').616 - 620
Found 'DU'-anomaly for variable 'answer' (lines '616'-'709').616 - 709
Found 'DD'-anomaly for variable 'virusFound' (lines '617'-'626').617 - 626
Found 'DD'-anomaly for variable 'logMessage' (lines '618'-'627').618 - 627
Found 'DU'-anomaly for variable 'logMessage' (lines '618'-'709').618 - 709
Found 'DD'-anomaly for variable 'virusFound' (lines '626'-'626').626
Avoid empty catch blocks686
Avoid empty catch blocks691
Avoid empty catch blocks696
Avoid empty catch blocks701
Avoid empty catch blocks706
Found 'UR'-anomaly for variable 'bad' (lines '717'-'717').717
Found 'DU'-anomaly for variable 'allowed' (lines '723'-'742').723 - 742
A method should have only one exit point, and that should be the last statement in the method749
Found 'DD'-anomaly for variable 'socket' (lines '781'-'789').781 - 789
Found 'DU'-anomaly for variable 'ping' (lines '783'-'842').783 - 842
Found 'DD'-anomaly for variable 'pongReceived' (lines '817'-'825').817 - 825
Found 'DD'-anomaly for variable 'pongReceived' (lines '825'-'825').825
Found 'DD'-anomaly for variable 'port' (lines '851'-'854').851 - 854
Avoid empty catch blocks855 - 857
Found 'DD'-anomaly for variable 'sender' (lines '912'-'913').912 - 913
Found 'DD'-anomaly for variable 'rcpts' (lines '921'-'922').921 - 922

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

ViolationLine
It is a good practice to call super() in a constructor66 - 74

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 constructor70 - 575
Found 'DU'-anomaly for variable 'listservAddr' (lines '196'-'231').196 - 231
A method should have only one exit point, and that should be the last statement in the method200
A method should have only one exit point, and that should be the last statement in the method205
Found 'DU'-anomaly for variable 'message' (lines '242'-'260').242 - 260
Found 'DD'-anomaly for variable 'subj' (lines '255'-'257').255 - 257
A method should have only one exit point, and that should be the last statement in the method301
A method should have only one exit point, and that should be the last statement in the method320
A method should have only one exit point, and that should be the last statement in the method340
Found 'DD'-anomaly for variable 'value' (lines '373'-'375').373 - 375
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.375
Avoid empty catch blocks376 - 378
Found 'DD'-anomaly for variable 'value' (lines '389'-'391').389 - 391
Avoid empty catch blocks392 - 394
Found 'UR'-anomaly for variable 'repName' (lines '409'-'409').409
Found 'DU'-anomaly for variable 'name' (lines '522'-'538').522 - 538
A method should have only one exit point, and that should be the last statement in the method531
A method should have only one exit point, and that should be the last statement in the method566

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

ViolationLine
Each class should declare at least one constructor28 - 54
Use explicit scoping instead of the default package private level33

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

ViolationLine
Each class should declare at least one constructor31 - 57

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

ViolationLine
Each class should declare at least one constructor30 - 51
Found 'DU'-anomaly for variable 'mail' (lines '39'-'41').39 - 41

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

ViolationLine
Each class should declare at least one constructor27 - 46
Found 'DU'-anomaly for variable 'mail' (lines '34'-'36').34 - 36

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

ViolationLine
Each class should declare at least one constructor87 - 673
Found 'DD'-anomaly for variable 'machine' (lines '256'-'259').256 - 259
Found 'DD'-anomaly for variable 'machine' (lines '259'-'261').259 - 261
Found 'DU'-anomaly for variable 'machine' (lines '261'-'285').261 - 285
Found 'DD'-anomaly for variable 'nameType' (lines '298'-'316').298 - 316
Found 'DD'-anomaly for variable 'diagnosticType' (lines '371'-'383').371 - 383
Found 'DD'-anomaly for variable 'diagnosticType' (lines '371'-'385').371 - 385
Found 'DU'-anomaly for variable 'sb' (lines '455'-'542').455 - 542
A method should have only one exit point, and that should be the last statement in the method459
A method should have only one exit point, and that should be the last statement in the method463
A method should have only one exit point, and that should be the last statement in the method468
A method should have only one exit point, and that should be the last statement in the method471
Found 'DU'-anomaly for variable 'sb' (lines '475'-'542').475 - 542
A method should have only one exit point, and that should be the last statement in the method479
Found 'DD'-anomaly for variable 'smtpCode' (lines '482'-'484').482 - 484
Avoid empty catch blocks485 - 486
A method should have only one exit point, and that should be the last statement in the method491
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 method510
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 method524
A method should have only one exit point, and that should be the last statement in the method525
A method should have only one exit point, and that should be the last statement in the method530
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 method536
A method should have only one exit point, and that should be the last statement in the method551
A method should have only one exit point, and that should be the last statement in the method563
Found 'DD'-anomaly for variable 'count' (lines '591'-'594').591 - 594
Found 'DD'-anomaly for variable 'index' (lines '592'-'593').592 - 593
Avoid assignments in operands593
Found 'DD'-anomaly for variable 'count' (lines '594'-'594').594
Found 'DD'-anomaly for variable 'apparentlyTo' (lines '657'-'658').657 - 658
Found 'DU'-anomaly for variable 'originalMail' (lines '665'-'667').665 - 667

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

ViolationLine
Each class should declare at least one constructor57 - 200
Found 'UR'-anomaly for variable 'addressList' (lines '109'-'109').109

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

ViolationLine
Each class should declare at least one constructor44 - 147
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.74
Avoid empty catch blocks75 - 77
Found 'DU'-anomaly for variable 'processed' (lines '102'-'137').102 - 137

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

ViolationLine
Each class should declare at least one constructor39 - 238
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
Found 'DD'-anomaly for variable 'subj' (lines '214'-'216').214 - 216

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

ViolationLine
Each class should declare at least one constructor30 - 88
Found 'DU'-anomaly for variable 'address' (lines '36'-'36').36
Found 'DU'-anomaly for variable 'address' (lines '42'-'42').42
Found 'DU'-anomaly for variable 'address' (lines '49'-'49').49

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

ViolationLine
Avoid modifiers which are implied by the context46
Avoid modifiers which are implied by the context53
Found 'DU'-anomaly for variable 'displayFormat' (lines '53'-'53').53
Avoid modifiers which are implied by the context59
Avoid modifiers which are implied by the context65
Avoid modifiers which are implied by the context72
Found 'DU'-anomaly for variable 'name' (lines '72'-'72').72
Avoid modifiers which are implied by the context78
Avoid modifiers which are implied by the context85
Found 'DU'-anomaly for variable 'mailAddress' (lines '85'-'85').85
Avoid modifiers which are implied by the context91
Avoid modifiers which are implied by the context99
Found 'DU'-anomaly for variable 'errorMessage' (lines '99'-'99').99
Found 'DU'-anomaly for variable 'mail' (lines '99'-'99').99
Found 'DU'-anomaly for variable 'subject' (lines '99'-'99').99
Avoid modifiers which are implied by the context104
Avoid modifiers which are implied by the context110
Avoid modifiers which are implied by the context118
Found 'DU'-anomaly for variable 'names' (lines '118'-'118').118

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

ViolationLine
Each class should declare at least one constructor54 - 198
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '70'-'70').70
Found 'DD'-anomaly for variable 'conn' (lines '78'-'92').78 - 92
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '99'-'127').99 - 127
Found 'DD'-anomaly for variable 'conn' (lines '132'-'140').132 - 140
Found 'DD'-anomaly for variable 'mappingStmt' (lines '133'-'141').133 - 141
Found 'DD'-anomaly for variable 'mappingRS' (lines '134'-'148').134 - 148
Found 'DU'-anomaly for variable 'mappingRS' (lines '134'-'187').134 - 187
Assigning an Object to null is a code smell. Consider refactoring.174
Found 'DD'-anomaly for variable 'mappingRS' (lines '174'-'148').174 - 148
Found 'DU'-anomaly for variable 'mappingRS' (lines '174'-'187').174 - 187

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

ViolationLine
Each class should declare at least one constructor57 - 330
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '90'-'90').90
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.111
Avoid empty catch blocks112 - 114
Found 'DD'-anomaly for variable 'conn' (lines '117'-'126').117 - 126
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '133'-'179').133 - 179
Found 'DD'-anomaly for variable 'conn' (lines '238'-'244').238 - 244
Found 'DD'-anomaly for variable 'stmt' (lines '239'-'246').239 - 246
Found 'DD'-anomaly for variable 'rs' (lines '240'-'248').240 - 248
Assigning an Object to null is a code smell. Consider refactoring.271
Found 'DD'-anomaly for variable 'rs' (lines '271'-'281').271 - 281
Assigning an Object to null is a code smell. Consider refactoring.275
Found 'DD'-anomaly for variable 'stmt' (lines '275'-'279').275 - 279
Assigning an Object to null is a code smell. Consider refactoring.296
Assigning an Object to null is a code smell. Consider refactoring.310

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

ViolationLine
Each class should declare at least one constructor89 - 201
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '110'-'110').110
Found 'DD'-anomaly for variable 'conn' (lines '121'-'130').121 - 130
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '137'-'156').137 - 156
Found 'DD'-anomaly for variable 'conn' (lines '165'-'171').165 - 171
Found 'DD'-anomaly for variable 'mappingStmt' (lines '166'-'172').166 - 172
Found 'DD'-anomaly for variable 'mappingRS' (lines '175'-'181').175 - 181

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

ViolationLine
Each class should declare at least one constructor56 - 223

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

ViolationLine
Each class should declare at least one constructor51 - 103

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

ViolationLine
Avoid modifiers which are implied by the context68
Avoid modifiers which are implied by the context75
Found 'DU'-anomaly for variable 'configuration' (lines '75'-'75').75
Found 'DU'-anomaly for variable 'commandListservManager' (lines '75'-'75').75
Avoid modifiers which are implied by the context82
Found 'DU'-anomaly for variable 'mail' (lines '82'-'82').82

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

ViolationLine
Each class should declare at least one constructor55 - 118

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

ViolationLine
Found 'DD'-anomaly for variable 'inputStream' (lines '108'-'109').108 - 109
Found 'DD'-anomaly for variable 'outputStream' (lines '108'-'110').108 - 110
Avoid assignments in operands115
Found 'DU'-anomaly for variable 'len' (lines '115'-'121').115 - 121

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

ViolationLine
Each class should declare at least one constructor37 - 63

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

ViolationLine
Each class should declare at least one constructor56 - 106
A method should have only one exit point, and that should be the last statement in the method102

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

ViolationLine
Each class should declare at least one constructor57 - 119
A method should have only one exit point, and that should be the last statement in the method115

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

ViolationLine
Each class should declare at least one constructor56 - 103
A method should have only one exit point, and that should be the last statement in the method99

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

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

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

ViolationLine
Each class should declare at least one constructor40 - 131
Found 'DU'-anomaly for variable 'm' (lines '85'-'129').85 - 129
Found 'DU'-anomaly for variable 'm' (lines '85'-'129').85 - 129
Found 'DU'-anomaly for variable 'm' (lines '85'-'129').85 - 129
Found 'DU'-anomaly for variable 'm' (lines '85'-'129').85 - 129
Found 'DU'-anomaly for variable 'm' (lines '85'-'129').85 - 129

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

ViolationLine
Each class should declare at least one constructor38 - 120
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.55
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.56
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.57
Avoid empty catch blocks60 - 62
Found 'DD'-anomaly for variable 'read' (lines '81'-'82').81 - 82
Avoid assignments in operands82
Found 'DU'-anomaly for variable 'read' (lines '82'-'97').82 - 97

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

ViolationLine
Each class should declare at least one constructor81 - 162
Found 'DD'-anomaly for variable 'iaarray' (lines '131'-'132').131 - 132
Found 'DD'-anomaly for variable 'iaarray' (lines '132'-'137').132 - 137
A method should have only one exit point, and that should be the last statement in the method153
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.155

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

ViolationLine
Each class should declare at least one constructor80 - 161
Found 'DD'-anomaly for variable 'iaarray' (lines '130'-'131').130 - 131
Found 'DD'-anomaly for variable 'iaarray' (lines '131'-'136').131 - 136
A method should have only one exit point, and that should be the last statement in the method152
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.154

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

ViolationLine
Each class should declare at least one constructor27 - 47

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

ViolationLine
Each class should declare at least one constructor37 - 81
Found 'DU'-anomaly for variable 'recipientsToRemove' (lines '49'-'71').49 - 71
Found 'DU'-anomaly for variable 'mailetContext' (lines '50'-'71').50 - 71
Found 'DD'-anomaly for variable 'postmasterAddressed' (lines '51'-'64').51 - 64
Found 'DD'-anomaly for variable 'postmasterAddressed' (lines '64'-'64').64

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

ViolationLine
Each class should declare at least one constructor267 - 441
Found 'UR'-anomaly for variable 'addressList' (lines '329'-'329').329
Found 'DU'-anomaly for variable 'newRecipients' (lines '330'-'358').330 - 358
A method should have only one exit point, and that should be the last statement in the method335
Found 'UR'-anomaly for variable 'addressList' (lines '369'-'369').369
Found 'DU'-anomaly for variable 'iaarray' (lines '370'-'396').370 - 396
Found 'DD'-anomaly for variable 'iaarray' (lines '370'-'379').370 - 379
A method should have only one exit point, and that should be the last statement in the method375
Found 'UR'-anomaly for variable 'addressString' (lines '405'-'405').405
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 method415

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

ViolationLine
Each class should declare at least one constructor97 - 1374
Each class should declare at least one constructor140 - 197
Use explicit scoping instead of the default package private level145
Found 'DU'-anomaly for variable 'key' (lines '155'-'182').155 - 182
Found 'DU'-anomaly for variable 'lastUpdated' (lines '155'-'182').155 - 182
Found 'DU'-anomaly for variable 'errorMessage' (lines '155'-'182').155 - 182
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 method169
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 method190
Perhaps 'deliveryThreadCount' could be replaced by a local variable.210
Perhaps 'bindAddress' could be replaced by a local variable.214
Perhaps 'delayFilter' could be replaced by a local variable.223
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.230
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.292
Found 'DU'-anomaly for variable 'gatewayPort' (lines '297'-'360').297 - 360
Avoid empty catch blocks340 - 341
Found 'DU'-anomaly for variable 'sfe' (lines '365'-'400').365 - 400
Avoid assignments in operands379
Found 'UR'-anomaly for variable 'lastError' (lines '413'-'413').413
Found 'DU'-anomaly for variable 'session' (lines '413'-'753').413 - 753
Found 'DU'-anomaly for variable 'message' (lines '418'-'753').418 - 753
Found 'DD'-anomaly for variable 'addr' (lines '422'-'426').422 - 426
Found 'DU'-anomaly for variable 'j' (lines '424'-'753').424 - 753
Found 'DD'-anomaly for variable 'addr' (lines '426'-'426').426
A method should have only one exit point, and that should be the last statement in the method431
Found 'DD'-anomaly for variable 'targetServers' (lines '436'-'442').436 - 442
Found 'DD'-anomaly for variable 'targetServers' (lines '436'-'453').436 - 453
A method should have only one exit point, and that should be the last statement in the method450
Found 'DU'-anomaly for variable 'lastError' (lines '456'-'753').456 - 753
Found 'DD'-anomaly for variable 'transport' (lines '487'-'489').487 - 489
Found 'DU'-anomaly for variable 'smtpTransport' (lines '507'-'753').507 - 753
Assigning an Object to null is a code smell. Consider refactoring.556
Found 'DU'-anomaly for variable 'transport' (lines '556'-'753').556 - 753
A method should have only one exit point, and that should be the last statement in the method570
Found 'DU'-anomaly for variable 'recipients' (lines '642'-'753').642 - 753
Found 'DD'-anomaly for variable 'deleteMessage' (lines '644'-'670').644 - 670
Found 'DD'-anomaly for variable 'deleteMessage' (lines '644'-'679').644 - 679
Found 'DD'-anomaly for variable 'deleteMessage' (lines '644'-'702').644 - 702
Found 'DD'-anomaly for variable 'deleteMessage' (lines '644'-'723').644 - 723
Found 'DD'-anomaly for variable 'deleteMessage' (lines '644'-'725').644 - 725
Found 'DD'-anomaly for variable 'deleteMessage' (lines '670'-'702').670 - 702
Found 'DD'-anomaly for variable 'deleteMessage' (lines '670'-'723').670 - 723
Found 'DD'-anomaly for variable 'deleteMessage' (lines '670'-'725').670 - 725
Avoid assignments in operands675
Found 'DD'-anomaly for variable 'deleteMessage' (lines '679'-'679').679
Found 'DD'-anomaly for variable 'deleteMessage' (lines '679'-'702').679 - 702
Found 'DD'-anomaly for variable 'deleteMessage' (lines '679'-'723').679 - 723
Found 'DD'-anomaly for variable 'deleteMessage' (lines '679'-'725').679 - 725
Found 'DD'-anomaly for variable 'deleteMessage' (lines '702'-'723').702 - 723
Found 'DD'-anomaly for variable 'deleteMessage' (lines '702'-'725').702 - 725
A method should have only one exit point, and that should be the last statement in the method730
A method should have only one exit point, and that should be the last statement in the method743
Found 'DU'-anomaly for variable 'ex' (lines '814'-'885').814 - 885
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 method864
Found 'DD'-anomaly for variable 'machine' (lines '890'-'893').890 - 893
Found 'DD'-anomaly for variable 'machine' (lines '893'-'895').893 - 895
Found 'DU'-anomaly for variable 'recipients' (lines '958'-'1,017').958 - 1017
Found 'DU'-anomaly for variable 'name' (lines '978'-'1,017').978 - 1017
Avoid empty catch blocks1051
Found 'DU'-anomaly for variable 'session' (lines '1,096'-'1,147').1096 - 1147
Assigning an Object to null is a code smell. Consider refactoring.1128
Found 'DD'-anomaly for variable 'delays' (lines '1,175'-'1,181').1175 - 1181
Found 'DU'-anomaly for variable 'idx' (lines '1,177'-'1,185').1177 - 1185
Found 'DD'-anomaly for variable 'delays' (lines '1,181'-'1,181').1181
A method should have only one exit point, and that should be the last statement in the method1194
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,306'-'1,373').1306 - 1373
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,306'-'1,373').1306 - 1373
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,306'-'1,373').1306 - 1373
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,306'-'1,373').1306 - 1373
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,306'-'1,373').1306 - 1373
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,306'-'1,373').1306 - 1373
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,306'-'1,373').1306 - 1373
Found 'DU'-anomaly for variable 'gatewayServers' (lines '1,306'-'1,373').1306 - 1373
Assigning an Object to null is a code smell. Consider refactoring.1366

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

ViolationLine
Each class should declare at least one constructor51 - 133
Use explicit scoping instead of the default package private level56 - 59
Assigning an Object to null is a code smell. Consider refactoring.57

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

ViolationLine
Each class should declare at least one constructor33 - 56

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

ViolationLine
Each class should declare at least one constructor40 - 87
Found 'DU'-anomaly for variable 'mail' (lines '78'-'84').78 - 84

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

ViolationLine
Each class should declare at least one constructor243 - 286

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

ViolationLine
Each class should declare at least one constructor32 - 75

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

ViolationLine
Each class should declare at least one constructor42 - 91
Perhaps 'attributesToSet' could be replaced by a local variable.44
Found 'DU'-anomaly for variable 'mail' (lines '80'-'88').80 - 88

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

ViolationLine
Each class should declare at least one constructor38 - 93

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

ViolationLine
Each class should declare at least one constructor98 - 711
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.152
These nested if statements could be combined221 - 223
These nested if statements could be combined228 - 230
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.278
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.302
A method should have only one exit point, and that should be the last statement in the method513
A method should have only one exit point, and that should be the last statement in the method519
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 method531
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 method544
Found 'DU'-anomaly for variable 'mail' (lines '557'-'557').557
Found 'UR'-anomaly for variable 'bad' (lines '563'-'563').563
Found 'DU'-anomaly for variable 'allowed' (lines '569'-'588').569 - 588
A method should have only one exit point, and that should be the last statement in the method595
A method should have only one exit point, and that should be the last statement in the method620
Found 'DD'-anomaly for variable 'mailAddress' (lines '627'-'629').627 - 629
A method should have only one exit point, and that should be the last statement in the method635
Found 'DD'-anomaly for variable 'replacedExplanationText' (lines '675'-'677').675 - 677
Found 'DU'-anomaly for variable 'pattern' (lines '692'-'709').692 - 709
Avoid assignments in operands697
Found 'DU'-anomaly for variable 'index' (lines '697'-'709').697 - 709
A method should have only one exit point, and that should be the last statement in the method705

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you108 - 111
Found 'DU'-anomaly for variable 'type' (lines '126'-'140').126 - 140
Found 'DD'-anomaly for variable 'strippedMessage' (lines '151'-'165').151 - 165
Found 'DD'-anomaly for variable 'strippedMessage' (lines '151'-'205').151 - 205
Found 'DD'-anomaly for variable 'signers' (lines '153'-'164').153 - 164
Found 'DD'-anomaly for variable 'signers' (lines '153'-'174').153 - 174
Assigning an Object to null is a code smell. Consider refactoring.161
Found 'DD'-anomaly for variable 'signers' (lines '164'-'174').164 - 174
Found 'DD'-anomaly for variable 'strippedMessage' (lines '165'-'205').165 - 205
Assigning an Object to null is a code smell. Consider refactoring.174
Found 'DD'-anomaly for variable 'signers' (lines '174'-'177').174 - 177
Assigning an Object to null is a code smell. Consider refactoring.177
Found 'DD'-anomaly for variable 'signers' (lines '177'-'180').177 - 180
Assigning an Object to null is a code smell. Consider refactoring.180
Found 'DD'-anomaly for variable 'signers' (lines '180'-'184').180 - 184
Assigning an Object to null is a code smell. Consider refactoring.184
Assigning an Object to null is a code smell. Consider refactoring.205

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

ViolationLine
Each class should declare at least one constructor67 - 159
Found 'DD'-anomaly for variable 'strippedMessage' (lines '105'-'118').105 - 118
Found 'DD'-anomaly for variable 'strippedMessage' (lines '118'-'118').118

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

ViolationLine
Each class should declare at least one constructor54 - 198
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.128
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.135
Found 'DU'-anomaly for variable 'mail' (lines '153'-'196').153 - 196
A method should have only one exit point, and that should be the last statement in the method159

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

ViolationLine
Each class should declare at least one constructor67 - 317
Found 'DD'-anomaly for variable 'message' (lines '117'-'119').117 - 119
Found 'DU'-anomaly for variable 'message' (lines '119'-'186').119 - 186
Found 'DU'-anomaly for variable 'deliveredTo' (lines '133'-'186').133 - 186
These nested if statements could be combined155 - 165
Found 'UR'-anomaly for variable 'errorBuffer' (lines '204'-'204').204
Found 'DU'-anomaly for variable 'errorBuffer' (lines '223'-'232').223 - 232
Found 'DD'-anomaly for variable 'localCount' (lines '240'-'242').240 - 242
Found 'DD'-anomaly for variable 'userInbox' (lines '308'-'311').308 - 311
Assigning an Object to null is a code smell. Consider refactoring.311

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

ViolationLine
Each class should declare at least one constructor36 - 107
Use explicit scoping instead of the default package private level46
Use explicit scoping instead of the default package private level51
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.59

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

ViolationLine
Each class should declare at least one constructor37 - 110
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.61
Avoid empty catch blocks62 - 64

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

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

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

ViolationLine
Each class should declare at least one constructor57 - 275
Found 'UR'-anomaly for variable 'errorBuffer' (lines '169'-'169').169
Found 'DU'-anomaly for variable 'sender' (lines '169'-'231').169 - 231
Found 'DU'-anomaly for variable 'errorBuffer' (lines '184'-'231').184 - 231
A method should have only one exit point, and that should be the last statement in the method218
Found 'DU'-anomaly for variable 'usersStore' (lines '243'-'273').243 - 273
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.246 - 248
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.249 - 251
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.252 - 254

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

ViolationLine
Each class should declare at least one constructor90 - 829
Perhaps 'rfc822DateFormat' could be replaced by a local variable.104
Perhaps 'usersStore' could be replaced by a local variable.109
Perhaps 'sqlFile' could be replaced by a local variable.133
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.161
A method should have only one exit point, and that should be the last statement in the method253
A method should have only one exit point, and that should be the last statement in the method260
Found 'DD'-anomaly for variable 'senderUser' (lines '309'-'312').309 - 312
Found 'DD'-anomaly for variable 'dbUpdated' (lines '317'-'358').317 - 358
Found 'DD'-anomaly for variable 'selectRS' (lines '322'-'344').322 - 344
Found 'DD'-anomaly for variable 'dbUpdated' (lines '367'-'358').367 - 358
Found 'DU'-anomaly for variable 'dbUpdated' (lines '380'-'386').380 - 386
Avoid empty catch blocks383
Found 'DD'-anomaly for variable 'senderUser' (lines '393'-'396').393 - 396
Found 'DD'-anomaly for variable 'conn' (lines '398'-'410').398 - 410
Found 'DD'-anomaly for variable 'selectStmt' (lines '399'-'411').399 - 411
Found 'DD'-anomaly for variable 'selectRS' (lines '400'-'414').400 - 414
Found 'DD'-anomaly for variable 'senderUser' (lines '442'-'445').442 - 445
Found 'DD'-anomaly for variable 'dbUpdated' (lines '450'-'511').450 - 511
Found 'DD'-anomaly for variable 'selectRS' (lines '468'-'496').468 - 496
Found 'DD'-anomaly for variable 'dbUpdated' (lines '511'-'511').511
Found 'DU'-anomaly for variable 'dbUpdated' (lines '552'-'558').552 - 558
Avoid empty catch blocks555
Found 'DD'-anomaly for variable 'senderUser' (lines '565'-'568').565 - 568
Found 'DD'-anomaly for variable 'dbUpdated' (lines '573'-'634').573 - 634
Found 'DD'-anomaly for variable 'selectRS' (lines '591'-'619').591 - 619
Found 'DD'-anomaly for variable 'dbUpdated' (lines '634'-'634').634
Found 'DU'-anomaly for variable 'dbUpdated' (lines '675'-'681').675 - 681
Avoid empty catch blocks678
Found 'DD'-anomaly for variable 'rcptAddr' (lines '694'-'695').694 - 695
Found 'DD'-anomaly for variable 'username' (lines '746'-'750').746 - 750
Found 'DU'-anomaly for variable 'dbMetaData' (lines '779'-'793').779 - 793
Found 'DD'-anomaly for variable 'dbUpdated' (lines '783'-'785').783 - 785
Found 'DU'-anomaly for variable 'dbUpdated' (lines '790'-'793').790 - 793
Found 'DU'-anomaly for variable 'createSqlStringName' (lines '795'-'827').795 - 827
A method should have only one exit point, and that should be the last statement in the method802
Found 'DD'-anomaly for variable 'createStatement' (lines '805'-'808').805 - 808
Found 'DD'-anomaly for variable 'logBuffer' (lines '812'-'813').812 - 813

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

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

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

ViolationLine
Each class should declare at least one constructor45 - 83
Found 'DD'-anomaly for variable 'networks' (lines '67'-'70').67 - 70

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

ViolationLine
Each class should declare at least one constructor37 - 148
Found 'DD'-anomaly for variable 'matching' (lines '48'-'50').48 - 50
Found 'DU'-anomaly for variable 'user' (lines '71'-'80').71 - 80
A method should have only one exit point, and that should be the last statement in the method75
Found 'DU'-anomaly for variable 'address' (lines '112'-'112').112
Found 'DU'-anomaly for variable 'mail' (lines '112'-'112').112
Found 'DU'-anomaly for variable 'address' (lines '120'-'120').120
Found 'DU'-anomaly for variable 'mail' (lines '120'-'120').120

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

ViolationLine
Each class should declare at least one constructor44 - 130
Found 'DU'-anomaly for variable '_' (lines '91'-'107').91 - 107
Found 'DD'-anomaly for variable 'username' (lines '121'-'125').121 - 125

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

ViolationLine
Each class should declare at least one constructor28 - 32

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

ViolationLine
Each class should declare at least one constructor53 - 258
Each class should declare at least one constructor67 - 73
A method should have only one exit point, and that should be the last statement in the method129
Found 'UR'-anomaly for variable 'anException' (lines '145'-'145').145
A method should have only one exit point, and that should be the last statement in the method154
Found 'DU'-anomaly for variable 'content' (lines '160'-'203').160 - 203
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 method174
A method should have only one exit point, and that should be the last statement in the method189
A method should have only one exit point, and that should be the last statement in the method192
Found 'DU'-anomaly for variable 'fileName' (lines '208'-'224').208 - 224
A method should have only one exit point, and that should be the last statement in the method220
Found 'DU'-anomaly for variable 'zis' (lines '230'-'250').230 - 250
A method should have only one exit point, and that should be the last statement in the method243

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

ViolationLine
Each class should declare at least one constructor31 - 48
These nested if statements could be combined41 - 44
A method should have only one exit point, and that should be the last statement in the method43

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

ViolationLine
Each class should declare at least one constructor40 - 62
These nested if statements could be combined56 - 58
A method should have only one exit point, and that should be the last statement in the method57

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

ViolationLine
Each class should declare at least one constructor49 - 167
Found 'UR'-anomaly for variable 'comparisonOperatorString' (lines '62'-'62').62
Found 'UR'-anomaly for variable 'headerValueString' (lines '62'-'62').62
Found 'UR'-anomaly for variable 'headerArray' (lines '117'-'117').117
Found 'DU'-anomaly for variable 'comparison' (lines '128'-'166').128 - 166
A method should have only one exit point, and that should be the last statement in the method132
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 method142
A method should have only one exit point, and that should be the last statement in the method147
A method should have only one exit point, and that should be the last statement in the method152

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

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

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

ViolationLine
Each class should declare at least one constructor30 - 66
Found 'DD'-anomaly for variable 'patternSource' (lines '32'-'34').32 - 34
Found 'DD'-anomaly for variable 'lines' (lines '35'-'36').35 - 36
Found 'DD'-anomaly for variable 'lines' (lines '36'-'36').36
Avoid empty catch blocks60 - 62

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

ViolationLine
Each class should declare at least one constructor44 - 81
Found 'DU'-anomaly for variable 'message' (lines '59'-'80').59 - 80
Found 'DU'-anomaly for variable 'matcher' (lines '60'-'80').60 - 80
Found 'DU'-anomaly for variable 'pattern' (lines '67'-'80').67 - 80
Found 'DD'-anomaly for variable 'pattern' (lines '67'-'67').67
A method should have only one exit point, and that should be the last statement in the method74

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

ViolationLine
Each class should declare at least one constructor34 - 87
Found 'UR'-anomaly for variable 'anException' (lines '40'-'40').40
Found 'DU'-anomaly for variable 'anException' (lines '42'-'86').42 - 86
Found 'DD'-anomaly for variable 'anException' (lines '42'-'67').42 - 67
Found 'DD'-anomaly for variable 'anException' (lines '42'-'77').42 - 77
A method should have only one exit point, and that should be the last statement in the method53
A method should have only one exit point, and that should be the last statement in the method64
Found 'DU'-anomaly for variable 'anException' (lines '67'-'86').67 - 86
Found 'DD'-anomaly for variable 'anException' (lines '67'-'67').67
Found 'DD'-anomaly for variable 'anException' (lines '67'-'77').67 - 77
A method should have only one exit point, and that should be the last statement in the method73

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

ViolationLine
Each class should declare at least one constructor65 - 122
Found 'DU'-anomaly for variable 'message' (lines '82'-'107').82 - 107
A method should have only one exit point, and that should be the last statement in the method97
A method should have only one exit point, and that should be the last statement in the method102

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

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

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

ViolationLine
Each class should declare at least one constructor38 - 71
A method should have only one exit point, and that should be the last statement in the method66

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

ViolationLine
Each class should declare at least one constructor64 - 157
A method should have only one exit point, and that should be the last statement in the method90

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

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

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

ViolationLine
Each class should declare at least one constructor31 - 46

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

ViolationLine
Each class should declare at least one constructor27 - 32

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

ViolationLine
Each class should declare at least one constructor44 - 76
Use explicit scoping instead of the default package private level45
A method should have only one exit point, and that should be the last statement in the method70

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

ViolationLine
Each class should declare at least one constructor56 - 269
Perhaps 'usersStore' could be replaced by a local variable.63
Perhaps 'sqlFile' could be replaced by a local variable.87
Found 'DD'-anomaly for variable 'repositoryPath' (lines '113'-'116').113 - 116
A method should have only one exit point, and that should be the last statement in the method159
A method should have only one exit point, and that should be the last statement in the method166
Found 'DU'-anomaly for variable 'conn' (lines '176'-'226').176 - 226
Found 'DU'-anomaly for variable 'selectStmt' (lines '177'-'226').177 - 226
Found 'DD'-anomaly for variable 'selectRS' (lines '178'-'205').178 - 205
Found 'DU'-anomaly for variable 'selectRS' (lines '178'-'226').178 - 226
Found 'DD'-anomaly for variable 'recipientUser' (lines '184'-'192').184 - 192
Found 'DU'-anomaly for variable 'conn' (lines '195'-'226').195 - 226
Found 'DD'-anomaly for variable 'username' (lines '235'-'239').235 - 239

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

ViolationLine
Each class should declare at least one constructor28 - 37
A method should have only one exit point, and that should be the last statement in the method32

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

ViolationLine
Each class should declare at least one constructor31 - 98

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

ViolationLine
Each class should declare at least one constructor29 - 44

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

ViolationLine
Each class should declare at least one constructor28 - 36

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

ViolationLine
Each class should declare at least one constructor52 - 68
Found 'DU'-anomaly for variable '_' (lines '65'-'67').65 - 67
Found 'DU'-anomaly for variable 'recipient' (lines '65'-'67').65 - 67

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

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

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

ViolationLine
Each class should declare at least one constructor34 - 56
Use explicit scoping instead of the default package private level35
Found 'DD'-anomaly for variable 'count' (lines '43'-'47').43 - 47
Found 'DD'-anomaly for variable 'count' (lines '47'-'47').47
A method should have only one exit point, and that should be the last statement in the method51

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

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

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

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

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

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

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

ViolationLine
Each class should declare at least one constructor31 - 70
A method should have only one exit point, and that should be the last statement in the method35
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 method63

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

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

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

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

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

ViolationLine
Each class should declare at least one constructor54 - 84
Use explicit scoping instead of the default package private level55
Found 'UR'-anomaly for variable 'patternString' (lines '57'-'57').57
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

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

ViolationLine
Each class should declare at least one constructor37 - 68
Use explicit scoping instead of the default package private level39
A method should have only one exit point, and that should be the last statement in the method63

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

ViolationLine
Each class should declare at least one constructor31 - 48
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 method40
A method should have only one exit point, and that should be the last statement in the method44

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

ViolationLine
Each class should declare at least one constructor31 - 51
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 method40
A method should have only one exit point, and that should be the last statement in the method48

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

ViolationLine
Each class should declare at least one constructor47 - 96
Found 'UR'-anomaly for variable 'condition' (lines '51'-'51').51
Found 'DD'-anomaly for variable 'valid' (lines '72'-'82').72 - 82
Found 'DD'-anomaly for variable 'valid' (lines '82'-'82').82
A method should have only one exit point, and that should be the last statement in the method87
A method should have only one exit point, and that should be the last statement in the method89

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

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

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

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

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

ViolationLine
Each class should declare at least one constructor31 - 40
A method should have only one exit point, and that should be the last statement in the method36

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

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

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

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

org/apache/james/userrepository/AbstractJdbcUsersRepository.java

ViolationLine
Each class should declare at least one constructor74 - 619
Perhaps 'm_createUserTableSql' could be replaced by a local variable.105
Found 'DD'-anomaly for variable 'logBuffer' (lines '124'-'127').124 - 127
Found 'DU'-anomaly for variable 'logBuffer' (lines '124'-'136').124 - 136
Found 'DD'-anomaly for variable 'logBuffer' (lines '157'-'159').157 - 159
Found 'DD'-anomaly for variable 'logBuffer' (lines '157'-'200').157 - 200
Found 'DU'-anomaly for variable 'logBuffer' (lines '157'-'226').157 - 226
Found 'DD'-anomaly for variable 'logBuffer' (lines '240'-'242').240 - 242
Found 'DD'-anomaly for variable 'logBuffer' (lines '240'-'274').240 - 274
Found 'DD'-anomaly for variable 'logBuffer' (lines '240'-'336').240 - 336
Found 'DU'-anomaly for variable 'logBuffer' (lines '240'-'353').240 - 353
Found 'DD'-anomaly for variable 'logBuffer' (lines '240'-'242').240 - 242
Found 'DD'-anomaly for variable 'logBuffer' (lines '240'-'274').240 - 274
Found 'DD'-anomaly for variable 'logBuffer' (lines '240'-'336').240 - 336
Found 'DU'-anomaly for variable 'logBuffer' (lines '240'-'353').240 - 353
Found 'DD'-anomaly for variable 'sqlFile' (lines '264'-'267').264 - 267
Found 'DD'-anomaly for variable 'sqlFile' (lines '264'-'267').264 - 267
Found 'DD'-anomaly for variable 'createStatement' (lines '327'-'329').327 - 329
Found 'DD'-anomaly for variable 'createStatement' (lines '327'-'329').327 - 329
Found 'DD'-anomaly for variable 'getUsersStatement' (lines '389'-'393').389 - 393
Found 'DD'-anomaly for variable 'rsUsers' (lines '390'-'395').390 - 395
Found 'DD'-anomaly for variable 'addUserStatement' (lines '424'-'429').424 - 429
Found 'DD'-anomaly for variable 'removeUserStatement' (lines '456'-'460').456 - 460
Found 'DD'-anomaly for variable 'updateUserStatement' (lines '481'-'485').481 - 485
Found 'DU'-anomaly for variable 'ignoreCase' (lines '509'-'554').509 - 554
A method should have only one exit point, and that should be the last statement in the method514
Found 'DD'-anomaly for variable 'getUsersStatement' (lines '520'-'525').520 - 525
Found 'DD'-anomaly for variable 'rsUsers' (lines '521'-'529').521 - 529
Found 'DD'-anomaly for variable 'user' (lines '532'-'539').532 - 539
Found 'DU'-anomaly for variable 'rsUsers' (lines '568'-'569').568 - 569
Found 'DU'-anomaly for variable 'userInsert' (lines '583'-'585').583 - 585
Found 'DU'-anomaly for variable 'user' (lines '583'-'585').583 - 585
Found 'DU'-anomaly for variable 'user' (lines '599'-'601').599 - 601
Found 'DU'-anomaly for variable 'userUpdate' (lines '599'-'601').599 - 601

org/apache/james/userrepository/AbstractUsersRepository.java

ViolationLine
Each class should declare at least one constructor35 - 294
Found 'DU'-anomaly for variable 'user' (lines '53'-'53').53
Found 'DU'-anomaly for variable 'user' (lines '59'-'59').59
Found 'DU'-anomaly for variable 'user' (lines '64'-'64').64
Found 'DU'-anomaly for variable 'ignoreCase' (lines '97'-'110').97 - 110
Found 'DU'-anomaly for variable 'name' (lines '97'-'110').97 - 110
A method should have only one exit point, and that should be the last statement in the method105
A method should have only one exit point, and that should be the last statement in the method128
Found 'DU'-anomaly for variable 'name' (lines '142'-'151').142 - 151
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 method233
Found 'DU'-anomaly for variable 'password' (lines '267'-'274').267 - 274
A method should have only one exit point, and that should be the last statement in the method270

org/apache/james/userrepository/DefaultJamesUser.java

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

org/apache/james/userrepository/DefaultUsersJdbcRepository.java

ViolationLine
Each class should declare at least one constructor38 - 108

org/apache/james/userrepository/JamesUsersJdbcRepository.java

ViolationLine
Each class should declare at least one constructor32 - 175
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '42'-'42').42
Found 'DD'-anomaly for variable 'forwardAddress' (lines '53'-'56').53 - 56
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '59'-'78').59 - 78
Found 'UR'-anomaly for variable 'jamesUser' (lines '120'-'149').120 - 149
Found 'DD'-anomaly for variable 'nameIndex' (lines '123'-'126').123 - 126
Found 'DD'-anomaly for variable 'colOffset' (lines '124'-'127').124 - 127
Found 'DD'-anomaly for variable 'forwardDestination' (lines '155'-'157').155 - 157

org/apache/james/userrepository/ListUsersJdbcRepository.java

ViolationLine
Each class should declare at least one constructor32 - 91
Found 'DU'-anomaly for variable 'user' (lines '74'-'79').74 - 79
Found 'DU'-anomaly for variable 'userUpdate' (lines '74'-'79').74 - 79

org/apache/james/userrepository/UsersFileRepository.java

ViolationLine
Each class should declare at least one constructor50 - 261
Found 'UR'-anomaly for variable 'message' (lines '70'-'70').70
A method should have only one exit point, and that should be the last statement in the method148
Found 'DU'-anomaly for variable 'name' (lines '158'-'168').158 - 168
A method should have only one exit point, and that should be the last statement in the method179
A method should have only one exit point, and that should be the last statement in the method192
Found 'DU'-anomaly for variable 'name' (lines '197'-'206').197 - 206
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 method211
Found 'DU'-anomaly for variable 'name' (lines '229'-'237').229 - 237
A method should have only one exit point, and that should be the last statement in the method233
Found 'DU'-anomaly for variable 'password' (lines '239'-'251').239 - 251
A method should have only one exit point, and that should be the last statement in the method245
Found 'DD'-anomaly for variable 'count' (lines '254'-'256').254 - 256
Found 'DD'-anomaly for variable 'count' (lines '256'-'256').256

org/apache/james/userrepository/UsersLDAPRepository.java

ViolationLine
Each class should declare at least one constructor59 - 741
Perhaps 'baseURL' could be replaced by a local variable.70
Found 'DU'-anomaly for variable 'compMgr' (lines '94'-'96').94 - 96
Found 'DD'-anomaly for variable 'destination' (lines '168'-'183').168 - 183
Found 'DD'-anomaly for variable 'destination' (lines '168'-'211').168 - 211
Found 'DU'-anomaly for variable 'user' (lines '256'-'258').256 - 258
Found 'DU'-anomaly for variable 'name' (lines '260'-'262').260 - 262
Found 'DU'-anomaly for variable 'name' (lines '275'-'277').275 - 277
Found 'DU'-anomaly for variable 'user' (lines '279'-'281').279 - 281
Found 'DU'-anomaly for variable 'attributes' (lines '288'-'346').288 - 346
Found 'DD'-anomaly for variable 'mods' (lines '312'-'313').312 - 313
Found 'DU'-anomaly for variable 'password' (lines '351'-'358').351 - 358
A method should have only one exit point, and that should be the last statement in the method354
Found 'DD'-anomaly for variable 'rootCtx' (lines '367'-'369').367 - 369
Found 'DU'-anomaly for variable 'returnAttrs' (lines '371'-'428').371 - 428
Found 'DD'-anomaly for variable 'mods' (lines '449'-'450').449 - 450
Avoid empty if statements475 - 477
Found 'DD'-anomaly for variable 'rootCtx' (lines '488'-'490').488 - 490
Found 'DD'-anomaly for variable 'mods' (lines '529'-'530').529 - 530
Assigning an Object to null is a code smell. Consider refactoring.564
Found 'DU'-anomaly for variable 'rootCtx' (lines '564'-'566').564 - 566
Found 'DD'-anomaly for variable 'found' (lines '570'-'576').570 - 576
Found 'DU'-anomaly for variable 'testPassword' (lines '599'-'697').599 - 697
Found 'DD'-anomaly for variable 'result' (lines '600'-'666').600 - 666
Found 'DD'-anomaly for variable 'foundFlag' (lines '601'-'636').601 - 636
Found 'DD'-anomaly for variable 'userDN' (lines '602'-'635').602 - 635
Found 'DU'-anomaly for variable 'userDN' (lines '602'-'697').602 - 697
Found 'DD'-anomaly for variable 'rootCtx' (lines '621'-'624').621 - 624
Found 'DU'-anomaly for variable 'userDN' (lines '635'-'697').635 - 697
Found 'DD'-anomaly for variable 'testCtx' (lines '663'-'665').663 - 665
Found 'DD'-anomaly for variable 'result' (lines '666'-'669').666 - 669
Found 'DD'-anomaly for variable 'result' (lines '702'-'707').702 - 707
Found 'DD'-anomaly for variable 'result' (lines '702'-'709').702 - 709
Assigning an Object to null is a code smell. Consider refactoring.729

org/apache/james/util/Base64.java

ViolationLine
Each class should declare at least one constructor33 - 76
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 method49

org/apache/james/util/BayesianAnalyzer.java

ViolationLine
Each class should declare at least one constructor143 - 186
Use explicit scoping instead of the default package private level148
Use explicit scoping instead of the default package private level153
A method should have only one exit point, and that should be the last statement in the method166
Avoid unnecessary constructors - the compiler will generate these for you191 - 192
Found 'DU'-anomaly for variable 'target' (lines '367'-'409').367 - 409
Found 'DU'-anomaly for variable 'header' (lines '370'-'409').370 - 409
Avoid assignments in operands373
Found 'DU'-anomaly for variable 'token' (lines '373'-'409').373 - 409
Found 'DD'-anomaly for variable 'endingLine' (lines '374'-'376').374 - 376
Found 'DD'-anomaly for variable 'value' (lines '394'-'397').394 - 397
Found 'DD'-anomaly for variable 'value' (lines '394'-'399').394 - 399
Found 'DU'-anomaly for variable 'header' (lines '406'-'409').406 - 409
Found 'DU'-anomaly for variable 'header' (lines '422'-'456').422 - 456
Avoid assignments in operands425
Found 'DU'-anomaly for variable 'token' (lines '425'-'456').425 - 456
Found 'DD'-anomaly for variable 'endingLine' (lines '426'-'428').426 - 428
Found 'DU'-anomaly for variable 'header' (lines '450'-'456').450 - 456
Found 'DU'-anomaly for variable 'token' (lines '459'-'534').459 - 534
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '462'-'534').462 - 534
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '462'-'492').462 - 492
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '462'-'496').462 - 496
Found 'DU'-anomaly for variable 'tokenCharFound' (lines '463'-'534').463 - 534
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '463'-'491').463 - 491
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '463'-'495').463 - 495
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '463'-'507').463 - 507
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '463'-'519').463 - 519
A method should have only one exit point, and that should be the last statement in the method466
Avoid assignments in operands469
Found 'DU'-anomaly for variable 'i' (lines '469'-'534').469 - 534
A method should have only one exit point, and that should be the last statement in the method480
Found 'DU'-anomaly for variable 'tokenCharFound' (lines '491'-'534').491 - 534
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '491'-'491').491
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '491'-'495').491 - 495
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '491'-'507').491 - 507
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '491'-'519').491 - 519
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '492'-'534').492 - 534
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '492'-'492').492
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '492'-'496').492 - 496
Found 'DU'-anomaly for variable 'tokenCharFound' (lines '495'-'534').495 - 534
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '495'-'491').495 - 491
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '495'-'495').495
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '495'-'507').495 - 507
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '495'-'519').495 - 519
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '496'-'534').496 - 534
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '496'-'492').496 - 492
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '496'-'496').496
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '500'-'534').500 - 534
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '500'-'508').500 - 508
Found 'DU'-anomaly for variable 'tokenCharFound' (lines '507'-'534').507 - 534
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '507'-'491').507 - 491
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '507'-'495').507 - 495
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '507'-'507').507
Found 'DD'-anomaly for variable 'tokenCharFound' (lines '507'-'519').507 - 519
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '508'-'534').508 - 534
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '508'-'492').508 - 492
Found 'DD'-anomaly for variable 'previousWasDigit' (lines '508'-'496').508 - 496
Avoid empty if statements515 - 517
Found 'DU'-anomaly for variable 'previousWasDigit' (lines '520'-'534').520 - 534
A method should have only one exit point, and that should be the last statement in the method530
Found 'DD'-anomaly for variable 'hamFactor' (lines '543'-'561').543 - 561
Found 'DD'-anomaly for variable 'spamFactor' (lines '544'-'568').544 - 568
Found 'DD'-anomaly for variable 'foundInHam' (lines '546'-'553').546 - 553
Found 'DD'-anomaly for variable 'foundInSpam' (lines '547'-'557').547 - 557
Found 'DD'-anomaly for variable 'minThreshold' (lines '549'-'563').549 - 563
Found 'DU'-anomaly for variable 'minThreshold' (lines '549'-'583').549 - 583
Found 'DD'-anomaly for variable 'maxThreshold' (lines '550'-'570').550 - 570
Found 'DU'-anomaly for variable 'maxThreshold' (lines '550'-'583').550 - 583
Found 'DU'-anomaly for variable 'minThreshold' (lines '563'-'583').563 - 583
Found 'DU'-anomaly for variable 'maxThreshold' (lines '570'-'583').570 - 583
A method should have only one exit point, and that should be the last statement in the method576
Found 'DD'-anomaly for variable 'corpusProbability' (lines '610'-'626').610 - 626
Found 'DU'-anomaly for variable 'corpusProbability' (lines '610'-'642').610 - 642
Found 'DD'-anomaly for variable 'corpusProbability' (lines '610'-'610').610
Found 'DD'-anomaly for variable 'isTokenDegeneratedFound' (lines '612'-'624').612 - 624
Found 'DD'-anomaly for variable 'tokenDegenerated' (lines '616'-'619').616 - 619
Found 'DU'-anomaly for variable 'tokenDegenerated' (lines '616'-'642').616 - 642
Found 'DD'-anomaly for variable 'tokenDegenerated' (lines '616'-'616').616
Found 'DD'-anomaly for variable 'isTokenDegeneratedFound' (lines '624'-'624').624
Found 'DD'-anomaly for variable 'corpusProbability' (lines '626'-'626').626
Found 'DU'-anomaly for variable 'corpusProbability' (lines '626'-'642').626 - 642
Found 'DD'-anomaly for variable 'corpusProbability' (lines '626'-'610').626 - 610
Found 'DU'-anomaly for variable 'workCorpus' (lines '697'-'721').697 - 721
Found 'DD'-anomaly for variable 'theDoubleValue' (lines '709'-'713').709 - 713
A method should have only one exit point, and that should be the last statement in the method725
Found 'DU'-anomaly for variable 'c' (lines '728'-'736').728 - 736
A method should have only one exit point, and that should be the last statement in the method732
A method should have only one exit point, and that should be the last statement in the method741

org/apache/james/util/CharTerminatedInputStream.java

ViolationLine
Perhaps 'endFound' could be replaced by a local variable.57
It is a good practice to call super() in a constructor68 - 82
A method should have only one exit point, and that should be the last statement in the method94
A method should have only one exit point, and that should be the last statement in the method105
A method should have only one exit point, and that should be the last statement in the method116
A method should have only one exit point, and that should be the last statement in the method131
A method should have only one exit point, and that should be the last statement in the method139

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

ViolationLine
Perhaps 'theRunnerFactory' could be replaced by a local variable.81
It is a good practice to call super() in a constructor121 - 131
Assigning an Object to null is a code smell. Consider refactoring.159
Avoid empty catch blocks163 - 169
Assigning an Object to null is a code smell. Consider refactoring.181
Assigning an Object to null is a code smell. Consider refactoring.191
Found 'DD'-anomaly for variable 'runner' (lines '191'-'189').191 - 189
Found 'DU'-anomaly for variable 'runner' (lines '191'-'198').191 - 198
Found 'DU'-anomaly for variable 'ioExceptionCount' (lines '253'-'342').253 - 342
Avoid empty catch blocks256 - 258
Found 'DD'-anomaly for variable 'clientSocket' (lines '270'-'272').270 - 272
Found 'DU'-anomaly for variable 'clientSocket' (lines '272'-'342').272 - 342
Avoid empty catch blocks303 - 305
Avoid empty catch blocks324 - 326
Assigning an Object to null is a code smell. Consider refactoring.338
Avoid unnecessary constructors - the compiler will generate these for you369 - 370
It is a good practice to call super() in a constructor369 - 370
Assigning an Object to null is a code smell. Consider refactoring.385
Found 'DD'-anomaly for variable 'handler' (lines '410'-'414').410 - 414
Found 'DD'-anomaly for variable 'connectionString' (lines '415'-'417').415 - 417
Found 'DU'-anomaly for variable 'connectionString' (lines '415'-'466').415 - 466
Assigning an Object to null is a code smell. Consider refactoring.442
Assigning an Object to null is a code smell. Consider refactoring.447
Assigning an Object to null is a code smell. Consider refactoring.457
Found 'DU'-anomaly for variable 'handler' (lines '457'-'466').457 - 466
A method should have only one exit point, and that should be the last statement in the method475
Each class should declare at least one constructor494 - 517
Found 'DU'-anomaly for variable 'object' (lines '514'-'516').514 - 516
Avoid unnecessary return statements515

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

ViolationLine
Each class should declare at least one constructor40 - 257
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '82'-'82').82
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '86'-'105').86 - 105
Found 'DU'-anomaly for variable 'tearDown' (lines '239'-'246').239 - 246

org/apache/james/util/CRLFTerminatedReader.java

ViolationLine
Use explicit scoping instead of the default package private level64
It is a good practice to call super() in a constructor66 - 68
Perhaps 'lineBuffer' could be replaced by a local variable.74
Perhaps 'EOF' could be replaced by a local variable.75 - 78
Perhaps 'tainted' could be replaced by a local variable.80
Found 'UR'-anomaly for variable 'pos' (lines '96'-'96').96
Found 'DU'-anomaly for variable 'cr_just_received' (lines '105'-'148').105 - 148
Found 'DU'-anomaly for variable 'cr_just_received' (lines '116'-'148').116 - 148
Found 'DD'-anomaly for variable 'cr_just_received' (lines '116'-'143').116 - 143
A method should have only one exit point, and that should be the last statement in the method118
Found 'DU'-anomaly for variable 'pos' (lines '129'-'148').129 - 148
A method should have only one exit point, and that should be the last statement in the method133
Found 'DU'-anomaly for variable 'cr_just_received' (lines '143'-'148').143 - 148
Found 'DD'-anomaly for variable 'cbuf' (lines '158'-'161').158 - 161
Found 'DU'-anomaly for variable 'off' (lines '158'-'163').158 - 163
Found 'DU'-anomaly for variable 'cbuf' (lines '158'-'163').158 - 163
Found 'DD'-anomaly for variable 'cbuf' (lines '161'-'161').161
Found 'DU'-anomaly for variable 'cbuf' (lines '161'-'163').161 - 163

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

ViolationLine
Each class should declare at least one constructor79 - 220
Use explicit scoping instead of the default package private level84
Found 'DU'-anomaly for variable 'dburl' (lines '98'-'200').98 - 200
Found 'DU'-anomaly for variable 'dburl' (lines '98'-'200').98 - 200
Found 'DU'-anomaly for variable 'dburl' (lines '98'-'200').98 - 200
Found 'DU'-anomaly for variable 'user' (lines '99'-'200').99 - 200
Found 'DU'-anomaly for variable 'user' (lines '99'-'200').99 - 200
Found 'DU'-anomaly for variable 'user' (lines '99'-'200').99 - 200
Found 'DU'-anomaly for variable 'password' (lines '100'-'200').100 - 200
Found 'DU'-anomaly for variable 'password' (lines '100'-'200').100 - 200
Found 'DU'-anomaly for variable 'password' (lines '100'-'200').100 - 200
This statement may have some unnecessary parentheses109

org/apache/james/util/DebugInputStream.java

ViolationLine
Use explicit scoping instead of the default package private level31
It is a good practice to call super() in a constructor38 - 40

org/apache/james/util/DotStuffingInputStream.java

ViolationLine
Found 'DU'-anomaly for variable 'len' (lines '66'-'95').66 - 95
Found 'DU'-anomaly for variable 'off' (lines '66'-'95').66 - 95
A method should have only one exit point, and that should be the last statement in the method73
A method should have only one exit point, and that should be the last statement in the method78
Found 'DU'-anomaly for variable 'b' (lines '80'-'95').80 - 95
Found 'DU'-anomaly for variable 'b' (lines '90'-'95').90 - 95

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 method54

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

ViolationLine
Each class should declare at least one constructor39 - 46

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

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

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

ViolationLine
Avoid empty catch blocks143 - 145
Avoid empty catch blocks165 - 167
Avoid empty catch blocks186 - 188
Avoid empty catch blocks207 - 209
Found 'DU'-anomaly for variable 'output' (lines '229'-'240').229 - 240
Found 'DD'-anomaly for variable 'n' (lines '235'-'236').235 - 236
Avoid assignments in operands236
Found 'DU'-anomaly for variable 'n' (lines '236'-'240').236 - 240
Found 'DU'-anomaly for variable 'output' (lines '255'-'264').255 - 264
Found 'DD'-anomaly for variable 'n' (lines '259'-'260').259 - 260
Avoid assignments in operands260
Found 'DU'-anomaly for variable 'n' (lines '260'-'264').260 - 264
Found 'DU'-anomaly for variable 'bufferSize' (lines '704'-'710').704 - 710
A method should have only one exit point, and that should be the last statement in the method732
A method should have only one exit point, and that should be the last statement in the method740

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

ViolationLine
It is a good practice to call super() in a constructor53 - 61

org/apache/james/util/JDBCBayesianAnalyzer.java

ViolationLine
Found 'DU'-anomaly for variable 'errorString' (lines '57'-'57').57
Perhaps 'sqlFile' could be replaced by a local variable.78
Avoid unnecessary constructors - the compiler will generate these for you146 - 147
It is a good practice to call super() in a constructor146 - 147
Found 'DD'-anomaly for variable 'pstmt' (lines '156'-'160').156 - 160
Found 'DD'-anomaly for variable 'rs' (lines '157'-'161').157 - 161
Found 'DU'-anomaly for variable 'token' (lines '165'-'228').165 - 228
Found 'DD'-anomaly for variable 'token' (lines '165'-'165').165
Avoid empty catch blocks213 - 214
Assigning an Object to null is a code smell. Consider refactoring.216
Found 'DU'-anomaly for variable 'rs' (lines '216'-'228').216 - 228
Avoid empty catch blocks222 - 223
Assigning an Object to null is a code smell. Consider refactoring.225
Found 'DU'-anomaly for variable 'pstmt' (lines '225'-'228').225 - 228
Found 'DD'-anomaly for variable 'init' (lines '260'-'265').260 - 265
Found 'DD'-anomaly for variable 'update' (lines '261'-'266').261 - 266
Avoid empty catch blocks279 - 280
Avoid empty catch blocks285 - 286
Found 'DD'-anomaly for variable 'insert' (lines '293'-'298').293 - 298
Found 'DD'-anomaly for variable 'update' (lines '294'-'301').294 - 301
Avoid empty catch blocks324 - 325
Assigning an Object to null is a code smell. Consider refactoring.327
Found 'DU'-anomaly for variable 'insert' (lines '327'-'339').327 - 339
Avoid empty catch blocks333 - 334
Assigning an Object to null is a code smell. Consider refactoring.336
Found 'DU'-anomaly for variable 'update' (lines '336'-'339').336 - 339
Found 'DU'-anomaly for variable 'dbMetaData' (lines '365'-'383').365 - 383
Found 'DD'-anomaly for variable 'dbUpdated' (lines '369'-'371').369 - 371
Found 'DD'-anomaly for variable 'dbUpdated' (lines '371'-'373').371 - 373
Found 'DD'-anomaly for variable 'dbUpdated' (lines '373'-'375').373 - 375
Found 'DU'-anomaly for variable 'dbUpdated' (lines '380'-'383').380 - 383
Found 'DU'-anomaly for variable 'createSqlStringName' (lines '385'-'417').385 - 417
A method should have only one exit point, and that should be the last statement in the method392
Found 'DD'-anomaly for variable 'createStatement' (lines '395'-'398').395 - 398
Found 'DD'-anomaly for variable 'logBuffer' (lines '402'-'403').402 - 403

org/apache/james/util/JDBCUtil.java

ViolationLine
Each class should declare at least one constructor36 - 195
Found 'DU'-anomaly for variable 'errorString' (lines '44'-'44').44
Avoid empty catch blocks190 - 192

org/apache/james/util/Lock.java

ViolationLine
Each class should declare at least one constructor26 - 111
A method should have only one exit point, and that should be the last statement in the method52
A method should have only one exit point, and that should be the last statement in the method72
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 method93
A method should have only one exit point, and that should be the last statement in the method96

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

ViolationLine
Each class should declare at least one constructor25 - 345

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you40 - 43
Found 'DD'-anomaly for variable 'content' (lines '71'-'74').71 - 74
Avoid empty catch blocks76 - 79
Found 'DU'-anomaly for variable 'aDataSource' (lines '90'-'91').90 - 91
Found 'DU'-anomaly for variable 'aDataSource' (lines '97'-'104').97 - 104
Found 'DD'-anomaly for variable 'content' (lines '100'-'102').100 - 102
Found 'DD'-anomaly for variable 'dataFlavor' (lines '121'-'122').121 - 122
Avoid assignments in operands122
Found 'DU'-anomaly for variable 'dataFlavor' (lines '122'-'128').122 - 128
A method should have only one exit point, and that should be the last statement in the method125

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you52 - 55
Found 'UR'-anomaly for variable 'encoding' (lines '69'-'69').69
Found 'DD'-anomaly for variable 'reader' (lines '73'-'78').73 - 78
Found 'DD'-anomaly for variable 'content' (lines '75'-'83').75 - 83
Avoid empty catch blocks100 - 103
Found 'UR'-anomaly for variable 'encoding' (lines '112'-'112').112
Found 'DU'-anomaly for variable 'aMimeType' (lines '112'-'132').112 - 132
Found 'DD'-anomaly for variable 'writer' (lines '120'-'123').120 - 123
Found 'DD'-anomaly for variable 'characterSet' (lines '136'-'139').136 - 139
Avoid empty catch blocks141 - 144

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you40 - 43
Found 'DU'-anomaly for variable 'aMimeType' (lines '58'-'72').58 - 72

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

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

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

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

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you28 - 31

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you29 - 32

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

ViolationLine
Each class should declare at least one constructor60 - 542
Perhaps 'connCreationsInProgress' could be replaced by a local variable.75
Perhaps 'connErrorMessage' could be replaced by a local variable.77
Perhaps 'connLastCreated' could be replaced by a local variable.79
Perhaps 'connectionCount' could be replaced by a local variable.81
Perhaps 'jdbcDriver' could be replaced by a local variable.83
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 method172
Found 'DD'-anomaly for variable 'attempts' (lines '193'-'127').193 - 127
Found 'UR'-anomaly for variable 'exceptionMessage' (lines '221'-'221').221
Assigning an Object to null is a code smell. Consider refactoring.298
Found 'DD'-anomaly for variable 'stmt' (lines '320'-'322').320 - 322
Avoid empty catch blocks329 - 331
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 method362
Found 'UR'-anomaly for variable 'entry' (lines '375'-'375').375
Found 'DU'-anomaly for variable 'entry' (lines '467'-'526').467 - 526
Found 'DD'-anomaly for variable 'entry' (lines '467'-'500').467 - 500
A method should have only one exit point, and that should be the last statement in the method471
A method should have only one exit point, and that should be the last statement in the method479
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 method510
Avoid empty catch blocks537 - 538

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 method75
Do not use if statements that are always true or always false82
Assigning an Object to null is a code smell. Consider refactoring.112
Found 'DU'-anomaly for variable 'holdability' (lines '345'-'349').345 - 349
Found 'DU'-anomaly for variable 'savepoint' (lines '363'-'367').363 - 367
Found 'DU'-anomaly for variable 'savepoint' (lines '369'-'373').369 - 373
Found 'DU'-anomaly for variable 'savepoint' (lines '375'-'379').375 - 379
Found 'DU'-anomaly for variable 'resulSetType' (lines '381'-'387').381 - 387
Found 'DU'-anomaly for variable 'resultSetHoldability' (lines '381'-'387').381 - 387
Found 'DU'-anomaly for variable 'resultSetConcurrency' (lines '381'-'387').381 - 387
Found 'DU'-anomaly for variable 'resulSetType' (lines '389'-'396').389 - 396
Found 'DU'-anomaly for variable 'resultSetHoldability' (lines '389'-'396').389 - 396
Found 'DU'-anomaly for variable 'resultSetConcurrency' (lines '389'-'396').389 - 396
Found 'DU'-anomaly for variable 'sql' (lines '389'-'396').389 - 396
Found 'DU'-anomaly for variable 'resulSetType' (lines '398'-'405').398 - 405
Found 'DU'-anomaly for variable 'resultSetHoldability' (lines '398'-'405').398 - 405
Found 'DU'-anomaly for variable 'resultSetConcurrency' (lines '398'-'405').398 - 405
Found 'DU'-anomaly for variable 'sql' (lines '398'-'405').398 - 405
Found 'DU'-anomaly for variable 'autoGeneratedKeys' (lines '407'-'412').407 - 412
Found 'DU'-anomaly for variable 'sql' (lines '407'-'412').407 - 412
Found 'DU'-anomaly for variable 'sql' (lines '414'-'419').414 - 419
Found 'DU'-anomaly for variable 'columnIndexes' (lines '414'-'419').414 - 419
Found 'DU'-anomaly for variable 'columnNames' (lines '421'-'426').421 - 426
Found 'DU'-anomaly for variable 'sql' (lines '421'-'426').421 - 426

org/apache/james/util/NetMatcher.java

ViolationLine
Found 'DD'-anomaly for variable 'ip' (lines '61'-'65').61 - 65
Found 'DU'-anomaly for variable 'ip' (lines '82'-'92').82 - 92
Found 'DU'-anomaly for variable 's' (lines '112'-'112').112
A method should have only one exit point, and that should be the last statement in the method175 - 181
Avoid empty catch blocks183
Found 'DU'-anomaly for variable 'srcb' (lines '206'-'212').206 - 212
Found 'DD'-anomaly for variable 'octets' (lines '207'-'209').207 - 209
Found 'DD'-anomaly for variable 'octets' (lines '209'-'209').209
Assigning an Object to null is a code smell. Consider refactoring.240
Found 'DU'-anomaly for variable 'ip' (lines '244'-'263').244 - 263
Found 'DD'-anomaly for variable 'addr' (lines '246'-'248').246 - 248
Avoid empty catch blocks249 - 250
Avoid empty catch blocks250 - 251

org/apache/james/util/SchedulerNotifyInputStream.java

ViolationLine
Use explicit scoping instead of the default package private level36
Use explicit scoping instead of the default package private level41
Use explicit scoping instead of the default package private level46
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 constructor64 - 72

org/apache/james/util/SchedulerNotifyOutputStream.java

ViolationLine
Use explicit scoping instead of the default package private level36
Use explicit scoping instead of the default package private level41
Use explicit scoping instead of the default package private level46
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 constructor58 - 66

org/apache/james/util/SqlResources.java

ViolationLine
Each class should declare at least one constructor46 - 371
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '85'-'85').85
Found 'UR'-anomaly for variable 'sqlMap' (lines '85'-'220').85 - 220
Found 'DD'-anomaly for variable 'dbProduct' (lines '98'-'100').98 - 100
Assigning an Object to null is a code smell. Consider refactoring.101
Found 'DD'-anomaly for variable 'sectionElement' (lines '120'-'123').120 - 123
Found 'DD'-anomaly for variable 'found' (lines '121'-'126').121 - 126
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '132'-'226').132 - 226
Found 'DD'-anomaly for variable 'sqlString' (lines '194'-'207').194 - 207
Found 'DU'-anomaly for variable 'dbProductName' (lines '245'-'266').245 - 266
Found 'DD'-anomaly for variable 'dbMatchName' (lines '252'-'252').252
Found 'DU'-anomaly for variable 'dbMatchName' (lines '252'-'266').252 - 266
A method should have only one exit point, and that should be the last statement in the method262
Found 'DU'-anomaly for variable 'dbOptionsMap' (lines '277'-'292').277 - 292
Found 'DU'-anomaly for variable 'dbProduct' (lines '277'-'292').277 - 292
Found 'DU'-anomaly for variable 'find_length' (lines '305'-'321').305 - 321
Found 'DU'-anomaly for variable 'replace_length' (lines '306'-'321').306 - 321
Found 'DU'-anomaly for variable 'outputOffset' (lines '310'-'321').310 - 321
Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '345'-'345').345
Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '350'-'358').350 - 358

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

ViolationLine
Use explicit scoping instead of the default package private level49
It is a good practice to call super() in a constructor56 - 64

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

ViolationLine
Use explicit scoping instead of the default package private level34
Use explicit scoping instead of the default package private level44
Use explicit scoping instead of the default package private level49
It is a good practice to call super() in a constructor51 - 59

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

ViolationLine
It is a good practice to call super() in a constructor83 - 93
Assigning an Object to null is a code smell. Consider refactoring.151
Assigning an Object to null is a code smell. Consider refactoring.173
Assigning an Object to null is a code smell. Consider refactoring.191
Assigning an Object to null is a code smell. Consider refactoring.213
Assigning an Object to null is a code smell. Consider refactoring.217

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

ViolationLine
It is a good practice to call super() in a constructor54 - 60

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

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

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/mailet/dates/RFC977DateFormat.java

ViolationLine
Found 'DD'-anomaly for variable 'source' (lines '72'-'73').72 - 73
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 method135
A method should have only one exit point, and that should be the last statement in the method138

org/apache/mailet/dates/SimplifiedDateFormat.java

ViolationLine
Avoid modifiers which are implied by the context42
Found 'DU'-anomaly for variable 'd' (lines '42'-'42').42
Avoid modifiers which are implied by the context53
Found 'DU'-anomaly for variable 'source' (lines '53'-'53').53
Avoid modifiers which are implied by the context59
Found 'DU'-anomaly for variable 'zone' (lines '59'-'59').59
Avoid modifiers which are implied by the context65
Avoid modifiers which are implied by the context75
Found 'DU'-anomaly for variable 'lenient' (lines '75'-'75').75
Avoid modifiers which are implied by the context81

org/apache/mailet/dates/SynchronizedDateFormat.java

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

org/apache/mailet/GenericMailet.java

ViolationLine
Each class should declare at least one constructor34 - 210
Found 'DU'-anomaly for variable 'defValue' (lines '71'-'78').71 - 78
A method should have only one exit point, and that should be the last statement in the method74
Found 'DU'-anomaly for variable 'mail' (lines '209'-'209').209

org/apache/mailet/GenericMatcher.java

ViolationLine
Each class should declare at least one constructor34 - 176
Use explicit scoping instead of the default package private level35
Found 'DU'-anomaly for variable 'mail' (lines '175'-'175').175

org/apache/mailet/GenericRecipientMatcher.java

ViolationLine
Each class should declare at least one constructor33 - 62
Found 'DU'-anomaly for variable 'recipient' (lines '61'-'61').61

org/apache/mailet/Mail.java

ViolationLine
Found 'DU'-anomaly for variable 'newName' (lines '53'-'53').53
Found 'DU'-anomaly for variable 'recipients' (lines '72'-'72').72
Found 'DU'-anomaly for variable 'msg' (lines '108'-'108').108
Found 'DU'-anomaly for variable 'message' (lines '114'-'114').114
Found 'DU'-anomaly for variable 'state' (lines '120'-'120').120
Found 'DU'-anomaly for variable 'name' (lines '138'-'138').138
Found 'DU'-anomaly for variable 'name' (lines '164'-'164').164
Found 'DU'-anomaly for variable 'object' (lines '187'-'187').187
Found 'DU'-anomaly for variable 'name' (lines '187'-'187').187
Found 'DU'-anomaly for variable 'lastUpdated' (lines '202'-'202').202

org/apache/mailet/MailAddress.java

ViolationLine
These nested if statements could be combined79 - 84
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 method221
A method should have only one exit point, and that should be the last statement in the method230
A method should have only one exit point, and that should be the last statement in the method233
A method should have only one exit point, and that should be the last statement in the method236
Found 'DU'-anomaly for variable 'address' (lines '254'-'289').254 - 289
Found 'DU'-anomaly for variable 'address' (lines '291'-'342').291 - 342
Found 'DD'-anomaly for variable 'lastCharDot' (lines '294'-'308').294 - 308
Found 'DD'-anomaly for variable 'lastCharDot' (lines '294'-'312').294 - 312
Found 'DD'-anomaly for variable 'lastCharDot' (lines '294'-'335').294 - 335
Found 'DD'-anomaly for variable 'lastCharDot' (lines '308'-'308').308
Found 'DD'-anomaly for variable 'lastCharDot' (lines '308'-'312').308 - 312
Found 'DD'-anomaly for variable 'lastCharDot' (lines '308'-'335').308 - 335
Found 'DD'-anomaly for variable 'lastCharDot' (lines '312'-'308').312 - 308
Found 'DD'-anomaly for variable 'lastCharDot' (lines '312'-'312').312
Found 'DD'-anomaly for variable 'lastCharDot' (lines '312'-'335').312 - 335
Found 'DD'-anomaly for variable 'lastCharDot' (lines '335'-'308').335 - 308
Found 'DD'-anomaly for variable 'lastCharDot' (lines '335'-'312').335 - 312
Found 'DD'-anomaly for variable 'lastCharDot' (lines '335'-'335').335
Found 'DU'-anomaly for variable 'address' (lines '344'-'365').344 - 365
Found 'DU'-anomaly for variable 'address' (lines '430'-'470').430 - 470

org/apache/mailet/Mailet.java

ViolationLine
Found 'DU'-anomaly for variable 'config' (lines '98'-'98').98
Found 'DU'-anomaly for variable 'mail' (lines '118'-'118').118

org/apache/mailet/MailetConfig.java

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

org/apache/mailet/MailetContext.java

ViolationLine
Found 'DU'-anomaly for variable 'message' (lines '48'-'48').48
Found 'DU'-anomaly for variable 'mail' (lines '48'-'48').48
Found 'DU'-anomaly for variable 'bouncer' (lines '58'-'58').58
Found 'DU'-anomaly for variable 'message' (lines '58'-'58').58
Found 'DU'-anomaly for variable 'mail' (lines '58'-'58').58
Found 'DU'-anomaly for variable 'host' (lines '69'-'69').69
Found 'DU'-anomaly for variable 'name' (lines '93'-'93').93
Found 'DU'-anomaly for variable 'serverName' (lines '142'-'142').142
Found 'DU'-anomaly for variable 'userAccount' (lines '150'-'150').150
Found 'DU'-anomaly for variable 'message' (lines '159'-'159').159
Found 'DU'-anomaly for variable 'message' (lines '168'-'168').168
Found 'DU'-anomaly for variable 't' (lines '168'-'168').168
Found 'DU'-anomaly for variable 'name' (lines '177'-'177').177
Found 'DU'-anomaly for variable 'msg' (lines '187'-'188').187 - 188
Found 'DU'-anomaly for variable 'sender' (lines '199'-'200').199 - 200
Found 'DU'-anomaly for variable 'recipients' (lines '199'-'200').199 - 200
Found 'DU'-anomaly for variable 'msg' (lines '199'-'200').199 - 200
Found 'DU'-anomaly for variable 'sender' (lines '213'-'214').213 - 214
Found 'DU'-anomaly for variable 'recipients' (lines '213'-'214').213 - 214
Found 'DU'-anomaly for variable 'msg' (lines '213'-'214').213 - 214
Found 'DU'-anomaly for variable 'state' (lines '213'-'214').213 - 214
Found 'DU'-anomaly for variable 'mail' (lines '224'-'225').224 - 225
Found 'DU'-anomaly for variable 'object' (lines '238'-'238').238
Found 'DU'-anomaly for variable 'name' (lines '238'-'238').238
Found 'DU'-anomaly for variable 'sender' (lines '251'-'252').251 - 252
Found 'DU'-anomaly for variable 'msg' (lines '251'-'252').251 - 252
Found 'DU'-anomaly for variable 'recipient' (lines '251'-'252').251 - 252
Found 'DU'-anomaly for variable 'domainName' (lines '267'-'267').267

org/apache/mailet/Matcher.java

ViolationLine
Found 'DU'-anomaly for variable 'config' (lines '115'-'115').115
Found 'DU'-anomaly for variable 'mail' (lines '136'-'136').136