langtools/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/MemberEnter.java
author vromero
Tue, 13 Jun 2017 11:21:09 -0700
changeset 45500 365640343c55
parent 37744 bf4fd5e022c5
child 45504 ea7475564d07
permissions -rw-r--r--
8181464: Invalid lambda in annotation causes NPE in Lint.augment 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.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    39
import com.sun.tools.javac.tree.JCTree.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    40
06bc494ca11e Initial load
duke
parents:
diff changeset
    41
import static com.sun.tools.javac.code.Flags.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    42
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
    43
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
    44
import static com.sun.tools.javac.code.TypeTag.TYPEVAR;
24604
7f68545b5128 8041422: Split javac ClassReader into ClassReader+ClassFinder
jjg
parents: 24396
diff changeset
    45
27857
7e913a535736 7101822: Compiling depends on order of imports
jlahoda
parents: 27854
diff changeset
    46
/** Resolves field, method and constructor header, and constructs corresponding Symbols.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    47
 *
5847
1908176fd6e3 6944312: Potential rebranding issues in openjdk/langtools repository sources
jjg
parents: 5520
diff changeset
    48
 *  <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
    49
 *  If you write code that depends on this, you do so at your own risk.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    50
 *  This code and its internal interfaces are subject to change or
06bc494ca11e Initial load
duke
parents:
diff changeset
    51
 *  deletion without notice.</b>
06bc494ca11e Initial load
duke
parents:
diff changeset
    52
 */
27857
7e913a535736 7101822: Compiling depends on order of imports
jlahoda
parents: 27854
diff changeset
    53
public class MemberEnter extends JCTree.Visitor {
22163
3651128c74eb 8030244: Update langtools to use Diamond
briangoetz
parents: 21498
diff changeset
    54
    protected static final Context.Key<MemberEnter> memberEnterKey = new Context.Key<>();
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    55
06bc494ca11e Initial load
duke
parents:
diff changeset
    56
    private final Enter enter;
06bc494ca11e Initial load
duke
parents:
diff changeset
    57
    private final Log log;
06bc494ca11e Initial load
duke
parents:
diff changeset
    58
    private final Check chk;
06bc494ca11e Initial load
duke
parents:
diff changeset
    59
    private final Attr attr;
06bc494ca11e Initial load
duke
parents:
diff changeset
    60
    private final Symtab syms;
06bc494ca11e Initial load
duke
parents:
diff changeset
    61
    private final Annotate annotate;
06bc494ca11e Initial load
duke
parents:
diff changeset
    62
    private final Types types;
8236
0d8646b7c602 6594914: @SuppressWarnings("deprecation") does not not work for the type of a variable
mcimadamore
parents: 8225
diff changeset
    63
    private final DeferredLintHandler deferredLintHandler;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    64
06bc494ca11e Initial load
duke
parents:
diff changeset
    65
    public static MemberEnter instance(Context context) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    66
        MemberEnter instance = context.get(memberEnterKey);
06bc494ca11e Initial load
duke
parents:
diff changeset
    67
        if (instance == null)
06bc494ca11e Initial load
duke
parents:
diff changeset
    68
            instance = new MemberEnter(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    69
        return instance;
06bc494ca11e Initial load
duke
parents:
diff changeset
    70
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    71
06bc494ca11e Initial load
duke
parents:
diff changeset
    72
    protected MemberEnter(Context context) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    73
        context.put(memberEnterKey, this);
06bc494ca11e Initial load
duke
parents:
diff changeset
    74
        enter = Enter.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    75
        log = Log.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    76
        chk = Check.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    77
        attr = Attr.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    78
        syms = Symtab.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    79
        annotate = Annotate.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    80
        types = Types.instance(context);
8236
0d8646b7c602 6594914: @SuppressWarnings("deprecation") does not not work for the type of a variable
mcimadamore
parents: 8225
diff changeset
    81
        deferredLintHandler = DeferredLintHandler.instance(context);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    82
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    83
06bc494ca11e Initial load
duke
parents:
diff changeset
    84
    /** Construct method type from method signature.
06bc494ca11e Initial load
duke
parents:
diff changeset
    85
     *  @param typarams    The method's type parameters.
06bc494ca11e Initial load
duke
parents:
diff changeset
    86
     *  @param params      The method's value parameters.
06bc494ca11e Initial load
duke
parents:
diff changeset
    87
     *  @param res             The method's result type,
06bc494ca11e Initial load
duke
parents:
diff changeset
    88
     *                 null if it is a constructor.
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
    89
     *  @param recvparam       The method's receiver parameter,
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
    90
     *                 null if none given; TODO: or already set here?
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    91
     *  @param thrown      The method's thrown exceptions.
06bc494ca11e Initial load
duke
parents:
diff changeset
    92
     *  @param env             The method's (local) environment.
06bc494ca11e Initial load
duke
parents:
diff changeset
    93
     */
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    94
    Type signature(MethodSymbol msym,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    95
                   List<JCTypeParameter> typarams,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    96
                   List<JCVariableDecl> params,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    97
                   JCTree res,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    98
                   JCVariableDecl recvparam,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
    99
                   List<JCExpression> thrown,
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   100
                   Env<AttrContext> env) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   101
06bc494ca11e Initial load
duke
parents:
diff changeset
   102
        // Enter and attribute type parameters.
06bc494ca11e Initial load
duke
parents:
diff changeset
   103
        List<Type> tvars = enter.classEnter(typarams, env);
06bc494ca11e Initial load
duke
parents:
diff changeset
   104
        attr.attribTypeVariables(typarams, env);
06bc494ca11e Initial load
duke
parents:
diff changeset
   105
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   106
        // Enter and attribute value parameters.
22163
3651128c74eb 8030244: Update langtools to use Diamond
briangoetz
parents: 21498
diff changeset
   107
        ListBuffer<Type> argbuf = new ListBuffer<>();
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   108
        for (List<JCVariableDecl> l = params; l.nonEmpty(); l = l.tail) {
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   109
            memberEnter(l.head, env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   110
            argbuf.append(l.head.vartype.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   111
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   112
06bc494ca11e Initial load
duke
parents:
diff changeset
   113
        // Attribute result type, if one is given.
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   114
        Type restype = res == null ? syms.voidType : attr.attribType(res, env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   115
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   116
        // Attribute receiver type, if one is given.
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   117
        Type recvtype;
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   118
        if (recvparam!=null) {
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   119
            memberEnter(recvparam, env);
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   120
            recvtype = recvparam.vartype.type;
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   121
        } else {
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   122
            recvtype = null;
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   123
        }
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   124
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   125
        // Attribute thrown exceptions.
22163
3651128c74eb 8030244: Update langtools to use Diamond
briangoetz
parents: 21498
diff changeset
   126
        ListBuffer<Type> thrownbuf = new ListBuffer<>();
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   127
        for (List<JCExpression> l = thrown; l.nonEmpty(); l = l.tail) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   128
            Type exc = attr.attribType(l.head, env);
18909
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   129
            if (!exc.hasTag(TYPEVAR)) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   130
                exc = chk.checkClassType(l.head.pos(), exc);
18909
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   131
            } else if (exc.tsym.owner == msym) {
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   132
                //mark inference variables in 'throws' clause
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   133
                exc.tsym.flags_field |= THROWS;
8f9fc5d876e4 8012242: Lambda compatibility and checked exceptions
mcimadamore
parents: 18643
diff changeset
   134
            }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   135
            thrownbuf.append(exc);
06bc494ca11e Initial load
duke
parents:
diff changeset
   136
        }
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   137
        MethodType mtype = new MethodType(argbuf.toList(),
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   138
                                    restype,
06bc494ca11e Initial load
duke
parents:
diff changeset
   139
                                    thrownbuf.toList(),
06bc494ca11e Initial load
duke
parents:
diff changeset
   140
                                    syms.methodClass);
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   141
        mtype.recvtype = recvtype;
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 14443
diff changeset
   142
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   143
        return tvars.isEmpty() ? mtype : new ForAll(tvars, mtype);
06bc494ca11e Initial load
duke
parents:
diff changeset
   144
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   145
06bc494ca11e Initial load
duke
parents:
diff changeset
   146
/* ********************************************************************
06bc494ca11e Initial load
duke
parents:
diff changeset
   147
 * Visitor methods for member enter
06bc494ca11e Initial load
duke
parents:
diff changeset
   148
 *********************************************************************/
06bc494ca11e Initial load
duke
parents:
diff changeset
   149
06bc494ca11e Initial load
duke
parents:
diff changeset
   150
    /** Visitor argument: the current environment
06bc494ca11e Initial load
duke
parents:
diff changeset
   151
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   152
    protected Env<AttrContext> env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   153
06bc494ca11e Initial load
duke
parents:
diff changeset
   154
    /** Enter field and method definitions and process import
06bc494ca11e Initial load
duke
parents:
diff changeset
   155
     *  clauses, catching any completion failure exceptions.
06bc494ca11e Initial load
duke
parents:
diff changeset
   156
     */
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   157
    protected void memberEnter(JCTree tree, Env<AttrContext> env) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   158
        Env<AttrContext> prevEnv = this.env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   159
        try {
06bc494ca11e Initial load
duke
parents:
diff changeset
   160
            this.env = env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   161
            tree.accept(this);
06bc494ca11e Initial load
duke
parents:
diff changeset
   162
        }  catch (CompletionFailure ex) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   163
            chk.completionError(tree.pos(), ex);
06bc494ca11e Initial load
duke
parents:
diff changeset
   164
        } finally {
06bc494ca11e Initial load
duke
parents:
diff changeset
   165
            this.env = prevEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   166
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   167
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   168
06bc494ca11e Initial load
duke
parents:
diff changeset
   169
    /** Enter members from a list of trees.
06bc494ca11e Initial load
duke
parents:
diff changeset
   170
     */
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   171
    void memberEnter(List<? extends JCTree> trees, Env<AttrContext> env) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   172
        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
   173
            memberEnter(l.head, env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   174
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   175
06bc494ca11e Initial load
duke
parents:
diff changeset
   176
    public void visitMethodDef(JCMethodDecl tree) {
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25007
diff changeset
   177
        WriteableScope enclScope = enter.enterScope(env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   178
        MethodSymbol m = new MethodSymbol(0, tree.name, null, enclScope.owner);
06bc494ca11e Initial load
duke
parents:
diff changeset
   179
        m.flags_field = chk.checkFlags(tree.pos(), tree.mods.flags, m, tree);
06bc494ca11e Initial load
duke
parents:
diff changeset
   180
        tree.sym = m;
14443
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   181
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   182
        //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
   183
        if ((tree.mods.flags & DEFAULT) != 0) {
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   184
            m.enclClass().flags_field |= DEFAULT;
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   185
        }
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   186
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   187
        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
   188
        DiagnosticPosition prevLintPos = deferredLintHandler.setPos(tree.pos());
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   189
        try {
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   190
            // Compute the method type
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   191
            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
   192
                               tree.restype, tree.recvparam,
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   193
                               tree.thrown,
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   194
                               localEnv);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   195
        } finally {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   196
            deferredLintHandler.setPos(prevLintPos);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   197
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   198
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   199
        if (types.isSignaturePolymorphic(m)) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   200
            m.flags_field |= SIGNATURE_POLYMORPHIC;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   201
        }
18389
a425d0819f36 8016569: javac, add new flag for polymorphic method signatures
vromero
parents: 18010
diff changeset
   202
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   203
        // Set m.params
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   204
        ListBuffer<VarSymbol> params = new ListBuffer<>();
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   205
        JCVariableDecl lastParam = null;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   206
        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
   207
            JCVariableDecl param = lastParam = l.head;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   208
            params.append(Assert.checkNonNull(param.sym));
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   209
        }
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   210
        m.params = params.toList();
21498
58c2486db8d0 8006732: support correct bytecode storage of type annotations in multicatch
emc
parents: 21040
diff changeset
   211
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   212
        // mark the method varargs, if necessary
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   213
        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
   214
            m.flags_field |= Flags.VARARGS;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   215
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   216
        localEnv.info.scope.leave();
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   217
        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
   218
        enclScope.enter(m);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   219
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   220
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   221
        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
   222
        // 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
   223
        // TypeAnnotate doesn't descend into the body.
29842
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   224
        annotate.queueScanTreeAndTypeAnnotate(tree, localEnv, m, tree.pos());
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   225
29842
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   226
        if (tree.defaultValue != null) {
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   227
            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
   228
            annotate.annotateDefaultValueLater(tree.defaultValue, localEnv, m, tree.pos());
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   229
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   230
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   231
06bc494ca11e Initial load
duke
parents:
diff changeset
   232
    /** Create a fresh environment for method bodies.
06bc494ca11e Initial load
duke
parents:
diff changeset
   233
     *  @param tree     The method definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   234
     *  @param env      The environment current outside of the method definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   235
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   236
    Env<AttrContext> methodEnv(JCMethodDecl tree, Env<AttrContext> env) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   237
        Env<AttrContext> localEnv =
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25007
diff changeset
   238
            env.dup(tree, env.info.dup(env.info.scope.dupUnshared(tree.sym)));
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   239
        localEnv.enclMethod = tree;
14057
b4b0377b8dba 7177387: Add target-typing support in method context
mcimadamore
parents: 14051
diff changeset
   240
        if (tree.sym.type != null) {
b4b0377b8dba 7177387: Add target-typing support in method context
mcimadamore
parents: 14051
diff changeset
   241
            //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
   242
            localEnv.info.returnResult = attr.new ResultInfo(KindSelector.VAL,
228abfa87080 8054457: Refactor Symbol kinds from small ints to an enum
emc
parents: 26781
diff changeset
   243
                                                             tree.sym.type.getReturnType());
14057
b4b0377b8dba 7177387: Add target-typing support in method context
mcimadamore
parents: 14051
diff changeset
   244
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   245
        if ((tree.mods.flags & STATIC) != 0) localEnv.info.staticLevel++;
06bc494ca11e Initial load
duke
parents:
diff changeset
   246
        return localEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   247
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   248
06bc494ca11e Initial load
duke
parents:
diff changeset
   249
    public void visitVarDef(JCVariableDecl tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   250
        Env<AttrContext> localEnv = env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   251
        if ((tree.mods.flags & STATIC) != 0 ||
06bc494ca11e Initial load
duke
parents:
diff changeset
   252
            (env.info.scope.owner.flags() & INTERFACE) != 0) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   253
            localEnv = env.dup(tree, env.info.dup());
06bc494ca11e Initial load
duke
parents:
diff changeset
   254
            localEnv.info.staticLevel++;
06bc494ca11e Initial load
duke
parents:
diff changeset
   255
        }
19942
e9dae0e41075 8021112: Spurious unchecked warning reported by javac
jlahoda
parents: 19941
diff changeset
   256
        DiagnosticPosition prevLintPos = deferredLintHandler.setPos(tree.pos());
29842
826ac2519523 8031744: Annotations on many Language Model elements are not returned
jjg
parents: 28590
diff changeset
   257
8236
0d8646b7c602 6594914: @SuppressWarnings("deprecation") does not not work for the type of a variable
mcimadamore
parents: 8225
diff changeset
   258
        try {
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   259
            if (TreeInfo.isEnumInit(tree)) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   260
                attr.attribIdentAsEnumType(localEnv, (JCIdent)tree.vartype);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   261
            } else {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   262
                attr.attribType(tree.vartype, localEnv);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   263
                if (TreeInfo.isReceiverParam(tree))
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   264
                    checkReceiver(tree, localEnv);
21498
58c2486db8d0 8006732: support correct bytecode storage of type annotations in multicatch
emc
parents: 21040
diff changeset
   265
            }
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   266
        } finally {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   267
            deferredLintHandler.setPos(prevLintPos);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   268
        }
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
        if ((tree.mods.flags & VARARGS) != 0) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   271
            //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
   272
            //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
   273
            //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
   274
            //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
   275
            //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
   276
            //distinct type of array node.
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   277
            ArrayType atype = (ArrayType)tree.vartype.type;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   278
            tree.vartype.type = atype.makeVarargs();
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   279
        }
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   280
        WriteableScope enclScope = enter.enterScope(env);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   281
        VarSymbol v =
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   282
            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
   283
        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
   284
        tree.sym = v;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   285
        if (tree.init != null) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   286
            v.flags_field |= HASINIT;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   287
            if ((v.flags_field & FINAL) != 0 &&
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   288
                needsLazyConstValue(tree.init)) {
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   289
                Env<AttrContext> initEnv = getInitEnv(tree, env);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   290
                initEnv.info.enclVar = v;
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   291
                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
   292
            }
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   293
        }
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   294
        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
   295
            chk.checkTransparentVar(tree.pos(), v, enclScope);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   296
            enclScope.enter(v);
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   297
        }
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   298
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   299
        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
   300
        annotate.queueScanTreeAndTypeAnnotate(tree.vartype, localEnv, v, tree.pos());
26532
aa84b6606229 8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents: 26266
diff changeset
   301
27231
c1ca668b421e 8054448: (ann) Cannot reference field of inner class in an anonymous class
jfranck
parents: 27224
diff changeset
   302
        v.pos = tree.pos;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   303
    }
17578
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   304
    // where
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   305
    void checkType(JCTree tree, Type type, String diag) {
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   306
        if (!tree.type.isErroneous() && !types.isSameType(tree.type, type)) {
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   307
            log.error(tree, diag, type, tree.type);
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   308
        }
46ac954e4a84 8013852: update reference impl for type-annotations
jjg
parents: 16558
diff changeset
   309
    }
24903
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   310
    void checkReceiver(JCVariableDecl tree, Env<AttrContext> localEnv) {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   311
        attr.attribExpr(tree.nameexpr, localEnv);
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   312
        MethodSymbol m = localEnv.enclMethod.sym;
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   313
        if (m.isConstructor()) {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   314
            Type outertype = m.owner.owner.type;
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   315
            if (outertype.hasTag(TypeTag.METHOD)) {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   316
                // we have a local inner class
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   317
                outertype = m.owner.owner.owner.type;
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   318
            }
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   319
            if (outertype.hasTag(TypeTag.CLASS)) {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   320
                checkType(tree.vartype, outertype, "incorrect.constructor.receiver.type");
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   321
                checkType(tree.nameexpr, outertype, "incorrect.constructor.receiver.name");
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   322
            } else {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   323
                log.error(tree, "receiver.parameter.not.applicable.constructor.toplevel.class");
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   324
            }
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   325
        } else {
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   326
            checkType(tree.vartype, m.owner.type, "incorrect.receiver.type");
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   327
            checkType(tree.nameexpr, m.owner.type, "incorrect.receiver.name");
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   328
        }
41e1812d6b4c 8027886: javac allows illegal receiver parameters
pgovereau
parents: 24895
diff changeset
   329
    }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   330
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   331
    public boolean needsLazyConstValue(JCTree tree) {
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   332
        InitTreeVisitor initTreeVisitor = new InitTreeVisitor();
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   333
        tree.accept(initTreeVisitor);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   334
        return initTreeVisitor.result;
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   335
    }
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   336
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   337
    /** Visitor class for expressions which might be constant expressions,
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   338
     *  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
   339
     */
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   340
    static class InitTreeVisitor extends JCTree.Visitor {
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   341
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   342
        private static final Set<Tag> ALLOWED_OPERATORS =
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   343
                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
   344
                           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
   345
                           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
   346
                           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
   347
45500
365640343c55 8181464: Invalid lambda in annotation causes NPE in Lint.augment
vromero
parents: 37744
diff changeset
   348
        boolean result = true;
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   349
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   350
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   351
        public void visitTree(JCTree tree) {
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   352
            result = false;
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   353
        }
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
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   356
        public void visitLiteral(JCLiteral that) {}
19942
e9dae0e41075 8021112: Spurious unchecked warning reported by javac
jlahoda
parents: 19941
diff changeset
   357
e9dae0e41075 8021112: Spurious unchecked warning reported by javac
jlahoda
parents: 19941
diff changeset
   358
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   359
        public void visitTypeCast(JCTypeCast tree) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   360
            tree.expr.accept(this);
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   361
        }
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
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   364
        public void visitUnary(JCUnary that) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   365
            if (!ALLOWED_OPERATORS.contains(that.getTag())) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   366
                result = false;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   367
                return ;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   368
            }
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   369
            that.arg.accept(this);
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   370
        }
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
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   373
        public void visitBinary(JCBinary that) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   374
            if (!ALLOWED_OPERATORS.contains(that.getTag())) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   375
                result = false;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   376
                return ;
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   377
            }
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   378
            that.lhs.accept(this);
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   379
            that.rhs.accept(this);
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   380
        }
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
        @Override
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   383
        public void visitConditional(JCConditional tree) {
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   384
            tree.cond.accept(this);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   385
            tree.truepart.accept(this);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   386
            tree.falsepart.accept(this);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   387
        }
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
        @Override
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   390
        public void visitParens(JCParens tree) {
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   391
            tree.expr.accept(this);
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   392
        }
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
        @Override
30064
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   395
        public void visitIdent(JCIdent that) {}
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   396
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   397
        @Override
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   398
        public void visitSelect(JCFieldAccess tree) {
39493809b601 8077605: Initializing static fields causes unbounded recursion in javac
jlahoda
parents: 29842
diff changeset
   399
            tree.selected.accept(this);
19656
7f0afbdbf142 8023112: javac should not use lazy constant evaluation approach for method references
vromero
parents: 19651
diff changeset
   400
        }
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
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   403
    /** Create a fresh environment for a variable's initializer.
06bc494ca11e Initial load
duke
parents:
diff changeset
   404
     *  If the variable is a field, the owner of the environment's scope
06bc494ca11e Initial load
duke
parents:
diff changeset
   405
     *  is be the variable itself, otherwise the owner is the method
06bc494ca11e Initial load
duke
parents:
diff changeset
   406
     *  enclosing the variable definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   407
     *
06bc494ca11e Initial load
duke
parents:
diff changeset
   408
     *  @param tree     The variable definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   409
     *  @param env      The environment current outside of the variable definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   410
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   411
    Env<AttrContext> initEnv(JCVariableDecl tree, Env<AttrContext> env) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   412
        Env<AttrContext> localEnv = env.dupto(new AttrContextEnv(tree, env.info.dup()));
06bc494ca11e Initial load
duke
parents:
diff changeset
   413
        if (tree.sym.owner.kind == TYP) {
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25007
diff changeset
   414
            localEnv.info.scope = env.info.scope.dupUnshared(tree.sym);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   415
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   416
        if ((tree.mods.flags & STATIC) != 0 ||
14443
91c05eb19277 7192246: Add type-checking support for default methods
mcimadamore
parents: 14369
diff changeset
   417
                ((env.enclClass.sym.flags() & INTERFACE) != 0 && env.enclMethod == null))
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   418
            localEnv.info.staticLevel++;
06bc494ca11e Initial load
duke
parents:
diff changeset
   419
        return localEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   420
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   421
06bc494ca11e Initial load
duke
parents:
diff changeset
   422
    /** Default member enter visitor method: do nothing
06bc494ca11e Initial load
duke
parents:
diff changeset
   423
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   424
    public void visitTree(JCTree tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   425
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   426
06bc494ca11e Initial load
duke
parents:
diff changeset
   427
    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
   428
        if (tree.errs != null)
81a6a7c1bef1 6988407: javac crashes running processor on errant code; it used to print error message
jjg
parents: 6721
diff changeset
   429
            memberEnter(tree.errs, env);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   430
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   431
06bc494ca11e Initial load
duke
parents:
diff changeset
   432
    public Env<AttrContext> getMethodEnv(JCMethodDecl tree, Env<AttrContext> env) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   433
        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
   434
        mEnv.info.lint = mEnv.info.lint.augment(tree.sym);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   435
        for (List<JCTypeParameter> l = tree.typarams; l.nonEmpty(); l = l.tail)
06bc494ca11e Initial load
duke
parents:
diff changeset
   436
            mEnv.info.scope.enterIfAbsent(l.head.type.tsym);
06bc494ca11e Initial load
duke
parents:
diff changeset
   437
        for (List<JCVariableDecl> l = tree.params; l.nonEmpty(); l = l.tail)
06bc494ca11e Initial load
duke
parents:
diff changeset
   438
            mEnv.info.scope.enterIfAbsent(l.head.sym);
06bc494ca11e Initial load
duke
parents:
diff changeset
   439
        return mEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   440
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   441
06bc494ca11e Initial load
duke
parents:
diff changeset
   442
    public Env<AttrContext> getInitEnv(JCVariableDecl tree, Env<AttrContext> env) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   443
        Env<AttrContext> iEnv = initEnv(tree, env);
06bc494ca11e Initial load
duke
parents:
diff changeset
   444
        return iEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   445
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   446
}