RFR: 8315968: Consolidate java.util.Digits and StringLatin1::PACKED_DIGITS [v10]

Roger Riggs rriggs at openjdk.org
Mon Sep 11 19:03:42 UTC 2023


On Mon, 11 Sep 2023 15:57:22 GMT, 温绍锦 <duke at openjdk.org> wrote:

>> Some codes in core libs are duplicated, including:
>> java.util.DecimalDigits::DIGITS -> java.lang.StringLatin1.PACKED_DIGITS
>> java.util.DecimalDigits::size -> java.lang.Long.stringSize
>> 
>> We can reduce duplication through JavaLangAccess, which is also needed in other places, such as:
>> https://github.com/openjdk/jdk/pull/15555
>
> 温绍锦 has updated the pull request incrementally with one additional commit since the last revision:
> 
>   revert code format

Changes requested by rriggs (Reviewer).

src/java.base/share/classes/jdk/internal/util/DecimalDigits.java line 1:

> 1: /*

Can git be convinced to show this as a rename instead of a delete and add?
The history will be cleaner.

src/java.base/share/classes/jdk/internal/util/OctalDigits.java line 41:

> 39:      * Singleton instance of OctalDigits.
> 40:      */
> 41:     public static final Digits INSTANCE = new OctalDigits();

The constructor should be after the field definitions. 
Don't let some tool reformat the code and move things around.

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

PR Review: https://git.openjdk.org/jdk/pull/15651#pullrequestreview-1620644060
PR Review Comment: https://git.openjdk.org/jdk/pull/15651#discussion_r1321958613
PR Review Comment: https://git.openjdk.org/jdk/pull/15651#discussion_r1321960820



More information about the security-dev mailing list