Menu

#1409 Fix / clean code to remove compiler warnings

Code_Cleanup
closed-fixed
5
2008-12-12
2008-10-28
No

Currently there are approx 200 warnings thrown by compiler.

Code should be fixed / cleaned up to get rid of those warnings.

Most of them are javadoc warnings OR unused code / variables and should be fixed.

Discussion

  • Dave Blaschke

    Dave Blaschke - 2008-10-28
    • assigned_to: nobody --> raman_arora
     
  • Ramandeep Arora

    Ramandeep Arora - 2008-10-28

    This is a valid bug. We will create a patch a come back to you.

     
  • Ramandeep Arora

    Ramandeep Arora - 2008-10-28
    • assigned_to: raman_arora --> nobody
     
  • Ramandeep Arora

    Ramandeep Arora - 2008-10-28
    • status: open --> open-fixed
     
  • Ramandeep Arora

    Ramandeep Arora - 2008-10-28

    2204488 Fix code to remove compiler warnings

    Fixed code :
    - added javadoc comments
    - removed extra iimports
    - removed unused methods / variables
    - removed unthrows exceptions

    File Added: JDK1.4_WarningsFix_v5.txt

     
  • Ramandeep Arora

    Ramandeep Arora - 2008-10-28

    2204488 Fix code to remove compiler warnings

    Fixed code :
    - added javadoc comments
    - removed extra iimports
    - removed unused methods / variables
    - removed unthrows exceptions

    File Added: JDK1.4_WarningsFix_v5.txt

     
  • Ramandeep Arora

    Ramandeep Arora - 2008-10-28

    Fix code to remove compiler warnings

     
  • Ramandeep Arora

    Ramandeep Arora - 2008-10-28
    • assigned_to: nobody --> raman_arora
     
  • Ramandeep Arora

    Ramandeep Arora - 2008-10-30

    Patch against HEAD: Fix code to remove compiler warnings

     
  • Ramandeep Arora

    Ramandeep Arora - 2008-10-30

    File Added: 2204488_patch_HEAD_warnings_java4.txt

     
  • Dave Blaschke

    Dave Blaschke - 2008-10-30

    Patch sent for community review. During a 2 week period any exploiter may comment on the patch, request changes or turn it down completely (with good reason). For the time being the patch is part of the "Experimental" branch in CVS.

     
  • Dave Blaschke

    Dave Blaschke - 2008-11-24

    The community review is completed and we received no substantial critisism. Therefore the patch has been approved and merged into the "HEAD" branch. The next release will pick it up.

     
  • Dave Blaschke

    Dave Blaschke - 2008-11-24
    • status: open-fixed --> pending-fixed
     
  • Dave Blaschke

    Dave Blaschke - 2008-12-12

    The patch was picked up by release 2.0.9 and will be closed.

     
  • Dave Blaschke

    Dave Blaschke - 2008-12-12
    • status: pending-fixed --> closed-fixed
     

Log in to post a comment.