[CAMEL-4796] Code Clean-Up

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.8.3
• Fix Version/s: 2.10.0
• Estimated Complexity: Unknown
• Assignee: Babak Vahdat
• Reporter: Babak Vahdat
• Created: 19/Dec/11 10:47
• Updated: 19/Jan/12 15:20
• Resolved: 16/Jan/12 11:30

Description

This ticket aims to resolve the issues already mentioned in [1], but just to name them ALL explicitly here:

• Removal of the unused imports
• Convert for loops to the enhanced one (supported since Java Tiger)
• Removal of the obsolete "super();" call by the Constructors (Since JDK 1.0 compilers already do it inside the byte-code for free!)
• Remove of all $NON-NLS$ tags
• Removal of the unnecessary casts
• Usage of the @Deprecated & @Override annotations where required (will be skipped because of the reasons Claus has already mentioned in his comment for the already provided patch)
• Avoidance of the raw type declarations by the generified classes as much as possible
• Removal of unused private memebers: Types, Constructors, Fields and Methods where applicable
• Remove the trailing whitespace on all lines, even on the empty ones

Add of missing serialVersionUID where required and default it to 1L where serialVersionUID has been already declared should be better postponed to the major 3.0.x release as otherwise the backward-compatility would be broken on the 2.x.y release branches.

If you can think of other possible clean-ups as well you may like to propose for including, then please first better discuss it in [1] so that other commiters can better/faster/easier react on it.

I'll provide the patches exactly in the same order as mentioned above.

Right now on the trunk my used IDE (eclipse) reports 1991 warnings!

[1] http://camel.465427.n5.nabble.com/DISCUSS-Trunk-Code-Cleanup-td5071594.html

Issue Links

• incorporates: CAMEL-4899 Code Clean-Up (continued)
• is related to: CAMEL-4921 Code Clean-Up (continued 2)