RFR JDK-6321472: Add CRC-32C API
Ulf Zibis
Ulf.Zibis at CoSoCo.de
Fri Oct 17 19:56:17 UTC 2014
Am 17.10.2014 um 20:58 schrieb Staffan Friberg:
> Here is a new webrev with the updates from Alan's and Peter's suggestions.
> http://cr.openjdk.java.net/~sfriberg/JDK-6321472/webrev.01
I would not remove:
86 public void update(byte[] b) {
87 adler = updateBytes(adler, b, 0, b.length);
88 }
The interfaces default method invokes update(b, 0, b.length), which unnecessarily wastes performance
with superfluous bound checks, but
86a if (b == null) {
86b throw new NullPointerException();
86c }
should be added, to behave same as
71 public void update(byte[] b, int off, int len) {
in case of null array.
My 2cc.
-Ulf
More information about the core-libs-dev
mailing list