I see Serguei has reviewed this. It looks okay to me too.
-Alan.
On 18/03/2013 14:14, Staffan Larsen wrote:
I still need an official Review for this change.
Thanks,
Staffan
On 7 mar 2013, at 09:10, Staffan Larsen<[email protected]> wrote:
Adding core-libs-dev and re-asking for a review.
Thanks,
/Staffan
On 27 feb 2013, at 15:59, Staffan Larsen<[email protected]> wrote:
Please review the following test fix.
webrev: http://cr.openjdk.java.net/~sla/8006637/webrev.00/
bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8006637
The problem is that the test assumes that no other threads call String.intern()
while the tests is running. This is normally ok, but when running with JFR
there are other threads calling String.intern() in the background and the test
fails. The solution is to add a thread filter to the method exit requests so
that only calls to String.inter() in the main thread are reported.
Thanks,
/Staffan
