RFR 8251989: Hex encoder and decoder utility

Chris Hegarty chris.hegarty at oracle.com
Thu Aug 20 15:33:47 UTC 2020


> On 19 Aug 2020, at 22:14, Roger Riggs <Roger.Riggs at oracle.com> wrote:
> 
> ..
> JavaDoc:
> http://cr.openjdk.java.net/~rriggs/hex-javadoc/java.base/java/util/Hex.html

I like it Roger, very nice. A few minor comments/quibbles:

Hex:

- "Utilities to encode bytes to hex strings and decode hex strings to bytes.” - This class consists solely of “factory methods" for creating encoders and decoders to ..

- In the example, do you want to assert that the byte arrays `bytes` and `dec` are equal?

- RFC 4752 or RFC 2752. Which one is it? ;-)

Decoder:

- decode​(char[] chars, int index, int length) - " A valid character array consists only of the delimiters, prefix, suffix, and an even number of hex digits.” - its just the portion of the array that is required to consist of …, right. You need to add “range”, or similar.

Please use “ReturnS” consistently ( I see a few Return (no S) ). Actually, there are a few others, e.g. “Convert”. Can you take pass over all these, in all three classes? 

-Chris.







More information about the core-libs-dev mailing list