RFR : 7162902 / 6893617 Umbrella port of a number of corba bug fixes from JDK 6 to jdk7u/8 (original) (raw)

Sean Coffey sean.coffey at oracle.com
Mon Jun 25 20:26:15 UTC 2012

Hi,

I'm looking for a code review around the following corba changes. It turns out that we've a few bug fixes in corba area for jdk6 that were never forward ported to jdk7 or 8. The port is pretty much identical to what was used in JDK6. Some formatting and diamond operator changes introduced but that's about it. There were a few bug fixes to follow up on regressions around the initial fix but this umbrella port should cover all issues.

The initial bugs fixes were fixed under the "jets" category in the bug tool and that category entered read only mode over a year ago. Hence the requirement for a new bug ID.

The main bug fix is 6725987 which is an issue where references to the ORB were being kept after ORB.destroy was called.

http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7162902 An accompanying CNCtx fix in JDK repo is also made to correct an issue where the java.naming.corba.orb system property wasn't used correctly. (CR 6893617) http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6893617

Corba testsuites run and no issues seen. I've also run some JCK testing and saw no issues.

http://cr.openjdk.java.net/~coffeys/webrev.7162902.jdk8/ (corba) http://cr.openjdk.java.net/~coffeys/webrev.6893617.jdk8/ (jdk)

The corba codebase code formatting seems to vary quite a bit. I've reformatted in a small number of areas but the whole corba repo probably warrants it's own clean up project at a later date.

regards, Sean.

More information about the core-libs-dev mailing list