6918625: handle annotations on array class literals
Co-authored-by: Michael Ernst <mernst@cs.washington.edu> Reviewed-by: jjg, darcy
This commit is contained in:
parent
378a8861c0
commit
b0d7cf8472
@ -744,6 +744,7 @@ public class ClassWriter {
|
||||
out.writeByte(p.type_index);
|
||||
break;
|
||||
case CLASS_LITERAL:
|
||||
case CLASS_LITERAL_GENERIC_OR_ARRAY:
|
||||
out.writeShort(p.offset);
|
||||
break;
|
||||
// method parameter: not specified
|
||||
|
@ -139,6 +139,7 @@ public class TypeAnnotationPosition {
|
||||
sb.append(type_index);
|
||||
break;
|
||||
case CLASS_LITERAL:
|
||||
case CLASS_LITERAL_GENERIC_OR_ARRAY:
|
||||
sb.append(", offset = ");
|
||||
sb.append(offset);
|
||||
break;
|
||||
|
@ -162,6 +162,7 @@ public class AnnotationWriter extends BasicWriter {
|
||||
print(pos.type_index);
|
||||
break;
|
||||
case CLASS_LITERAL:
|
||||
case CLASS_LITERAL_GENERIC_OR_ARRAY:
|
||||
if (showOffsets) {
|
||||
print(", offset=");
|
||||
print(pos.offset);
|
||||
|
@ -0,0 +1,82 @@
|
||||
/*
|
||||
* Copyright 2009 Sun Microsystems, Inc. All Rights Reserved.
|
||||
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
|
||||
*
|
||||
* This code is free software; you can redistribute it and/or modify it
|
||||
* under the terms of the GNU General Public License version 2 only, as
|
||||
* published by the Free Software Foundation.
|
||||
*
|
||||
* This code is distributed in the hope that it will be useful, but WITHOUT
|
||||
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
|
||||
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
|
||||
* version 2 for more details (a copy is included in the LICENSE file that
|
||||
* accompanied this code).
|
||||
*
|
||||
* You should have received a copy of the GNU General Public License version
|
||||
* 2 along with this work; if not, write to the Free Software Foundation,
|
||||
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
|
||||
*
|
||||
* Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
|
||||
* CA 95054 USA or visit www.sun.com if you need additional information or
|
||||
* have any questions.
|
||||
*/
|
||||
|
||||
/*
|
||||
* @test
|
||||
* @bug 6918625
|
||||
* @summary javap dumps type information of array class literals
|
||||
*/
|
||||
|
||||
import java.io.*;
|
||||
|
||||
public class ArrayClassLiterals2 {
|
||||
public static void main(String[] args) throws Exception {
|
||||
new ArrayClassLiterals2().run();
|
||||
}
|
||||
|
||||
public void run() throws IOException {
|
||||
File classFile = new File(System.getProperty("test.classes"), "ArrayClassLiterals2$Test.class");
|
||||
|
||||
verify(classFile,
|
||||
"RuntimeInvisibleTypeAnnotations:",
|
||||
"CLASS_LITERAL_GENERIC_OR_ARRAY"
|
||||
);
|
||||
|
||||
if (errors > 0)
|
||||
throw new Error(errors + " found.");
|
||||
}
|
||||
|
||||
String javap(File f) {
|
||||
StringWriter sw = new StringWriter();
|
||||
PrintWriter out = new PrintWriter(sw);
|
||||
int rc = com.sun.tools.javap.Main.run(new String[] { "-v", f.getPath() }, out);
|
||||
if (rc != 0)
|
||||
throw new Error("javap failed. rc=" + rc);
|
||||
out.close();
|
||||
return sw.toString();
|
||||
}
|
||||
|
||||
void verify(File classFile, String... expects) {
|
||||
String output = javap(classFile);
|
||||
for (String expect: expects) {
|
||||
if (output.indexOf(expect)< 0)
|
||||
error(expect + " not found");
|
||||
}
|
||||
}
|
||||
|
||||
void error(String msg) {
|
||||
System.err.println(msg);
|
||||
errors++;
|
||||
}
|
||||
|
||||
int errors;
|
||||
|
||||
|
||||
/*********************** Test class *************************/
|
||||
static class Test {
|
||||
@interface A { }
|
||||
void test() {
|
||||
Object a = @A String @A [] @A [].class;
|
||||
}
|
||||
}
|
||||
}
|
Loading…
Reference in New Issue
Block a user