<i18n dev> RFR: JDK-8282625 Formatter caches Locale/DecimalFormatSymbols poorly [v7]

[Jim Laskey]

On Mon, 7 Mar 2022 21:24:32 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Jim Laskey has updated the pull request incrementally with four additional commits since the last revision:
>> 
>>  - Add comment about DecimalFormatSymbols being mutable objects.
>>  - Revert "Cache DecimalFormatSymbols in DecimalFormatSymbols instead of Formatter. No significant performance degradation."
>>    
>>    This reverts commit fcbf66a2fe9641d3c3349f12cc7b32d8b84c6f72.
>>  - Revert "Drop DecimalFormatSymbols.getLocale change"
>>    
>>    This reverts commit b93cdb03ec68f24f4b8851c0966bb144c30b5110.
>>  - Revert "Correct caching test"
>>    
>>    This reverts commit bf7975396aaad4ed58d053bde8f54984215eeba5.
>
> src/java.base/share/classes/java/util/Formatter.java line 4550:
> 
>> 4548: 
>> 4549:                 if (l == null || l.equals(Locale.US)) {
>> 4550:                      grpSize = 3;
> 
> unintentional indentation

Noted

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

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