<div dir="ltr">Y'all should import GcFinalization for use as a testing utility.<br><br><a href="http://code.google.com/p/guava-libraries/source/browse/guava-testlib/src/com/google/common/testing/GcFinalization.java">http://code.google.com/p/guava-libraries/source/browse/guava-testlib/src/com/google/common/testing/GcFinalization.java</a><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 13, 2015 at 4:55 AM, Mattias Tobiasson <span dir="ltr"><<a href="mailto:mattias.tobiasson@oracle.com" target="_blank">mattias.tobiasson@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
Could I please have a review of this test bug fix.<br>
<br>
bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8068613" target="_blank">https://bugs.openjdk.java.net/<u></u>browse/JDK-8068613</a><br>
webrev: <a href="http://cr.openjdk.java.net/~ykantser/8068613/webrev.00/" target="_blank">http://cr.openjdk.java.net/~<u></u>ykantser/8068613/webrev.00/</a> <<a href="http://cr.openjdk.java.net/%7Eykantser/8068613/webrev.00/" target="_blank">http://cr.openjdk.java.net/%<u></u>7Eykantser/8068613/webrev.00/</a>><br>
<br>
On line 90 and on line 246 (in the old version) the test does a short wait between<br>
Runtime.getRuntime().<u></u>runFinalization() and<br>
MemoryMXBean.<u></u>getObjectPendingFinalizationCo<u></u>unt()<br>
<br>
The test expects getObjectPendingFinalizationCo<u></u>unt() to return 0,<br>
but that can fail on really slow servers.<br>
<br>
The fix will loop until getObjectPendingFinalizationCo<u></u>unt() returns 0,<br>
or until it gives up after 100 tries.<br>
<br>
I have also moved all logging and finalization count verification to a function.<br>
This makes the diff look like everything have been changed, but it is mostly<br>
duplicate logging code that has been removed.<br>
<br>
I have tested it on all platforms, except embedded.<br>
<br>
Thanks,<br>
Mattias<br>
<br>
</blockquote></div><br></div>