On Mon, 15 Sep 2025 21:18:12 GMT, Justin Lu <[email protected]> wrote:

> During parse routines, DecimalFormat uses DigitList to append digits from the 
> parsed text.
> 
> Note that `digit` is always the int value 0 through 9 (and subsequently the 
> code point 48 through 57) when passed to `append`.
> 
> Currently, `append` accepts a char which forces an int -> char -> byte 
> conversion to be stored in `digits`. This can be simplified to int -> byte if 
> the parameter type for the method is updated. The two call sites can safely 
> make this swap.
> 
> Tiers 1-3 and java.text JCK tests continue to pass with this change.

Looks ok.
Mostly this is a small change that it is not worthy of a PR. It is just churn 
in the codebase.

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

Marked as reviewed by rriggs (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27300#pullrequestreview-3242216681

Reply via email to