Looks good to me Staffan! (Those methods should have been called getXXXCount not Counter though :( )
David On 8/05/2013 6:17 PM, Staffan Larsen wrote:
When JDK-8010862 (Method counter fields used for profiling can be allocated lazily) was fixed, SA was not updated to reflect this change. The problem is with Method.getInvocationCounter() and Method.getBackedgeCounter() which do not handle the case where MethodCounters are not allocated. The fix is to check for null and return 0 as the counter value. webrev: http://cr.openjdk.java.net/~sla/8013591/webrev.00/ bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8013591 Thanks, /Staffan