RFR: 8366224: Introduce DecimalDigits.appendPair for efficient two-digit formatting and refactor DateTimeHelper [v5]

Per Minborg pminborg at openjdk.org
Mon Sep 15 09:05:24 UTC 2025


On Mon, 15 Sep 2025 06:45:25 GMT, Per Minborg <pminborg at openjdk.org> wrote:

>> Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update test/jdk/jdk/internal/util/DecimalDigitsTest.java
>>   
>>   Co-authored-by: Chen Liang <liach at openjdk.org>
>
> We have, for long, established that changes like this should be discussed before putting forward any code. Yet, I am unable to find any discussion in the appropriate mailing list. Please send me the link to the mailing list discussion. I must have missed it for some reason.

> @minborg
> 
> The process went like this:
> 
> On August 23rd, I posted an email to the core-libs-dev mailing list titled "Introduce and utilize DecimalDigits.appendPair for efficient two-digit formatting." The message included a link to the pull request (PR) #26911. At the time, the PR was in draft status.
> 
> On August 26th, @jodastephen commented on the PR.
> 
> He's an expert in java.time, and based on his comments, I created an issue and changed the PR status to RFR.

This mail appears to merely point out that there is a PR with code. I am unable to locate any discussion *prior* to any code being presented. E.g., whether we should do this or not. What are the pros and cons, etc. What did I miss?

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

PR Comment: https://git.openjdk.org/jdk/pull/26911#issuecomment-3291135881


More information about the security-dev mailing list