RFR: 8263754: HexFormat 'fromHex' methods should be static

Claes Redestad redestad at openjdk.java.net
Thu Mar 25 22:49:26 UTC 2021


On Thu, 25 Mar 2021 21:50:46 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/tools/keytool/Main.java line 4582:
>> 
>>> 4580:                             int pos = 0;
>>> 4581:                             for (char c: value.toCharArray()) {
>>> 4582:                                 if (!HexFormat.isHexDigit(c)) {
>> 
>> `hexFmt` created on line 4576 appears to be unused after this.
>
> The toHexDigit methods need access to the uppercase/lowercase distinction.
> So there is no plan to change them.
> I'll amend to include the above and re-check for uncaught uses.

Hmm, I missed that unfortunate detail. It looks mis-matched to have `fromHex*` be static while `toHex*` variants aren't.

-------------

PR: https://git.openjdk.java.net/jdk/pull/3205


More information about the core-libs-dev mailing list