<Sound Dev>  Review Request: 8167028 SunCodec.java can be removed
alexey.menkov at oracle.com
Mon Oct 3 16:29:59 UTC 2016
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 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