On 20/11/15 03:13, Mandy Chung wrote:
Cross-posting to core-libs-dev.
Delta webrev incorporating feedback from Coleen and Brent and also a fix that
Daniel made:
http://cr.openjdk.java.net/~mchung/jdk9/jep259/webrev.03-delta/
Full webrev:
http://cr.openjdk.java.net/~mchung/jdk9/jep259/webrev.03/
This also includes a couple of new tests Hamlin has contributed.
Looks good to me Mandy!
-- daniel
Mandy
On Nov 19, 2015, at 1:38 PM, Coleen Phillimore <coleen.phillim...@oracle.com>
wrote:
Hi Mandy,
Thank you for making these changes. I think it looks really good!
On 11/19/15 4:09 PM, Mandy Chung wrote:
Hi Coleen,
Here is the revised webrev incorporating your suggestions and a couple other
minor java side change:
http://cr.openjdk.java.net/~mchung/jdk9/jep259/webrev.03-delta
I got some idea how to prepare Throwable to switch to StackWalker API to
address the footprint/performance concern. I agree with you that it would have
to use mid/cpref in the short term for Throwable. StackWalker::walk should use
MemberName and should improve the footprint/performance issue in the long term.
Are you okay to follow that up with JDK-8141239 and Throwable continues to use
the VM backtrace until JDK-8141239 is resolved?
Yes, and as we discussed, I'd like to help with this.
Thanks!
Coleen