<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