Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.3
    • Fix Version/s: 2.10.0
    • Component/s: None
    • Labels:
      None
    • Estimated Complexity:
      Unknown

      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

      1. CAMEL-4796-remove-unnecessary-casts.patch
        48 kB
        Babak Vahdat
      2. CAMEL-4796-remove-$NON-NLS$.patch
        6 kB
        Babak Vahdat
      3. CAMEL-4796-organize-imports-2.patch
        10 kB
        Babak Vahdat
      4. CAMEL-4796-organize-imports.patch
        157 kB
        Babak Vahdat
      5. CAMEL-4796-obsolete-super-call.patch
        59 kB
        Babak Vahdat
      6. CAMEL-4796-enhanced-for-loops.patch
        32 kB
        Babak Vahdat
      7. CAMEL-4796-avoid-rawtypes-3.patch
        199 kB
        Babak Vahdat
      8. CAMEL-4796-avoid-rawtypes-2.patch
        230 kB
        Babak Vahdat
      9. CAMEL-4796-avoid-rawtypes.patch
        169 kB
        Babak Vahdat
      10. CAMEL-4796-@Deprecated-@Override.patch
        3.67 MB
        Babak Vahdat

        Issue Links

          Activity

          Babak Vahdat created issue -
          Babak Vahdat made changes -
          Field Original Value New Value
          Description This ticket aims to resolve the issues mentioned in [1], just to name some of them:

          - organize imports
          - avoid unnecessary cast
          - add of missing serialVersionUID where required

          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.it.

          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
          This ticket aims to resolve the issues mentioned in [1], just to name some of them:

          - organize imports
          - avoid unnecessary cast
          - add of missing serialVersionUID where required

          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.

          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
          Babak Vahdat made changes -
          Attachment CAMEL-4796-organize-imports.patch [ 12508315 ]
          Hadrian Zbarcea made changes -
          Affects Version/s 2.8.3 [ 12318649 ]
          Affects Version/s 2.9.1 [ 12319191 ]
          Hadrian Zbarcea made changes -
          Fix Version/s 2.9.1 [ 12319191 ]
          Babak Vahdat made changes -
          Attachment CAMEL-4796-organize-imports-2.patch [ 12508445 ]
          Babak Vahdat made changes -
          Description This ticket aims to resolve the issues mentioned in [1], just to name some of them:

          - organize imports
          - avoid unnecessary cast
          - add of missing serialVersionUID where required

          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.

          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
          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
          - 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
          Babak Vahdat made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Babak Vahdat made changes -
          Attachment CAMEL-4796-enhanced-for-loops.patch [ 12508991 ]
          Babak Vahdat made changes -
          Attachment CAMEL-4796-enhanced-for-loops.patch [ 12508991 ]
          Babak Vahdat made changes -
          Attachment CAMEL-4796-enhanced-for-loops.patch [ 12509417 ]
          Claus Ibsen made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Fix Version/s 2.10 [ 12317612 ]
          Fix Version/s 2.9.1 [ 12319191 ]
          Resolution Fixed [ 1 ]
          Claus Ibsen made changes -
          Fix Version/s 2.9.1 [ 12319191 ]
          Babak Vahdat made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Babak Vahdat made changes -
          Attachment CAMEL-4796-obsolete-super-call.patch [ 12509471 ]
          Babak Vahdat made changes -
          Attachment CAMEL-4796-remove-$NON-NLS$.patch [ 12509533 ]
          Babak Vahdat made changes -
          Babak Vahdat made changes -
          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
          - 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
          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
          - 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
          Babak Vahdat made changes -
          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
          - 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
          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
          - 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
          Babak Vahdat made changes -
          Attachment CAMEL-4796-@Deprecated-@Override.patch [ 12509603 ]
          Babak Vahdat made changes -
          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
          - 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
          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
          Babak Vahdat made changes -
          Attachment CAMEL-4796-avoid-rawtypes.patch [ 12510298 ]
          Babak Vahdat made changes -
          Attachment CAMEL-4796-avoid-rawtypes-2.patch [ 12510495 ]
          Babak Vahdat made changes -
          Attachment CAMEL-4796-avoid-rawtypes-3.patch [ 12510611 ]
          Babak Vahdat made changes -
          Link This issue incorporates CAMEL-4899 [ CAMEL-4899 ]
          Claus Ibsen made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Fix Version/s 2.9.1 [ 12319191 ]
          Resolution Fixed [ 1 ]
          Babak Vahdat made changes -
          Link This issue is related to CAMEL-4921 [ CAMEL-4921 ]

            People

            • Assignee:
              Babak Vahdat
              Reporter:
              Babak Vahdat
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development