ExternalVendorCode/corda
1
0
Fork 0
mirror of https://github.com/corda/corda.git synced 2025-01-03 19:54:13 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Merge pull request #234 from dicej/field_getAnnotations

Browse Source 
fix NPE in Field.getAnnotations
This commit is contained in:
Joshua Warner 2014-04-17 15:18:07 -06:00
commit b01515ac84
2 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
classpath/java/lang/reflect/Field.java
View File

@ -348,8 +348,13 @@ public class Field<T> extends AccessibleObject {
return (Annotation) a[0];
}
private boolean hasAnnotations() {
return vmField.addendum != null
&& vmField.addendum.annotationTable != null;
}
public <T extends Annotation> T getAnnotation(Class<T> class_) {
if (vmField.addendum != null && vmField.addendum.annotationTable != null) {
if (hasAnnotations()) {
Object[] table = (Object[]) vmField.addendum.annotationTable;
for (int i = 0; i < table.length; ++i) {
Object[] a = (Object[]) table[i];
@ -362,7 +367,7 @@ public class Field<T> extends AccessibleObject {
}
public Annotation[] getAnnotations() {
if (vmField.addendum.annotationTable != null) {
if (hasAnnotations()) {
Object[] table = (Object[]) vmField.addendum.annotationTable;
Annotation[] array = new Annotation[table.length];
for (int i = 0; i < table.length; ++i) {

5
test/Reflection.java
View File

@ -252,6 +252,9 @@ public class Reflection {
expect(avian.testing.annotations.Test.class.getPackage().getName().equals
("avian.testing.annotations"));
expect(Baz.class.getField("foo").getAnnotation(Ann.class) == null);
expect(Baz.class.getField("foo").getAnnotations().length == 0);
}
protected static class Baz {
@ -280,3 +283,5 @@ interface A {
}
interface B extends A { }
@interface Ann { }