<Sound Dev>  Review Request: 8167028 SunCodec.java can be removed
philip.race at oracle.com
Mon Oct 3 22:26:32 UTC 2016
On 10/03/2016 09:29 AM, Alex Menkov wrote:
> Looks okay to me.
> On 03.10.2016 17:45, Sergey Bylokhov wrote:
>> Hello, Audio Guru.
>> This is a request to small cleanup of the classes related to
>> - 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
>> - 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:
More information about the sound-dev