The following document contains the results of PMD 3.9.
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 28 - 83 |
| Violation | Line |
|---|---|
| Found 'DD'-anomaly for variable 'baseDirectory' (lines '74'-'78'). | 74 - 78 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 52 - 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 method | 385 |
| 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 method | 525 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 47 - 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 method | 241 |
| 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 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 43 - 171 |
| Found 'UR'-anomaly for variable 'error' (lines '95'-'95'). | 95 |
| Found 'DU'-anomaly for variable 'theClassLoader' (lines '102'-'142'). | 102 - 142 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 30 - 99 |
| Violation | Line |
|---|---|
| Avoid unnecessary constructors - the compiler will generate these for you | 55 - 56 |
| Violation | Line |
|---|---|
| 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 method | 269 |
| Avoid empty catch blocks | 270 - 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 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 27 - 101 |
| Violation | Line |
|---|---|
| Assigning an Object to null is a code smell. Consider refactoring. | 77 |
| Assigning an Object to null is a code smell. Consider refactoring. | 166 |
| Violation | Line |
|---|---|
| Use explicit scoping instead of the default package private level | 49 |
| Use explicit scoping instead of the default package private level | 54 |
| It is a good practice to call super() in a constructor | 67 - 117 |
| Avoid assignments in operands | 76 |
| Assigning an Object to null is a code smell. Consider refactoring. | 89 |
| Avoid empty catch blocks | 90 - 92 |
| Assigning an Object to null is a code smell. Consider refactoring. | 96 |
| Avoid empty catch blocks | 105 - 107 |
| Avoid empty catch blocks | 113 - 115 |
| Avoid empty catch blocks | 156 - 158 |
| Assigning an Object to null is a code smell. Consider refactoring. | 159 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 30 - 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 operands | 64 |
| Found 'DU'-anomaly for variable 'read' (lines '64'-'78'). | 64 - 78 |
| Avoid empty catch blocks | 72 - 75 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 41 - 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 operands | 165 |
| Found 'DU'-anomaly for variable 'read' (lines '165'-'169'). | 165 - 169 |
| Each class should declare at least one constructor | 220 - 238 |
| Use explicit scoping instead of the default package private level | 221 |
| 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 |
| Violation | Line |
|---|---|
| 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 method | 305 |
| A method should have only one exit point, and that should be the last statement in the method | 308 |
| Avoid empty while statements | 321 - 323 |
| A method should have only one exit point, and that should be the last statement in the method | 324 |
| A method should have only one exit point, and that should be the last statement in the method | 338 |
| 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 method | 512 |
| A method should have only one exit point, and that should be the last statement in the method | 538 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 59 - 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 method | 203 |
| 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 method | 306 |
| A method should have only one exit point, and that should be the last statement in the method | 317 |
| A method should have only one exit point, and that should be the last statement in the method | 320 |
| A method should have only one exit point, and that should be the last statement in the method | 323 |
| A method should have only one exit point, and that should be the last statement in the method | 339 |
| A method should have only one exit point, and that should be the last statement in the method | 346 |
| A method should have only one exit point, and that should be the last statement in the method | 351 |
| 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 constructor | 397 - 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 |
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 30 |
| Violation | Line |
|---|---|
| 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 operands | 279 - 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 method | 283 |
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 113 |
| A method should have only one exit point, and that should be the last statement in the method | 115 |
| Avoid unnecessary constructors - the compiler will generate these for you | 402 - 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 method | 680 |
| 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 method | 790 |
| 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 method | 874 |
| Found 'DD'-anomaly for variable 'session' (lines '903'-'904'). | 903 - 904 |
| Avoid assignments in operands | 904 |
| 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 method | 907 |
| Found 'DU'-anomaly for variable 'properties' (lines '960'-'980'). | 960 - 980 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 44 - 151 |
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 33 |
| Violation | Line |
|---|---|
| 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 blocks | 124 - 127 |
| Avoid unnecessary return statements | 151 |
| 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 operands | 267 |
| 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 method | 270 |
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 304 |
| A method should have only one exit point, and that should be the last statement in the method | 322 |
| A method should have only one exit point, and that should be the last statement in the method | 341 |
| A method should have only one exit point, and that should be the last statement in the method | 347 |
| A method should have only one exit point, and that should be the last statement in the method | 353 |
| A method should have only one exit point, and that should be the last statement in the method | 360 |
| A method should have only one exit point, and that should be the last statement in the method | 367 |
| 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 method | 386 |
| A method should have only one exit point, and that should be the last statement in the method | 391 |
| Avoid unnecessary return statements | 429 |
| Avoid unnecessary return statements | 452 |
| Avoid unnecessary return statements | 479 |
| Avoid unnecessary return statements | 503 |
| Avoid unnecessary return statements | 527 |
| Avoid unnecessary return statements | 549 |
| 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 operands | 781 - 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 method | 995 |
| 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 method | 1053 |
| A method should have only one exit point, and that should be the last statement in the method | 1082 |
| A method should have only one exit point, and that should be the last statement in the method | 1096 |
| A method should have only one exit point, and that should be the last statement in the method | 1113 |
| 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 Strings | 1336 |
| Found 'DU'-anomaly for variable 'mail' (lines '1,363'-'1,375'). | 1363 - 1375 |
| Avoid assignments in operands | 1411 |
| 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 method | 1414 |
| 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 operands | 1502 |
| 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 method | 1505 |
| Avoid assignments in operands | 1527 |
| 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 method | 1530 |
| Found 'DD'-anomaly for variable 'isMaxMessageSizeExceeded' (lines '1,595'-'1,596'). | 1595 - 1596 |
| Avoid assignments in operands | 1596 - 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 method | 1600 |
| A method should have only one exit point, and that should be the last statement in the method | 1620 |
| 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 operands | 1649 |
| 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 method | 1652 |
| Found 'DD'-anomaly for variable 'isInvalid' (lines '1,664'-'1,671'). | 1664 - 1671 |
| Violation | Line |
|---|---|
| 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 operands | 807 |
| 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 method | 810 |
| Found 'DD'-anomaly for variable 'hostName' (lines '862'-'870'). | 862 - 870 |
| Found 'DD'-anomaly for variable 'hostName' (lines '870'-'874'). | 870 - 874 |
| Violation | Line |
|---|---|
| Found 'DD'-anomaly for variable 'value' (lines '516'-'517'). | 516 - 517 |
| Avoid assignments in operands | 517 |
| 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 method | 520 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 31 - 33 |
| Violation | Line |
|---|---|
| Found 'DD'-anomaly for variable 'store' (lines '51'-'67'). | 51 - 67 |
| Found 'DD'-anomaly for variable 'folder' (lines '52'-'78'). | 52 - 78 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 87 - 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 method | 526 |
| 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 method | 728 |
| Found 'DD'-anomaly for variable 'dnsServer' (lines '851'-'853'). | 851 - 853 |
| Violation | Line |
|---|---|
| Found 'DU'-anomaly for variable 'password' (lines '37'-'37'). | 37 |
| Found 'DU'-anomaly for variable 'userName' (lines '37'-'37'). | 37 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 60 - 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 method | 204 |
| A method should have only one exit point, and that should be the last statement in the method | 233 |
| 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 method | 360 |
| Found 'UR'-anomaly for variable 'exceptionBuffer' (lines '400'-'400'). | 400 |
| Found 'DU'-anomaly for variable 'exceptionBuffer' (lines '410'-'417'). | 410 - 417 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 37 - 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 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 47 - 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 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 32 - 140 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 33 - 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 method | 79 |
| Violation | Line |
|---|---|
| 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 method | 41 |
| A method should have only one exit point, and that should be the last statement in the method | 47 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 42 - 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 method | 115 |
| 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 method | 161 |
| Found 'DU'-anomaly for variable 'message' (lines '165'-'214'). | 165 - 214 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 89 - 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 method | 504 |
| A method should have only one exit point, and that should be the last statement in the method | 530 |
| 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 method | 813 |
| 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 method | 1025 |
| 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 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 91 - 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 operands | 196 |
| 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 method | 257 |
| 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 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 49 - 176 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 113 - 868 |
| Use explicit scoping instead of the default package private level | 118 |
| Use explicit scoping instead of the default package private level | 119 |
| Use explicit scoping instead of the default package private level | 120 |
| Use explicit scoping instead of the default package private level | 121 |
| Use explicit scoping instead of the default package private level | 122 |
| Use explicit scoping instead of the default package private level | 123 |
| Use explicit scoping instead of the default package private level | 124 |
| Avoid modifiers which are implied by the context | 150 |
| Avoid modifiers which are implied by the context | 151 |
| 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 operands | 267 |
| 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 method | 277 |
| 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 operands | 295 |
| 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 method | 305 |
| 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 method | 329 |
| 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 method | 391 |
| 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 combined | 433 - 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 method | 582 |
| 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 - |