PMD Results

The following document contains the results of PMD 3.9.

Files

org/apache/james/mime4j/AbstractContentHandler.java

ViolationLine
Each class should declare at least one constructor34 - 115
Found 'DU'-anomaly for variable 'bd' (lines '45'-'46').45 - 46
Found 'DU'-anomaly for variable 'is' (lines '51'-'53').51 - 53
Found 'DU'-anomaly for variable 'bd' (lines '51'-'53').51 - 53
Found 'DU'-anomaly for variable 'is' (lines '76'-'77').76 - 77
Found 'DU'-anomaly for variable 'fieldData' (lines '82'-'83').82 - 83
Found 'DU'-anomaly for variable 'is' (lines '88'-'89').88 - 89
Found 'DU'-anomaly for variable 'is' (lines '112'-'113').112 - 113

org/apache/james/mime4j/BodyDescriptor.java

ViolationLine
Found 'DU'-anomaly for variable 'pFieldValue' (lines '42'-'42').42
Found 'DU'-anomaly for variable 'pFieldName' (lines '42'-'42').42

org/apache/james/mime4j/CloseShieldInputStream.java

ViolationLine
It is a good practice to call super() in a constructor39 - 41
Assigning an Object to null is a code smell. Consider refactoring.68
A method should have only one exit point, and that should be the last statement in the method84
Found 'DU'-anomaly for variable 'readlimit' (lines '91'-'94').91 - 94

org/apache/james/mime4j/ContentHandler.java

ViolationLine
Found 'DU'-anomaly for variable 'fieldData' (lines '129'-'129').129
Found 'DU'-anomaly for variable 'is' (lines '146'-'146').146
Found 'DU'-anomaly for variable 'is' (lines '156'-'156').156
Found 'DU'-anomaly for variable 'bd' (lines '168'-'168').168
Found 'DU'-anomaly for variable 'is' (lines '189'-'190').189 - 190
Found 'DU'-anomaly for variable 'bd' (lines '189'-'190').189 - 190
Found 'DU'-anomaly for variable 'is' (lines '201'-'201').201

org/apache/james/mime4j/Cursor.java

ViolationLine
Avoid modifiers which are implied by the context33
Avoid modifiers which are implied by the context40
Avoid modifiers which are implied by the context48
Avoid modifiers which are implied by the context56
Avoid modifiers which are implied by the context62
Avoid modifiers which are implied by the context70
Avoid modifiers which are implied by the context79
Avoid modifiers which are implied by the context87
Found 'DU'-anomaly for variable 'boundary' (lines '87'-'87').87
Avoid modifiers which are implied by the context94
Avoid modifiers which are implied by the context102
Avoid modifiers which are implied by the context108
Avoid modifiers which are implied by the context114

org/apache/james/mime4j/decoder/Base64InputStream.java

ViolationLine
Perhaps 'done' could be replaced by a local variable.39
It is a good practice to call super() in a constructor41 - 43
A method should have only one exit point, and that should be the last statement in the method58
A method should have only one exit point, and that should be the last statement in the method64
Found 'DU'-anomaly for variable 'data' (lines '75'-'103').75 - 103
Found 'DD'-anomaly for variable 'data' (lines '75'-'95').75 - 95
Found 'DU'-anomaly for variable 'pos' (lines '76'-'103').76 - 103
Found 'DU'-anomaly for variable 'i' (lines '80'-'103').80 - 103
A method should have only one exit point, and that should be the last statement in the method86

org/apache/james/mime4j/decoder/DecoderUtil.java

ViolationLine
Each class should declare at least one constructor37 - 276
Found 'DD'-anomaly for variable 'b' (lines '55'-'56').55 - 56
Avoid assignments in operands56
Found 'DU'-anomaly for variable 'b' (lines '56'-'67').56 - 67
Found 'DD'-anomaly for variable 'b' (lines '84'-'85').84 - 85
Avoid assignments in operands85
Found 'DU'-anomaly for variable 'b' (lines '85'-'96').85 - 96
Found 'DD'-anomaly for variable 'p1' (lines '155'-'170').155 - 170
Found 'DD'-anomaly for variable 'decodedWord' (lines '222'-'227').222 - 227
Found 'DD'-anomaly for variable 'decodedWord' (lines '222'-'239').222 - 239
Found 'DD'-anomaly for variable 'decodedWord' (lines '222'-'246').222 - 246
Found 'DD'-anomaly for variable 'decodedWord' (lines '222'-'255').222 - 255
Found 'DD'-anomaly for variable 'decodedWord' (lines '222'-'257').222 - 257
Found 'DD'-anomaly for variable 'decodedWord' (lines '222'-'259').222 - 259
Found 'DU'-anomaly for variable 'enc' (lines '231'-'275').231 - 275
Found 'DD'-anomaly for variable 'enc' (lines '231'-'231').231
Found 'DU'-anomaly for variable 'encodedWord' (lines '232'-'275').232 - 275
Found 'DD'-anomaly for variable 'encodedWord' (lines '232'-'232').232

org/apache/james/mime4j/decoder/QuotedPrintableInputStream.java

ViolationLine
Use explicit scoping instead of the default package private level39
Use explicit scoping instead of the default package private level40
Perhaps 'state' could be replaced by a local variable.41
It is a good practice to call super() in a constructor43 - 45
A method should have only one exit point, and that should be the last statement in the method59
A method should have only one exit point, and that should be the last statement in the method63
A method should have only one exit point, and that should be the last statement in the method82
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 method99
Found 'DU'-anomaly for variable 'msdChar' (lines '116'-'203').116 - 203
Found 'DD'-anomaly for variable 'msdChar' (lines '116'-'141').116 - 141
Found 'DU'-anomaly for variable 'msdChar' (lines '141'-'203').141 - 203
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 method214

org/apache/james/mime4j/decoder/UnboundedFifoByteBuffer.java

ViolationLine
Found 'DD'-anomaly for variable 'size' (lines '95'-'98').95 - 98
Found 'DD'-anomaly for variable 'size' (lines '95'-'100').95 - 100
Found 'DD'-anomaly for variable 'tmp' (lines '124'-'128').124 - 128
Found 'DD'-anomaly for variable 'tmp' (lines '128'-'128').128
Found 'DD'-anomaly for variable 'index' (lines '192'-'193').192 - 193
Found 'DD'-anomaly for variable 'index' (lines '206'-'207').206 - 207

org/apache/james/mime4j/DefaultBodyDescriptor.java

ViolationLine
Perhaps 'contentTypeSet' could be replaced by a local variable.43
Perhaps 'contentTransferEncSet' could be replaced by a local variable.44
Found 'DD'-anomaly for variable 'name' (lines '75'-'77').75 - 77
Found 'DD'-anomaly for variable 'value' (lines '75'-'82').75 - 82
Found 'DD'-anomaly for variable 'value' (lines '75'-'96').75 - 96
Found 'DU'-anomaly for variable 'value' (lines '75'-'159').75 - 159
Found 'DD'-anomaly for variable 'valid' (lines '116'-'122').116 - 122
Found 'DD'-anomaly for variable 'main' (lines '121'-'127').121 - 127
Assigning an Object to null is a code smell. Consider refactoring.127

org/apache/james/mime4j/EOLConvertingInputStream.java

ViolationLine
Perhaps 'previous' could be replaced by a local variable.43
A method should have only one exit point, and that should be the last statement in the method87

org/apache/james/mime4j/field/address/Address.java

ViolationLine
Each class should declare at least one constructor32 - 52
Use explicit scoping instead of the default package private level41 - 43
Found 'DU'-anomaly for variable 'results' (lines '50'-'50').50

org/apache/james/mime4j/field/address/AddressList.java

ViolationLine
Found 'DD'-anomaly for variable 'groupDetected' (lines '71'-'74').71 - 74
A method should have only one exit point, and that should be the last statement in the method80
Found 'DU'-anomaly for variable 'args' (lines '121'-'139').121 - 139
Found 'DU'-anomaly for variable 'reader' (lines '122'-'139').122 - 139

org/apache/james/mime4j/field/address/Builder.java

ViolationLine
Each class should declare at least one constructor48 - 244
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 method75
Found 'DD'-anomaly for variable 'name' (lines '78'-'84').78 - 84
Found 'DU'-anomaly for variable 'name' (lines '78'-'94').78 - 94
A method should have only one exit point, and that should be the last statement in the method81
A method should have only one exit point, and that should be the last statement in the method115
A method should have only one exit point, and that should be the last statement in the method118
Found 'DD'-anomaly for variable 'name' (lines '133'-'141').133 - 141
Found 'DU'-anomaly for variable 'name' (lines '133'-'147').133 - 147
Found 'DD'-anomaly for variable 'route' (lines '151'-'154').151 - 154
Found 'DU'-anomaly for variable 'route' (lines '151'-'166').151 - 166
Found 'DU'-anomaly for variable 'route' (lines '154'-'166').154 - 166
Avoid empty if statements158
An empty statement (semicolon) not part of a loop158
Found 'DU'-anomaly for variable 'stripSpaces' (lines '192'-'206').192 - 206
Found 'DU'-anomaly for variable 'out' (lines '208'-'213').208 - 213

org/apache/james/mime4j/field/address/Group.java

ViolationLine
It is a good practice to call super() in a constructor37 - 40
Found 'DU'-anomaly for variable 'results' (lines '69'-'72').69 - 72

org/apache/james/mime4j/field/address/Mailbox.java

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

org/apache/james/mime4j/field/address/parser/BaseNode.java

ViolationLine
Each class should declare at least one constructor25 - 30

org/apache/james/mime4j/field/AddressListField.java

ViolationLine
Each class should declare at least one constructor45 - 62
Found 'DD'-anomaly for variable 'addressList' (lines '49'-'52').49 - 52
Found 'DD'-anomaly for variable 'parseException' (lines '50'-'58').50 - 58

org/apache/james/mime4j/field/ContentTransferEncodingField.java

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

org/apache/james/mime4j/field/ContentTypeField.java

ViolationLine
Found 'DU'-anomaly for variable 'parent' (lines '109'-'123').109 - 123
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 method118
These nested if statements could be combined177 - 179
A method should have only one exit point, and that should be the last statement in the method178
Each class should declare at least one constructor205 - 255
Found 'DD'-anomaly for variable 'parseException' (lines '209'-'221').209 - 221
Found 'DD'-anomaly for variable 'mimeType' (lines '210'-'235').210 - 235
Found 'DD'-anomaly for variable 'parseException' (lines '221'-'227').221 - 227
Avoid empty catch blocks251 - 252

org/apache/james/mime4j/field/datetime/DateTime.java

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

org/apache/james/mime4j/field/DateTimeField.java

ViolationLine
Each class should declare at least one constructor49 - 71
Found 'DD'-anomaly for variable 'date' (lines '53'-'57').53 - 57
Found 'DD'-anomaly for variable 'parseException' (lines '54'-'67').54 - 67

org/apache/james/mime4j/field/DefaultFieldParser.java

ViolationLine
It is a good practice to call super() in a constructor20 - 44

org/apache/james/mime4j/field/DelegatingFieldParser.java

ViolationLine
Each class should declare at least one constructor21 - 47
Perhaps 'defaultParser' could be replaced by a local variable.24
A method should have only one exit point, and that should be the last statement in the method38

org/apache/james/mime4j/field/FieldParser.java

ViolationLine
Found 'DU'-anomaly for variable 'name' (lines '20'-'20').20
Found 'DU'-anomaly for variable 'raw' (lines '20'-'20').20
Found 'DU'-anomaly for variable 'body' (lines '20'-'20').20

org/apache/james/mime4j/field/MailboxField.java

ViolationLine
Each class should declare at least one constructor47 - 67
Found 'DD'-anomaly for variable 'mailbox' (lines '51'-'56').51 - 56
Found 'DD'-anomaly for variable 'parseException' (lines '52'-'63').52 - 63

org/apache/james/mime4j/field/MailboxListField.java

ViolationLine
Each class should declare at least one constructor47 - 64
Found 'DD'-anomaly for variable 'mailboxList' (lines '51'-'54').51 - 54
Found 'DD'-anomaly for variable 'parseException' (lines '52'-'60').52 - 60

org/apache/james/mime4j/field/UnstructuredField.java

ViolationLine
Each class should declare at least one constructor43 - 48

org/apache/james/mime4j/message/AbstractBody.java

ViolationLine
Each class should declare at least one constructor30 - 47

org/apache/james/mime4j/message/Body.java

ViolationLine
Found 'DU'-anomaly for variable 'parent' (lines '45'-'45').45
Found 'DU'-anomaly for variable 'out' (lines '53'-'53').53

org/apache/james/mime4j/message/BodyPart.java

ViolationLine
Each class should declare at least one constructor32 - 42

org/apache/james/mime4j/message/Entity.java

ViolationLine
Each class should declare at least one constructor35 - 170
Found 'DU'-anomaly for variable 'out' (lines '169'-'169').169

org/apache/james/mime4j/message/Header.java

ViolationLine
Found 'UR'-anomaly for variable 'ise' (lines '64'-'64').64
Found 'UR'-anomaly for variable 'ise' (lines '64'-'64').64
A method should have only one exit point, and that should be the last statement in the method119
Found 'UR'-anomaly for variable 'fs' (lines '167'-'167').167
Found 'DD'-anomaly for variable 'charset' (lines '168'-'172').168 - 172
Found 'DD'-anomaly for variable 'charset' (lines '168'-'174').168 - 174
Found 'DD'-anomaly for variable 'charset' (lines '168'-'177').168 - 177

org/apache/james/mime4j/message/Message.java

ViolationLine
It is a good practice to call super() in a constructor56 - 57
It is a good practice to call super() in a constructor66 - 77
Avoid unnecessary constructors - the compiler will generate these for you110 - 111
Found 'DU'-anomaly for variable 'bd' (lines '171'-'178').171 - 178
Found 'DD'-anomaly for variable 'is' (lines '183'-'188').183 - 188
Found 'DD'-anomaly for variable 'is' (lines '183'-'190').183 - 190
Found 'DD'-anomaly for variable 'body' (lines '193'-'195').193 - 195
Found 'DD'-anomaly for variable 'body' (lines '193'-'197').193 - 197
Avoid assignments in operands236
Found 'DU'-anomaly for variable 'b' (lines '236'-'240').236 - 240
Avoid assignments in operands249
Found 'DU'-anomaly for variable 'b' (lines '249'-'253').249 - 253
Found 'DU'-anomaly for variable 'is' (lines '260'-'262').260 - 262

org/apache/james/mime4j/message/Multipart.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you58 - 59
Found 'DD'-anomaly for variable 'charset' (lines '186'-'189').186 - 189
Found 'DD'-anomaly for variable 'charset' (lines '186'-'191').186 - 191
Found 'DD'-anomaly for variable 'charset' (lines '186'-'194').186 - 194

org/apache/james/mime4j/message/SimpleContentHandler.java

ViolationLine
Each class should declare at least one constructor39 - 102
Found 'DU'-anomaly for variable 'header' (lines '44'-'44').44
Found 'DU'-anomaly for variable 'is' (lines '58'-'58').58
Found 'DU'-anomaly for variable 'bd' (lines '58'-'58').58
Assigning an Object to null is a code smell. Consider refactoring.84

org/apache/james/mime4j/message/TempFileBinaryBody.java

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

org/apache/james/mime4j/message/TempFileTextBody.java

ViolationLine
It is a good practice to call super() in a constructor54 - 65
Found 'DD'-anomaly for variable 'javaCharset' (lines '71'-'73').71 - 73

org/apache/james/mime4j/MimeBoundaryInputStream.java

ViolationLine
Perhaps 'first' could be replaced by a local variable.41
It is a good practice to call super() in a constructor51 - 70
Avoid empty while statements109 - 110
A method should have only one exit point, and that should be the last statement in the method118
A method should have only one exit point, and that should be the last statement in the method124
These nested if statements could be combined132 - 134
A method should have only one exit point, and that should be the last statement in the method133
A method should have only one exit point, and that should be the last statement in the method158
Use bitwise inversion to invert boolean values167
Found 'DU'-anomaly for variable 'prev' (lines '172'-'183').172 - 183

org/apache/james/mime4j/MimeStreamParser.java

ViolationLine
Each class should declare at least one constructor45 - 160
Found 'UR'-anomaly for variable 'state' (lines '56'-'58').56 - 58
Found 'DU'-anomaly for variable 'raw' (lines '130'-'132').130 - 132

org/apache/james/mime4j/MimeTokenStream.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method293
A method should have only one exit point, and that should be the last statement in the method295
A method should have only one exit point, and that should be the last statement in the method297
Each class should declare at least one constructor307 - 311
Use explicit scoping instead of the default package private level308
Use explicit scoping instead of the default package private level309
Use explicit scoping instead of the default package private level310
It is a good practice to call super() in a constructor315 - 318
Use explicit scoping instead of the default package private level319 - 322
Use explicit scoping instead of the default package private level323 - 325
Perhaps 'pos' could be replaced by a local variable.333
Use explicit scoping instead of the default package private level337
It is a good practice to call super() in a constructor339 - 344
A method should have only one exit point, and that should be the last statement in the method366
Use explicit scoping instead of the default package private level373 - 435
Found 'DD'-anomaly for variable 'nextCursor' (lines '391'-'395').391 - 395
Found 'DD'-anomaly for variable 'nextCursor' (lines '391'-'398').391 - 398
A method should have only one exit point, and that should be the last statement in the method401
A method should have only one exit point, and that should be the last statement in the method416
A method should have only one exit point, and that should be the last statement in the method418
Use explicit scoping instead of the default package private level437 - 448
A method should have only one exit point, and that should be the last statement in the method442
Found 'DD'-anomaly for variable 'curr' (lines '454'-'456').454 - 456
Found 'DU'-anomaly for variable 'prev' (lines '455'-'471').455 - 471
Avoid assignments in operands456
Found 'DD'-anomaly for variable 'valid' (lines '494'-'496').494 - 496
Found 'DU'-anomaly for variable 'valid' (lines '494'-'523').494 - 523
Found 'DD'-anomaly for variable 'valid' (lines '494'-'494').494
Found 'DD'-anomaly for variable 'valid' (lines '496'-'500').496 - 500
A method should have only one exit point, and that should be the last statement in the method510
Found 'DU'-anomaly for variable 'parent' (lines '570'-'583').570 - 583
Found 'DU'-anomaly for variable 'cursor' (lines '570'-'583').570 - 583
Found 'DD'-anomaly for variable 'preamble' (lines '734'-'736').734 - 736
Found 'DU'-anomaly for variable 'event' (lines '756'-'760').756 - 760
Found 'DU'-anomaly for variable 'event' (lines '767'-'771').767 - 771
A method should have only one exit point, and that should be the last statement in the method861
Assigning an Object to null is a code smell. Consider refactoring.865

org/apache/james/mime4j/RootInputStream.java

ViolationLine
It is a good practice to call super() in a constructor46 - 48
A method should have only one exit point, and that should be the last statement in the method75
Found 'DU'-anomaly for variable 'len' (lines '90'-'103').90 - 103
Found 'DU'-anomaly for variable 'b' (lines '90'-'103').90 - 103
Found 'DU'-anomaly for variable 'off' (lines '90'-'103').90 - 103
A method should have only one exit point, and that should be the last statement in the method92

org/apache/james/mime4j/StreamCursor.java

ViolationLine
Use explicit scoping instead of the default package private level33
Use explicit scoping instead of the default package private level34
Use explicit scoping instead of the default package private level35
Use explicit scoping instead of the default package private level36
Assigning an Object to null is a code smell. Consider refactoring.102

org/apache/james/mime4j/util/CharsetUtil.java

ViolationLine
Each class should declare at least one constructor791 - 1178
Perhaps 'mime' could be replaced by a local variable.796
Perhaps 'aliases' could be replaced by a local variable.797
Avoid empty catch blocks1026 - 1027
Avoid empty catch blocks1027 - 1028
Avoid empty catch blocks1032 - 1033
Avoid empty catch blocks1033 - 1034
A method should have only one exit point, and that should be the last statement in the method1099
A method should have only one exit point, and that should be the last statement in the method1118
Found 'DU'-anomaly for variable 'defaultCharset' (lines '1,124'-'1,141').1124 - 1141
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 method1134

org/apache/james/mime4j/util/MessageUtils.java

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

org/apache/james/mime4j/util/MimeUtil.java

ViolationLine
Found 'UR'-anomaly for variable 'READY_FOR_NAME' (lines '107'-'107').107
Found 'UR'-anomaly for variable 'ERROR' (lines '107'-'107').107
Found 'UR'-anomaly for variable 'READY_FOR_VALUE' (lines '107'-'107').107
Found 'UR'-anomaly for variable 'VALUE_DONE' (lines '107'-'107').107
Found 'UR'-anomaly for variable 'IN_QUOTED_VALUE' (lines '107'-'107').107
Found 'UR'-anomaly for variable 'IN_NAME' (lines '107'-'107').107
Found 'UR'-anomaly for variable 'IN_VALUE' (lines '107'-'107').107
Assigning an Object to null is a code smell. Consider refactoring.115
Found 'DU'-anomaly for variable 'paramName' (lines '124'-'272').124 - 272
Found 'DD'-anomaly for variable 'paramName' (lines '124'-'153').124 - 153
Found 'DU'-anomaly for variable 'paramValue' (lines '125'-'272').125 - 272
Found 'DD'-anomaly for variable 'paramValue' (lines '125'-'154').125 - 154
Found 'DU'-anomaly for variable 'IN_NAME' (lines '128'-'272').128 - 272
Found 'DU'-anomaly for variable 'READY_FOR_VALUE' (lines '129'-'272').129 - 272
Found 'DU'-anomaly for variable 'IN_QUOTED_VALUE' (lines '131'-'272').131 - 272
Found 'DU'-anomaly for variable 'VALUE_DONE' (lines '132'-'272').132 - 272
Found 'DU'-anomaly for variable 'ERROR' (lines '133'-'272').133 - 272
Found 'DU'-anomaly for variable 'escaped' (lines '136'-'272').136 - 272
Found 'DD'-anomaly for variable 'c' (lines '138'-'138').138
Found 'DU'-anomaly for variable 'c' (lines '138'-'272').138 - 272
Found 'DU'-anomaly for variable 'paramValue' (lines '154'-'272').154 - 272
Found 'DD'-anomaly for variable 'state' (lines '156'-'162').156 - 162
Found 'DD'-anomaly for variable 'state' (lines '156'-'164').156 - 164
Found 'DD'-anomaly for variable 'fallThrough' (lines '173'-'185').173 - 185
Found 'DD'-anomaly for variable 'state' (lines '180'-'202').180 - 202
Found 'DD'-anomaly for variable 'state' (lines '180'-'215').180 - 215
Found 'DD'-anomaly for variable 'state' (lines '180'-'223').180 - 223
Found 'DD'-anomaly for variable 'state' (lines '184'-'202').184 - 202
Found 'DD'-anomaly for variable 'state' (lines '184'-'215').184 - 215
Found 'DD'-anomaly for variable 'state' (lines '184'-'223').184 - 223
Found 'DD'-anomaly for variable 'fallThrough' (lines '194'-'203').194 - 203
Found 'DD'-anomaly for variable 'state' (lines '202'-'215').202 - 215
Found 'DD'-anomaly for variable 'state' (lines '202'-'223').202 - 223
Found 'DU'-anomaly for variable 'escaped' (lines '238'-'272').238 - 272
Use bitwise inversion to invert boolean values247
Found 'DU'-anomaly for variable 'escaped' (lines '254'-'272').254 - 272

org/apache/james/mime4j/util/PartialInputStream.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method41
Found 'DD'-anomaly for variable 'len' (lines '46'-'47').46 - 47
Found 'DD'-anomaly for variable 'n' (lines '51'-'52').51 - 52

org/apache/james/mime4j/util/PositionInputStream.java

ViolationLine
It is a good practice to call super() in a constructor32 - 34

org/apache/james/mime4j/util/SimpleTempStorage.java

ViolationLine
It is a good practice to call super() in a constructor48 - 50
Found 'UR'-anomaly for variable 'p' (lines '52'-'52').52
Found 'DD'-anomaly for variable 'p' (lines '59'-'63').59 - 63
Found 'DD'-anomaly for variable 'count' (lines '60'-'64').60 - 64
Found 'UR'-anomaly for variable 'f' (lines '76'-'76').76
Found 'DD'-anomaly for variable 'f' (lines '86'-'92').86 - 92
Found 'DD'-anomaly for variable 'count' (lines '88'-'93').88 - 93
Found 'DU'-anomaly for variable 'allowInMemory' (lines '149'-'154').149 - 154

org/apache/james/mime4j/util/StringArrayMap.java

ViolationLine
Each class should declare at least one constructor36 - 239
A method should have only one exit point, and that should be the last statement in the method50
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 method56
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 method77
A method should have only one exit point, and that should be the last statement in the method80
A method should have only one exit point, and that should be the last statement in the method99
A method should have only one exit point, and that should be the last statement in the method102 - 115
Assigning an Object to null is a code smell. Consider refactoring.112
Found 'DU'-anomaly for variable 'values' (lines '118'-'136').118 - 136
Found 'DU'-anomaly for variable 'values' (lines '118'-'136').118 - 136
Found 'DU'-anomaly for variable 'values' (lines '118'-'136').118 - 136
Found 'DU'-anomaly for variable 'values' (lines '118'-'136').118 - 136
Found 'DU'-anomaly for variable 'values' (lines '118'-'136').118 - 136
A method should have only one exit point, and that should be the last statement in the method119 - 130
Found 'UR'-anomaly for variable 'o' (lines '154'-'154').154
Found 'DU'-anomaly for variable 'pValue' (lines '154'-'177').154 - 177

org/apache/james/mime4j/util/TempPath.java

ViolationLine
Found 'DU'-anomaly for variable 'prefix' (lines '30'-'30').30
Found 'DU'-anomaly for variable 'suffix' (lines '52'-'52').52
Found 'DU'-anomaly for variable 'prefix' (lines '52'-'52').52
Found 'DU'-anomaly for variable 'allowInMemory' (lines '69'-'70').69 - 70
Found 'DU'-anomaly for variable 'suffix' (lines '69'-'70').69 - 70
Found 'DU'-anomaly for variable 'prefix' (lines '69'-'70').69 - 70

org/apache/james/mime4j/util/TempStorage.java

ViolationLine
Each class should declare at least one constructor29 - 70