Standards Based Linux Instrumentation / Bugs / #1558 Code cleanup from empty statement et al

#1558 Code cleanup from empty statement et al

Milestone: Code_Cleanup

Status: closed-fixed

Owner: Dave Blaschke

Labels: Java Client (JSR48) (490)

Component:

Priority: 5

Updated: 2009-06-15

Created: 2009-04-10

Creator: Dave Blaschke

Private: No

Currently the "empty statement", "redundant null check", "redundant super interface" and "boxing and unboxing conversions" checks for the Java compiler are ignored. Checking them results in a few more fixable items.

Discussion

Dave Blaschke - 2009-04-10

Point of clarification, the redundant null checks were fixed in 2714989, this patch will actually change the .settings file as well...

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Blaschke - 2009-04-10

status: open --> open-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Blaschke - 2009-04-10

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.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Blaschke - 2009-04-10

Patch against Exp

2750520-patch.txt

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Blaschke - 2009-04-27

Patch against HEAD

2750520-patch.txt

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Blaschke - 2009-04-27

status: open-fixed --> pending-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Blaschke - 2009-04-27

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.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Blaschke - 2009-06-15

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

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Blaschke - 2009-06-15

status: pending-fixed --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.