Hi Sergey, Minor: Why the import of CountDownLatch? I don’t see where it is used.
import java.util.concurrent.CountDownLatch; -Dan > On Mar 21, 2017, at 9:38 AM, Sergey Bylokhov <sergey.bylok...@oracle.com> > wrote: > > Hello, Audio Guru. > > Please review the fix for the test in jdk9 > > The test tries to check that the Clip.close() method does not lose some audio > data(position). > It assumes that the time spent in the Clip.close() should be: realtime + 50 > ms > time in the close(); > > But time in the close methods is calculated incorrectly because position is > 22050 frames per second(22.05 per ms). So the test should convert frames to > ms first. > > Bug: https://bugs.openjdk.java.net/browse/JDK-6574989 > Webrev can be found at: http://cr.openjdk.java.net/~serb/6574989/webrev.01
