<Sound Dev> [9] Review Request: 8133677 Specification of AudioFileReader should be clarifed

[alexey menkov]

Looks good for me.

regards
Alex

On 19.08.2015 18:24, Sergey Bylokhov wrote:
> Hello.
> 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 [1].
>
> 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
>
> [1]
> http://docs.oracle.com/javase/7/docs/api/javax/sound/midi/MidiSystem.html#getMidiFileFormat%28java.io.InputStream%29
>
>