Re: RFR: 8282319: java.util.Locale method to stream available Locales [v3]

2023-02-24 Thread Roger Riggs
On Fri, 24 Feb 2023 21:57:41 GMT, Justin Lu wrote: >> OK but then we should be consistent and update all of the methods to use the >> new tag or revert the change for consistency > > Makes sense, will look at both Locale and LocaleServiceProviderPool and > update the methods that are applicabl

Re: RFR: 8282319: java.util.Locale method to stream available Locales [v3]

2023-02-24 Thread Justin Lu
On Fri, 24 Feb 2023 18:58:45 GMT, Lance Andersen wrote: >> From what I saw in other methods, `@return ...` goes on the bottom (like the >> line I removed), but the javadoc 16 tag `{@return ... }` goes on the top. > > OK but then we should be consistent and update all of the methods to use the >

Re: RFR: 8282319: java.util.Locale method to stream available Locales [v3]

2023-02-24 Thread Lance Andersen
On Fri, 24 Feb 2023 18:39:55 GMT, Justin Lu wrote: >> src/java.base/share/classes/sun/util/locale/provider/LocaleServiceProviderPool.java >> line 155: >> >>> 153: * that are provided by each provider, including the JRE. >>> 154: * >>> 155: * @return a stream of the available loca

Re: RFR: 8282319: java.util.Locale method to stream available Locales [v3]

2023-02-24 Thread Justin Lu
> This PR adds a new method to java.util.Locale which returns Stream > > The contents of the Stream are composed of the same underlying elements as > Locale.getAvailableLocales(). > > This method allows streaming of the Locale array without creating a defensive > copy. Justin Lu has updated th