<Sound Dev>  Review Request: 8133677 Specification of AudioFileReader should be clarifed
alexey.menkov at oracle.com
Thu Aug 20 13:01:48 UTC 2015
Looks good for me.
On 19.08.2015 18:24, Sergey Bylokhov wrote:
> Please review the fix for jdk9.
> This is specification update of AudioFileReader. CCC request will be
> filed after technical review.
> According to specification if AudioFileReader.getAudioFileFormat()
> returns a correct format then 'reset' of stream is not strictly
> necessary. But it was implemented like that in 1.3.0 - 1.8 in most of
> our readers except SoftMidiAudioFileReader which was added in jdk7. In
> the fix I removed the sentence that the reader should reset the stream
> only if it is not supported. Note that the same sentence already used in
> the similar API for midi .
> The fix of specification also required some changes in
> SoftMidiAudioFileReader.java, it now uses the general implementations of
> some methods from SunFileReader.java.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8133677
> Webrev can be found at: http://cr.openjdk.java.net/~serb/8133677/webrev.00
More information about the sound-dev