Looks good.
Best regards,
Vladimir Ivanov
On 11/17/15 2:43 PM, Claes Redestad wrote:
On 2015-11-17 12:22, Vladimir Ivanov wrote:
src/java.base/share/classes/java/lang/invoke/LambdaForm.java:
+ private static void createFormsFor(BasicType type) {
+ synchronized (LF_identity) {
Looks a bit confusing since the method touches both LF_identity and
LF_zero array now. Why not simply make the method synchronized?
LambdaForm is package-private and shouldn't leak outside j.l.i.
Fixed.
test/sun/invoke/util/WrapperTest.java:
@run junit/othervm
Why do you specify /othervm mode?
Changed to @run junit
http://cr.openjdk.java.net/~redestad/8142487/webrev.05
Thanks!
/Claes
