<Sound Dev> [9] Review Request: 8167028 SunCodec.java can be removed
Phil Race
philip.race at oracle.com
Mon Oct 3 22:26:32 UTC 2016
+1
-phil.
On 10/03/2016 09:29 AM, Alex Menkov wrote:
> Looks okay to me.
>
> --alex
>
> On 03.10.2016 17:45, Sergey Bylokhov wrote:
>> Hello, Audio Guru.
>>
>> This is a request to small cleanup of the classes related to
>> FormatConversionProvider.
>> - SunCodec.java is removed. It was used as a parent class for other
>> providers, but it contained only the methods to clone the array of
>> supported formats. But we can have the same functionality if instead of
>> cloning the array just create the new one each time it will be
>> requested.
>> - All our providers supports the same encodings as input and output. So
>> getTargetEncodings() can be implemented via getSourceEncodings();
>> - FormatConversionProvider contains a few methods to filter, search the
>> data in arrays, and this functionality can be simplified by Stream API.
>>
>> No regressions were found by jtreg/jck tests.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8167028
>> Webrev can be found at:
>> http://cr.openjdk.java.net/~serb/8167028/webrev.00
>>
More information about the sound-dev
mailing list