Fri, 24 May 2013 15:27:12 +0100
8014649: Regression: bug in Resolve.resolveOperator
Summary: Missing curly braces causes Resolve.findMethod to be called spuriously
Reviewed-by: jjg, vromero
duke@1 | 1 | /* |
ohair@798 | 2 | * Copyright (c) 2006, 2010, 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 6423583 |
duke@1 | 27 | * @summary LiteralTree.getValue() should return Boolean for Kind.BOOLEAN_LITERAL literals |
duke@1 | 28 | * @build T6423583 |
duke@1 | 29 | * @compile -proc:only -processor T6423583 T6423583.java |
duke@1 | 30 | */ |
duke@1 | 31 | |
duke@1 | 32 | import java.util.*; |
duke@1 | 33 | import javax.annotation.processing.*; |
duke@1 | 34 | import javax.lang.model.*; |
duke@1 | 35 | import javax.lang.model.element.*; |
duke@1 | 36 | import com.sun.source.tree.*; |
duke@1 | 37 | import com.sun.source.util.*; |
duke@1 | 38 | |
duke@1 | 39 | @SupportedAnnotationTypes("*") |
duke@1 | 40 | public class T6423583 extends AbstractProcessor { |
duke@1 | 41 | boolean b1 = true; |
duke@1 | 42 | boolean b2 = false; |
duke@1 | 43 | String s = "s"; |
duke@1 | 44 | char c = 'c'; |
duke@1 | 45 | int i = 0; |
duke@1 | 46 | long l = 0l; |
duke@1 | 47 | float f = 0f; |
duke@1 | 48 | double d = 0d; |
duke@1 | 49 | Void v = null; |
duke@1 | 50 | |
duke@1 | 51 | public boolean process(Set<? extends TypeElement> annos, RoundEnvironment rEnv) { |
duke@1 | 52 | Trees trees = Trees.instance(processingEnv); |
duke@1 | 53 | Test test = new Test(); |
duke@1 | 54 | for (Element e: rEnv.getRootElements()) { |
duke@1 | 55 | Tree t = trees.getTree(e); |
duke@1 | 56 | test.scan(t, null); |
duke@1 | 57 | } |
duke@1 | 58 | return true; |
duke@1 | 59 | } |
duke@1 | 60 | |
darcy@495 | 61 | @Override |
darcy@495 | 62 | public SourceVersion getSupportedSourceVersion() { |
darcy@495 | 63 | return SourceVersion.latest(); |
darcy@495 | 64 | } |
duke@1 | 65 | |
duke@1 | 66 | private static class Test extends TreeScanner<Void,Void> { |
duke@1 | 67 | |
duke@1 | 68 | private static Map<Tree.Kind, Class> map = new HashMap<Tree.Kind, Class>(); |
duke@1 | 69 | static { |
duke@1 | 70 | map.put(Tree.Kind.BOOLEAN_LITERAL, Boolean.class); |
duke@1 | 71 | map.put(Tree.Kind.CHAR_LITERAL, Character.class); |
duke@1 | 72 | map.put(Tree.Kind.STRING_LITERAL, String.class); |
duke@1 | 73 | map.put(Tree.Kind.INT_LITERAL, Integer.class); |
duke@1 | 74 | map.put(Tree.Kind.LONG_LITERAL, Long.class); |
duke@1 | 75 | map.put(Tree.Kind.FLOAT_LITERAL, Float.class); |
duke@1 | 76 | map.put(Tree.Kind.DOUBLE_LITERAL, Double.class); |
duke@1 | 77 | } |
duke@1 | 78 | |
duke@1 | 79 | public Void visitLiteral(LiteralTree tree, Void ignore) { |
duke@1 | 80 | System.err.println(tree); |
duke@1 | 81 | Class expect = map.get(tree.getKind()); |
duke@1 | 82 | if (!check(tree.getValue(), expect)) { |
duke@1 | 83 | System.err.println("tree: " + tree); |
duke@1 | 84 | System.err.println("expected class: " + expect); |
duke@1 | 85 | if (tree.getValue() != null) |
duke@1 | 86 | System.err.println("actual class: " + tree.getValue().getClass()); |
duke@1 | 87 | throw new AssertionError("unexpected value for literal"); |
duke@1 | 88 | } |
duke@1 | 89 | return null; |
duke@1 | 90 | } |
duke@1 | 91 | |
duke@1 | 92 | private boolean check(Object value, Class<?> expectedClass) { |
duke@1 | 93 | return (value == null ? expectedClass == null : value.getClass().equals(expectedClass)); |
duke@1 | 94 | } |
duke@1 | 95 | } |
duke@1 | 96 | } |