One of the things that I'm struggling with is that StackFrameInfo
contains both the collected information from walking the stack frames,
method id, bci, mirror, version and cpref, and the digested information:
interned string for class name, method name, line number and source file
name.
If this is to replace stack walking for exceptions, this will more than
double the footprint of the information collected but rarely used. I
don't understand why the digested information isn't still StackFrameElement?
That's just a high level comment. I haven't read the java code yet for
the rationale why this type is used for two different things.
Coleen
On 11/16/15 7:13 PM, Mandy Chung wrote:
I’d like to get the code review done by this week.
I renamed the static factory method from create to getInstance since “create”
implies to create a new instance but the method returns a cached instance
instead. I also changed the spec of getCallerClass per [1]. There is not much
change since webrev.01.
Webrev:
http://cr.openjdk.java.net/~mchung/jdk9/jep259/webrev.02
javadoc:
http://cr.openjdk.java.net/~mchung/jdk9/jep259/api/
Mandy
[1]
http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-November/036589.html
