PMD Results

The following document contains the results of PMD 4.2.2.

Files

org/apache/jsieve/mailet/ActionContext.java

ViolationLine
Avoid modifiers which are implied by the context38
Avoid modifiers which are implied by the context49
Avoid modifiers which are implied by the context58
Avoid modifiers which are implied by the context64

org/apache/jsieve/mailet/ActionUtils.java

ViolationLine
Avoid unnecessary temporaries when converting primitives to Strings49

org/apache/jsieve/mailet/FileIntoAction.java

ViolationLine
Avoid using final local variables, turn them into fields36
Found 'DD'-anomaly for variable 'delivered' (lines '70'-'91').70 - 91
Avoid using final local variables, turn them into fields81
Avoid using final local variables, turn them into fields82
Avoid using final local variables, turn them into fields88
Found 'DU'-anomaly for variable 'delivered' (lines '91'-'116').91 - 116
Avoid using final local variables, turn them into fields95
Avoid using final local variables, turn them into fields108

org/apache/jsieve/mailet/KeepAction.java

ViolationLine
Avoid using final local variables, turn them into fields35
Avoid using final local variables, turn them into fields57

org/apache/jsieve/mailet/MailAction.java

ViolationLine
Avoid modifiers which are implied by the context38

org/apache/jsieve/mailet/Poster.java

ViolationLine
Avoid modifiers which are implied by the context39

org/apache/jsieve/mailet/RedirectAction.java

ViolationLine
Avoid using final local variables, turn them into fields38

org/apache/jsieve/mailet/RejectAction.java

ViolationLine
Avoid using final local variables, turn them into fields49
Found 'DD'-anomaly for variable 'reporting_UA_name' (lines '83'-'86').83 - 86
Found 'DD'-anomaly for variable 'reporting_UA_name' (lines '86'-'91').86 - 91
Found 'DD'-anomaly for variable 'original_recipient' (lines '98'-'101').98 - 101

org/apache/jsieve/mailet/ResourceLocator.java

ViolationLine
Avoid modifiers which are implied by the context65

org/apache/jsieve/mailet/SieveMailAdapter.java

ViolationLine
Found 'DD'-anomaly for variable 'actions' (lines '118'-'119').118 - 119
Avoid assignments in operands119
Found 'DU'-anomaly for variable 'actions' (lines '119'-'125').119 - 125
A method should have only one exit point, and that should be the last statement in the method122
Found 'DD'-anomaly for variable 'recipient' (lines '316'-'319').316 - 319
Found 'DD'-anomaly for variable 'messageID' (lines '363'-'366').363 - 366
Found 'DD'-anomaly for variable 'messageID' (lines '366'-'370').366 - 370
Avoid using final local variables, turn them into fields398
Avoid using final local variables, turn them into fields399
Avoid using final local variables, turn them into fields400
Found 'DD'-anomaly for variable 'results' (lines '400'-'403').400 - 403
Found 'DU'-anomaly for variable 'results' (lines '400'-'409').400 - 409
Avoid using final local variables, turn them into fields402
Found 'DD'-anomaly for variable 'results' (lines '403'-'403').403
Found 'DU'-anomaly for variable 'results' (lines '403'-'409').403 - 409

org/apache/jsieve/mailet/SieveMailboxMailet.java

ViolationLine
It is a good practice to call super() in a constructor88
Avoid using final local variables, turn them into fields195
Avoid using final local variables, turn them into fields196
Found 'DD'-anomaly for variable 'message' (lines '225'-'227').225 - 227
Found 'DU'-anomaly for variable 'deliveredTo' (lines '241'-'296').241 - 296
These nested if statements could be combined263 - 273
Use explicit scoping instead of the default package private level332 - 362
Avoid using final local variables, turn them into fields337
Use explicit scoping instead of the default package private level364 - 367

org/apache/jsieve/mailet/mdn/ActionModeAutomatic.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you34 - 37

org/apache/jsieve/mailet/mdn/ActionModeManual.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you34 - 37

org/apache/jsieve/mailet/mdn/ModifierError.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/ModifierExpired.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/ModifierFailed.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/ModifierMailboxTerminated.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/ModifierSuperseded.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/ModifierWarning.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/SendingModeAutomatic.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/SendingModeManual.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/TypeDeleted.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/TypeDenied.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/TypeDispatched.java

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

org/apache/jsieve/mailet/mdn/TypeDisplayed.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/TypeFailed.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35

org/apache/jsieve/mailet/mdn/TypeProcessed.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you32 - 35