On Fri, 3 Sep 2021 13:56:04 GMT, lawrence.andrews 
<[email protected]> wrote:

>> 1) Testcase was failing with Parse Exception: Arguments to `manual' option 
>> not supported: yesno due to yesno
>> 2) After removing =yesno once again test case was failing with following 
>> exception
>> ----------System.err:(15/782)----------
>> java.io.IOException: Illegal seek
>> at java.io.FileInputStream.skip(Native Method)
>> at java.io.BufferedInputStream.skip(BufferedInputStream.java:366)
>> at bug6186488.main(bug6186488.java:36)
>> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>> at 
>> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
>> at 
>> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>> at java.lang.reflect.Method.invoke(Method.java:474)
>> at com.sun.javatest.regtest.MainWrapper$MainThread.run(MainWrapper.java:96)
>> at java.lang.Thread.run(Thread.java:722)
>> 
>> 3) If test case is run independently user has to enter y or n to make the 
>> test case pass or fail that is when user heard the sound or no. The same 
>> action was able to perform via JTREG.  
>> 4) Rewrote the testcase . Now user can listen to sound when he press "Start 
>> Test" and press "pass"  button if user hears the sound  and press "fail" 
>> button when user does not hear the sound. Test UI also shows a timer that 
>> get timeout if user does not perform any action.
>> 5) Add try-with-resource to close the synth.
>> 
>> @shurymury
>
> lawrence.andrews has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Added extra instruction for the user

test/jdk/javax/sound/midi/Synthesizer/Receiver/bug6186488.java line 2:

> 1: /*
> 2:  * Copyright (c) 2005, 2016, 2021, Oracle and/or its affiliates. All 
> rights reserved.

You don't need a third year, but instead the second year should be updated.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5338

Reply via email to