[9] Review request : JDK-8058728: TEST_BUG: Make java/lang/invoke/LFCaching/LFGarbageCollectedTest.java skip arrayElementSetter and arrayElementGetter methods

2014-09-19 Thread Konstantin Shefov
Hello, Please review the test bug fix https://bugs.openjdk.java.net/browse/JDK-8058728 Webrev is http://cr.openjdk.java.net/~kshefov/8058728/webrev.00 Thanks -Konstantin ___ mlvm-dev mailing list mlvm-dev@openjdk.java.net

Re: [9] Review request : JDK-8058728: TEST_BUG: Make java/lang/invoke/LFCaching/LFGarbageCollectedTest.java skip arrayElementSetter and arrayElementGetter methods

2014-09-19 Thread Vladimir Ivanov
Looks good. Best regards, Vladimir Ivanov On 9/19/14, 1:50 PM, Konstantin Shefov wrote: Hello, Please review the test bug fix https://bugs.openjdk.java.net/browse/JDK-8058728 Webrev is http://cr.openjdk.java.net/~kshefov/8058728/webrev.00 Thanks -Konstantin

Re: [9] Review request : JDK-8058728: TEST_BUG: Make java/lang/invoke/LFCaching/LFGarbageCollectedTest.java skip arrayElementSetter and arrayElementGetter methods

2014-09-19 Thread Paul Sandoz
On Sep 19, 2014, at 11:54 AM, Vladimir Ivanov vladimir.x.iva...@oracle.com wrote: Looks good. Small typo in the comment: s/filed/field Otherwise +1, Paul. Best regards, Vladimir Ivanov On 9/19/14, 1:50 PM, Konstantin Shefov wrote: Hello, Please review the test bug fix

Re: [9] Review request : JDK-8058728: TEST_BUG: Make java/lang/invoke/LFCaching/LFGarbageCollectedTest.java skip arrayElementSetter and arrayElementGetter methods

2014-09-19 Thread Igor Ignatyev
Hi, looks good to me. how have you tested your changes? Thanks, Igor On 09/19/2014 01:50 PM, Konstantin Shefov wrote: Hello, Please review the test bug fix https://bugs.openjdk.java.net/browse/JDK-8058728 Webrev is http://cr.openjdk.java.net/~kshefov/8058728/webrev.00 Thanks -Konstantin

Re: [9] Review request : JDK-8058728: TEST_BUG: Make java/lang/invoke/LFCaching/LFGarbageCollectedTest.java skip arrayElementSetter and arrayElementGetter methods

2014-09-19 Thread Konstantin Shefov
Yes, the test compiles and runs ok, but fails because of 8057020. If we remove -Djava.lang.invoke.MethodHandle.USE_LF_EDITOR=true, the test passes. -Konstantin On 19.09.2014 16:01, Igor Ignatyev wrote: Hi, looks good to me. how have you tested your changes? Thanks, Igor On 09/19/2014

Re: [9] Review request : JDK-8058728: TEST_BUG: Make java/lang/invoke/LFCaching/LFGarbageCollectedTest.java skip arrayElementSetter and arrayElementGetter methods

2014-09-19 Thread igor.ignat...@oracle.com
thx. Reviewed. -- II - Reply message - From: Konstantin Shefov konstantin.she...@oracle.com To: Igor Ignatyev igor.ignat...@oracle.com Cc: VLADIMIR.X.IVANOV vladimir.x.iva...@oracle.com, core-libs-...@openjdk.java.net, mlvm-dev@openjdk.java.net Subject: [9] Review request : JDK-8058728: