code review request : 8003147 port fix for BCEL bug 39695

David Buck david.buck at oracle.com
Mon Dec 10 06:25:54 UTC 2012


Hi!

I would like to request a code review of my JDK8 fix for the following 
issue:

[ 8003147 : port fix for BCEL bug 39695 to our copy bundled as part of 
jaxp ]
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8003147

In addition to the fix that the BCEL project had for this issue, I 
needed to "port" some of their support for LocalVariableTypeTable:s to 
our copy of BCEL as well. Implementing support for LVTT is very 
straightforward and does not add much to the complexity or risk of this 
change (especially considering the limited scope of jaxp's use of BCEL).

Here is the webrev for my fix:

[ Code Review for jaxp ]
http://cr.openjdk.java.net/~dbuck/8003147/webrev.00/

My understanding is that the test cases for our copy of the jaxp code 
are handled in a separate repository / test suite. I have been in 
contact with Patrick Zhang (Oracle QA) and Joe Wang and have provided a 
junit test for this issue as requested. Please see bug report for a 
simpler (non-junit) test-case. If for some reason anyone wants to see 
the junit-based test, please let me know and I can provide a copy of 
that as well.

Best Regards,
-Buck



More information about the core-libs-dev mailing list