langtools/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/MemberEnter.java
author jlahoda
Thu, 15 Jun 2017 13:44:42 +0200
changeset 45504 ea7475564d07
parent 45500 365640343c55
permissions -rw-r--r--
8170326: Inconsistencies between code, compiler.properties and comments Summary: Converting uses of Log and JCDiagnostic.Factory methods to use CompilerProperties instead of plain Strings, fixing inconsistencies, adding crules analyzer to ensure CompilerProperties are used whenever possible. Reviewed-by: mcimadamore
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
10
06bc494ca11e Initial load
duke
parents:
diff changeset
     1
/*
45500
365640343c55 8181464: Invalid lambda in annotation causes NPE in Lint.augment
vromero
parents: 37744
diff changeset
     2
 * Copyright (c) 2003, 2017, Oracle and/or its affiliates. All rights reserved.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
06bc494ca11e Initial load
duke
parents:
diff changeset
     4
 *
06bc494ca11e Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
06bc494ca11e Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
5520
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 4877
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
10
06bc494ca11e Initial load
duke
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
5520
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 4877
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    10
 *
06bc494ca11e Initial load
duke
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
06bc494ca11e Initial load
duke
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
06bc494ca11e Initial load
duke
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
06bc494ca11e Initial load
duke
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
06bc494ca11e Initial load
duke
parents:
diff changeset
    15
 * accompanied this code).
06bc494ca11e Initial load
duke
parents:
diff changeset
    16
 *
06bc494ca11e Initial load
duke
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
06bc494ca11e Initial load
duke
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
06bc494ca11e Initial load
duke
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
06bc494ca11e Initial load
duke
parents:
diff changeset
    20
 *
5520
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 4877
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 4877
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 4877
diff changeset
    23
 * questions.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    24
 */
06bc494ca11e Initial load
duke
parents:
diff changeset
    25
06bc494ca11e Initial load
duke
parents:
diff changeset
    26
package com.sun.tools.javac.comp;
06bc494ca11e Initial load
duke
parents:
diff changeset
    27
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
    28
import java.util.EnumSet;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
    29
import java.util.Set;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
    30
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    31
import com.sun.tools.javac.code.*;
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25007
diff changeset
    32
import com.sun.tools.javac.code.Scope.WriteableScope;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    33
import com.sun.tools.javac.tree.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    34
import com.sun.tools.javac.util.*;
27857
7e913a535736 7101822: Compiling depends on order of imports
jlahoda
parents: 27854
diff changeset
    35
import com.sun.tools.javac.util.JCDiagnostic.DiagnosticPosition;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    36
24895
dd091d389fbf 8027262: Determine location for type annotations earlier in compiler pipeline
emc
parents: 24604
diff changeset
    37
import com.sun.tools.javac.code.Symbol.*;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    38
import com.sun.tools.javac.code.Type.*;
45504
ea7475564d07 8170326: Inconsistencies between code, compiler.properties and comments
jlahoda
parents: 45500
diff changeset
    39
import com.sun.tools.javac.resources.CompilerProperties.Errors;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    40
import com.sun.tools.javac.tree.JCTree.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    41
06bc494ca11e Initial load
duke
parents:
diff changeset
    42
import static com.sun.tools.javac.code.Flags.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    43
import static com.sun.tools.javac.code.Kinds.*;
27224
228abfa87080 8054457: Refactor Symbol kinds from small ints to an enum
emc
parents: 26781
diff changeset
    44
import static com.sun.tools.javac.code.Kinds.Kind.*;
14359
d4099818ab70 7200915: convert TypeTags from a series of small ints to an enum
jjg
parents: 14259
diff changeset
    45
import static com.sun.tools.javac.code.TypeTag.TYPEVAR;
24604
7f68545b5128 8041422: Split javac ClassReader into ClassReader+ClassFinder
jjg
parents: 24396
diff changeset
    46
27857
7e913a535736 7101822: Compiling depends on order of imports
jlahoda
parents: 27854
diff changeset
    47
/** Resolves field, method and constructor header, and constructs corresponding Symbols.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    48
 *
5847
1908176fd6e3 6944312: Potential rebranding issues in openjdk/langtools repository sources
jjg
parents: 5520
diff changeset
    49
 *  <p><b>This is NOT part of any supported API.
1908176fd6e3 6944312: Potential rebranding issues in openjdk/langtools repository sources
jjg
parents: 5520
diff changeset
    50
 *  If you write code that depends on this, you do so at your own risk.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    51
 *  This code and its internal interfaces are subject to change or
06bc494ca11e Initial load
duke
parents:
diff changeset
    52
 *  deletion without notice.</b>
06bc494ca11e Initial load
duke
parents:
diff changeset
    53
 */
27857
7e913a535736 7101822: Compiling depends on order of imports
jlahoda
parents: 27854
diff changeset
    54
public class MemberEnter extends JCTree.Visitor {
22163
3651128c74eb 8030244: Update langtools to use Diamond
briangoetz
parents: 21498
diff changeset
    55
    protected static final Context.Key<MemberEnter> memberEnterKey = new Context.Key<>();
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    56
06bc494ca11e Initial load
duke
parents:
diff changeset
    57
    private final Enter enter;
06bc494ca11e Initial load
duke
parents:
diff changeset
    58
    private final Log log;
06bc494ca11e Initial load
duke
parents:
diff changeset
    59
    private final Check chk;
06bc494ca11e Initial load
duke
parents:
diff changeset
    60
    private final Attr attr;
06bc494ca11e Initial load
duke
parents:
diff changeset
    61
    private final Symtab syms;
06bc494ca11e Initial load
duke
parents:
diff changeset
    62
    private final Annotate annotate;
06bc494ca11e Initial load
duke
parents:
diff changeset
    63
    private final Types types;
8236
0d8646b7c602 6594914: @SuppressWarnings("deprecation") does not not work for the type of a variable
mcimadamore
parents: 8225
diff changeset
    64
    private final DeferredLintHandler deferredLintHandler;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    65
06bc494ca11e Initial load
duke
parents:
diff changeset
    66
    public static MemberEnter instance(Context context) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    67
        MemberEnter instance = context.get(memberEnterKey);
06bc494ca11e Initial load
duke
parents:
diff changeset
    68
        if (instance == null)
06bc494ca11e Initial load
duke
parents:
diff changeset
    69
            instance = new MemberEnter(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    70
        return instance;
06bc494ca11e Initial load
duke
parents:
diff changeset
    71
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    72
06bc494ca11e Initial load
duke
parents:
diff changeset
    73
    protected MemberEnter(Context context) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    74
        context.put(memberEnterKey, this);
06bc494ca11e Initial load
duke
parents:
diff changeset
    75
        enter = Enter.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    76
        log = Log.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    77
        chk = Check.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    78
        attr = Attr.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    79
        syms = Symtab.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    80
        annotate = Annotate.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    81
        types = Types.instance(context);
8236
0d8646b7c602 6594914: @SuppressWarnings("deprecation") does not not work for the type of a variable
mcimadamore
parents: 8225
diff changeset
    82
        deferredLintHandler = DeferredLintHandler.instance(context);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    83
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    84
06bc494ca11e Initial load
duke
parents:
diff changeset
    85
    /** Construct method type from method signature.
06bc494ca11e Initial load
duke
parents:
diff changeset
    86
     *  @param typarams    The method's type parameters.
06bc494ca11e Initial load
duke
parents:
diff changeset
    87
     *  @param params      The method's value parameters.
06bc494ca11e Initial load
duke
parents:
diff changeset
    88
     *  @param res             The method's result type,
06bc494ca11e Initial load
duke
parents:
diff changeset
    89
     *                 null if it is a constructor.
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
    90
     *  @param recvparam       The method's receiver parameter,
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
    91
     *                 null if none given; TODO: or already set here?
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    92
     *  @param thrown      The method's thrown exceptions.
06bc494ca11e Initial load
duke
parents:
diff changeset
    93
     *  @param env             The method's (local) environment.
06bc494ca11e Initial load
duke
parents:
diff changeset
    94
     */
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    95
    Type signature(MethodSymbol msym,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    96
                   List<JCTypeParameter> typarams,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    97
                   List<JCVariableDecl> params,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    98
                   JCTree res,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    99
                   JCVariableDecl recvparam,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   100
                   List<JCExpression> thrown,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   101
                   Env<AttrContext> env) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   102
06bc494ca11e Initial load
duke
parents:
diff changeset
   103
        // Enter and attribute type parameters.
06bc494ca11e Initial load
duke
parents:
diff changeset
   104
        List<Type> tvars = enter.classEnter(typarams, env);
06bc494ca11e Initial load
duke
parents:
diff changeset
   105
        attr.attribTypeVariables(typarams, env);
06bc494ca11e Initial load
duke
parents:
diff changeset
   106
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   107
        // Enter and attribute value parameters.
22163
3651128c74eb 8030244: Update langtools to use Diamond
briangoetz
parents: 21498
diff changeset
   108
        ListBuffer<Type> argbuf = new ListBuffer<>();
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   109
        for (List<JCVariableDecl> l = params; l.nonEmpty(); l = l.tail) {
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   110
            memberEnter(l.head, env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   111
            argbuf.append(l.head.vartype.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   112
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   113
06bc494ca11e Initial load
duke
parents:
diff changeset
   114
        // Attribute result type, if one is given.
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   115
        Type restype = res == null ? syms.voidType : attr.attribType(res, env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   116
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   117
        // Attribute receiver type, if one is given.
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   118
        Type recvtype;
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   119
        if (recvparam!=null) {
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   120
            memberEnter(recvparam, env);
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   121
            recvtype = recvparam.vartype.type;
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   122
        } else {
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   123
            recvtype = null;
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   124
        }
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   125
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   126
        // Attribute thrown exceptions.
22163
3651128c74eb 8030244: Update langtools to use Diamond
briangoetz
parents: 21498
diff changeset
   127
        ListBuffer<Type> thrownbuf = new ListBuffer<>();
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   128
        for (List<JCExpression> l = thrown; l.nonEmpty(); l = l.tail) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   129
            Type exc = attr.attribType(l.head, env);
18909
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   130
            if (!exc.hasTag(TYPEVAR)) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   131
                exc = chk.checkClassType(l.head.pos(), exc);
18909
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   132
            } else if (exc.tsym.owner == msym) {
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   133
                //mark inference variables in 'throws' clause
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   134
                exc.tsym.flags_field |= THROWS;
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   135
            }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   136
            thrownbuf.append(exc);
06bc494ca11e Initial load
duke
parents:
diff changeset
   137
        }
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   138
        MethodType mtype = new MethodType(argbuf.toList(),
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   139
                                    restype,
06bc494ca11e Initial load
duke
parents:
diff changeset
   140
                                    thrownbuf.toList(),
06bc494ca11e Initial load
duke
parents:
diff changeset
   141
                                    syms.methodClass);
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   142
        mtype.recvtype = recvtype;
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   143
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   144
        return tvars.isEmpty() ? mtype : new ForAll(tvars, mtype);
06bc494ca11e Initial load
duke
parents:
diff changeset
   145
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   146
06bc494ca11e Initial load
duke
parents:
diff changeset
   147
/* ********************************************************************
06bc494ca11e Initial load
duke
parents:
diff changeset
   148
 * Visitor methods for member enter
06bc494ca11e Initial load
duke
parents:
diff changeset
   149
 *********************************************************************/
06bc494ca11e Initial load
duke
parents:
diff changeset
   150
06bc494ca11e Initial load
duke
parents:
diff changeset
   151
    /** Visitor argument: the current environment
06bc494ca11e Initial load
duke
parents:
diff changeset
   152
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   153
    protected Env<AttrContext> env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   154
06bc494ca11e Initial load
duke
parents:
diff changeset
   155
    /** Enter field and method definitions and process import
06bc494ca11e Initial load
duke
parents:
diff changeset
   156
     *  clauses, catching any completion failure exceptions.
06bc494ca11e Initial load
duke
parents:
diff changeset
   157
     */
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   158
    protected void memberEnter(JCTree tree, Env<AttrContext> env) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   159
        Env<AttrContext> prevEnv = this.env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   160
        try {
06bc494ca11e Initial load
duke
parents:
diff changeset
   161
            this.env = env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   162
            tree.accept(this);
06bc494ca11e Initial load
duke
parents:
diff changeset
   163
        }  catch (CompletionFailure ex) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   164
            chk.completionError(tree.pos(), ex);
06bc494ca11e Initial load
duke
parents:
diff changeset
   165
        } finally {
06bc494ca11e Initial load
duke
parents:
diff changeset
   166
            this.env = prevEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   167
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   168
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   169
06bc494ca11e Initial load
duke
parents:
diff changeset
   170
    /** Enter members from a list of trees.
06bc494ca11e Initial load
duke
parents:
diff changeset
   171
     */
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   172
    void memberEnter(List<? extends JCTree> trees, Env<AttrContext> env) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   173
        for (List<? extends JCTree> l = trees; l.nonEmpty(); l = l.tail)
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   174
            memberEnter(l.head, env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   175
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   176
06bc494ca11e Initial load
duke
parents:
diff changeset
   177
    public void visitMethodDef(JCMethodDecl tree) {
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25007
diff changeset
   178
        WriteableScope enclScope = enter.enterScope(env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   179
        MethodSymbol m = new MethodSymbol(0, tree.name, null, enclScope.owner);
06bc494ca11e Initial load
duke
parents:
diff changeset
   180
        m.flags_field = chk.checkFlags(tree.pos(), tree.mods.flags, m, tree);
06bc494ca11e Initial load
duke
parents:
diff changeset
   181
        tree.sym = m;
14443
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   182
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   183
        //if this is a default method, add the DEFAULT flag to the enclosing interface
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   184
        if ((tree.mods.flags & DEFAULT) != 0) {
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   185
            m.enclClass().flags_field |= DEFAULT;
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   186
        }
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   187
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   188
        Env<AttrContext> localEnv = methodEnv(tree, env);
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   189
        DiagnosticPosition prevLintPos = deferredLintHandler.setPos(tree.pos());
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   190
        try {
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   191
            // Compute the method type
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   192
            m.type = signature(m, tree.typarams, tree.params,
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   193
                               tree.restype, tree.recvparam,
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   194
                               tree.thrown,
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   195
                               localEnv);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   196
        } finally {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   197
            deferredLintHandler.setPos(prevLintPos);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   198
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   199
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   200
        if (types.isSignaturePolymorphic(m)) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   201
            m.flags_field |= SIGNATURE_POLYMORPHIC;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   202
        }
18389
a425d0819f36 8016569: javac, add new flag for polymorphic method signatures
vromero
parents: 18010
diff changeset
   203
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   204
        // Set m.params
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   205
        ListBuffer<VarSymbol> params = new ListBuffer<>();
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   206
        JCVariableDecl lastParam = null;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   207
        for (List<JCVariableDecl> l = tree.params; l.nonEmpty(); l = l.tail) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   208
            JCVariableDecl param = lastParam = l.head;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   209
            params.append(Assert.checkNonNull(param.sym));
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   210
        }
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   211
        m.params = params.toList();
21498
58c2486db8d0 8006732: support correct bytecode storage of type annotations in multicatch
emc
parents: 21040
diff changeset
   212
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   213
        // mark the method varargs, if necessary
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   214
        if (lastParam != null && (lastParam.mods.flags & Flags.VARARGS) != 0)
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   215
            m.flags_field |= Flags.VARARGS;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   216
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   217
        localEnv.info.scope.leave();
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   218
        if (chk.checkUnique(tree.pos(), m, enclScope)) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   219
        enclScope.enter(m);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   220
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   221
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   222
        annotate.annotateLater(tree.mods.annotations, localEnv, m, tree.pos());
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   223
        // Visit the signature of the method. Note that
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   224
        // TypeAnnotate doesn't descend into the body.
29842
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   225
        annotate.queueScanTreeAndTypeAnnotate(tree, localEnv, m, tree.pos());
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   226
29842
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   227
        if (tree.defaultValue != null) {
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   228
            m.defaultValue = annotate.unfinishedDefaultValue(); // set it to temporary sentinel for now
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   229
            annotate.annotateDefaultValueLater(tree.defaultValue, localEnv, m, tree.pos());
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   230
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   231
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   232
06bc494ca11e Initial load
duke
parents:
diff changeset
   233
    /** Create a fresh environment for method bodies.
06bc494ca11e Initial load
duke
parents:
diff changeset
   234
     *  @param tree     The method definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   235
     *  @param env      The environment current outside of the method definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   236
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   237
    Env<AttrContext> methodEnv(JCMethodDecl tree, Env<AttrContext> env) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   238
        Env<AttrContext> localEnv =
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25007
diff changeset
   239
            env.dup(tree, env.info.dup(env.info.scope.dupUnshared(tree.sym)));
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   240
        localEnv.enclMethod = tree;
14057
b4b0377b8dba 7177387: Add target-typing support in method context
mcimadamore
parents: 14051
diff changeset
   241
        if (tree.sym.type != null) {
b4b0377b8dba 7177387: Add target-typing support in method context
mcimadamore
parents: 14051
diff changeset
   242
            //when this is called in the enter stage, there's no type to be set
27224
228abfa87080 8054457: Refactor Symbol kinds from small ints to an enum
emc
parents: 26781
diff changeset
   243
            localEnv.info.returnResult = attr.new ResultInfo(KindSelector.VAL,
228abfa87080 8054457: Refactor Symbol kinds from small ints to an enum
emc
parents: 26781
diff changeset
   244
                                                             tree.sym.type.getReturnType());
14057
b4b0377b8dba 7177387: Add target-typing support in method context
mcimadamore
parents: 14051
diff changeset
   245
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   246
        if ((tree.mods.flags & STATIC) != 0) localEnv.info.staticLevel++;
06bc494ca11e Initial load
duke
parents:
diff changeset
   247
        return localEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   248
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   249
06bc494ca11e Initial load
duke
parents:
diff changeset
   250
    public void visitVarDef(JCVariableDecl tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   251
        Env<AttrContext> localEnv = env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   252
        if ((tree.mods.flags & STATIC) != 0 ||
06bc494ca11e Initial load
duke
parents:
diff changeset
   253
            (env.info.scope.owner.flags() & INTERFACE) != 0) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   254
            localEnv = env.dup(tree, env.info.dup());
06bc494ca11e Initial load
duke
parents:
diff changeset
   255
            localEnv.info.staticLevel++;
06bc494ca11e Initial load
duke
parents:
diff changeset
   256
        }
19942
e9dae0e41075 8021112: Spurious unchecked warning reported by javac
jlahoda
parents: 19941
diff changeset
   257
        DiagnosticPosition prevLintPos = deferredLintHandler.setPos(tree.pos());
29842
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   258
8236
0d8646b7c602 6594914: @SuppressWarnings("deprecation") does not not work for the type of a variable
mcimadamore
parents: 8225
diff changeset
   259
        try {
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   260
            if (TreeInfo.isEnumInit(tree)) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   261
                attr.attribIdentAsEnumType(localEnv, (JCIdent)tree.vartype);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   262
            } else {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   263
                attr.attribType(tree.vartype, localEnv);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   264
                if (TreeInfo.isReceiverParam(tree))
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   265
                    checkReceiver(tree, localEnv);
21498
58c2486db8d0 8006732: support correct bytecode storage of type annotations in multicatch
emc
parents: 21040
diff changeset
   266
            }
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   267
        } finally {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   268
            deferredLintHandler.setPos(prevLintPos);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   269
        }
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   270
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   271
        if ((tree.mods.flags & VARARGS) != 0) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   272
            //if we are entering a varargs parameter, we need to
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   273
            //replace its type (a plain array type) with the more
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   274
            //precise VarargsType --- we need to do it this way
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   275
            //because varargs is represented in the tree as a
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   276
            //modifier on the parameter declaration, and not as a
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   277
            //distinct type of array node.
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   278
            ArrayType atype = (ArrayType)tree.vartype.type;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   279
            tree.vartype.type = atype.makeVarargs();
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   280
        }
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   281
        WriteableScope enclScope = enter.enterScope(env);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   282
        VarSymbol v =
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   283
            new VarSymbol(0, tree.name, tree.vartype.type, enclScope.owner);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   284
        v.flags_field = chk.checkFlags(tree.pos(), tree.mods.flags, v, tree);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   285
        tree.sym = v;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   286
        if (tree.init != null) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   287
            v.flags_field |= HASINIT;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   288
            if ((v.flags_field & FINAL) != 0 &&
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   289
                needsLazyConstValue(tree.init)) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   290
                Env<AttrContext> initEnv = getInitEnv(tree, env);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   291
                initEnv.info.enclVar = v;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   292
                v.setLazyConstValue(initEnv(tree, initEnv), attr, tree);
12916
021c069e8e27 7160084: javac fails to compile an apparently valid class/interface combination
mcimadamore
parents: 12335
diff changeset
   293
            }
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   294
        }
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   295
        if (chk.checkUnique(tree.pos(), v, enclScope)) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   296
            chk.checkTransparentVar(tree.pos(), v, enclScope);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   297
            enclScope.enter(v);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   298
        }
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   299
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   300
        annotate.annotateLater(tree.mods.annotations, localEnv, v, tree.pos());
29842
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   301
        annotate.queueScanTreeAndTypeAnnotate(tree.vartype, localEnv, v, tree.pos());
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   302
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   303
        v.pos = tree.pos;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   304
    }
17578
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   305
    // where
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   306
    void checkType(JCTree tree, Type type, String diag) {
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   307
        if (!tree.type.isErroneous() && !types.isSameType(tree.type, type)) {
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   308
            log.error(tree, diag, type, tree.type);
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   309
        }
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   310
    }
24903
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   311
    void checkReceiver(JCVariableDecl tree, Env<AttrContext> localEnv) {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   312
        attr.attribExpr(tree.nameexpr, localEnv);
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   313
        MethodSymbol m = localEnv.enclMethod.sym;
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   314
        if (m.isConstructor()) {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   315
            Type outertype = m.owner.owner.type;
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   316
            if (outertype.hasTag(TypeTag.METHOD)) {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   317
                // we have a local inner class
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   318
                outertype = m.owner.owner.owner.type;
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   319
            }
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   320
            if (outertype.hasTag(TypeTag.CLASS)) {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   321
                checkType(tree.vartype, outertype, "incorrect.constructor.receiver.type");
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   322
                checkType(tree.nameexpr, outertype, "incorrect.constructor.receiver.name");
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   323
            } else {
45504
ea7475564d07 8170326: Inconsistencies between code, compiler.properties and comments
jlahoda
parents: 45500
diff changeset
   324
                log.error(tree, Errors.ReceiverParameterNotApplicableConstructorToplevelClass);
24903
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   325
            }
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   326
        } else {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   327
            checkType(tree.vartype, m.owner.type, "incorrect.receiver.type");
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   328
            checkType(tree.nameexpr, m.owner.type, "incorrect.receiver.name");
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   329
        }
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   330
    }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   331
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   332
    public boolean needsLazyConstValue(JCTree tree) {
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   333
        InitTreeVisitor initTreeVisitor = new InitTreeVisitor();
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   334
        tree.accept(initTreeVisitor);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   335
        return initTreeVisitor.result;
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   336
    }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   337
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   338
    /** Visitor class for expressions which might be constant expressions,
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   339
     *  as per JLS 15.28 (Constant Expressions).
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   340
     */
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   341
    static class InitTreeVisitor extends JCTree.Visitor {
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   342
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   343
        private static final Set<Tag> ALLOWED_OPERATORS =
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   344
                EnumSet.of(Tag.POS, Tag.NEG, Tag.NOT, Tag.COMPL, Tag.PLUS, Tag.MINUS,
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   345
                           Tag.MUL, Tag.DIV, Tag.MOD, Tag.SL, Tag.SR, Tag.USR,
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   346
                           Tag.LT, Tag.LE, Tag.GT, Tag.GE, Tag.EQ, Tag.NE,
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   347
                           Tag.BITAND, Tag.BITXOR, Tag.BITOR, Tag.AND, Tag.OR);
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   348
45500
365640343c55 8181464: Invalid lambda in annotation causes NPE in Lint.augment
vromero
parents: 37744
diff changeset
   349
        boolean result = true;
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   350
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   351
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   352
        public void visitTree(JCTree tree) {
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   353
            result = false;
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   354
        }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   355
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   356
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   357
        public void visitLiteral(JCLiteral that) {}
19942
e9dae0e41075 8021112: Spurious unchecked warning reported by javac
jlahoda
parents: 19941
diff changeset
   358
e9dae0e41075 8021112: Spurious unchecked warning reported by javac
jlahoda
parents: 19941
diff changeset
   359
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   360
        public void visitTypeCast(JCTypeCast tree) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   361
            tree.expr.accept(this);
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   362
        }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   363
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   364
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   365
        public void visitUnary(JCUnary that) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   366
            if (!ALLOWED_OPERATORS.contains(that.getTag())) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   367
                result = false;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   368
                return ;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   369
            }
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   370
            that.arg.accept(this);
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   371
        }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   372
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   373
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   374
        public void visitBinary(JCBinary that) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   375
            if (!ALLOWED_OPERATORS.contains(that.getTag())) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   376
                result = false;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   377
                return ;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   378
            }
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   379
            that.lhs.accept(this);
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   380
            that.rhs.accept(this);
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   381
        }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   382
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   383
        @Override
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   384
        public void visitConditional(JCConditional tree) {
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   385
            tree.cond.accept(this);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   386
            tree.truepart.accept(this);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   387
            tree.falsepart.accept(this);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   388
        }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   389
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   390
        @Override
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   391
        public void visitParens(JCParens tree) {
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   392
            tree.expr.accept(this);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   393
        }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   394
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   395
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   396
        public void visitIdent(JCIdent that) {}
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   397
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   398
        @Override
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   399
        public void visitSelect(JCFieldAccess tree) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   400
            tree.selected.accept(this);
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   401
        }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   402
    }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   403
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   404
    /** Create a fresh environment for a variable's initializer.
06bc494ca11e Initial load
duke
parents:
diff changeset
   405
     *  If the variable is a field, the owner of the environment's scope
06bc494ca11e Initial load
duke
parents:
diff changeset
   406
     *  is be the variable itself, otherwise the owner is the method
06bc494ca11e Initial load
duke
parents:
diff changeset
   407
     *  enclosing the variable definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   408
     *
06bc494ca11e Initial load
duke
parents:
diff changeset
   409
     *  @param tree     The variable definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   410
     *  @param env      The environment current outside of the variable definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   411
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   412
    Env<AttrContext> initEnv(JCVariableDecl tree, Env<AttrContext> env) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   413
        Env<AttrContext> localEnv = env.dupto(new AttrContextEnv(tree, env.info.dup()));
06bc494ca11e Initial load
duke
parents:
diff changeset
   414
        if (tree.sym.owner.kind == TYP) {
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25007
diff changeset
   415
            localEnv.info.scope = env.info.scope.dupUnshared(tree.sym);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   416
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   417
        if ((tree.mods.flags & STATIC) != 0 ||
14443
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   418
                ((env.enclClass.sym.flags() & INTERFACE) != 0 && env.enclMethod == null))
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   419
            localEnv.info.staticLevel++;
06bc494ca11e Initial load
duke
parents:
diff changeset
   420
        return localEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   421
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   422
06bc494ca11e Initial load
duke
parents:
diff changeset
   423
    /** Default member enter visitor method: do nothing
06bc494ca11e Initial load
duke
parents:
diff changeset
   424
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   425
    public void visitTree(JCTree tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   426
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   427
06bc494ca11e Initial load
duke
parents:
diff changeset
   428
    public void visitErroneous(JCErroneous tree) {
6929
81a6a7c1bef1 6988407: javac crashes running processor on errant code; it used to print error message
jjg
parents: 6721
diff changeset
   429
        if (tree.errs != null)
81a6a7c1bef1 6988407: javac crashes running processor on errant code; it used to print error message
jjg
parents: 6721
diff changeset
   430
            memberEnter(tree.errs, env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   431
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   432
06bc494ca11e Initial load
duke
parents:
diff changeset
   433
    public Env<AttrContext> getMethodEnv(JCMethodDecl tree, Env<AttrContext> env) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   434
        Env<AttrContext> mEnv = methodEnv(tree, env);
18010
604faee85350 8004643: Reduce javac space overhead introduced with compiler support for repeating annotations
jjg
parents: 17999
diff changeset
   435
        mEnv.info.lint = mEnv.info.lint.augment(tree.sym);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   436
        for (List<JCTypeParameter> l = tree.typarams; l.nonEmpty(); l = l.tail)
06bc494ca11e Initial load
duke
parents:
diff changeset
   437
            mEnv.info.scope.enterIfAbsent(l.head.type.tsym);
06bc494ca11e Initial load
duke
parents:
diff changeset
   438
        for (List<JCVariableDecl> l = tree.params; l.nonEmpty(); l = l.tail)
06bc494ca11e Initial load
duke
parents:
diff changeset
   439
            mEnv.info.scope.enterIfAbsent(l.head.sym);
06bc494ca11e Initial load
duke
parents:
diff changeset
   440
        return mEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   441
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   442
06bc494ca11e Initial load
duke
parents:
diff changeset
   443
    public Env<AttrContext> getInitEnv(JCVariableDecl tree, Env<AttrContext> env) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   444
        Env<AttrContext> iEnv = initEnv(tree, env);
06bc494ca11e Initial load
duke
parents:
diff changeset
   445
        return iEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   446
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   447
}