Mon, 16 Sep 2013 14:13:44 +0200
8021112: Spurious unchecked warning reported by javac
6480588: No way to suppress deprecation warnings when implementing deprecated interface
Summary: Fixing DeferredLintHandler configuration, so lint warnings are reported with correct @SuppressWarnings settings
Reviewed-by: jjg, vromero
duke@1 | 1 | /* |
jjg@1733 | 2 | * Copyright (c) 2006, 2013, Oracle and/or its affiliates. All rights reserved. |
duke@1 | 3 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
duke@1 | 4 | * |
duke@1 | 5 | * This code is free software; you can redistribute it and/or modify it |
duke@1 | 6 | * under the terms of the GNU General Public License version 2 only, as |
duke@1 | 7 | * published by the Free Software Foundation. |
duke@1 | 8 | * |
duke@1 | 9 | * This code is distributed in the hope that it will be useful, but WITHOUT |
duke@1 | 10 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
duke@1 | 11 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
duke@1 | 12 | * version 2 for more details (a copy is included in the LICENSE file that |
duke@1 | 13 | * accompanied this code). |
duke@1 | 14 | * |
duke@1 | 15 | * You should have received a copy of the GNU General Public License version |
duke@1 | 16 | * 2 along with this work; if not, write to the Free Software Foundation, |
duke@1 | 17 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
duke@1 | 18 | * |
ohair@554 | 19 | * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
ohair@554 | 20 | * or visit www.oracle.com if you need additional information or have any |
ohair@554 | 21 | * questions. |
duke@1 | 22 | */ |
duke@1 | 23 | |
duke@1 | 24 | /* |
duke@1 | 25 | * @test |
duke@1 | 26 | * @bug 6421111 |
duke@1 | 27 | * @summary NullPointerException thrown when retrieving bounds for the type parameter |
duke@1 | 28 | * @author Peter von der Ah\u00e9 |
duke@1 | 29 | * @library ../lib |
jjg@1733 | 30 | * @build ToolTester |
duke@1 | 31 | * @compile -Xlint:all T6421111.java |
duke@1 | 32 | * @run main T6421111 |
duke@1 | 33 | */ |
duke@1 | 34 | |
duke@1 | 35 | import java.io.File; |
duke@1 | 36 | import java.net.URI; |
duke@1 | 37 | import java.util.Arrays; |
duke@1 | 38 | import java.util.Collections; |
duke@1 | 39 | import java.util.Set; |
duke@1 | 40 | import javax.annotation.processing.AbstractProcessor; |
duke@1 | 41 | import javax.annotation.processing.RoundEnvironment; |
duke@1 | 42 | import javax.annotation.processing.SupportedAnnotationTypes; |
duke@1 | 43 | import javax.annotation.processing.SupportedSourceVersion; |
duke@1 | 44 | import javax.lang.model.SourceVersion; |
duke@1 | 45 | import javax.lang.model.element.TypeElement; |
duke@1 | 46 | import javax.lang.model.element.TypeParameterElement; |
duke@1 | 47 | import javax.lang.model.type.DeclaredType; |
duke@1 | 48 | import javax.lang.model.type.TypeVariable; |
duke@1 | 49 | import javax.tools.SimpleJavaFileObject; |
duke@1 | 50 | |
duke@1 | 51 | import static javax.tools.JavaFileObject.Kind.SOURCE; |
duke@1 | 52 | |
duke@1 | 53 | public class T6421111 extends ToolTester { |
duke@1 | 54 | void test(String... args) { |
duke@1 | 55 | class Test1 extends SimpleJavaFileObject { |
duke@1 | 56 | Test1() { |
duke@1 | 57 | super(URI.create("myfo:///Test1.java"), SOURCE); |
duke@1 | 58 | } |
duke@1 | 59 | @Override |
duke@1 | 60 | public String getCharContent(boolean ignoreEncodingErrors) { |
duke@1 | 61 | return "class Test1<T extends Thread & Runnable> {}"; |
duke@1 | 62 | } |
duke@1 | 63 | } |
duke@1 | 64 | class Test2 extends SimpleJavaFileObject { |
duke@1 | 65 | Test2() { |
duke@1 | 66 | super(URI.create("myfo:///Test2.java"), SOURCE); |
duke@1 | 67 | } |
duke@1 | 68 | @Override |
duke@1 | 69 | public String getCharContent(boolean ignoreEncodingErrors) { |
duke@1 | 70 | return "class Test2<T extends Test2<T> & Runnable> {}"; |
duke@1 | 71 | } |
duke@1 | 72 | } |
duke@1 | 73 | task = tool.getTask(null, fm, null, Collections.singleton("-Xlint:all"), null, |
duke@1 | 74 | Arrays.asList(new Test1(), new Test2())); |
duke@1 | 75 | task.setProcessors(Collections.singleton(new MyProcessor())); |
duke@1 | 76 | if (!task.call()) |
duke@1 | 77 | throw new AssertionError("Annotation processor failed"); |
duke@1 | 78 | } |
duke@1 | 79 | @SupportedAnnotationTypes("*") |
duke@1 | 80 | static class MyProcessor extends AbstractProcessor { |
duke@1 | 81 | void test(TypeElement element, boolean fbound) { |
duke@1 | 82 | TypeParameterElement tpe = element.getTypeParameters().iterator().next(); |
duke@1 | 83 | tpe.getBounds().getClass(); |
duke@1 | 84 | if (fbound) { |
duke@1 | 85 | DeclaredType type = (DeclaredType)tpe.getBounds().get(0); |
duke@1 | 86 | if (type.asElement() != element) |
duke@1 | 87 | throw error("%s != %s", type.asElement(), element); |
duke@1 | 88 | TypeVariable tv = (TypeVariable)type.getTypeArguments().get(0); |
duke@1 | 89 | if (tv.asElement() != tpe) |
duke@1 | 90 | throw error("%s != %s", tv.asElement(), tpe); |
duke@1 | 91 | } |
duke@1 | 92 | } |
duke@1 | 93 | public boolean process(Set<? extends TypeElement> annotations, |
duke@1 | 94 | RoundEnvironment roundEnv) { |
duke@1 | 95 | test(processingEnv.getElementUtils().getTypeElement("Test1"), false); |
duke@1 | 96 | test(processingEnv.getElementUtils().getTypeElement("Test2"), true); |
duke@1 | 97 | return false; |
duke@1 | 98 | } |
darcy@495 | 99 | @Override |
darcy@495 | 100 | public SourceVersion getSupportedSourceVersion() { |
darcy@495 | 101 | return SourceVersion.latest(); |
darcy@495 | 102 | } |
duke@1 | 103 | } |
duke@1 | 104 | public static void main(String... args) { |
duke@1 | 105 | new T6421111().test(args); |
duke@1 | 106 | } |
duke@1 | 107 | public static AssertionError error(String format, Object... args) { |
duke@1 | 108 | return new AssertionError(String.format(format, args)); |
duke@1 | 109 | } |
duke@1 | 110 | } |