Re: review request (L): 6984311 JSR 292 needs optional bootstrap method parameters

2010-10-30 Thread John Rose
On Oct 29, 2010, at 4:09 AM, Christian Thalinger wrote: > On Oct 28, 2010, at 8:40 PM, John Rose wrote: >> 6984311: JSR 292 needs optional bootstrap method parameters >> >> http://cr.openjdk.java.net/~jrose/6984311/webrev.02/ > > src/share/vm/oops/constantPoolOop.cpp: > 576 // Benign race co

Re: review request (XL): 6981777: implement JSR 292 EG adjustments from summer 2010

2010-10-30 Thread John Rose
Thanks for the review. I will answer it in detail shortly. Here's a related (tiny) review request for JVM code. http://cr.openjdk.java.net/~jrose/6984311/webrev.03/src/share/vm/prims/methodHandles.cpp.udiff.html A tiny JVM bug was uncovered by unit-testing the big JDK changes for 6981777.

hg: mlvm/mlvm/hotspot: meth, indy: incorporate reviewer comments

2010-10-30 Thread john . r . rose
Changeset: 18f563972447 Author:jrose Date: 2010-10-30 13:14 -0700 URL: http://hg.openjdk.java.net/mlvm/mlvm/hotspot/rev/18f563972447 meth, indy: incorporate reviewer comments ! indy-args-6984311.patch ! meth-api-6981777.patch ! meth.patch ! series _

Re: review request (XL): 6981777: implement JSR 292 EG adjustments from summer 2010

2010-10-30 Thread John Rose
On Oct 28, 2010, at 4:30 AM, Christian Thalinger wrote: > On Oct 26, 2010, at 10:39 AM, John Rose wrote: >> This is a large set of miscellaneous API adjustments that takes account of >> intensive JSR 292 Expert Group work since the JVM Language Summit and >> JavaOne. >> >> http://cr.openjdk.jav

hg: mlvm/mlvm/jdk: meth: incorporate reviewer comments

2010-10-30 Thread john . r . rose
Changeset: 6446c829a965 Author:jrose Date: 2010-10-30 13:50 -0700 URL: http://hg.openjdk.java.net/mlvm/mlvm/jdk/rev/6446c829a965 meth: incorporate reviewer comments ! meth-api-6981777.patch ___ mlvm-dev mailing list mlvm-dev@openjdk.java