Thanks Nils. Looks good to me too.
David
On 19/02/2013 10:52 PM, Nils Loodin wrote:
Here's the updated webrev:
http://cr.openjdk.java.net/~nloodin/8007804/webrev.01/
Regards,
Nils Loodin
On 02/19/2013 12:09 PM, Nils Loodin wrote:
On 02/09/2013 03:13 AM, David Holmes wrote:
On 9/02/2013 2:40 AM, Nils Loodin wrote:
Please review the following very small and insignificant change:
Basically, I need to access a performance-counter by name from within
the JVM.
The method name by_name seems a little odd to me. find_by_name would
seem more consistent (particularly if you consider the strange
PerfDataList::by_name method - which compares names)
I think you're right. I'll rename it to find_by_name instead and send
out another webrev.
Nisse
David
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8007804
http://cr.openjdk.java.net/~nloodin/8007804/webrev.00/
Regards,
Nils Loodin
