changeset 3246:ed4c306ec942

8129740: Incorrect class file created when passing lambda in inner class constructor Summary: Lambda implementation method must be static when lambda is an argument to an explicit constructor call. Reviewed-by: mcimadamore, jlahoda
author sadayapalam
date Mon, 09 Nov 2015 05:45:22 +0530
parents 16d09290bb2a
children 424fba7cabb0
files src/jdk.compiler/share/classes/com/sun/tools/javac/comp/LambdaToMethod.java src/jdk.compiler/share/classes/com/sun/tools/javac/tree/TreeMaker.java src/jdk.compiler/share/classes/com/sun/tools/javac/util/Names.java test/tools/javac/lambda/T8129740/AllowEnclosingVarCaptureTest.java test/tools/javac/lambda/T8129740/CaptureInCtorChainingTest.java test/tools/javac/lambda/T8129740/QualifiedThisAccessTest.java test/tools/javac/lambda/T8129740/SourceForTranslation.java test/tools/javac/lambda/T8129740/SourceToSourceTranslationTest.java test/tools/javac/lambda/T8129740/Universe.java.out
diffstat 9 files changed, 690 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/LambdaToMethod.java	Fri Nov 06 14:45:44 2015 +0530
+++ b/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/LambdaToMethod.java	Mon Nov 09 05:45:22 2015 +0530
@@ -338,6 +338,11 @@
                 syntheticInits.append((JCExpression) captured_local);
             }
         }
+        // add captured outer this instances (used only when `this' capture itself is illegal)
+        for (Symbol fv : localContext.getSymbolMap(CAPTURED_OUTER_THIS).keySet()) {
+            JCTree captured_local = make.QualThis(fv.type);
+            syntheticInits.append((JCExpression) captured_local);
+        }
 
         //then, determine the arguments to the indy call
         List<JCExpression> indy_args = translate(syntheticInits.toList(), localContext.prev);
@@ -434,6 +439,32 @@
         }
     }
 
+    /**
+     * Translate qualified `this' references within a lambda to the mapped identifier
+     * @param tree
+     */
+    @Override
+    public void visitSelect(JCFieldAccess tree) {
+        if (context == null || !analyzer.lambdaFieldAccessFilter(tree)) {
+            super.visitSelect(tree);
+        } else {
+            int prevPos = make.pos;
+            try {
+                make.at(tree);
+
+                LambdaTranslationContext lambdaContext = (LambdaTranslationContext) context;
+                JCTree ltree = lambdaContext.translate(tree);
+                if (ltree != null) {
+                    result = ltree;
+                } else {
+                    super.visitSelect(tree);
+                }
+            } finally {
+                make.at(prevPos);
+            }
+        }
+    }
+
     @Override
     public void visitVarDef(JCVariableDecl tree) {
         LambdaTranslationContext lambdaContext = (LambdaTranslationContext)context;
@@ -1128,6 +1159,11 @@
         private int lambdaCount = 0;
 
         /**
+         * List of types undergoing construction via explicit constructor chaining.
+         */
+        private List<ClassSymbol> typesUnderConstruction;
+
+        /**
          * keep the count of lambda expression defined in given context (used to
          * generate unambiguous names for serializable lambdas)
          */
@@ -1157,11 +1193,36 @@
 
         private JCClassDecl analyzeAndPreprocessClass(JCClassDecl tree) {
             frameStack = List.nil();
+            typesUnderConstruction = List.nil();
             localClassDefs = new HashMap<>();
             return translate(tree);
         }
 
         @Override
+        public void visitApply(JCMethodInvocation tree) {
+            List<ClassSymbol> previousNascentTypes = typesUnderConstruction;
+            try {
+                Name methName = TreeInfo.name(tree.meth);
+                if (methName == names._this || methName == names._super) {
+                    typesUnderConstruction = typesUnderConstruction.prepend(currentClass());
+                }
+                super.visitApply(tree);
+            } finally {
+                typesUnderConstruction = previousNascentTypes;
+            }
+        }
+            // where
+            private ClassSymbol currentClass() {
+                for (Frame frame : frameStack) {
+                    if (frame.tree.hasTag(JCTree.Tag.CLASSDEF)) {
+                        JCClassDecl cdef = (JCClassDecl) frame.tree;
+                        return cdef.sym;
+                    }
+                }
+                return null;
+            }
+
+        @Override
         public void visitBlock(JCBlock tree) {
             List<Frame> prevStack = frameStack;
             try {
@@ -1633,6 +1694,22 @@
         }
 
         /**
+         *  This is used to filter out those select nodes that need to be adjusted
+         *  when translating away lambda expressions - at the moment, this is the
+         *  set of nodes that select `this' (qualified this)
+         */
+        private boolean lambdaFieldAccessFilter(JCFieldAccess fAccess) {
+            LambdaTranslationContext lambdaContext =
+                    context instanceof LambdaTranslationContext ?
+                            (LambdaTranslationContext) context : null;
+            return lambdaContext != null
+                    && !fAccess.sym.isStatic()
+                    && fAccess.name == names._this
+                    && (fAccess.sym.owner.kind == TYP)
+                    && !lambdaContext.translatedSymbols.get(CAPTURED_OUTER_THIS).isEmpty();
+        }
+
+        /**
          * This is used to filter out those new class expressions that need to
          * be qualified with an enclosing tree
          */
@@ -1806,6 +1883,7 @@
                 translatedSymbols.put(LOCAL_VAR, new LinkedHashMap<Symbol, Symbol>());
                 translatedSymbols.put(CAPTURED_VAR, new LinkedHashMap<Symbol, Symbol>());
                 translatedSymbols.put(CAPTURED_THIS, new LinkedHashMap<Symbol, Symbol>());
+                translatedSymbols.put(CAPTURED_OUTER_THIS, new LinkedHashMap<Symbol, Symbol>());
                 translatedSymbols.put(TYPE_VAR, new LinkedHashMap<Symbol, Symbol>());
 
                 freeVarProcessedLocalClasses = new HashSet<>();
@@ -1918,6 +1996,16 @@
                             }
                         };
                         break;
+                    case CAPTURED_OUTER_THIS:
+                        Name name = names.fromString(new String(sym.flatName().toString() + names.dollarThis));
+                        ret = new VarSymbol(SYNTHETIC | FINAL | PARAMETER, name, types.erasure(sym.type), translatedSym) {
+                            @Override
+                            public Symbol baseSymbol() {
+                                //keep mapping with original captured symbol
+                                return sym;
+                            }
+                        };
+                        break;
                     case LOCAL_VAR:
                         ret = new VarSymbol(sym.flags() & FINAL, sym.name, sym.type, translatedSym);
                         ((VarSymbol) ret).pos = ((VarSymbol) sym).pos;
@@ -1938,6 +2026,14 @@
             }
 
             void addSymbol(Symbol sym, LambdaSymbolKind skind) {
+                if (skind == CAPTURED_THIS && sym != null && sym.kind == TYP && !typesUnderConstruction.isEmpty()) {
+                    ClassSymbol currentClass = currentClass();
+                    if (currentClass != null && typesUnderConstruction.contains(currentClass)) {
+                        // reference must be to enclosing outer instance, mutate capture kind.
+                        Assert.check(sym != currentClass); // should have been caught right in Attr
+                        skind = CAPTURED_OUTER_THIS;
+                    }
+                }
                 Map<Symbol, Symbol> transMap = getSymbolMap(skind);
                 if (!transMap.containsKey(sym)) {
                     transMap.put(sym, translate(sym, skind));
@@ -1951,17 +2047,49 @@
             }
 
             JCTree translate(JCIdent lambdaIdent) {
-                for (Map<Symbol, Symbol> m : translatedSymbols.values()) {
-                    if (m.containsKey(lambdaIdent.sym)) {
-                        Symbol tSym = m.get(lambdaIdent.sym);
-                        JCTree t = make.Ident(tSym).setType(lambdaIdent.type);
-                        tSym.setTypeAttributes(lambdaIdent.sym.getRawTypeAttributes());
-                        return t;
+                for (LambdaSymbolKind kind : LambdaSymbolKind.values()) {
+                    Map<Symbol, Symbol> m = getSymbolMap(kind);
+                    switch(kind) {
+                        default:
+                            if (m.containsKey(lambdaIdent.sym)) {
+                                Symbol tSym = m.get(lambdaIdent.sym);
+                                JCTree t = make.Ident(tSym).setType(lambdaIdent.type);
+                                tSym.setTypeAttributes(lambdaIdent.sym.getRawTypeAttributes());
+                                return t;
+                            }
+                            break;
+                        case CAPTURED_OUTER_THIS:
+                            if (lambdaIdent.sym.owner.kind == TYP && m.containsKey(lambdaIdent.sym.owner)) {
+                                // Transform outer instance variable references anchoring them to the captured synthetic.
+                                Symbol tSym = m.get(lambdaIdent.sym.owner);
+                                JCExpression t = make.Ident(tSym).setType(lambdaIdent.sym.owner.type);
+                                tSym.setTypeAttributes(lambdaIdent.sym.owner.getRawTypeAttributes());
+                                t = make.Select(t, lambdaIdent.name);
+                                t.setType(lambdaIdent.type);
+                                TreeInfo.setSymbol(t, lambdaIdent.sym);
+                                return t;
+                            }
+                            break;
                     }
                 }
                 return null;
             }
 
+            /* Translate away qualified this expressions, anchoring them to synthetic parameters that
+               capture the qualified this handle. `fieldAccess' is guaranteed to one such.
+            */
+            public JCTree translate(JCFieldAccess fieldAccess) {
+                Assert.check(fieldAccess.name == names._this);
+                Map<Symbol, Symbol> m = translatedSymbols.get(LambdaSymbolKind.CAPTURED_OUTER_THIS);
+                if (m.containsKey(fieldAccess.sym.owner)) {
+                    Symbol tSym = m.get(fieldAccess.sym.owner);
+                    JCExpression t = make.Ident(tSym).setType(fieldAccess.sym.owner.type);
+                    tSym.setTypeAttributes(fieldAccess.sym.owner.getRawTypeAttributes());
+                    return t;
+                }
+                return null;
+            }
+
             /**
              * The translatedSym is not complete/accurate until the analysis is
              * finished.  Once the analysis is finished, the translatedSym is
@@ -1999,6 +2127,10 @@
                     params.append(make.VarDef((VarSymbol) thisSym, null));
                     parameterSymbols.append((VarSymbol) thisSym);
                 }
+                for (Symbol thisSym : getSymbolMap(CAPTURED_OUTER_THIS).values()) {
+                    params.append(make.VarDef((VarSymbol) thisSym, null));
+                    parameterSymbols.append((VarSymbol) thisSym);
+                }
                 for (Symbol thisSym : getSymbolMap(PARAM).values()) {
                     params.append(make.VarDef((VarSymbol) thisSym, null));
                     parameterSymbols.append((VarSymbol) thisSym);
@@ -2147,6 +2279,7 @@
         LOCAL_VAR,      // original to translated lambda locals
         CAPTURED_VAR,   // variables in enclosing scope to translated synthetic parameters
         CAPTURED_THIS,  // class symbols to translated synthetic parameters (for captured member access)
+        CAPTURED_OUTER_THIS, // used when `this' capture is illegal, but outer this capture is legit (JDK-8129740)
         TYPE_VAR       // original to translated lambda type variables
     }
 
--- a/src/jdk.compiler/share/classes/com/sun/tools/javac/tree/TreeMaker.java	Fri Nov 06 14:45:44 2015 +0530
+++ b/src/jdk.compiler/share/classes/com/sun/tools/javac/tree/TreeMaker.java	Mon Nov 09 05:45:22 2015 +0530
@@ -628,6 +628,12 @@
         return Ident(new VarSymbol(FINAL, names._this, t, t.tsym));
     }
 
+    /** Create a tree representing qualified `this' given its type
+     */
+    public JCExpression QualThis(Type t) {
+        return Select(Type(t), new VarSymbol(FINAL, names._this, t, t.tsym));
+    }
+
     /** Create a tree representing a class literal.
      */
     public JCExpression ClassLiteral(ClassSymbol clazz) {
--- a/src/jdk.compiler/share/classes/com/sun/tools/javac/util/Names.java	Fri Nov 06 14:45:44 2015 +0530
+++ b/src/jdk.compiler/share/classes/com/sun/tools/javac/util/Names.java	Mon Nov 09 05:45:22 2015 +0530
@@ -177,6 +177,7 @@
     public final Name lambda;
     public final Name metafactory;
     public final Name altMetafactory;
+    public final Name dollarThis;
 
     public final Name.Table table;
 
@@ -235,6 +236,7 @@
         value = fromString("value");
         valueOf = fromString("valueOf");
         values = fromString("values");
+        dollarThis = fromString("$this");
 
         // class names
         java_io_Serializable = fromString("java.io.Serializable");
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/test/tools/javac/lambda/T8129740/AllowEnclosingVarCaptureTest.java	Mon Nov 09 05:45:22 2015 +0530
@@ -0,0 +1,59 @@
+/*
+ * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+/*
+ * @test
+ * @bug 8129740 8133111
+ * @summary Incorrect class file created when passing lambda in inner class constructor
+ * @run main AllowEnclosingVarCaptureTest
+ */
+
+public class AllowEnclosingVarCaptureTest {
+
+    int var = 0;
+
+    void foo() {
+        var *= 2;
+    }
+
+    public static void main(String[] args) {
+        new AllowEnclosingVarCaptureTest().new Inner(9764);
+    }
+
+    public class Inner {
+        public Inner(Runnable r) {
+            r.run();
+            if (var != 66704)
+                throw new AssertionError("Unexpected output: " + var);
+        }
+
+        public Inner(int x) {
+            this(() -> {
+                var = x + 1234;
+                AllowEnclosingVarCaptureTest.this.var += 5678;
+                foo();
+                AllowEnclosingVarCaptureTest.this.foo();
+            });
+        }
+    }
+}
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/test/tools/javac/lambda/T8129740/CaptureInCtorChainingTest.java	Mon Nov 09 05:45:22 2015 +0530
@@ -0,0 +1,54 @@
+/*
+ * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+/*
+ * @test
+ * @bug 8129740 8133111
+ * @summary Incorrect class file created when passing lambda in inner class constructor
+ * @run main CaptureInCtorChainingTest
+ */
+
+import java.util.function.Consumer;
+import java.util.function.Function;
+
+public class CaptureInCtorChainingTest {
+
+    CaptureInCtorChainingTest(Function<Function<Function<Consumer<Void>, Void>, Void>, Void> innerClass) {
+        new InnerClass(innerClass);
+    }
+
+    void foo(Void v) { }
+
+    class InnerClass {
+
+        InnerClass(Function<Function<Function<Consumer<Void>, Void>, Void>, Void> factory) {
+            this(factory.apply(o -> o.apply(CaptureInCtorChainingTest.this::foo)));
+        }
+
+        InnerClass(Void unused) { }
+    }
+
+    public static void main(String[] args) {
+        new CaptureInCtorChainingTest(o -> null);
+    }
+}
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/test/tools/javac/lambda/T8129740/QualifiedThisAccessTest.java	Mon Nov 09 05:45:22 2015 +0530
@@ -0,0 +1,157 @@
+/*
+ * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+/*
+ * @test
+ * @bug 8129740 8133111
+ * @summary Incorrect class file created when passing lambda in inner class constructor
+ * @run main QualifiedThisAccessTest
+ */
+
+public class QualifiedThisAccessTest { // Not referenced by lambda, so should not be captured.
+
+    public class Universe { // Not referenced by lambda, so should not be captured.
+
+        public String name;
+        public int galaxiesCount;
+
+        public Universe(String name, int galaxiesCount) {
+            this.name = name;
+            this.galaxiesCount = galaxiesCount;
+        }
+
+        public String toString() {
+            return "Universe" + name + " of " + galaxiesCount + " galaxies";
+        }
+
+        class Galaxy {
+
+            String name;
+            private int starsCount;
+
+            Galaxy(String name, int starsCount) {
+                this.name = name;
+                this.starsCount = starsCount;
+            }
+
+            public String toString() {
+                return "galaxy " + name + " of " + starsCount + " solar systems";
+            }
+
+            int starsCount() {
+                return starsCount;
+            }
+
+            private String name() {
+                return name;
+            }
+
+            class SolarSystem {
+
+                String name;
+                int planetsCount;
+
+                SolarSystem(String name, int planetsCount) {
+                    this.name = name;
+                    this.planetsCount = planetsCount;
+                }
+
+                public String toString() {
+                    return "Solar System of " + name + " with " + planetsCount + " planets";
+                }
+
+                int planetsCount() {
+                    return planetsCount;
+                }
+
+                SolarSystem copy(SolarSystem s) {
+                    return s;
+                }
+
+                class Planet {
+
+                    String name;
+                    int moonsCount;
+
+                    Planet(String name, int moonsCount, Runnable r) {
+                        this.name = name;
+                        this.moonsCount = moonsCount;
+                        r.run();
+                    }
+                    Planet (String name, int moonsCount) {
+                        this(name, moonsCount, ()-> {
+                            StringBuffer buf = new StringBuffer();
+                            buf.append("This planet belongs to the galaxy "
+                                        + Galaxy.this.name + " with " + starsCount + " stars\n");
+                            buf.append("This planet belongs to the galaxy "
+                                        + Universe.Galaxy.this.name + " with " + starsCount() + " stars\n");
+                            buf.append("This planet belongs to the galaxy "
+                                        + Galaxy.this.name() + " with " + starsCount() + " stars\n");
+                            buf.append("This planet belongs to the galaxy "
+                                        + Universe.Galaxy.this.name() + " with "
+                                        + (Universe.Galaxy.this).starsCount() + " stars\n");
+
+                            buf.append("This planet belongs to the solar system "
+                                        + SolarSystem.this.name + " with " + planetsCount + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + Galaxy.SolarSystem.this.name + " with " + planetsCount() + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + (SolarSystem.this).name + " with " + planetsCount + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + Universe.Galaxy.SolarSystem.this.name + " with "
+                                        + Universe.Galaxy.SolarSystem.this.planetsCount + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + Universe.Galaxy.SolarSystem.this.name.toLowerCase().toUpperCase()
+                                        + " with " + Universe.Galaxy.SolarSystem.this.planetsCount + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + copy(Universe.Galaxy.SolarSystem.this).name.toLowerCase().toUpperCase()
+                                        + " with " + Universe.Galaxy.SolarSystem.this.planetsCount + " planets\n");
+                            if (!buf.toString().equals(output))
+                                throw new AssertionError("Unexpected value\n" + buf);
+                        });
+                    }
+
+                    static final String output =
+                        "This planet belongs to the galaxy Mily way with 23456789 stars\n" +
+                        "This planet belongs to the galaxy Mily way with 23456789 stars\n" +
+                        "This planet belongs to the galaxy Mily way with 23456789 stars\n" +
+                        "This planet belongs to the galaxy Mily way with 23456789 stars\n" +
+                        "This planet belongs to the solar system Sun with 9 planets\n" +
+                        "This planet belongs to the solar system Sun with 9 planets\n" +
+                        "This planet belongs to the solar system Sun with 9 planets\n" +
+                        "This planet belongs to the solar system Sun with 9 planets\n" +
+                        "This planet belongs to the solar system SUN with 9 planets\n" +
+                        "This planet belongs to the solar system SUN with 9 planets\n";
+
+
+                    public String toString() {
+                        return "Planet " + name + " with " + moonsCount + " moon(s)";
+                    }
+                }
+            }
+        }
+    }
+    public static void main(String[] args) {
+        new QualifiedThisAccessTest().new Universe("Universe", 12345678).new Galaxy("Mily way", 23456789).new SolarSystem("Sun", 9).new Planet("Earth", 1);
+    }
+}
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/test/tools/javac/lambda/T8129740/SourceForTranslation.java	Mon Nov 09 05:45:22 2015 +0530
@@ -0,0 +1,122 @@
+    class Universe { // Not referenced by lambda, so should not be captured.
+
+        public String name;
+        public int galaxiesCount;
+
+        public Universe(String name, int galaxiesCount) {
+            this.name = name;
+            this.galaxiesCount = galaxiesCount;
+        }
+
+        public String toString() {
+            return "Universe" + name + " of " + galaxiesCount + " galaxies";
+        }
+
+        class Galaxy {
+
+            String name;
+            private int starsCount;
+
+            Galaxy(String name, int starsCount) {
+                this.name = name;
+                this.starsCount = starsCount;
+            }
+
+            public String toString() {
+                return "galaxy " + name + " of " + starsCount + " solar systems";
+            }
+
+            int starsCount() {
+                return starsCount;
+            }
+
+            private String name() {
+                return name;
+            }
+
+            class SolarSystem {
+
+                String name;
+                int planetsCount;
+
+                SolarSystem(String name, int planetsCount) {
+                    this.name = name;
+                    this.planetsCount = planetsCount;
+                }
+
+                public String toString() {
+                    return "Solar System of " + name + " with " + planetsCount + " planets";
+                }
+
+                int planetsCount() {
+                    return planetsCount;
+                }
+
+                SolarSystem copy(SolarSystem s) {
+                    return s;
+                }
+
+                class Planet {
+
+                    String name;
+                    int moonsCount;
+
+                    Planet(String name, int moonsCount, Runnable r) {
+                        this.name = name;
+                        this.moonsCount = moonsCount;
+                        r.run();
+                    }
+                    Planet (String name, int moonsCount) {
+                        this(name, moonsCount, ()-> {
+                            String n = name;
+                            StringBuffer buf = new StringBuffer();
+                            buf.append("This planet belongs to the galaxy "
+                                        + Galaxy.this.name + " with " + starsCount + " stars\n");
+                            buf.append("This planet belongs to the galaxy "
+                                        + Universe.Galaxy.this.name + " with " + starsCount() + " stars\n");
+                            buf.append("This planet belongs to the galaxy "
+                                        + Galaxy.this.name() + " with " + starsCount() + " stars\n");
+                            buf.append("This planet belongs to the galaxy "
+                                        + Universe.Galaxy.this.name() + " with "
+                                        + (Universe.Galaxy.this).starsCount() + " stars\n");
+
+                            buf.append("This planet belongs to the solar system "
+                                        + SolarSystem.this.name + " with " + planetsCount + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + Galaxy.SolarSystem.this.name + " with " + planetsCount() + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + (SolarSystem.this).name + " with " + planetsCount + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + Universe.Galaxy.SolarSystem.this.name + " with "
+                                        + Universe.Galaxy.SolarSystem.this.planetsCount + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + Universe.Galaxy.SolarSystem.this.name.toLowerCase().toUpperCase()
+                                        + " with " + Universe.Galaxy.SolarSystem.this.planetsCount + " planets\n");
+                            buf.append("This planet belongs to the solar system "
+                                        + copy(Universe.Galaxy.SolarSystem.this).name.toLowerCase().toUpperCase()
+                                        + " with " + Universe.Galaxy.SolarSystem.this.planetsCount + " planets\n");
+                            if (!buf.toString().equals(output))
+                                throw new AssertionError("Unexpected value\n" + buf);
+                        });
+                    }
+
+                    static final String output =
+                        "This planet belongs to the galaxy Mily way with 23456789 stars\n" +
+                        "This planet belongs to the galaxy Mily way with 23456789 stars\n" +
+                        "This planet belongs to the galaxy Mily way with 23456789 stars\n" +
+                        "This planet belongs to the galaxy Mily way with 23456789 stars\n" +
+                        "This planet belongs to the solar system Sun with 9 planets\n" +
+                        "This planet belongs to the solar system Sun with 9 planets\n" +
+                        "This planet belongs to the solar system Sun with 9 planets\n" +
+                        "This planet belongs to the solar system Sun with 9 planets\n" +
+                        "This planet belongs to the solar system SUN with 9 planets\n" +
+                        "This planet belongs to the solar system SUN with 9 planets\n";
+
+
+                    public String toString() {
+                        return "Planet " + name + " with " + moonsCount + " moon(s)";
+                    }
+                }
+            }
+        }
+    }
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/test/tools/javac/lambda/T8129740/SourceToSourceTranslationTest.java	Mon Nov 09 05:45:22 2015 +0530
@@ -0,0 +1,53 @@
+/*
+ * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+/*
+ * @test
+ * @bug 8129740 8133111
+ * @summary Incorrect class file created when passing lambda in inner class constructor
+ * @library /tools/lib
+ * @modules jdk.compiler/com.sun.tools.javac.api
+ *          jdk.compiler/com.sun.tools.javac.file
+ *          jdk.compiler/com.sun.tools.javac.main
+ * @build ToolBox
+ * @run compile -XD-printsource SourceForTranslation.java
+ * @run main SourceToSourceTranslationTest
+ */
+
+import java.nio.file.Path;
+import java.nio.file.Paths;
+import java.util.List;
+
+public class SourceToSourceTranslationTest {
+
+    public static void main(String[] args) throws Exception {
+        ToolBox tb = new ToolBox();
+        Path path1 = Paths.get(ToolBox.testClasses, "Universe.java");
+        List<String> file1 = tb.readAllLines(path1);
+
+        Path path2 = Paths.get(ToolBox.testSrc, "Universe.java.out");
+        List<String> file2 = tb.readAllLines(path2);
+        tb.checkEqual(file1, file2);
+    }
+
+}
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/test/tools/javac/lambda/T8129740/Universe.java.out	Mon Nov 09 05:45:22 2015 +0530
@@ -0,0 +1,98 @@
+
+class Universe {
+    public String name;
+    public int galaxiesCount;
+    
+    public Universe(String name, int galaxiesCount) {
+        super();
+        this.name = name;
+        this.galaxiesCount = galaxiesCount;
+    }
+    
+    public String toString() {
+        return "Universe" + name + " of " + galaxiesCount + " galaxies";
+    }
+    
+    class Galaxy {
+        String name;
+        private int starsCount;
+        
+        Galaxy(String name, int starsCount) {
+            super();
+            this.name = name;
+            this.starsCount = starsCount;
+        }
+        
+        public String toString() {
+            return "galaxy " + name + " of " + starsCount + " solar systems";
+        }
+        
+        int starsCount() {
+            return starsCount;
+        }
+        
+        private String name() {
+            return name;
+        }
+        
+        class SolarSystem {
+            String name;
+            int planetsCount;
+            
+            SolarSystem(String name, int planetsCount) {
+                super();
+                this.name = name;
+                this.planetsCount = planetsCount;
+            }
+            
+            public String toString() {
+                return "Solar System of " + name + " with " + planetsCount + " planets";
+            }
+            
+            int planetsCount() {
+                return planetsCount;
+            }
+            
+            SolarSystem copy(SolarSystem s) {
+                return s;
+            }
+            
+            class Planet {
+                String name;
+                int moonsCount;
+                
+                Planet(String name, int moonsCount, Runnable r) {
+                    super();
+                    this.name = name;
+                    this.moonsCount = moonsCount;
+                    r.run();
+                }
+                
+                Planet(String name, int moonsCount) {
+                    this(name, moonsCount, java.lang.invoke.LambdaMetafactory.metafactory(name, Universe.Galaxy.this, Universe.Galaxy.SolarSystem.this));
+                }
+                static final String output = "This planet belongs to the galaxy Mily way with 23456789 stars\nThis planet belongs to the galaxy Mily way with 23456789 stars\nThis planet belongs to the galaxy Mily way with 23456789 stars\nThis planet belongs to the galaxy Mily way with 23456789 stars\nThis planet belongs to the solar system Sun with 9 planets\nThis planet belongs to the solar system Sun with 9 planets\nThis planet belongs to the solar system Sun with 9 planets\nThis planet belongs to the solar system Sun with 9 planets\nThis planet belongs to the solar system SUN with 9 planets\nThis planet belongs to the solar system SUN with 9 planets\n";
+                
+                public String toString() {
+                    return "Planet " + name + " with " + moonsCount + " moon(s)";
+                }
+                
+                /*synthetic*/ private static void lambda$new$0(/*synthetic*/ final String name, /*synthetic*/ final Universe.Galaxy Universe$Galaxy$this, /*synthetic*/ final Universe.Galaxy.SolarSystem Universe$Galaxy$SolarSystem$this) {
+                    String n = name;
+                    StringBuffer buf = new StringBuffer();
+                    buf.append("This planet belongs to the galaxy " + Universe$Galaxy$this.name + " with " + Universe$Galaxy$this.starsCount + " stars\n");
+                    buf.append("This planet belongs to the galaxy " + Universe$Galaxy$this.name + " with " + Universe$Galaxy$this.starsCount() + " stars\n");
+                    buf.append("This planet belongs to the galaxy " + Universe$Galaxy$this.name() + " with " + Universe$Galaxy$this.starsCount() + " stars\n");
+                    buf.append("This planet belongs to the galaxy " + Universe$Galaxy$this.name() + " with " + (Universe$Galaxy$this).starsCount() + " stars\n");
+                    buf.append("This planet belongs to the solar system " + Universe$Galaxy$SolarSystem$this.name + " with " + Universe$Galaxy$SolarSystem$this.planetsCount + " planets\n");
+                    buf.append("This planet belongs to the solar system " + Universe$Galaxy$SolarSystem$this.name + " with " + Universe$Galaxy$SolarSystem$this.planetsCount() + " planets\n");
+                    buf.append("This planet belongs to the solar system " + (Universe$Galaxy$SolarSystem$this).name + " with " + Universe$Galaxy$SolarSystem$this.planetsCount + " planets\n");
+                    buf.append("This planet belongs to the solar system " + Universe$Galaxy$SolarSystem$this.name + " with " + Universe$Galaxy$SolarSystem$this.planetsCount + " planets\n");
+                    buf.append("This planet belongs to the solar system " + Universe$Galaxy$SolarSystem$this.name.toLowerCase().toUpperCase() + " with " + Universe$Galaxy$SolarSystem$this.planetsCount + " planets\n");
+                    buf.append("This planet belongs to the solar system " + Universe$Galaxy$SolarSystem$this.copy(Universe$Galaxy$SolarSystem$this).name.toLowerCase().toUpperCase() + " with " + Universe$Galaxy$SolarSystem$this.planetsCount + " planets\n");
+                    if (!buf.toString().equals(output)) throw new AssertionError("Unexpected value\n" + buf);
+                }
+            }
+        }
+    }
+}