On Thu, 12 Jun 2025 23:00:19 GMT, Sergey Bylokhov <[email protected]> wrote:
>> Phil Race has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8359053
>
> test/jdk/javax/sound/sampled/Clip/DataPusherThreadCheck.java line 73:
>
>> 71:
>> 72: private static void checkThread(File file) throws Exception {
>> 73: SoundClip clip = SoundClip.createSoundClip(file);
>
> I think we should either support content handlers via the new SoundClip
> object or remove the existing implementation, as it is currently tied to
> AudioClip.
was this comment meant to be on some other (test) code ?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25698#discussion_r2146001842
