Thu, 25 Jul 2013 14:51:40 +0100
8020843: javac crashes on accessibility check with method reference with typevar receiver
Summary: method reference overload check doesn't walk through type-variable receivers
Reviewed-by: jjg
aoqi@0 | 1 | /* |
aoqi@0 | 2 | * Copyright (c) 2012, Oracle and/or its affiliates. All rights reserved. |
aoqi@0 | 3 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
aoqi@0 | 4 | * |
aoqi@0 | 5 | * This code is free software; you can redistribute it and/or modify it |
aoqi@0 | 6 | * under the terms of the GNU General Public License version 2 only, as |
aoqi@0 | 7 | * published by the Free Software Foundation. Oracle designates this |
aoqi@0 | 8 | * particular file as subject to the "Classpath" exception as provided |
aoqi@0 | 9 | * by Oracle in the LICENSE file that accompanied this code. |
aoqi@0 | 10 | * |
aoqi@0 | 11 | * This code is distributed in the hope that it will be useful, but WITHOUT |
aoqi@0 | 12 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
aoqi@0 | 13 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
aoqi@0 | 14 | * version 2 for more details (a copy is included in the LICENSE file that |
aoqi@0 | 15 | * accompanied this code). |
aoqi@0 | 16 | * |
aoqi@0 | 17 | * You should have received a copy of the GNU General Public License version |
aoqi@0 | 18 | * 2 along with this work; if not, write to the Free Software Foundation, |
aoqi@0 | 19 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
aoqi@0 | 20 | * |
aoqi@0 | 21 | * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
aoqi@0 | 22 | * or visit www.oracle.com if you need additional information or have any |
aoqi@0 | 23 | * questions. |
aoqi@0 | 24 | */ |
aoqi@0 | 25 | |
aoqi@0 | 26 | /** |
aoqi@0 | 27 | * @test |
aoqi@0 | 28 | * @bug 8003639 |
aoqi@0 | 29 | * @summary convert lambda testng tests to jtreg and add them |
aoqi@0 | 30 | * @run testng MethodReferenceTestVarArgs |
aoqi@0 | 31 | */ |
aoqi@0 | 32 | |
aoqi@0 | 33 | import org.testng.annotations.Test; |
aoqi@0 | 34 | import java.lang.reflect.Array; |
aoqi@0 | 35 | |
aoqi@0 | 36 | import static org.testng.Assert.assertEquals; |
aoqi@0 | 37 | |
aoqi@0 | 38 | /** |
aoqi@0 | 39 | * @author Robert Field |
aoqi@0 | 40 | */ |
aoqi@0 | 41 | |
aoqi@0 | 42 | @Test |
aoqi@0 | 43 | public class MethodReferenceTestVarArgs { |
aoqi@0 | 44 | |
aoqi@0 | 45 | interface SII { |
aoqi@0 | 46 | |
aoqi@0 | 47 | String m(Integer a, Integer b); |
aoqi@0 | 48 | } |
aoqi@0 | 49 | |
aoqi@0 | 50 | interface Siii { |
aoqi@0 | 51 | |
aoqi@0 | 52 | String m(int a, int b, int c); |
aoqi@0 | 53 | } |
aoqi@0 | 54 | |
aoqi@0 | 55 | interface Si { |
aoqi@0 | 56 | |
aoqi@0 | 57 | String m(int a); |
aoqi@0 | 58 | } |
aoqi@0 | 59 | |
aoqi@0 | 60 | interface SaO { |
aoqi@0 | 61 | |
aoqi@0 | 62 | String m(Object[] a); |
aoqi@0 | 63 | } |
aoqi@0 | 64 | |
aoqi@0 | 65 | interface Sai { |
aoqi@0 | 66 | |
aoqi@0 | 67 | String m(int[] a); |
aoqi@0 | 68 | } |
aoqi@0 | 69 | |
aoqi@0 | 70 | interface Svi { |
aoqi@0 | 71 | |
aoqi@0 | 72 | String m(int... va); |
aoqi@0 | 73 | } |
aoqi@0 | 74 | |
aoqi@0 | 75 | // These should be processed as var args |
aoqi@0 | 76 | |
aoqi@0 | 77 | static String xvI(Integer... vi) { |
aoqi@0 | 78 | StringBuilder sb = new StringBuilder("xvI:"); |
aoqi@0 | 79 | for (Integer i : vi) { |
aoqi@0 | 80 | sb.append(i); |
aoqi@0 | 81 | sb.append("-"); |
aoqi@0 | 82 | } |
aoqi@0 | 83 | return sb.toString(); |
aoqi@0 | 84 | } |
aoqi@0 | 85 | |
aoqi@0 | 86 | static String xIvI(Integer f, Integer... vi) { |
aoqi@0 | 87 | StringBuilder sb = new StringBuilder("xIvI:"); |
aoqi@0 | 88 | sb.append(f); |
aoqi@0 | 89 | for (Integer i : vi) { |
aoqi@0 | 90 | sb.append(i); |
aoqi@0 | 91 | sb.append("-"); |
aoqi@0 | 92 | } |
aoqi@0 | 93 | return sb.toString(); |
aoqi@0 | 94 | } |
aoqi@0 | 95 | |
aoqi@0 | 96 | static String xvi(int... vi) { |
aoqi@0 | 97 | int sum = 0; |
aoqi@0 | 98 | for (int i : vi) { |
aoqi@0 | 99 | sum += i; |
aoqi@0 | 100 | } |
aoqi@0 | 101 | return "xvi:" + sum; |
aoqi@0 | 102 | } |
aoqi@0 | 103 | |
aoqi@0 | 104 | static String xIvi(Integer f, int... vi) { |
aoqi@0 | 105 | int sum = 0; |
aoqi@0 | 106 | for (int i : vi) { |
aoqi@0 | 107 | sum += i; |
aoqi@0 | 108 | } |
aoqi@0 | 109 | return "xIvi:(" + f + ")" + sum; |
aoqi@0 | 110 | } |
aoqi@0 | 111 | |
aoqi@0 | 112 | static String xvO(Object... vi) { |
aoqi@0 | 113 | StringBuilder sb = new StringBuilder("xvO:"); |
aoqi@0 | 114 | for (Object i : vi) { |
aoqi@0 | 115 | if (i.getClass().isArray()) { |
aoqi@0 | 116 | sb.append("["); |
aoqi@0 | 117 | int len = Array.getLength(i); |
aoqi@0 | 118 | for (int x = 0; x < len; ++x) { |
aoqi@0 | 119 | sb.append(Array.get(i, x)); |
aoqi@0 | 120 | sb.append(","); |
aoqi@0 | 121 | } |
aoqi@0 | 122 | sb.append("]"); |
aoqi@0 | 123 | |
aoqi@0 | 124 | } else { |
aoqi@0 | 125 | sb.append(i); |
aoqi@0 | 126 | } |
aoqi@0 | 127 | sb.append("*"); |
aoqi@0 | 128 | } |
aoqi@0 | 129 | return sb.toString(); |
aoqi@0 | 130 | } |
aoqi@0 | 131 | |
aoqi@0 | 132 | public void testVarArgsSuperclass() { |
aoqi@0 | 133 | SII q; |
aoqi@0 | 134 | |
aoqi@0 | 135 | q = MethodReferenceTestVarArgs::xvO; |
aoqi@0 | 136 | assertEquals(q.m(55,66), "xvO:55*66*"); |
aoqi@0 | 137 | } |
aoqi@0 | 138 | |
aoqi@0 | 139 | public void testVarArgsArray() { |
aoqi@0 | 140 | Sai q; |
aoqi@0 | 141 | |
aoqi@0 | 142 | q = MethodReferenceTestVarArgs::xvO; |
aoqi@0 | 143 | assertEquals(q.m(new int[] { 55,66 } ), "xvO:[55,66,]*"); |
aoqi@0 | 144 | } |
aoqi@0 | 145 | |
aoqi@0 | 146 | public void testVarArgsII() { |
aoqi@0 | 147 | SII q; |
aoqi@0 | 148 | |
aoqi@0 | 149 | q = MethodReferenceTestVarArgs::xvI; |
aoqi@0 | 150 | assertEquals(q.m(33,7), "xvI:33-7-"); |
aoqi@0 | 151 | |
aoqi@0 | 152 | q = MethodReferenceTestVarArgs::xIvI; |
aoqi@0 | 153 | assertEquals(q.m(50,40), "xIvI:5040-"); |
aoqi@0 | 154 | |
aoqi@0 | 155 | q = MethodReferenceTestVarArgs::xvi; |
aoqi@0 | 156 | assertEquals(q.m(100,23), "xvi:123"); |
aoqi@0 | 157 | |
aoqi@0 | 158 | q = MethodReferenceTestVarArgs::xIvi; |
aoqi@0 | 159 | assertEquals(q.m(9,21), "xIvi:(9)21"); |
aoqi@0 | 160 | } |
aoqi@0 | 161 | |
aoqi@0 | 162 | public void testVarArgsiii() { |
aoqi@0 | 163 | Siii q; |
aoqi@0 | 164 | |
aoqi@0 | 165 | q = MethodReferenceTestVarArgs::xvI; |
aoqi@0 | 166 | assertEquals(q.m(3, 2, 1), "xvI:3-2-1-"); |
aoqi@0 | 167 | |
aoqi@0 | 168 | q = MethodReferenceTestVarArgs::xIvI; |
aoqi@0 | 169 | assertEquals(q.m(888, 99, 2), "xIvI:88899-2-"); |
aoqi@0 | 170 | |
aoqi@0 | 171 | q = MethodReferenceTestVarArgs::xvi; |
aoqi@0 | 172 | assertEquals(q.m(900,80,7), "xvi:987"); |
aoqi@0 | 173 | |
aoqi@0 | 174 | q = MethodReferenceTestVarArgs::xIvi; |
aoqi@0 | 175 | assertEquals(q.m(333,27, 72), "xIvi:(333)99"); |
aoqi@0 | 176 | } |
aoqi@0 | 177 | |
aoqi@0 | 178 | public void testVarArgsi() { |
aoqi@0 | 179 | Si q; |
aoqi@0 | 180 | |
aoqi@0 | 181 | q = MethodReferenceTestVarArgs::xvI; |
aoqi@0 | 182 | assertEquals(q.m(3), "xvI:3-"); |
aoqi@0 | 183 | |
aoqi@0 | 184 | q = MethodReferenceTestVarArgs::xIvI; |
aoqi@0 | 185 | assertEquals(q.m(888), "xIvI:888"); |
aoqi@0 | 186 | |
aoqi@0 | 187 | q = MethodReferenceTestVarArgs::xvi; |
aoqi@0 | 188 | assertEquals(q.m(900), "xvi:900"); |
aoqi@0 | 189 | |
aoqi@0 | 190 | q = MethodReferenceTestVarArgs::xIvi; |
aoqi@0 | 191 | assertEquals(q.m(333), "xIvi:(333)0"); |
aoqi@0 | 192 | } |
aoqi@0 | 193 | |
aoqi@0 | 194 | // These should NOT be processed as var args |
aoqi@0 | 195 | |
aoqi@0 | 196 | public void testVarArgsaO() { |
aoqi@0 | 197 | SaO q; |
aoqi@0 | 198 | |
aoqi@0 | 199 | q = MethodReferenceTestVarArgs::xvO; |
aoqi@0 | 200 | assertEquals(q.m(new String[] { "yo", "there", "dude" }), "xvO:yo*there*dude*"); |
aoqi@0 | 201 | } |
aoqi@0 | 202 | |
aoqi@0 | 203 | |
aoqi@0 | 204 | } |