Mercurial > hg > truffle

annotate graal/com.oracle.truffle.dsl.processor/src/com/oracle/truffle/dsl/processor/verify/VerifyTruffleProcessor.java @ 21480:c2b006c5e15f

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
author Jaroslav Tulach <jaroslav.tulach@oracle.com>
date Mon, 25 May 2015 10:36:30 +0200
parents fb17e716b03c
children bb51b9a142b3
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
18977
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
1 /*
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
2 * Copyright (c) 2014, Oracle and/or its affiliates. All rights reserved.
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
4 *
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
7 * published by the Free Software Foundation.
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
8 *
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
13 * accompanied this code).
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
14 *
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
18 *
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
21 * questions.
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
22 */
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
23 package com.oracle.truffle.dsl.processor.verify;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
24
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
25 import static com.oracle.truffle.dsl.processor.java.ElementUtils.*;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
26 import static java.util.Collections.*;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
27
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
28 import java.io.*;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
29 import java.util.*;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
30
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
31 import javax.annotation.processing.*;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
32 import javax.lang.model.*;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
33 import javax.lang.model.element.*;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
34 import javax.tools.Diagnostic.Kind;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
35
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
36 import com.oracle.truffle.api.CompilerDirectives.TruffleBoundary;
21480
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
37 import com.oracle.truffle.api.dsl.ExpectError;
21420
fb17e716b03c The annotation processor should verify proper use of @Child annotation during compilation time and prevent usage of final.
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 18977
diff changeset
38 import com.oracle.truffle.api.nodes.Node.Child;
18977
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
39
21420
fb17e716b03c The annotation processor should verify proper use of @Child annotation during compilation time and prevent usage of final.
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 18977
diff changeset
40 @SupportedAnnotationTypes({"com.oracle.truffle.api.CompilerDirectives.TruffleBoundary", "com.oracle.truffle.api.nodes.Node.Child"})
18977
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
41 public class VerifyTruffleProcessor extends AbstractProcessor {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
42 @Override
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
43 public SourceVersion getSupportedSourceVersion() {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
44 return SourceVersion.latest();
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
45 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
46
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
47 /**
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
48 * Node class currently being processed.
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
49 */
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
50 private Element scope;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
51
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
52 public static boolean isEnclosedIn(Element e, Element scopeElement) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
53 List<Element> elementHierarchy = getElementHierarchy(e);
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
54 return elementHierarchy.contains(scopeElement);
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
55 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
56
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
57 void errorMessage(Element element, String format, Object... args) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
58 message(Kind.ERROR, element, format, args);
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
59 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
60
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
61 void message(Kind kind, Element element, String format, Object... args) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
62 if (scope != null && !isEnclosedIn(element, scope)) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
63 // See https://bugs.eclipse.org/bugs/show_bug.cgi?id=428357#c1
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
64 List<Element> elementHierarchy = getElementHierarchy(element);
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
65 reverse(elementHierarchy);
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
66
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
67 StringBuilder str = new StringBuilder();
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
68 for (Element e : elementHierarchy) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
69 if (e.getKind() != ElementKind.PACKAGE) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
70 str.append(str.length() == 0 ? "" : ".");
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
71 str.append(e);
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
72 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
73 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
74 processingEnv.getMessager().printMessage(kind, String.format(str + ": " + format, args), scope);
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
75 } else {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
76 processingEnv.getMessager().printMessage(kind, String.format(format, args), element);
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
77 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
78 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
79
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
80 /**
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
81 * Bugs in an annotation processor can cause silent failure so try to report any exception
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
82 * throws as errors.
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
83 */
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
84 private void reportException(Kind kind, Element element, Throwable t) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
85 StringWriter buf = new StringWriter();
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
86 t.printStackTrace(new PrintWriter(buf));
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
87 buf.toString();
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
88 message(kind, element, "Exception thrown during processing: %s", buf.toString());
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
89 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
90
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
91 @Override
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
92 public boolean process(Set<? extends TypeElement> annotations, RoundEnvironment roundEnv) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
93 if (roundEnv.processingOver()) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
94 return false;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
95 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
96
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
97 TypeElement virtualFrameType = processingEnv.getElementUtils().getTypeElement("com.oracle.truffle.api.frame.VirtualFrame");
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
98
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
99 for (Element element : roundEnv.getElementsAnnotatedWith(TruffleBoundary.class)) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
100 scope = element;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
101 try {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
102 ExecutableElement method = (ExecutableElement) element;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
103
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
104 for (VariableElement parameter : method.getParameters()) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
105 Element paramType = processingEnv.getTypeUtils().asElement(parameter.asType());
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
106 if (paramType != null && paramType.equals(virtualFrameType)) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
107 errorMessage(element, "Method %s cannot be annotated with @%s and have a parameter of type %s", method.getSimpleName(), TruffleBoundary.class.getSimpleName(),
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
108 paramType.getSimpleName());
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
109 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
110 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
111 } catch (Throwable t) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
112 reportException(isBug367599(t) ? Kind.NOTE : Kind.ERROR, element, t);
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
113 } finally {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
114 scope = null;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
115 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
116 }
21420
fb17e716b03c The annotation processor should verify proper use of @Child annotation during compilation time and prevent usage of final.
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 18977
diff changeset
117
fb17e716b03c The annotation processor should verify proper use of @Child annotation during compilation time and prevent usage of final.
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 18977
diff changeset
118 for (Element e : roundEnv.getElementsAnnotatedWith(Child.class)) {
fb17e716b03c The annotation processor should verify proper use of @Child annotation during compilation time and prevent usage of final.
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 18977
diff changeset
119 if (e.getModifiers().contains(Modifier.FINAL)) {
21480
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
120 emitError("@Child field cannot be final", e);
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
121 continue;
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
122 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
123 assertNoErrorExpected(e);
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
124 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
125 return false;
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
126 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
127
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
128 void assertNoErrorExpected(Element e) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
129 TypeElement eee = processingEnv.getElementUtils().getTypeElement(ExpectError.class.getName());
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
130 for (AnnotationMirror am : e.getAnnotationMirrors()) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
131 if (am.getAnnotationType().asElement() == eee) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
132 processingEnv.getMessager().printMessage(Kind.ERROR, "Expected an error, but none found!", e);
21420
fb17e716b03c The annotation processor should verify proper use of @Child annotation during compilation time and prevent usage of final.
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 18977
diff changeset
133 }
fb17e716b03c The annotation processor should verify proper use of @Child annotation during compilation time and prevent usage of final.
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 18977
diff changeset
134 }
21480
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
135 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
136
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
137 void emitError(String msg, Element e) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
138 TypeElement eee = processingEnv.getElementUtils().getTypeElement(ExpectError.class.getName());
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
139 for (AnnotationMirror am : e.getAnnotationMirrors()) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
140 if (am.getAnnotationType().asElement() == eee) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
141 Map<? extends ExecutableElement, ? extends AnnotationValue> vals = am.getElementValues();
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
142 if (vals.size() == 1) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
143 AnnotationValue av = vals.values().iterator().next();
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
144 if (av.getValue() instanceof List) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
145 List<?> arr = (List<?>) av.getValue();
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
146 for (Object o : arr) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
147 if (o instanceof AnnotationValue) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
148 AnnotationValue ov = (AnnotationValue) o;
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
149 if (msg.equals(ov.getValue())) {
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
150 return;
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
151 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
152 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
153 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
154 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
155 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
156 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
157 }
c2b006c5e15f Testing the annotation processor using the @ExpectedError annotation as suggested by Christian Humer
Jaroslav Tulach <jaroslav.tulach@oracle.com>
parents: 21420
diff changeset
158 processingEnv.getMessager().printMessage(Kind.ERROR, msg, e);
18977
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
159 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
160
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
161 /**
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
162 * Determines if a given exception is (most likely) caused by <a
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
163 * href="https://bugs.eclipse.org/bugs/show_bug.cgi?id=367599">Bug 367599</a>.
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
164 */
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
165 public static boolean isBug367599(Throwable t) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
166 if (t instanceof FilerException) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
167 for (StackTraceElement ste : t.getStackTrace()) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
168 if (ste.toString().contains("org.eclipse.jdt.internal.apt.pluggable.core.filer.IdeFilerImpl.create")) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
169 // See: https://bugs.eclipse.org/bugs/show_bug.cgi?id=367599
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
170 return true;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
171 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
172 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
173 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
174 if (t.getCause() != null) {
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
175 return isBug367599(t.getCause());
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
176 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
177 return false;
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
178 }
8fd7354353b7 add annotation processor to verify TruffleBoundary/VirtualFrame usages
Lukas Stadler <lukas.stadler@oracle.com>
parents:
diff changeset
179 }