On Thu, 24 Mar 2022 22:01:30 GMT, Naoto Sato <na...@openjdk.org> wrote:
> Proposing to deprecate the constructors in the `java.util.Locale` class.
> There is already a factory method and a builder to return singletons, so
> there is no need to have constructors anymore unless one purposefully wants
> to create `ill-formed` Locale objects, which is discouraged. We cannot
> terminally deprecate those constructors for the compatibility to serialized
> objects created with older JDKs. Please see the draft CSR for more detail.
src/java.base/share/classes/java/util/Locale.java line 245:
> 243: * <h4>Factory Method</h4>
> 244: *
> 245: * <p>The method {@link #forLanguageTag} obtains a {@code Locale}
The factory name `forLanguageTag` is a bit off-putting, it doesn't seem like
the best name for the factory.
Yes, it already exists and does what's required but you might get better uptake
with a more natural name.
Some alternatives:
- `Locale.of("en_US")` - short and conventional
- `Locale.ofLanguage("en_US")` - 'of' prefix is used in other factories
- `Locale.forLanguage("en_US")` - natural but less conventional
-------------
PR: https://git.openjdk.java.net/jdk/pull/7947
