src/share/classes/com/sun/tools/javac/code/Lint.java

Thu, 25 Feb 2010 09:42:35 -0800

author
jjg
date
Thu, 25 Feb 2010 09:42:35 -0800
changeset 505
87eb6edd4f21
parent 377
d9febdd5ae21
child 554
9d9f26857129
permissions
-rw-r--r--

4880220: Add a warning when accessing a static method via an reference
Reviewed-by: darcy

     1 /*
     2  * Copyright 2005-2008 Sun Microsystems, Inc.  All Rights Reserved.
     3  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
     4  *
     5  * This code is free software; you can redistribute it and/or modify it
     6  * under the terms of the GNU General Public License version 2 only, as
     7  * published by the Free Software Foundation.  Sun designates this
     8  * particular file as subject to the "Classpath" exception as provided
     9  * by Sun in the LICENSE file that accompanied this code.
    10  *
    11  * This code is distributed in the hope that it will be useful, but WITHOUT
    12  * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
    13  * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
    14  * version 2 for more details (a copy is included in the LICENSE file that
    15  * accompanied this code).
    16  *
    17  * You should have received a copy of the GNU General Public License version
    18  * 2 along with this work; if not, write to the Free Software Foundation,
    19  * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
    20  *
    21  * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
    22  * CA 95054 USA or visit www.sun.com if you need additional information or
    23  * have any questions.
    24  */
    26 package com.sun.tools.javac.code;
    28 import java.util.EnumSet;
    29 import java.util.HashMap;
    30 import java.util.Map;
    31 import com.sun.tools.javac.code.Symbol.*;
    32 import com.sun.tools.javac.util.Context;
    33 import com.sun.tools.javac.util.List;
    34 import com.sun.tools.javac.util.Options;
    35 import com.sun.tools.javac.util.Pair;
    36 import static com.sun.tools.javac.code.Flags.*;
    39 /**
    40  * A class for handling -Xlint suboptions and @SuppresssWarnings.
    41  *
    42  *  <p><b>This is NOT part of any API supported by Sun Microsystems.  If
    43  *  you write code that depends on this, you do so at your own risk.
    44  *  This code and its internal interfaces are subject to change or
    45  *  deletion without notice.</b>
    46  */
    47 public class Lint
    48 {
    49     /** The context key for the root Lint object. */
    50     protected static final Context.Key<Lint> lintKey = new Context.Key<Lint>();
    52     /** Get the root Lint instance. */
    53     public static Lint instance(Context context) {
    54         Lint instance = context.get(lintKey);
    55         if (instance == null)
    56             instance = new Lint(context);
    57         return instance;
    58     }
    60     /**
    61      * Returns the result of combining the values in this object with
    62      * the given annotation.
    63      */
    64     public Lint augment(Attribute.Compound attr) {
    65         return augmentor.augment(this, attr);
    66     }
    69     /**
    70      * Returns the result of combining the values in this object with
    71      * the given annotations.
    72      */
    73     public Lint augment(List<Attribute.Compound> attrs) {
    74         return augmentor.augment(this, attrs);
    75     }
    77     /**
    78      * Returns the result of combining the values in this object with
    79      * the given annotations and flags.
    80      */
    81     public Lint augment(List<Attribute.Compound> attrs, long flags) {
    82         Lint l = augmentor.augment(this, attrs);
    83         if ((flags & DEPRECATED) != 0) {
    84             if (l == this)
    85                 l = new Lint(this);
    86             l.values.remove(LintCategory.DEPRECATION);
    87             l.suppressedValues.add(LintCategory.DEPRECATION);
    88         }
    89         return l;
    90     }
    93     private final AugmentVisitor augmentor;
    95     private final EnumSet<LintCategory> values;
    96     private final EnumSet<LintCategory> suppressedValues;
    98     private static Map<String, LintCategory> map = new HashMap<String,LintCategory>();
   101     protected Lint(Context context) {
   102         // initialize values according to the lint options
   103         Options options = Options.instance(context);
   104         values = EnumSet.noneOf(LintCategory.class);
   105         for (Map.Entry<String, LintCategory> e: map.entrySet()) {
   106             if (options.lint(e.getKey()))
   107                 values.add(e.getValue());
   108         }
   110         suppressedValues = EnumSet.noneOf(LintCategory.class);
   112         context.put(lintKey, this);
   113         augmentor = new AugmentVisitor(context);
   114     }
   116     protected Lint(Lint other) {
   117         this.augmentor = other.augmentor;
   118         this.values = other.values.clone();
   119         this.suppressedValues = other.suppressedValues.clone();
   120     }
   122     public String toString() {
   123         return "Lint:[values" + values + " suppressedValues" + suppressedValues + "]";
   124     }
   126     /**
   127      * Categories of warnings that can be generated by the compiler.
   128      */
   129     public enum LintCategory {
   130         /**
   131          * Warn about use of unnecessary casts.
   132          */
   133         CAST("cast"),
   135         /**
   136          * Warn about use of deprecated items.
   137          */
   138         DEPRECATION("deprecation"),
   140         /**
   141          * Warn about items which are documented with an {@code @deprecated} JavaDoc
   142          * comment, but which do not have {@code @Deprecated} annotation.
   143          */
   144         DEP_ANN("dep-ann"),
   146         /**
   147          * Warn about division by constant integer 0.
   148          */
   149         DIVZERO("divzero"),
   151         /**
   152          * Warn about empty statement after if.
   153          */
   154         EMPTY("empty"),
   156         /**
   157          * Warn about falling through from one case of a switch statement to the next.
   158          */
   159         FALLTHROUGH("fallthrough"),
   161         /**
   162          * Warn about finally clauses that do not terminate normally.
   163          */
   164         FINALLY("finally"),
   166         /**
   167          * Warn about issues regarding method overrides.
   168          */
   169         OVERRIDES("overrides"),
   171         /**
   172          * Warn about invalid path elements on the command line.
   173          * Such warnings cannot be suppressed with the SuppressWarnings
   174          * annotation.
   175          */
   176         PATH("path"),
   178         /**
   179          * Warn about issues regarding annotation processing.
   180          */
   181         PROCESSING("processing"),
   183         /**
   184          * Warn about Serializable classes that do not provide a serial version ID.
   185          */
   186         SERIAL("serial"),
   188         /**
   189          * Warn about unchecked operations on raw types.
   190          */
   191         UNCHECKED("unchecked"),
   193         /**
   194          * Warn about unchecked operations on raw types.
   195          */
   196         RAW("rawtypes"),
   198         /**
   199          * Warn about Sun proprietary API that may be removed in a future release.
   200          */
   201         SUNAPI("sunapi", true),
   203         /**
   204          * Warn about issues relating to use of statics
   205          */
   206         STATIC("static");
   208         LintCategory(String option) {
   209             this(option, false);
   210         }
   212         LintCategory(String option, boolean hidden) {
   213             this.option = option;
   214             this.hidden = hidden;
   215             map.put(option, this);
   216         }
   218         static LintCategory get(String option) {
   219             return map.get(option);
   220         }
   222         public final String option;
   223         public final boolean hidden;
   224     };
   226     /**
   227      * Checks if a warning category is enabled. A warning category may be enabled
   228      * on the command line, or by default, and can be temporarily disabled with
   229      * the SuppressWarnings annotation.
   230      */
   231     public boolean isEnabled(LintCategory lc) {
   232         return values.contains(lc);
   233     }
   235     /**
   236      * Checks is a warning category has been specifically suppressed, by means
   237      * of the SuppressWarnings annotation, or, in the case of the deprecated
   238      * category, whether it has been implicitly suppressed by virtue of the
   239      * current entity being itself deprecated.
   240      */
   241     public boolean isSuppressed(LintCategory lc) {
   242         return suppressedValues.contains(lc);
   243     }
   245     protected static class AugmentVisitor implements Attribute.Visitor {
   246         private final Context context;
   247         private Symtab syms;
   248         private Lint parent;
   249         private Lint lint;
   251         AugmentVisitor(Context context) {
   252             // to break an ugly sequence of initialization dependencies,
   253             // we defer the initialization of syms until it is needed
   254             this.context = context;
   255         }
   257         Lint augment(Lint parent, Attribute.Compound attr) {
   258             initSyms();
   259             this.parent = parent;
   260             lint = null;
   261             attr.accept(this);
   262             return (lint == null ? parent : lint);
   263         }
   265         Lint augment(Lint parent, List<Attribute.Compound> attrs) {
   266             initSyms();
   267             this.parent = parent;
   268             lint = null;
   269             for (Attribute.Compound a: attrs) {
   270                 a.accept(this);
   271             }
   272             return (lint == null ? parent : lint);
   273         }
   275         private void initSyms() {
   276             if (syms == null)
   277                 syms = Symtab.instance(context);
   278         }
   280         private void suppress(LintCategory lc) {
   281             if (lint == null)
   282                 lint = new Lint(parent);
   283             lint.suppressedValues.add(lc);
   284             lint.values.remove(lc);
   285         }
   287         public void visitConstant(Attribute.Constant value) {
   288             if (value.type.tsym == syms.stringType.tsym) {
   289                 LintCategory lc = LintCategory.get((String) (value.value));
   290                 if (lc != null)
   291                     suppress(lc);
   292             }
   293         }
   295         public void visitClass(Attribute.Class clazz) {
   296         }
   298         // If we find a @SuppressWarnings annotation, then we continue
   299         // walking the tree, in order to suppress the individual warnings
   300         // specified in the @SuppressWarnings annotation.
   301         public void visitCompound(Attribute.Compound compound) {
   302             if (compound.type.tsym == syms.suppressWarningsType.tsym) {
   303                 for (List<Pair<MethodSymbol,Attribute>> v = compound.values;
   304                      v.nonEmpty(); v = v.tail) {
   305                     Pair<MethodSymbol,Attribute> value = v.head;
   306                     if (value.fst.name.toString().equals("value"))
   307                         value.snd.accept(this);
   308                 }
   310             }
   311         }
   313         public void visitArray(Attribute.Array array) {
   314             for (Attribute value : array.values)
   315                 value.accept(this);
   316         }
   318         public void visitEnum(Attribute.Enum e) {
   319         }
   321         public void visitError(Attribute.Error e) {
   322         }
   323     };
   324 }

mercurial