Re: RFR(L): 8008688: Make MethodHandleInfo public

2013-07-02 Thread John Rose
Thanks, Chris. -- John (on my iPhone) On Jul 2, 2013, at 9:17 PM, Christian Thalinger wrote: > Couldn't find any obvious problems. Looks good. > > -- Chris > > On Jul 2, 2013, at 6:26 PM, John Rose wrote: > >> Thanks for the helpful review, Vladimir. >> >> I have incorporated all your

Re: RFR(L): 8008688: Make MethodHandleInfo public

2013-07-02 Thread Christian Thalinger
Couldn't find any obvious problems. Looks good. -- Chris On Jul 2, 2013, at 6:26 PM, John Rose wrote: > Thanks for the helpful review, Vladimir. > > I have incorporated all your comments and updated the webrev here: > > http://cr.openjdk.java.net/~jrose/8008688/webrev.05 > > Detailed repli

Re: RFR(L): 8008688: Make MethodHandleInfo public

2013-07-02 Thread John Rose
Thanks for the helpful review, Vladimir. I have incorporated all your comments and updated the webrev here: http://cr.openjdk.java.net/~jrose/8008688/webrev.05 Detailed replies follow. On Jul 1, 2013, at 3:36 PM, Vladimir Ivanov wrote: > John, > > I have some minor suggestions about code

hg: mlvm/mlvm/jdk: meth-info-8008688.patch: incorporate review comments

2013-07-02 Thread john . r . rose
Changeset: 5e0f932f8ef0 Author:jrose Date: 2013-07-02 18:23 -0700 URL: http://hg.openjdk.java.net/mlvm/mlvm/jdk/rev/5e0f932f8ef0 meth-info-8008688.patch: incorporate review comments ! meth-info-8008688.patch ___ mlvm-dev mailing list mlv

hg: mlvm/mlvm/jdk: meth-info-8008688.patch: fix regression on BigArityTest

2013-07-02 Thread john . r . rose
Changeset: a49289c99587 Author:jrose Date: 2013-07-02 17:15 -0700 URL: http://hg.openjdk.java.net/mlvm/mlvm/jdk/rev/a49289c99587 meth-info-8008688.patch: fix regression on BigArityTest ! meth-info-8008688.patch ___ mlvm-dev mailing list

Re: sun.reflect.Reflection.getCallerClass(int) is going to be removed... how to replace?

2013-07-02 Thread John Rose
On Jul 2, 2013, at 7:00 AM, Jochen Theodorou wrote: > for example... Class.forName(String) That's a good example. If you call Class.forName via reflection, it will sense its immediate caller, which (depending on your system) might be a random frame in your language runtime. This approach get

Re: sun.reflect.Reflection.getCallerClass(int) is going to be removed... how to replace?

2013-07-02 Thread Jochen Theodorou
Am 02.07.2013 15:10, schrieb Alessio Stalla: > On Tue, Jul 2, 2013 at 2:56 PM, Jochen Theodorou wrote: >> I imagine other languages will have similar problems here and there... >> or not? > > Hmm... what are you using that method for? I'm not aware of uses of it > in ABCL, but that may be because

Re: sun.reflect.Reflection.getCallerClass(int) is going to be removed... how to replace?

2013-07-02 Thread Jochen Theodorou
Am 02.07.2013 15:10, schrieb Attila Szegedi: > FWIW, I'm currently working on Dynalink correctly supporting methods > that are marked with @CallerSensitive; My tests from Nashorn show > things work as expected, i.e. you can give some permissions to your > script based on its URL in the security pol

Re: sun.reflect.Reflection.getCallerClass(int) is going to be removed... how to replace?

2013-07-02 Thread Alessio Stalla
On Tue, Jul 2, 2013 at 2:56 PM, Jochen Theodorou wrote: > I imagine other languages will have similar problems here and there... > or not? Hmm... what are you using that method for? I'm not aware of uses of it in ABCL, but that may be because we aren't smart enough... ;) _

Re: sun.reflect.Reflection.getCallerClass(int) is going to be removed... how to replace?

2013-07-02 Thread Attila Szegedi
FWIW, I'm currently working on Dynalink correctly supporting methods that are marked with @CallerSensitive; My tests from Nashorn show things work as expected, i.e. you can give some permissions to your script based on its URL in the security policy, and e.g. invoking AccessController.doPrivileg

sun.reflect.Reflection.getCallerClass(int) is going to be removed... how to replace?

2013-07-02 Thread Jochen Theodorou
Hi all, """The Oracle JDK internal method sun.reflect.Reflection. getCallerClass(int) is planned to be disabled in Oracle JDK 7u40. It is considered for removal in a later 7 update release. As you may be aware, the Oracle JDK internal sun.reflect.Reflection.getCallerClass(int) method has been r