archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-09-20 19:14:38 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

6983646: javap should identify why a DefaultAttribute is being used

Browse source 
Reviewed-by: jjg
This commit is contained in:
Vicente Romero 2013-06-29 20:12:24 +01:00
parent c5e36903f7
commit a33129c6af
3 changed files with 26 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

10
langtools/src/share/classes/com/sun/tools/classfile/Attribute.java
View file

@ -77,10 +77,12 @@ public abstract class Attribute {
public Attribute createAttribute(ClassReader cr, int name_index, byte[] data)
throws IOException {
if (standardAttributes == null)
if (standardAttributes == null) {
init();
}
ConstantPool cp = cr.getConstantPool();
String reasonForDefaultAttr;
try {
String name = cp.getUTF8Value(name_index);
Class<? extends Attribute> attrClass = standardAttributes.get(name);
@ -90,14 +92,18 @@ public abstract class Attribute {
Constructor<? extends Attribute> constr = attrClass.getDeclaredConstructor(constrArgTypes);
return constr.newInstance(new Object[] { cr, name_index, data.length });
} catch (Throwable t) {
reasonForDefaultAttr = t.toString();
// fall through and use DefaultAttribute
// t.printStackTrace();
}
} else {
reasonForDefaultAttr = "unknown attribute";
}
} catch (ConstantPoolException e) {
reasonForDefaultAttr = e.toString();
// fall through and use DefaultAttribute
}
return new DefaultAttribute(cr, name_index, data);
return new DefaultAttribute(cr, name_index, data, reasonForDefaultAttr);
}
protected void init() {

16
langtools/src/share/classes/com/sun/tools/classfile/DefaultAttribute.java
View file

@ -1,5 +1,5 @@
/*
* Copyright (c) 2007, 2008, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2007, 2013, Oracle and/or its affiliates. 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
@ -33,13 +33,24 @@ package com.sun.tools.classfile;
*/
public class DefaultAttribute extends Attribute {
DefaultAttribute(ClassReader cr, int name_index, byte[] data) {
this(cr, name_index, data, null);
}
DefaultAttribute(ClassReader cr, int name_index, byte[] data, String reason) {
super(name_index, data.length);
info = data;
this.reason = reason;
}
public DefaultAttribute(ConstantPool constant_pool, int name_index, byte[] info) {
this(constant_pool, name_index, info, null);
}
public DefaultAttribute(ConstantPool constant_pool, int name_index,
byte[] info, String reason) {
super(name_index, info.length);
this.info = info;
this.reason = reason;
}
public <R, P> R accept(Visitor<R, P> visitor, P p) {
@ -47,4 +58,7 @@ public class DefaultAttribute extends Attribute {
}
public final byte[] info;
/** Why did we need to generate a DefaultAttribute
*/
public final String reason;
}

3
langtools/src/share/classes/com/sun/tools/javap/AttributeWriter.java
View file

@ -114,6 +114,9 @@ public class AttributeWriter extends BasicWriter
}
public Void visitDefault(DefaultAttribute attr, Void ignore) {
if (attr.reason != null) {
report(attr.reason);
}
byte[] data = attr.info;
int i = 0;
int j = 0;