Apache Commons logo Commons Imaging™ logo

PMD Results

The following document contains the results of PMD 6.55.0.

Violations By Priority

Priority 1

org/apache/commons/imaging/formats/png/chunks/PngChunkIhdr.java
Rule Violation Line
ConstructorCallsOverridableMethod Overridable method 'getPngColorType' called during object construction 48
org/apache/commons/imaging/formats/png/chunks/PngChunkScal.java
Rule Violation Line
ConstructorCallsOverridableMethod Overridable method 'getUnitSpecifier' called during object construction 34
ConstructorCallsOverridableMethod Overridable method 'getUnitSpecifier' called during object construction 34
ConstructorCallsOverridableMethod Overridable method 'getUnitSpecifier' called during object construction 35

Priority 2

org/apache/commons/imaging/formats/webp/WebPImageParser.java
Rule Violation Line
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 122

Priority 3

org/apache/commons/imaging/ColorTools.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 43221
org/apache/commons/imaging/FormatCompliance.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 41
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 85
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 99
org/apache/commons/imaging/ImageDump.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 3299
org/apache/commons/imaging/ImageInfo.java
Rule Violation Line
DataClass The class 'ImageInfo' is suspected to be a Data Class (WOC=13.636%, NOPA=0, NOAM=18, WMC=25) 29365
org/apache/commons/imaging/Imaging.java
Rule Violation Line
AvoidProtectedMethodInFinalClassNotExtending Avoid protected methods in a final class that doesnt extend anything other than Object. Change to private or package access. 314330
InefficientEmptyStringCheck String.trim().length() == 0 / String.trim().isEmpty() is an inefficient way to validate a blank String. 806
org/apache/commons/imaging/ImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 3198
org/apache/commons/imaging/PixelDensity.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 44
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 46
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 49
org/apache/commons/imaging/bytesource/ByteSource.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 55
org/apache/commons/imaging/bytesource/InputStreamByteSource.java
Rule Violation Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 51
AtLeastOneConstructor Each class should declare at least one constructor 57170
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 68
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 105
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 143
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 172
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 173
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 174
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 175
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 176
CloseResource Ensure that resources like this InputStream object are closed after use 191
org/apache/commons/imaging/common/AllocationRequestException.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 35
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 37
org/apache/commons/imaging/common/Allocator.java
Rule Violation Line
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 27249
org/apache/commons/imaging/common/BasicCParser.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 385
org/apache/commons/imaging/common/BinaryFunctions.java
Rule Violation Line
CompareObjectsWithEquals Use equals() to compare object references. 124
CompareObjectsWithEquals Use equals() to compare object references. 142
CompareObjectsWithEquals Use equals() to compare object references. 161
CompareObjectsWithEquals Use equals() to compare object references. 196
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 255
org/apache/commons/imaging/common/BinaryOutputStream.java
Rule Violation Line
CompareObjectsWithEquals Use equals() to compare object references. 35
CompareObjectsWithEquals Use equals() to compare object references. 38
UnusedFormalParameter Avoid unused constructor parameters such as 'byteOrder'. 52
org/apache/commons/imaging/common/ByteConversions.java
Rule Violation Line
CompareObjectsWithEquals Use equals() to compare object references. 34
CompareObjectsWithEquals Use equals() to compare object references. 75
CompareObjectsWithEquals Use equals() to compare object references. 107
CompareObjectsWithEquals Use equals() to compare object references. 146
CompareObjectsWithEquals Use equals() to compare object references. 174
CompareObjectsWithEquals Use equals() to compare object references. 214
CompareObjectsWithEquals Use equals() to compare object references. 249
CompareObjectsWithEquals Use equals() to compare object references. 277
CompareObjectsWithEquals Use equals() to compare object references. 306
CompareObjectsWithEquals Use equals() to compare object references. 344
CompareObjectsWithEquals Use equals() to compare object references. 391
CompareObjectsWithEquals Use equals() to compare object references. 438
org/apache/commons/imaging/common/GenericImageMetadata.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 22103
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 59
org/apache/commons/imaging/common/RationalNumber.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 35
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 37
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 50
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 173
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 175
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 177
org/apache/commons/imaging/common/RgbBufferedImageFactory.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2236
org/apache/commons/imaging/common/SimpleBufferedImageFactory.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2239
org/apache/commons/imaging/common/XmpImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2841
org/apache/commons/imaging/formats/bmp/BmpHeaderInfo.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 42
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 43
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 44
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 45
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 46
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 47
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 48
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 49
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 50
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 51
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 52
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 55
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 56
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 59
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 60
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 61
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 62
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 63
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 64
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 65
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 66
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 67
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 69
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 71
org/apache/commons/imaging/formats/bmp/BmpImageParser.java
Rule Violation Line
CloseResource Ensure that resources like this BinaryOutputStream object are closed after use 634
org/apache/commons/imaging/formats/bmp/BmpImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2729
org/apache/commons/imaging/formats/bmp/BmpWriterRgb.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2582
org/apache/commons/imaging/formats/dcx/DcxImageParser.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 64
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 66
CloseResource Ensure that resources like this BinaryOutputStream object are closed after use 177
org/apache/commons/imaging/formats/gif/GifImageMetadata.java
Rule Violation Line
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 57
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 58
org/apache/commons/imaging/formats/gif/GifImageMetadataItem.java
Rule Violation Line
DataClass The class 'GifImageMetadataItem' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=4, WMC=7) 2161
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 57
org/apache/commons/imaging/formats/gif/GifImageParser.java
Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 171058
UnusedFormalParameter Avoid unused method parameters such as 'headerInfo'. 252
org/apache/commons/imaging/formats/gif/GifImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2743
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 33
org/apache/commons/imaging/formats/icns/IcnsImageParser.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 40267
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 95
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 97
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 99
org/apache/commons/imaging/formats/icns/IcnsImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2527
org/apache/commons/imaging/formats/icns/IcnsType.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 112
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 114
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 116
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 118
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 120
org/apache/commons/imaging/formats/ico/IcoImageParser.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 227
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 229
org/apache/commons/imaging/formats/ico/IcoImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2527
org/apache/commons/imaging/formats/jpeg/JpegImageMetadata.java
Rule Violation Line
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 221
org/apache/commons/imaging/formats/jpeg/JpegImageParser.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 65943
SwitchDensity A high ratio of statements to labels in a switch statement. Consider refactoring. 480605
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 899
org/apache/commons/imaging/formats/jpeg/JpegImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2527
org/apache/commons/imaging/formats/jpeg/decoder/JpegDecoder.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 48588
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 129
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 130
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 131
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 132
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 133
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 134
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 135
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 136
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 137
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 139
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 141
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 143
SingularField Perhaps 'block' could be replaced by a local variable. 143
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 145
AvoidArrayLoops System.arraycopy is more efficient 367369
org/apache/commons/imaging/formats/jpeg/decoder/ZigZag.java
Rule Violation Line
AvoidArrayLoops System.arraycopy is more efficient 2325
org/apache/commons/imaging/formats/jpeg/iptc/IptcParser.java
Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 18485
org/apache/commons/imaging/formats/jpeg/iptc/IptcTypes.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 58
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 60
org/apache/commons/imaging/formats/jpeg/iptc/JpegIptcRewriter.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 37284
org/apache/commons/imaging/formats/jpeg/segments/DhtSegment.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 136
org/apache/commons/imaging/formats/jpeg/segments/DqtSegment.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 49
org/apache/commons/imaging/formats/jpeg/segments/SofnSegment.java
Rule Violation Line
DataClass The class 'Component' is suspected to be a Data Class (WOC=0.000%, NOPA=4, NOAM=0, WMC=1) 3447
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 49
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 50
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 51
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 52
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 55
org/apache/commons/imaging/formats/jpeg/segments/SosSegment.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 44
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 45
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 46
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 47
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 48
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 49
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 51
org/apache/commons/imaging/formats/jpeg/xmp/JpegRewriter.java
Rule Violation Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 83
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 83
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 154
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 156
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 158
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 160
org/apache/commons/imaging/formats/jpeg/xmp/JpegXmpRewriter.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 35182
org/apache/commons/imaging/formats/pcx/PcxImageParser.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 150
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 152
org/apache/commons/imaging/formats/pcx/PcxImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2558
org/apache/commons/imaging/formats/pcx/PcxWriter.java
Rule Violation Line
CloseResource Ensure that resources like this BinaryOutputStream object are closed after use 55
org/apache/commons/imaging/formats/png/AbstractPngText.java
Rule Violation Line
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 1960
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 51
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 53
org/apache/commons/imaging/formats/png/PhysicalScale.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 35
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 37
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39
org/apache/commons/imaging/formats/png/PngColorType.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 55
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 59
org/apache/commons/imaging/formats/png/PngCrc.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2085
org/apache/commons/imaging/formats/png/PngImageParser.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 68694
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 77
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 78
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 79
org/apache/commons/imaging/formats/png/PngImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 28131
org/apache/commons/imaging/formats/png/PngWriter.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 35581
org/apache/commons/imaging/formats/png/chunks/PngChunk.java
Rule Violation Line
DataClass The class 'PngChunk' is suspected to be a Data Class (WOC=10.000%, NOPA=0, NOAM=8, WMC=12) 29131
org/apache/commons/imaging/formats/png/chunks/PngChunkIccp.java
Rule Violation Line
DataClass The class 'PngChunkIccp' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=7) 38124
org/apache/commons/imaging/formats/png/chunks/PngChunkIhdr.java
Rule Violation Line
DataClass The class 'PngChunkIhdr' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=7, WMC=13) 2987
org/apache/commons/imaging/formats/png/chunks/PngChunkScal.java
Rule Violation Line
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 68
org/apache/commons/imaging/formats/png/scanlinefilters/ScanlineFilterNone.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2329
org/apache/commons/imaging/formats/png/scanlinefilters/ScanlineFilterUp.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2343
org/apache/commons/imaging/formats/pnm/AbstractFileInfo.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 56
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 58
org/apache/commons/imaging/formats/pnm/PamFileInfo.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2953
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 37
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 38
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 39
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 41
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 41
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 42
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 42
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 43
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 43
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 46
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 47
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 48
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 48
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 69
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 70
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 70
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 73
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 74
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 75
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 75
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 88
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 89
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 90
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 92
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 94
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 96
org/apache/commons/imaging/formats/pnm/PamWriter.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2772
org/apache/commons/imaging/formats/pnm/PnmImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2752
org/apache/commons/imaging/formats/psd/ImageResourceType.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 85
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 86
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 88
org/apache/commons/imaging/formats/psd/PsdImageParser.java
Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17638
AtLeastOneConstructor Each class should declare at least one constructor 56638
org/apache/commons/imaging/formats/psd/PsdImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2729
org/apache/commons/imaging/formats/psd/dataparsers/DataParser.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2544
org/apache/commons/imaging/formats/psd/dataparsers/DataParserBitmap.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2142
org/apache/commons/imaging/formats/psd/dataparsers/DataParserCmyk.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2244
org/apache/commons/imaging/formats/psd/dataparsers/DataParserGrayscale.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2135
org/apache/commons/imaging/formats/psd/dataparsers/DataParserLab.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2240
org/apache/commons/imaging/formats/psd/dataparsers/DataParserRgb.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2137
org/apache/commons/imaging/formats/psd/dataparsers/DataParserStub.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2132
org/apache/commons/imaging/formats/rgbe/RgbeImageParser.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 43112
org/apache/commons/imaging/formats/rgbe/RgbeImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2527
org/apache/commons/imaging/formats/rgbe/RgbeInfo.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 66
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 67
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 68
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 70
org/apache/commons/imaging/formats/tiff/AbstractTiffElement.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 52
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 56
org/apache/commons/imaging/formats/tiff/AbstractTiffImageData.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 30159
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 60
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 103
org/apache/commons/imaging/formats/tiff/TiffDirectory.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 99
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 105
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 107
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 109
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 111
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 113
CompareObjectsWithEquals Use equals() to compare object references. 442
org/apache/commons/imaging/formats/tiff/TiffField.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 55
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 56
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 58
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 59
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 60
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 61
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 62
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 63
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 65
CompareObjectsWithEquals Use equals() to compare object references. 394
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 436
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 469
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 487
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 505
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 523
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 541
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 559
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 577
org/apache/commons/imaging/formats/tiff/TiffImageMetadata.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 227
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 238
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 508
org/apache/commons/imaging/formats/tiff/TiffImageParser.java
Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17875
AtLeastOneConstructor Each class should declare at least one constructor 73875
org/apache/commons/imaging/formats/tiff/TiffImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 27257
org/apache/commons/imaging/formats/tiff/TiffRasterDataInt.java
Rule Violation Line
AvoidArrayLoops System.arraycopy is more efficient 106108
org/apache/commons/imaging/formats/tiff/TiffReader.java
Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17483
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 130
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 131
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 132
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 133
org/apache/commons/imaging/formats/tiff/constants/TiffDirectoryType.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 58
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 59
org/apache/commons/imaging/formats/tiff/constants/TiffPlanarConfiguration.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 53
org/apache/commons/imaging/formats/tiff/datareaders/BitInputStream.java
Rule Violation Line
CompareObjectsWithEquals Use equals() to compare object references. 104
org/apache/commons/imaging/formats/tiff/datareaders/DataInterpreterJpeg.java
Rule Violation Line
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 37129
org/apache/commons/imaging/formats/tiff/datareaders/ImageDataReader.java
Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17766
CompareObjectsWithEquals Use equals() to compare object references. 653
CompareObjectsWithEquals Use equals() to compare object references. 677
CompareObjectsWithEquals Use equals() to compare object references. 735
CompareObjectsWithEquals Use equals() to compare object references. 745
org/apache/commons/imaging/formats/tiff/fieldtypes/AbstractFieldType.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 74
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 76
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 78
org/apache/commons/imaging/formats/tiff/itu_t4/HuffmanTree.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2891
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 34
org/apache/commons/imaging/formats/tiff/itu_t4/T4AndT6Compression.java
Rule Violation Line
CloseResource Ensure that resources like this BitArrayOutputStream object are closed after use 200
CloseResource Ensure that resources like this BitArrayOutputStream object are closed after use 278
CompareObjectsWithEquals Use equals() to compare object references. 442
CompareObjectsWithEquals Use equals() to compare object references. 445
CompareObjectsWithEquals Use equals() to compare object references. 455
CompareObjectsWithEquals Use equals() to compare object references. 457
CompareObjectsWithEquals Use equals() to compare object references. 459
CompareObjectsWithEquals Use equals() to compare object references. 461
CompareObjectsWithEquals Use equals() to compare object references. 463
CompareObjectsWithEquals Use equals() to compare object references. 465
CompareObjectsWithEquals Use equals() to compare object references. 467
CompareObjectsWithEquals Use equals() to compare object references. 539
CompareObjectsWithEquals Use equals() to compare object references. 542
CompareObjectsWithEquals Use equals() to compare object references. 552
CompareObjectsWithEquals Use equals() to compare object references. 554
CompareObjectsWithEquals Use equals() to compare object references. 556
CompareObjectsWithEquals Use equals() to compare object references. 558
CompareObjectsWithEquals Use equals() to compare object references. 560
CompareObjectsWithEquals Use equals() to compare object references. 562
CompareObjectsWithEquals Use equals() to compare object references. 564
AvoidInstanceofChecksInCatchClause An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 592
CompareObjectsWithEquals Use equals() to compare object references. 608
CompareObjectsWithEquals Use equals() to compare object references. 612
CompareObjectsWithEquals Use equals() to compare object references. 612
CompareObjectsWithEquals Use equals() to compare object references. 612
CompareObjectsWithEquals Use equals() to compare object references. 612
CompareObjectsWithEquals Use equals() to compare object references. 613
CompareObjectsWithEquals Use equals() to compare object references. 613
CompareObjectsWithEquals Use equals() to compare object references. 613
org/apache/commons/imaging/formats/tiff/itu_t4/T4_T6_Tables.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 4052
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 5468
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 7075
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 7783
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 8588
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 89
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 90
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 91
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 92
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 93
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 94
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 95
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 96
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 97
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 98
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 99
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 100
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 101
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 102
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 103
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 104
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 106
org/apache/commons/imaging/formats/tiff/taginfos/TagInfoGpsText.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 4748
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 4950
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 5152
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 5354
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 5557
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 5964
CompareObjectsWithEquals Use equals() to compare object references. 90
CompareObjectsWithEquals Use equals() to compare object references. 107
CompareObjectsWithEquals Use equals() to compare object references. 122
CompareObjectsWithEquals Use equals() to compare object references. 122
org/apache/commons/imaging/formats/tiff/taginfos/TagInfoXpString.java
Rule Violation Line
CompareObjectsWithEquals Use equals() to compare object references. 48
org/apache/commons/imaging/formats/tiff/write/AbstractTiffImageWriter.java
Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17618
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 68
CompareObjectsWithEquals Use equals() to compare object references. 605
org/apache/commons/imaging/formats/tiff/write/AbstractTiffOutputItem.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2477
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 60
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 62
org/apache/commons/imaging/formats/tiff/write/TiffImageWriterLossless.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 80
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 81
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 83
org/apache/commons/imaging/formats/tiff/write/TiffImageWriterLossy.java
Rule Violation Line
CloseResource Ensure that resources like this BinaryOutputStream object are closed after use 62
org/apache/commons/imaging/formats/tiff/write/TiffOutputField.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 37
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 41
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 42
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 44
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 125
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 126
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 127
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 129
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 130
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 131
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 132
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 134
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 135
org/apache/commons/imaging/formats/tiff/write/TiffOutputSet.java
Rule Violation Line
AvoidProtectedMethodInFinalClassNotExtending Avoid protected methods in a final class that doesnt extend anything other than Object. Change to private or package access. 153161
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 254
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 255
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 256
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 258
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 259
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 260
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 273
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 275
org/apache/commons/imaging/formats/tiff/write/TiffOutputSummary.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 41
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 43
org/apache/commons/imaging/formats/wbmp/WbmpImageParser.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 42226
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 66
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 68
org/apache/commons/imaging/formats/wbmp/WbmpImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2527
org/apache/commons/imaging/formats/webp/WebPChunkType.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 91
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 107
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 108
org/apache/commons/imaging/formats/webp/WebPImageParser.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 54328
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 71
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 132
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 134
org/apache/commons/imaging/formats/webp/WebPImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2627
org/apache/commons/imaging/formats/webp/chunks/WebPChunkVp8.java
Rule Violation Line
DataClass The class 'WebPChunkVp8' is suspected to be a Data Class (WOC=14.286%, NOPA=0, NOAM=5, WMC=17) 42166
org/apache/commons/imaging/formats/webp/chunks/WebPChunkVp8l.java
Rule Violation Line
DataClass The class 'WebPChunkVp8l' is suspected to be a Data Class (WOC=16.667%, NOPA=0, NOAM=4, WMC=11) 42114
org/apache/commons/imaging/formats/webp/chunks/WebPChunkVp8x.java
Rule Violation Line
DataClass The class 'WebPChunkVp8x' is suspected to be a Data Class (WOC=11.111%, NOPA=0, NOAM=7, WMC=13) 49148
org/apache/commons/imaging/formats/xbm/XbmImageParser.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 49369
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 80
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 82
SimplifyStartsWith This call to String.startsWith can be rewritten using String.charAt(0) 85
org/apache/commons/imaging/formats/xbm/XbmImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2527
org/apache/commons/imaging/formats/xpm/XpmImageParser.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 54668
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 125
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 126
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 128
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 130133
CloseResource Ensure that resources like this InputStream object are closed after use 142
org/apache/commons/imaging/formats/xpm/XpmImagingParameters.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 2426
org/apache/commons/imaging/icc/IccTagDataTypes.java
Rule Violation Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 43
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 76
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 92
org/apache/commons/imaging/internal/Debug.java
Rule Violation Line
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 57
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 58
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 59
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 60
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 61
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 62
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 63
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 64
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 65
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 66
InsufficientStringBufferDeclaration StringBuilder has been initialized with size 16, but has at least 18 characters appended. 238
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 265
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 267
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 268
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 269
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 270
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 272
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 274
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 275
org/apache/commons/imaging/mylzw/MyBitInputStream.java
Rule Violation Line
CompareObjectsWithEquals Use equals() to compare object references. 65
CompareObjectsWithEquals Use equals() to compare object references. 78
org/apache/commons/imaging/mylzw/MyBitOutputStream.java
Rule Violation Line
CompareObjectsWithEquals Use equals() to compare object references. 46
CompareObjectsWithEquals Use equals() to compare object references. 72
CompareObjectsWithEquals Use equals() to compare object references. 82
org/apache/commons/imaging/mylzw/MyLzwCompressor.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 88
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 90
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 91
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 92
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 93
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 94
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 95
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 97
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 99
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 176
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 186
org/apache/commons/imaging/mylzw/MyLzwDecompressor.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 41
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 42
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 43
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 44
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 45
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 46
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 47
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 48
org/apache/commons/imaging/palette/ColorSpaceSubset.java
Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 33
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 34
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 35
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 36
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 37
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 42
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 44
org/apache/commons/imaging/palette/LongestAxisMedianCut.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 25113
org/apache/commons/imaging/palette/MostPopulatedBoxesMedianCut.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 24126
org/apache/commons/imaging/palette/PaletteFactory.java
Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 36512
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 50
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 52
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 134
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 135
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 156
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 157

Priority 4

org/apache/commons/imaging/Imaging.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 112
org/apache/commons/imaging/formats/dcx/DcxImageParser.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 50
org/apache/commons/imaging/formats/icns/IcnsDecoder.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 156
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 202
org/apache/commons/imaging/formats/icns/IcnsImageParser.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 45
org/apache/commons/imaging/formats/ico/IcoImageParser.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 201
org/apache/commons/imaging/formats/jpeg/JpegImageParser.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 855
org/apache/commons/imaging/formats/jpeg/decoder/Dct.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 73
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 114
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 195
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 242
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 319
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 327
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 333
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 341
org/apache/commons/imaging/formats/jpeg/decoder/JpegDecoder.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 70
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 108
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 207
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 302
org/apache/commons/imaging/formats/jpeg/decoder/JpegInputStream.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 32
org/apache/commons/imaging/formats/jpeg/decoder/ZigZag.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 22
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 28
org/apache/commons/imaging/formats/jpeg/segments/DhtSegment.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 44
org/apache/commons/imaging/formats/jpeg/segments/DqtSegment.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 35
org/apache/commons/imaging/formats/png/PngColorType.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 61
org/apache/commons/imaging/formats/png/PngImageParser.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 591
org/apache/commons/imaging/formats/psd/PsdImageParser.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 491
org/apache/commons/imaging/formats/tiff/TiffRasterDataFloat.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 61
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 90
org/apache/commons/imaging/formats/tiff/TiffRasterDataInt.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 72
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 88
org/apache/commons/imaging/formats/tiff/datareaders/ImageDataReader.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 176
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 284
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 359
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 476
org/apache/commons/imaging/formats/tiff/itu_t4/T4AndT6Compression.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 129
org/apache/commons/imaging/formats/tiff/photometricinterpreters/PhotometricInterpreterPalette.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 50
org/apache/commons/imaging/icc/IccProfileInfo.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 50
org/apache/commons/imaging/internal/Debug.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 93
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 115
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 207
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 222
org/apache/commons/imaging/palette/ColorSpaceSubset.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 58
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 135
org/apache/commons/imaging/palette/MedianCutPalette.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 22
org/apache/commons/imaging/palette/PaletteFactory.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 85
org/apache/commons/imaging/palette/SimplePalette.java
Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 30

Files

org/apache/commons/imaging/ColorTools.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 43221

org/apache/commons/imaging/FormatCompliance.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 41
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 85
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 99

org/apache/commons/imaging/ImageDump.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 3299

org/apache/commons/imaging/ImageInfo.java

Rule Violation Priority Line
DataClass The class 'ImageInfo' is suspected to be a Data Class (WOC=13.636%, NOPA=0, NOAM=18, WMC=25) 3 29365

org/apache/commons/imaging/Imaging.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 112
AvoidProtectedMethodInFinalClassNotExtending Avoid protected methods in a final class that doesnt extend anything other than Object. Change to private or package access. 3 314330
InefficientEmptyStringCheck String.trim().length() == 0 / String.trim().isEmpty() is an inefficient way to validate a blank String. 3 806

org/apache/commons/imaging/ImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 3198

org/apache/commons/imaging/PixelDensity.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 44
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 46
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 49

org/apache/commons/imaging/bytesource/ByteSource.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 55

org/apache/commons/imaging/bytesource/InputStreamByteSource.java

Rule Violation Priority Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 51
AtLeastOneConstructor Each class should declare at least one constructor 3 57170
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 68
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 105
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 143
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 172
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 173
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 174
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 175
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 176
CloseResource Ensure that resources like this InputStream object are closed after use 3 191

org/apache/commons/imaging/common/AllocationRequestException.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 35
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 37

org/apache/commons/imaging/common/Allocator.java

Rule Violation Priority Line
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 3 27249

org/apache/commons/imaging/common/BasicCParser.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 385

org/apache/commons/imaging/common/BinaryFunctions.java

Rule Violation Priority Line
CompareObjectsWithEquals Use equals() to compare object references. 3 124
CompareObjectsWithEquals Use equals() to compare object references. 3 142
CompareObjectsWithEquals Use equals() to compare object references. 3 161
CompareObjectsWithEquals Use equals() to compare object references. 3 196
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 255

org/apache/commons/imaging/common/BinaryOutputStream.java

Rule Violation Priority Line
CompareObjectsWithEquals Use equals() to compare object references. 3 35
CompareObjectsWithEquals Use equals() to compare object references. 3 38
UnusedFormalParameter Avoid unused constructor parameters such as 'byteOrder'. 3 52

org/apache/commons/imaging/common/ByteConversions.java

Rule Violation Priority Line
CompareObjectsWithEquals Use equals() to compare object references. 3 34
CompareObjectsWithEquals Use equals() to compare object references. 3 75
CompareObjectsWithEquals Use equals() to compare object references. 3 107
CompareObjectsWithEquals Use equals() to compare object references. 3 146
CompareObjectsWithEquals Use equals() to compare object references. 3 174
CompareObjectsWithEquals Use equals() to compare object references. 3 214
CompareObjectsWithEquals Use equals() to compare object references. 3 249
CompareObjectsWithEquals Use equals() to compare object references. 3 277
CompareObjectsWithEquals Use equals() to compare object references. 3 306
CompareObjectsWithEquals Use equals() to compare object references. 3 344
CompareObjectsWithEquals Use equals() to compare object references. 3 391
CompareObjectsWithEquals Use equals() to compare object references. 3 438

org/apache/commons/imaging/common/GenericImageMetadata.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 22103
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 59

org/apache/commons/imaging/common/RationalNumber.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 35
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 37
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 50
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 173
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 175
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 177

org/apache/commons/imaging/common/RgbBufferedImageFactory.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2236

org/apache/commons/imaging/common/SimpleBufferedImageFactory.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2239

org/apache/commons/imaging/common/XmpImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2841

org/apache/commons/imaging/formats/bmp/BmpHeaderInfo.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 42
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 43
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 44
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 45
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 46
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 47
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 48
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 49
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 50
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 51
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 52
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 55
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 56
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 59
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 60
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 61
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 62
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 63
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 64
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 65
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 66
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 67
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 69
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 71

org/apache/commons/imaging/formats/bmp/BmpImageParser.java

Rule Violation Priority Line
CloseResource Ensure that resources like this BinaryOutputStream object are closed after use 3 634

org/apache/commons/imaging/formats/bmp/BmpImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2729

org/apache/commons/imaging/formats/bmp/BmpWriterRgb.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2582

org/apache/commons/imaging/formats/dcx/DcxImageParser.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 50
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 64
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 66
CloseResource Ensure that resources like this BinaryOutputStream object are closed after use 3 177

org/apache/commons/imaging/formats/gif/GifImageMetadata.java

Rule Violation Priority Line
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 55
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 56
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 57
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 58

org/apache/commons/imaging/formats/gif/GifImageMetadataItem.java

Rule Violation Priority Line
DataClass The class 'GifImageMetadataItem' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=4, WMC=7) 3 2161
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 55
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 56
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 57

org/apache/commons/imaging/formats/gif/GifImageParser.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 171058
UnusedFormalParameter Avoid unused method parameters such as 'headerInfo'. 3 252

org/apache/commons/imaging/formats/gif/GifImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2743
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 33

org/apache/commons/imaging/formats/icns/IcnsDecoder.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 156
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 202

org/apache/commons/imaging/formats/icns/IcnsImageParser.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 40267
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 45
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 95
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 97
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 99

org/apache/commons/imaging/formats/icns/IcnsImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2527

org/apache/commons/imaging/formats/icns/IcnsType.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 112
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 114
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 116
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 118
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 120

org/apache/commons/imaging/formats/ico/IcoImageParser.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 201
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 227
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 229

org/apache/commons/imaging/formats/ico/IcoImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2527

org/apache/commons/imaging/formats/jpeg/JpegImageMetadata.java

Rule Violation Priority Line
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 221

org/apache/commons/imaging/formats/jpeg/JpegImageParser.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 65943
SwitchDensity A high ratio of statements to labels in a switch statement. Consider refactoring. 3 480605
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 855
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 899

org/apache/commons/imaging/formats/jpeg/JpegImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2527

org/apache/commons/imaging/formats/jpeg/decoder/Dct.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 73
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 114
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 195
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 242
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 319
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 327
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 333
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 341

org/apache/commons/imaging/formats/jpeg/decoder/JpegDecoder.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 48588
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 70
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 108
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 129
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 130
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 131
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 132
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 133
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 134
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 135
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 136
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 137
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 139
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 141
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 143
SingularField Perhaps 'block' could be replaced by a local variable. 3 143
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 145
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 207
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 302
AvoidArrayLoops System.arraycopy is more efficient 3 367369

org/apache/commons/imaging/formats/jpeg/decoder/JpegInputStream.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 32

org/apache/commons/imaging/formats/jpeg/decoder/ZigZag.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 22
AvoidArrayLoops System.arraycopy is more efficient 3 2325
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 28

org/apache/commons/imaging/formats/jpeg/iptc/IptcParser.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 18485

org/apache/commons/imaging/formats/jpeg/iptc/IptcTypes.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 58
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 60

org/apache/commons/imaging/formats/jpeg/iptc/JpegIptcRewriter.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 37284

org/apache/commons/imaging/formats/jpeg/segments/DhtSegment.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 44
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 136

org/apache/commons/imaging/formats/jpeg/segments/DqtSegment.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 35
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 49

org/apache/commons/imaging/formats/jpeg/segments/SofnSegment.java

Rule Violation Priority Line
DataClass The class 'Component' is suspected to be a Data Class (WOC=0.000%, NOPA=4, NOAM=0, WMC=1) 3 3447
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 49
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 50
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 51
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 52
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 55

org/apache/commons/imaging/formats/jpeg/segments/SosSegment.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 44
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 45
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 46
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 47
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 48
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 49
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 51

org/apache/commons/imaging/formats/jpeg/xmp/JpegRewriter.java

Rule Violation Priority Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 83
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 83
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 154
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 156
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 158
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 160

org/apache/commons/imaging/formats/jpeg/xmp/JpegXmpRewriter.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 35182

org/apache/commons/imaging/formats/pcx/PcxImageParser.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 150
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 152

org/apache/commons/imaging/formats/pcx/PcxImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2558

org/apache/commons/imaging/formats/pcx/PcxWriter.java

Rule Violation Priority Line
CloseResource Ensure that resources like this BinaryOutputStream object are closed after use 3 55

org/apache/commons/imaging/formats/png/AbstractPngText.java

Rule Violation Priority Line
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 3 1960
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 51
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 53

org/apache/commons/imaging/formats/png/PhysicalScale.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 35
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 37
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 39

org/apache/commons/imaging/formats/png/PngColorType.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 55
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 59
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 61

org/apache/commons/imaging/formats/png/PngCrc.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2085

org/apache/commons/imaging/formats/png/PngImageParser.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 68694
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 77
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 78
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 79
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 591

org/apache/commons/imaging/formats/png/PngImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 28131

org/apache/commons/imaging/formats/png/PngWriter.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 35581

org/apache/commons/imaging/formats/png/chunks/PngChunk.java

Rule Violation Priority Line
DataClass The class 'PngChunk' is suspected to be a Data Class (WOC=10.000%, NOPA=0, NOAM=8, WMC=12) 3 29131

org/apache/commons/imaging/formats/png/chunks/PngChunkIccp.java

Rule Violation Priority Line
DataClass The class 'PngChunkIccp' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=7) 3 38124

org/apache/commons/imaging/formats/png/chunks/PngChunkIhdr.java

Rule Violation Priority Line
DataClass The class 'PngChunkIhdr' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=7, WMC=13) 3 2987
ConstructorCallsOverridableMethod Overridable method 'getPngColorType' called during object construction 1 48

org/apache/commons/imaging/formats/png/chunks/PngChunkScal.java

Rule Violation Priority Line
ConstructorCallsOverridableMethod Overridable method 'getUnitSpecifier' called during object construction 1 34
ConstructorCallsOverridableMethod Overridable method 'getUnitSpecifier' called during object construction 1 34
ConstructorCallsOverridableMethod Overridable method 'getUnitSpecifier' called during object construction 1 35
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 68

org/apache/commons/imaging/formats/png/scanlinefilters/ScanlineFilterNone.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2329

org/apache/commons/imaging/formats/png/scanlinefilters/ScanlineFilterUp.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2343

org/apache/commons/imaging/formats/pnm/AbstractFileInfo.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 56
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 58

org/apache/commons/imaging/formats/pnm/PamFileInfo.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2953
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 37
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 38
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 39
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 41
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 41
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 42
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 42
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 43
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 43
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 46
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 47
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 48
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 48
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 69
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 70
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 70
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 73
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 74
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 75
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 75
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 88
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 89
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 90
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 92
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 94
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 96

org/apache/commons/imaging/formats/pnm/PamWriter.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2772

org/apache/commons/imaging/formats/pnm/PnmImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2752

org/apache/commons/imaging/formats/psd/ImageResourceType.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 85
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 86
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 88

org/apache/commons/imaging/formats/psd/PsdImageParser.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17638
AtLeastOneConstructor Each class should declare at least one constructor 3 56638
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 491

org/apache/commons/imaging/formats/psd/PsdImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2729

org/apache/commons/imaging/formats/psd/dataparsers/DataParser.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2544

org/apache/commons/imaging/formats/psd/dataparsers/DataParserBitmap.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2142

org/apache/commons/imaging/formats/psd/dataparsers/DataParserCmyk.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2244

org/apache/commons/imaging/formats/psd/dataparsers/DataParserGrayscale.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2135

org/apache/commons/imaging/formats/psd/dataparsers/DataParserLab.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2240

org/apache/commons/imaging/formats/psd/dataparsers/DataParserRgb.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2137

org/apache/commons/imaging/formats/psd/dataparsers/DataParserStub.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2132

org/apache/commons/imaging/formats/rgbe/RgbeImageParser.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 43112

org/apache/commons/imaging/formats/rgbe/RgbeImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2527

org/apache/commons/imaging/formats/rgbe/RgbeInfo.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 66
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 67
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 68
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 70

org/apache/commons/imaging/formats/tiff/AbstractTiffElement.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 52
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 56

org/apache/commons/imaging/formats/tiff/AbstractTiffImageData.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 30159
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 60
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 103

org/apache/commons/imaging/formats/tiff/TiffDirectory.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 99
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 105
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 107
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 109
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 111
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 113
CompareObjectsWithEquals Use equals() to compare object references. 3 442

org/apache/commons/imaging/formats/tiff/TiffField.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 55
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 56
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 58
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 59
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 60
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 61
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 62
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 63
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 65
CompareObjectsWithEquals Use equals() to compare object references. 3 394
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 436
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 469
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 487
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 505
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 523
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 541
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 559
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 577

org/apache/commons/imaging/formats/tiff/TiffImageMetadata.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 227
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 238
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 508

org/apache/commons/imaging/formats/tiff/TiffImageParser.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17875
AtLeastOneConstructor Each class should declare at least one constructor 3 73875

org/apache/commons/imaging/formats/tiff/TiffImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 27257

org/apache/commons/imaging/formats/tiff/TiffRasterDataFloat.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 61
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 90

org/apache/commons/imaging/formats/tiff/TiffRasterDataInt.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 72
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 88
AvoidArrayLoops System.arraycopy is more efficient 3 106108

org/apache/commons/imaging/formats/tiff/TiffReader.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17483
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 130
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 131
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 132
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 133

org/apache/commons/imaging/formats/tiff/constants/TiffDirectoryType.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 58
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 59

org/apache/commons/imaging/formats/tiff/constants/TiffPlanarConfiguration.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 53

org/apache/commons/imaging/formats/tiff/datareaders/BitInputStream.java

Rule Violation Priority Line
CompareObjectsWithEquals Use equals() to compare object references. 3 104

org/apache/commons/imaging/formats/tiff/datareaders/DataInterpreterJpeg.java

Rule Violation Priority Line
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 3 37129

org/apache/commons/imaging/formats/tiff/datareaders/ImageDataReader.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17766
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 176
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 284
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 359
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 476
CompareObjectsWithEquals Use equals() to compare object references. 3 653
CompareObjectsWithEquals Use equals() to compare object references. 3 677
CompareObjectsWithEquals Use equals() to compare object references. 3 735
CompareObjectsWithEquals Use equals() to compare object references. 3 745

org/apache/commons/imaging/formats/tiff/fieldtypes/AbstractFieldType.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 74
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 76
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 78

org/apache/commons/imaging/formats/tiff/itu_t4/HuffmanTree.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2891
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 34

org/apache/commons/imaging/formats/tiff/itu_t4/T4AndT6Compression.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 129
CloseResource Ensure that resources like this BitArrayOutputStream object are closed after use 3 200
CloseResource Ensure that resources like this BitArrayOutputStream object are closed after use 3 278
CompareObjectsWithEquals Use equals() to compare object references. 3 442
CompareObjectsWithEquals Use equals() to compare object references. 3 445
CompareObjectsWithEquals Use equals() to compare object references. 3 455
CompareObjectsWithEquals Use equals() to compare object references. 3 457
CompareObjectsWithEquals Use equals() to compare object references. 3 459
CompareObjectsWithEquals Use equals() to compare object references. 3 461
CompareObjectsWithEquals Use equals() to compare object references. 3 463
CompareObjectsWithEquals Use equals() to compare object references. 3 465
CompareObjectsWithEquals Use equals() to compare object references. 3 467
CompareObjectsWithEquals Use equals() to compare object references. 3 539
CompareObjectsWithEquals Use equals() to compare object references. 3 542
CompareObjectsWithEquals Use equals() to compare object references. 3 552
CompareObjectsWithEquals Use equals() to compare object references. 3 554
CompareObjectsWithEquals Use equals() to compare object references. 3 556
CompareObjectsWithEquals Use equals() to compare object references. 3 558
CompareObjectsWithEquals Use equals() to compare object references. 3 560
CompareObjectsWithEquals Use equals() to compare object references. 3 562
CompareObjectsWithEquals Use equals() to compare object references. 3 564
AvoidInstanceofChecksInCatchClause An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 3 592
CompareObjectsWithEquals Use equals() to compare object references. 3 608
CompareObjectsWithEquals Use equals() to compare object references. 3 612
CompareObjectsWithEquals Use equals() to compare object references. 3 612
CompareObjectsWithEquals Use equals() to compare object references. 3 612
CompareObjectsWithEquals Use equals() to compare object references. 3 612
CompareObjectsWithEquals Use equals() to compare object references. 3 613
CompareObjectsWithEquals Use equals() to compare object references. 3 613
CompareObjectsWithEquals Use equals() to compare object references. 3 613

org/apache/commons/imaging/formats/tiff/itu_t4/T4_T6_Tables.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 4052
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 5468
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 7075
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 7783
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 8588
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 89
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 90
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 91
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 92
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 93
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 94
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 95
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 96
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 97
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 98
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 99
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 100
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 101
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 102
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 103
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 104
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 106

org/apache/commons/imaging/formats/tiff/photometricinterpreters/PhotometricInterpreterPalette.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 50

org/apache/commons/imaging/formats/tiff/taginfos/TagInfoGpsText.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 4748
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 4950
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 5152
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 5354
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 5557
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 5964
CompareObjectsWithEquals Use equals() to compare object references. 3 90
CompareObjectsWithEquals Use equals() to compare object references. 3 107
CompareObjectsWithEquals Use equals() to compare object references. 3 122
CompareObjectsWithEquals Use equals() to compare object references. 3 122

org/apache/commons/imaging/formats/tiff/taginfos/TagInfoXpString.java

Rule Violation Priority Line
CompareObjectsWithEquals Use equals() to compare object references. 3 48

org/apache/commons/imaging/formats/tiff/write/AbstractTiffImageWriter.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17618
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 68
CompareObjectsWithEquals Use equals() to compare object references. 3 605

org/apache/commons/imaging/formats/tiff/write/AbstractTiffOutputItem.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2477
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 60
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 62

org/apache/commons/imaging/formats/tiff/write/TiffImageWriterLossless.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 80
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 81
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 83

org/apache/commons/imaging/formats/tiff/write/TiffImageWriterLossy.java

Rule Violation Priority Line
CloseResource Ensure that resources like this BinaryOutputStream object are closed after use 3 62

org/apache/commons/imaging/formats/tiff/write/TiffOutputField.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 37
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 41
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 42
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 44
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 125
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 126
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 127
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 129
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 130
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 131
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 132
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 134
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 135

org/apache/commons/imaging/formats/tiff/write/TiffOutputSet.java

Rule Violation Priority Line
AvoidProtectedMethodInFinalClassNotExtending Avoid protected methods in a final class that doesnt extend anything other than Object. Change to private or package access. 3 153161
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 254
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 255
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 256
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 258
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 259
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 260
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 273
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 275

org/apache/commons/imaging/formats/tiff/write/TiffOutputSummary.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 41
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 43

org/apache/commons/imaging/formats/wbmp/WbmpImageParser.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 42226
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 66
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 68

org/apache/commons/imaging/formats/wbmp/WbmpImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2527

org/apache/commons/imaging/formats/webp/WebPChunkType.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 91
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 107
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 108

org/apache/commons/imaging/formats/webp/WebPImageParser.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 54328
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 71
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 2 122
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 132
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 134

org/apache/commons/imaging/formats/webp/WebPImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2627

org/apache/commons/imaging/formats/webp/chunks/WebPChunkVp8.java

Rule Violation Priority Line
DataClass The class 'WebPChunkVp8' is suspected to be a Data Class (WOC=14.286%, NOPA=0, NOAM=5, WMC=17) 3 42166

org/apache/commons/imaging/formats/webp/chunks/WebPChunkVp8l.java

Rule Violation Priority Line
DataClass The class 'WebPChunkVp8l' is suspected to be a Data Class (WOC=16.667%, NOPA=0, NOAM=4, WMC=11) 3 42114

org/apache/commons/imaging/formats/webp/chunks/WebPChunkVp8x.java

Rule Violation Priority Line
DataClass The class 'WebPChunkVp8x' is suspected to be a Data Class (WOC=11.111%, NOPA=0, NOAM=7, WMC=13) 3 49148

org/apache/commons/imaging/formats/xbm/XbmImageParser.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 49369
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 80
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 82
SimplifyStartsWith This call to String.startsWith can be rewritten using String.charAt(0) 3 85

org/apache/commons/imaging/formats/xbm/XbmImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2527

org/apache/commons/imaging/formats/xpm/XpmImageParser.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 54668
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 125
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 126
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 128
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 130133
CloseResource Ensure that resources like this InputStream object are closed after use 3 142

org/apache/commons/imaging/formats/xpm/XpmImagingParameters.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 2426

org/apache/commons/imaging/icc/IccProfileInfo.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 50

org/apache/commons/imaging/icc/IccTagDataTypes.java

Rule Violation Priority Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 43
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 76
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 92

org/apache/commons/imaging/internal/Debug.java

Rule Violation Priority Line
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 57
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 58
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 59
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 60
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 61
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 62
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 63
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 64
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 65
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 66
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 93
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 115
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 207
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 222
InsufficientStringBufferDeclaration StringBuilder has been initialized with size 16, but has at least 18 characters appended. 3 238
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 265
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 267
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 268
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 269
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 270
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 272
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 274
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 275

org/apache/commons/imaging/mylzw/MyBitInputStream.java

Rule Violation Priority Line
CompareObjectsWithEquals Use equals() to compare object references. 3 65
CompareObjectsWithEquals Use equals() to compare object references. 3 78

org/apache/commons/imaging/mylzw/MyBitOutputStream.java

Rule Violation Priority Line
CompareObjectsWithEquals Use equals() to compare object references. 3 46
CompareObjectsWithEquals Use equals() to compare object references. 3 72
CompareObjectsWithEquals Use equals() to compare object references. 3 82

org/apache/commons/imaging/mylzw/MyLzwCompressor.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 88
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 90
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 91
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 92
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 93
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 94
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 95
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 97
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 99
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 176
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 186

org/apache/commons/imaging/mylzw/MyLzwDecompressor.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 41
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 42
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 43
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 44
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 45
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 46
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 47
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 48

org/apache/commons/imaging/palette/ColorSpaceSubset.java

Rule Violation Priority Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 33
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 34
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 35
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 36
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 37
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 38
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 39
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 40
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 42
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 44
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 58
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 135

org/apache/commons/imaging/palette/LongestAxisMedianCut.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 25113

org/apache/commons/imaging/palette/MedianCutPalette.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 22

org/apache/commons/imaging/palette/MostPopulatedBoxesMedianCut.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 24126

org/apache/commons/imaging/palette/PaletteFactory.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 36512
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 50
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 52
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 85
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 134
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 135
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 156
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 157

org/apache/commons/imaging/palette/SimplePalette.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 30