Review Request: 8006506: Add test for redefining methods in backtraces to java/lang/instrument tests (original) (raw)
Stefan Karlsson stefan.karlsson at oracle.com
Fri Feb 1 01:57:13 PST 2013
- Previous message: Review Request: 8006506: Add test for redefining methods in backtraces to java/lang/instrument tests
- Next message: Review Request: 8006506: Add test for redefining methods in backtraces to java/lang/instrument tests
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 2013-02-01 10:22, serguei.spitsyn at oracle.com wrote:
Nice test! It looks good. Thanks for reviewing!
As the original bug and the test are non-trivial, it'd make sense to add a comment to the class RedefineMethodInBacktraceApp and explain a little bit what the test is doing, and what behavior is expected. http://cr.openjdk.java.net/~stefank/8006506/webrev.04/
Tell me if you think this is good enough.
thanks, StefanK
Thanks, Sergueri On 2/1/13 12:13 AM, Stefan Karlsson wrote: http://cr.openjdk.java.net/~stefank/8006506/webrev.03/
1) Reverted the ProblemList change, since the fix has already propagaged to jdk8/tl 2) Renamed doredefine -> doRedefine 3) Updated the .sh file with the bug number of the original CR instead of the test CR. thanks, StefanK On 2013-01-22 14:11, Stefan Karlsson wrote: http://cr.openjdk.java.net/~stefank/8006506/webrev.00/
This test provokes the JVM crash described in bug: JDK-7174978. I intend to push this to: http://hg.openjdk.java.net/jdk8/tl/jdk thanks, StefanK
- Previous message: Review Request: 8006506: Add test for redefining methods in backtraces to java/lang/instrument tests
- Next message: Review Request: 8006506: Add test for redefining methods in backtraces to java/lang/instrument tests
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]