PMD Results

The following document contains the results of PMD

Files

javax/mail/Address.java

ViolationLine
Ensure you override both equals() and hashCode()35

javax/mail/FetchProfile.java

ViolationLine
Avoid unused constructor parameters such as 'name'63 - 66

javax/mail/Flags.java

ViolationLine
Avoid unused imports such as 'java.util.Iterator'22
Avoid unused imports such as 'java.util.LinkedList'23
Avoid unused imports such as 'java.util.List'24
Avoid unused private fields such as 'mask'72

javax/mail/Folder.java

ViolationLine
Avoid unnecessary return statements518

javax/mail/internet/AddressException.java

ViolationLine
Avoid unused constructor parameters such as 'ref'35 - 37

javax/mail/internet/AddressParser.java

ViolationLine
Avoid unused imports such as 'java.lang.reflect.Array'21
Avoid empty catch blocks456 - 457
Avoid unused local variables such as 'startPosition'783
These nested if statements could be combined1141 - 1143
Avoid unused private methods such as 'isSpace(char)'1524

javax/mail/internet/HeaderTokenizer.java

ViolationLine
Avoid empty while statements280

javax/mail/internet/InternetAddress.java

ViolationLine
Avoid unused imports such as 'java.lang.reflect.Array'20
Avoid unused imports such as 'java.util.ArrayList'23
Avoid unused imports such as 'java.util.List'24
Avoid unused imports such as 'java.util.StringTokenizer'25
Avoid unused imports such as 'org.apache.geronimo.mail.util.SessionUtil'30
Avoid empty catch blocks422 - 424
Avoid empty catch blocks424 - 426
Avoid empty catch blocks426 - 428

javax/mail/internet/InternetHeaders.java

ViolationLine
Avoid unused imports such as 'java.util.Arrays'24
Avoid unused local variables such as 'name'416

javax/mail/internet/MimeBodyPart.java

ViolationLine
Avoid unused imports such as 'javax.swing.text.AbstractDocument.Content'34
Avoid empty catch blocks111 - 112
Avoid empty catch blocks265 - 267
Avoid empty catch blocks306 - 307
These nested if statements could be combined537 - 556
These nested if statements could be combined539 - 555
These nested if statements could be combined542 - 554

javax/mail/internet/MimeMessage.java

ViolationLine
Avoid empty catch blocks523 - 525
Avoid unused local variables such as 's'554
Avoid empty catch blocks624 - 626
Avoid empty catch blocks778 - 780
Avoid empty catch blocks843 - 844
These nested if statements could be combined1352 - 1371
These nested if statements could be combined1354 - 1370
These nested if statements could be combined1357 - 1369

javax/mail/internet/MimeUtility.java

ViolationLine
Avoid unused imports such as 'java.io.BufferedInputStream'20
Avoid unused imports such as 'org.apache.geronimo.mail.util.QuotedPrintable'46
Avoid unused private fields such as 'defaultJavaCharset'67
These nested if statements could be combined185 - 188
Avoid empty catch blocks198 - 199
These nested if statements could be combined341 - 344
Avoid empty catch blocks354 - 355
Avoid empty catch blocks671 - 672
Avoid unused local variables such as 'requiresQuoting'691
Avoid empty catch blocks840 - 841
Avoid empty catch blocks949 - 951
These nested if statements could be combined992 - 994

javax/mail/internet/NewsAddress.java

ViolationLine
Avoid unused imports such as 'sun.security.provider.Sun'26

javax/mail/internet/ParameterList.java

ViolationLine
Avoid unused imports such as 'java.util.StringTokenizer'25
Avoid unused imports such as 'java.util.List'26
Avoid unused imports such as 'java.util.ArrayList'27
Avoid unused private fields such as 'encodeParameters'49

javax/mail/search/ComparisonTerm.java

ViolationLine
Overriding method merely calls super38 - 40
Overriding method merely calls super42 - 44

javax/mail/search/MessageNumberTerm.java

ViolationLine
Ensure you override both equals() and hashCode()34
Overriding method merely calls super34 - 36

javax/mail/Service.java

ViolationLine
Avoid empty catch blocks160 - 162
Avoid empty catch blocks217 - 218
Avoid empty catch blocks225 - 226

javax/mail/Session.java

ViolationLine
These nested if statements could be combined134 - 136
Avoid empty catch blocks531 - 533
Avoid empty catch blocks533 - 535
Avoid empty catch blocks549 - 551
Avoid empty catch blocks551 - 553
Avoid empty catch blocks569 - 571
Avoid empty catch blocks571 - 573
Avoid empty catch blocks671 - 673
Avoid empty catch blocks673 - 675
Avoid empty catch blocks690 - 692
Avoid empty catch blocks692 - 694
Avoid empty catch blocks706 - 708
Avoid empty catch blocks708 - 710
Avoid empty catch blocks724 - 726
Avoid empty catch blocks726 - 728
Avoid unused private fields such as 'byClassName'750
Avoid unused private fields such as 'byProtocol'751
Avoid unused private fields such as 'all'752

javax/mail/StoreClosedException.java

ViolationLine
Avoid unused constructor parameters such as 'store'31 - 33

org/apache/geronimo/mail/handlers/MessageHandler.java

ViolationLine
Avoid unused imports such as 'javax.mail.internet.ContentType'21
Avoid unused imports such as 'javax.mail.internet.MimeUtility'27
Avoid unused imports such as 'javax.mail.internet.ParseException'28
Avoid unused imports such as 'java.io.InputStreamReader'31
Avoid unused imports such as 'java.io.OutputStreamWriter'33
Avoid unused imports such as 'java.io.StringWriter'34
Avoid unused imports such as 'java.io.UnsupportedEncodingException'35

org/apache/geronimo/mail/handlers/MultipartHandler.java

ViolationLine
Avoid unused imports such as 'javax.mail.internet.MimeMessage'26

org/apache/geronimo/mail/util/ASCIIUtil.java

ViolationLine
These nested if statements could be combined303 - 305
Avoid unused local variables such as 'lineBreak'425
These nested if statements could be combined430 - 434

org/apache/geronimo/mail/util/Base64DecoderStream.java

ViolationLine
Avoid unused private methods such as 'getBufferedChar()'77
These nested if statements could be combined108 - 110
These nested if statements could be combined123 - 125

org/apache/geronimo/mail/util/Base64EncoderStream.java

ViolationLine
These nested if statements could be combined163 - 166

org/apache/geronimo/mail/util/HexEncoder.java

ViolationLine
Avoid unused local variables such as 'b3'152
Avoid unused local variables such as 'b4'152

org/apache/geronimo/mail/util/QuotedPrintable.java

ViolationLine
Avoid unused imports such as 'java.io.InputStream'22

org/apache/geronimo/mail/util/QuotedPrintableDecoderStream.java

ViolationLine
Avoid unused imports such as 'java.io.ByteArrayOutputStream'20
Avoid unused imports such as 'java.io.UnsupportedEncodingException'24

org/apache/geronimo/mail/util/QuotedPrintableEncoder.java

ViolationLine
Avoid unused imports such as 'java.io.EOFException'20

org/apache/geronimo/mail/util/RFC2231Encoder.java

ViolationLine
Avoid unused local variables such as 'language'248

org/apache/geronimo/mail/util/SessionUtil.java

ViolationLine
Avoid unused imports such as 'java.security.Security'20
Avoid empty catch blocks122 - 123
Avoid empty catch blocks167 - 169
Avoid empty catch blocks191 - 193
Avoid empty catch blocks210 - 212

org/apache/geronimo/mail/util/UUDecoderStream.java

ViolationLine
Avoid unused private methods such as 'getBufferedChar()'71
These nested if statements could be combined96 - 98
These nested if statements could be combined111 - 113
An empty statement (semicolon) not part of a loop180

org/apache/geronimo/mail/util/XTextEncoder.java

ViolationLine
Avoid unused local variables such as 'b3'135
Avoid unused local variables such as 'b4'135