langtools/src/jdk.compiler/share/classes/com/sun/tools/javac/processing/JavacMessager.java
author jlahoda
Tue, 13 Jan 2015 19:13:42 +0100
changeset 28455 41245007c074
parent 26266 2d24bda701dc
child 45504 ea7475564d07
permissions -rw-r--r--
8066843: Messager.printMessage cannot print multiple errors for same source position Summary: Using a DiagnosticFlag to mark diagnostics that should be always printed even if multiple of them have the same source position. Reviewed-by: darcy, jjg, mcimadamore
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
10
06bc494ca11e Initial load
duke
parents:
diff changeset
     1
/*
28455
41245007c074 8066843: Messager.printMessage cannot print multiple errors for same source position
jlahoda
parents: 26266
diff changeset
     2
 * Copyright (c) 2005, 2015, 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: 1264
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: 1264
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: 1264
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 1264
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 1264
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.processing;
06bc494ca11e Initial load
duke
parents:
diff changeset
    27
06bc494ca11e Initial load
duke
parents:
diff changeset
    28
import com.sun.tools.javac.model.JavacElements;
06bc494ca11e Initial load
duke
parents:
diff changeset
    29
import com.sun.tools.javac.util.*;
26266
2d24bda701dc 8056061: Mark implementations of public interfaces with an annotation
jlahoda
parents: 25874
diff changeset
    30
import com.sun.tools.javac.util.DefinedBy.Api;
28455
41245007c074 8066843: Messager.printMessage cannot print multiple errors for same source position
jlahoda
parents: 26266
diff changeset
    31
import com.sun.tools.javac.util.JCDiagnostic.DiagnosticFlag;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    32
import com.sun.tools.javac.tree.JCTree;
06bc494ca11e Initial load
duke
parents:
diff changeset
    33
import com.sun.tools.javac.tree.JCTree.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    34
import javax.lang.model.element.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    35
import javax.tools.JavaFileObject;
06bc494ca11e Initial load
duke
parents:
diff changeset
    36
import javax.tools.Diagnostic;
06bc494ca11e Initial load
duke
parents:
diff changeset
    37
import javax.annotation.processing.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    38
06bc494ca11e Initial load
duke
parents:
diff changeset
    39
/**
06bc494ca11e Initial load
duke
parents:
diff changeset
    40
 * An implementation of the Messager built on top of log.
06bc494ca11e Initial load
duke
parents:
diff changeset
    41
 *
5847
1908176fd6e3 6944312: Potential rebranding issues in openjdk/langtools repository sources
jjg
parents: 5520
diff changeset
    42
 * <p><b>This is NOT part of any supported API.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    43
 * If you write code that depends on this, you do so at your own risk.
06bc494ca11e Initial load
duke
parents:
diff changeset
    44
 * This code and its internal interfaces are subject to change or
06bc494ca11e Initial load
duke
parents:
diff changeset
    45
 * deletion without notice.</b>
06bc494ca11e Initial load
duke
parents:
diff changeset
    46
 */
06bc494ca11e Initial load
duke
parents:
diff changeset
    47
public class JavacMessager implements Messager {
06bc494ca11e Initial load
duke
parents:
diff changeset
    48
    Log log;
06bc494ca11e Initial load
duke
parents:
diff changeset
    49
    JavacProcessingEnvironment processingEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
    50
    int errorCount = 0;
6153
277c719ff46e 6403456: -Werror should work with annotation processing
jjg
parents: 5847
diff changeset
    51
    int warningCount = 0;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    52
06bc494ca11e Initial load
duke
parents:
diff changeset
    53
    JavacMessager(Context context, JavacProcessingEnvironment processingEnv) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    54
        log = Log.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    55
        this.processingEnv = processingEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
    56
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    57
06bc494ca11e Initial load
duke
parents:
diff changeset
    58
    // processingEnv.getElementUtils()
06bc494ca11e Initial load
duke
parents:
diff changeset
    59
26266
2d24bda701dc 8056061: Mark implementations of public interfaces with an annotation
jlahoda
parents: 25874
diff changeset
    60
    @DefinedBy(Api.ANNOTATION_PROCESSING)
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    61
    public void printMessage(Diagnostic.Kind kind, CharSequence msg) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    62
        printMessage(kind, msg, null, null, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
    63
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    64
26266
2d24bda701dc 8056061: Mark implementations of public interfaces with an annotation
jlahoda
parents: 25874
diff changeset
    65
    @DefinedBy(Api.ANNOTATION_PROCESSING)
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    66
    public void printMessage(Diagnostic.Kind kind, CharSequence msg,
06bc494ca11e Initial load
duke
parents:
diff changeset
    67
                      Element e) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    68
        printMessage(kind, msg, e, null, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
    69
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    70
06bc494ca11e Initial load
duke
parents:
diff changeset
    71
    /**
06bc494ca11e Initial load
duke
parents:
diff changeset
    72
     * Prints a message of the specified kind at the location of the
06bc494ca11e Initial load
duke
parents:
diff changeset
    73
     * annotation mirror of the annotated element.
06bc494ca11e Initial load
duke
parents:
diff changeset
    74
     *
06bc494ca11e Initial load
duke
parents:
diff changeset
    75
     * @param kind the kind of message
06bc494ca11e Initial load
duke
parents:
diff changeset
    76
     * @param msg  the message, or an empty string if none
06bc494ca11e Initial load
duke
parents:
diff changeset
    77
     * @param e    the annotated element
06bc494ca11e Initial load
duke
parents:
diff changeset
    78
     * @param a    the annotation to use as a position hint
06bc494ca11e Initial load
duke
parents:
diff changeset
    79
     */
26266
2d24bda701dc 8056061: Mark implementations of public interfaces with an annotation
jlahoda
parents: 25874
diff changeset
    80
    @DefinedBy(Api.ANNOTATION_PROCESSING)
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    81
    public void printMessage(Diagnostic.Kind kind, CharSequence msg,
06bc494ca11e Initial load
duke
parents:
diff changeset
    82
                      Element e, AnnotationMirror a) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    83
        printMessage(kind, msg, e, a, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
    84
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    85
06bc494ca11e Initial load
duke
parents:
diff changeset
    86
    /**
06bc494ca11e Initial load
duke
parents:
diff changeset
    87
     * Prints a message of the specified kind at the location of the
06bc494ca11e Initial load
duke
parents:
diff changeset
    88
     * annotation value inside the annotation mirror of the annotated
06bc494ca11e Initial load
duke
parents:
diff changeset
    89
     * element.
06bc494ca11e Initial load
duke
parents:
diff changeset
    90
     *
06bc494ca11e Initial load
duke
parents:
diff changeset
    91
     * @param kind the kind of message
06bc494ca11e Initial load
duke
parents:
diff changeset
    92
     * @param msg  the message, or an empty string if none
06bc494ca11e Initial load
duke
parents:
diff changeset
    93
     * @param e    the annotated element
06bc494ca11e Initial load
duke
parents:
diff changeset
    94
     * @param a    the annotation containing the annotaiton value
06bc494ca11e Initial load
duke
parents:
diff changeset
    95
     * @param v    the annotation value to use as a position hint
06bc494ca11e Initial load
duke
parents:
diff changeset
    96
     */
26266
2d24bda701dc 8056061: Mark implementations of public interfaces with an annotation
jlahoda
parents: 25874
diff changeset
    97
    @DefinedBy(Api.ANNOTATION_PROCESSING)
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    98
    public void printMessage(Diagnostic.Kind kind, CharSequence msg,
06bc494ca11e Initial load
duke
parents:
diff changeset
    99
                      Element e, AnnotationMirror a, AnnotationValue v) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   100
        JavaFileObject oldSource = null;
06bc494ca11e Initial load
duke
parents:
diff changeset
   101
        JavaFileObject newSource = null;
06bc494ca11e Initial load
duke
parents:
diff changeset
   102
        JCDiagnostic.DiagnosticPosition pos = null;
06bc494ca11e Initial load
duke
parents:
diff changeset
   103
        JavacElements elemUtils = processingEnv.getElementUtils();
06bc494ca11e Initial load
duke
parents:
diff changeset
   104
        Pair<JCTree, JCCompilationUnit> treeTop = elemUtils.getTreeAndTopLevel(e, a, v);
06bc494ca11e Initial load
duke
parents:
diff changeset
   105
        if (treeTop != null) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   106
            newSource = treeTop.snd.sourcefile;
06bc494ca11e Initial load
duke
parents:
diff changeset
   107
            if (newSource != null) {
12714
73f1f7097c5e 7166010: (javac) JavacMessager incorrectly restores log source file
ksrini
parents: 7681
diff changeset
   108
                // save the old version and reinstate it later
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   109
                oldSource = log.useSource(newSource);
06bc494ca11e Initial load
duke
parents:
diff changeset
   110
                pos = treeTop.fst.pos();
06bc494ca11e Initial load
duke
parents:
diff changeset
   111
            }
06bc494ca11e Initial load
duke
parents:
diff changeset
   112
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   113
        try {
06bc494ca11e Initial load
duke
parents:
diff changeset
   114
            switch (kind) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   115
            case ERROR:
06bc494ca11e Initial load
duke
parents:
diff changeset
   116
                errorCount++;
28455
41245007c074 8066843: Messager.printMessage cannot print multiple errors for same source position
jlahoda
parents: 26266
diff changeset
   117
                log.error(DiagnosticFlag.MULTIPLE, pos, "proc.messager", msg.toString());
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   118
                break;
06bc494ca11e Initial load
duke
parents:
diff changeset
   119
06bc494ca11e Initial load
duke
parents:
diff changeset
   120
            case WARNING:
6153
277c719ff46e 6403456: -Werror should work with annotation processing
jjg
parents: 5847
diff changeset
   121
                warningCount++;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   122
                log.warning(pos, "proc.messager", msg.toString());
06bc494ca11e Initial load
duke
parents:
diff changeset
   123
                break;
06bc494ca11e Initial load
duke
parents:
diff changeset
   124
06bc494ca11e Initial load
duke
parents:
diff changeset
   125
            case MANDATORY_WARNING:
6153
277c719ff46e 6403456: -Werror should work with annotation processing
jjg
parents: 5847
diff changeset
   126
                warningCount++;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   127
                log.mandatoryWarning(pos, "proc.messager", msg.toString());
06bc494ca11e Initial load
duke
parents:
diff changeset
   128
                break;
06bc494ca11e Initial load
duke
parents:
diff changeset
   129
06bc494ca11e Initial load
duke
parents:
diff changeset
   130
            default:
06bc494ca11e Initial load
duke
parents:
diff changeset
   131
                log.note(pos, "proc.messager", msg.toString());
06bc494ca11e Initial load
duke
parents:
diff changeset
   132
                break;
06bc494ca11e Initial load
duke
parents:
diff changeset
   133
            }
06bc494ca11e Initial load
duke
parents:
diff changeset
   134
        } finally {
12714
73f1f7097c5e 7166010: (javac) JavacMessager incorrectly restores log source file
ksrini
parents: 7681
diff changeset
   135
            // reinstate the saved version, only if it was saved earlier
73f1f7097c5e 7166010: (javac) JavacMessager incorrectly restores log source file
ksrini
parents: 7681
diff changeset
   136
            if (newSource != null)
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   137
                log.useSource(oldSource);
06bc494ca11e Initial load
duke
parents:
diff changeset
   138
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   139
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   140
06bc494ca11e Initial load
duke
parents:
diff changeset
   141
    /**
06bc494ca11e Initial load
duke
parents:
diff changeset
   142
     * Prints an error message.
06bc494ca11e Initial load
duke
parents:
diff changeset
   143
     * Equivalent to {@code printError(null, msg)}.
06bc494ca11e Initial load
duke
parents:
diff changeset
   144
     * @param msg  the message, or an empty string if none
06bc494ca11e Initial load
duke
parents:
diff changeset
   145
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   146
    public void printError(String msg) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   147
        printMessage(Diagnostic.Kind.ERROR, msg);
06bc494ca11e Initial load
duke
parents:
diff changeset
   148
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   149
06bc494ca11e Initial load
duke
parents:
diff changeset
   150
    /**
06bc494ca11e Initial load
duke
parents:
diff changeset
   151
     * Prints a warning message.
06bc494ca11e Initial load
duke
parents:
diff changeset
   152
     * Equivalent to {@code printWarning(null, msg)}.
06bc494ca11e Initial load
duke
parents:
diff changeset
   153
     * @param msg  the message, or an empty string if none
06bc494ca11e Initial load
duke
parents:
diff changeset
   154
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   155
    public void printWarning(String msg) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   156
        printMessage(Diagnostic.Kind.WARNING, msg);
06bc494ca11e Initial load
duke
parents:
diff changeset
   157
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   158
06bc494ca11e Initial load
duke
parents:
diff changeset
   159
    /**
06bc494ca11e Initial load
duke
parents:
diff changeset
   160
     * Prints a notice.
06bc494ca11e Initial load
duke
parents:
diff changeset
   161
     * @param msg  the message, or an empty string if none
06bc494ca11e Initial load
duke
parents:
diff changeset
   162
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   163
    public void printNotice(String msg) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   164
        printMessage(Diagnostic.Kind.NOTE, msg);
06bc494ca11e Initial load
duke
parents:
diff changeset
   165
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   166
06bc494ca11e Initial load
duke
parents:
diff changeset
   167
    public boolean errorRaised() {
06bc494ca11e Initial load
duke
parents:
diff changeset
   168
        return errorCount > 0;
06bc494ca11e Initial load
duke
parents:
diff changeset
   169
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   170
06bc494ca11e Initial load
duke
parents:
diff changeset
   171
    public int errorCount() {
06bc494ca11e Initial load
duke
parents:
diff changeset
   172
        return errorCount;
06bc494ca11e Initial load
duke
parents:
diff changeset
   173
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   174
6153
277c719ff46e 6403456: -Werror should work with annotation processing
jjg
parents: 5847
diff changeset
   175
    public int warningCount() {
277c719ff46e 6403456: -Werror should work with annotation processing
jjg
parents: 5847
diff changeset
   176
        return warningCount;
277c719ff46e 6403456: -Werror should work with annotation processing
jjg
parents: 5847
diff changeset
   177
    }
277c719ff46e 6403456: -Werror should work with annotation processing
jjg
parents: 5847
diff changeset
   178
23810
b92eb80925f0 8038455: Use single Context for all rounds of annotation processing
jlahoda
parents: 12714
diff changeset
   179
    public void newRound() {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   180
        errorCount = 0;
06bc494ca11e Initial load
duke
parents:
diff changeset
   181
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   182
06bc494ca11e Initial load
duke
parents:
diff changeset
   183
    public String toString() {
06bc494ca11e Initial load
duke
parents:
diff changeset
   184
        return "javac Messager";
06bc494ca11e Initial load
duke
parents:
diff changeset
   185
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   186
}