Re: RFR: 8284780: Need methods to create pre-sized HashSet and LinkedHashSet [v17]

2022-07-07 Thread Stuart Marks
On Wed, 1 Jun 2022 16:45:35 GMT, liach wrote: >> XenoAmess has updated the pull request incrementally with one additional >> commit since the last revision: >> >> do it as naotoj said > > 'the new' fix should be applied to newHashMap etc. too. @liach > 'the new' fix should be applied to ne

Re: RFR: 8291660: Grapheme support in BreakIterator

2022-08-24 Thread Stuart Marks
On Tue, 23 Aug 2022 22:44:13 GMT, Naoto Sato wrote: > This is to enhance the character break analysis in `java.text.BreakIterator` > to conform to the extended grapheme cluster boundaries defined in > https://www.unicode.org/reports/tr29/#Grapheme_Cluster_Boundaries. A > corresponding CSR has

Re: RFR: 8291660: Grapheme support in BreakIterator

2022-08-24 Thread Stuart Marks
On Tue, 23 Aug 2022 22:44:13 GMT, Naoto Sato wrote: > This is to enhance the character break analysis in `java.text.BreakIterator` > to conform to the extended grapheme cluster boundaries defined in > https://www.unicode.org/reports/tr29/#Grapheme_Cluster_Boundaries. A > corresponding CSR has

Re: RFR: 8291660: Grapheme support in BreakIterator

2022-08-24 Thread Stuart Marks
On Tue, 23 Aug 2022 22:44:13 GMT, Naoto Sato wrote: > This is to enhance the character break analysis in `java.text.BreakIterator` > to conform to the extended grapheme cluster boundaries defined in > https://www.unicode.org/reports/tr29/#Grapheme_Cluster_Boundaries. A > corresponding CSR has

Re: RFR: 8291660: Grapheme support in BreakIterator [v3]

2022-08-26 Thread Stuart Marks
On Fri, 26 Aug 2022 18:12:04 GMT, Naoto Sato wrote: >> This is to enhance the character break analysis in `java.text.BreakIterator` >> to conform to the extended grapheme cluster boundaries defined in >> https://www.unicode.org/reports/tr29/#Grapheme_Cluster_Boundaries. A >> corresponding CSR

Re: RFR: 8291660: Grapheme support in BreakIterator [v4]

2022-09-07 Thread Stuart Marks
On Fri, 26 Aug 2022 21:48:14 GMT, Naoto Sato wrote: >> This is to enhance the character break analysis in `java.text.BreakIterator` >> to conform to the extended grapheme cluster boundaries defined in >> https://www.unicode.org/reports/tr29/#Grapheme_Cluster_Boundaries. A >> corresponding CSR

Re: RFR: 8291660: Grapheme support in BreakIterator [v6]

2022-09-09 Thread Stuart Marks
On Thu, 8 Sep 2022 18:32:56 GMT, Naoto Sato wrote: >> This is to enhance the character break analysis in `java.text.BreakIterator` >> to conform to the extended grapheme cluster boundaries defined in >> https://www.unicode.org/reports/tr29/#Grapheme_Cluster_Boundaries. A >> corresponding CSR h

Re: RFR: 8294008: Grapheme implementation of setText() throws IndexOutOfBoundsException [v2]

2022-09-19 Thread Stuart Marks
On Mon, 19 Sep 2022 22:18:42 GMT, Naoto Sato wrote: >> Yeah, I saw there's a mismatch between the src and limit in this call that >> led to the index check Exception: >> 286 for (int b = ci.getBeginIndex(); b < end;) { >> 287 boundaries.add(b); >> 288

Re: RFR: 8294008: Grapheme implementation of setText() throws IndexOutOfBoundsException [v3]

2022-09-20 Thread Stuart Marks
On Tue, 20 Sep 2022 01:14:38 GMT, Naoto Sato wrote: >> Fixing JCK failures caused by the new grapheme implementation. The length of >> a CharSequence should return the `endIndex`, not `endIndex - beginIndex`. > > Naoto Sato has updated the pull request incrementally with one additional > commit

Re: RFR: 8294008: Grapheme implementation of setText() throws IndexOutOfBoundsException [v3]

2022-09-20 Thread Stuart Marks
On Tue, 20 Sep 2022 01:09:29 GMT, Naoto Sato wrote: >> OK yeah this is really confusing. One might ask a similar question in >> `charAt` about why it doesn't call >> >> src.setIndex(index + src.getBeginIndex()); >> >> The answer is that this is a special-purpose `CharSequence` that represe

Re: RFR: JDK-8285932 Implementation of JEP-430 String Templates (Preview) [v7]

2022-10-31 Thread Stuart Marks
On Mon, 31 Oct 2022 20:11:34 GMT, Jim Laskey wrote: >> Enhance the Java programming language with string templates, which are >> similar to string literals but contain embedded expressions. A string >> template is interpreted at run time by replacing each expression with the >> result of evalu

Re: RFR: JDK-8300133: Use generalized see and link tags in core libs [v2]

2023-01-18 Thread Stuart Marks
On Fri, 13 Jan 2023 22:54:47 GMT, Joe Darcy wrote: >> With generalized see and link tags that can refer to anchors (JDK-8200337), >> the see and link tags in core libraries should be updated to use this >> feature when possible. This PR covers such updates for java.base. > > Joe Darcy has updat

Re: RFR: 8289220: Locale.forLanguageTag throws NPE due to soft ref used in locale cache being cleared [v3]

2023-06-06 Thread Stuart Marks
On Tue, 6 Jun 2023 18:48:35 GMT, John R Rose wrote: >> This reasoning seems invalid. There are method calls in there, and you rely >> on inlining heuristics for this not to break. Please use reachabilityFence >> instead. > > It appears you are assuming that some combination of bytecodes constit

Re: RFR: 8311188: Simplify and modernize equals and hashCode in java.text [v4]

2023-07-06 Thread Stuart Marks
On Tue, 4 Jul 2023 22:03:58 GMT, John R Rose wrote: >>> Hmm, I think that issue refers to code that have explicit non-Object >>> parameter types (like `X::equals(Object)boolean` in the issue's sample). >>> This method already have both arguments as `Object`, so I don't think >>> there's any ty

Re: RFR: 8320919: Clarify Locale related system properties [v3]

2023-12-12 Thread Stuart Marks
On Mon, 11 Dec 2023 23:17:01 GMT, Naoto Sato wrote: >> src/java.base/share/classes/java/util/Locale.java line 301: >> >>> 299: * is unparsable, it is ignored. The overriding values of other >>> properties are not >>> 300: * checked for syntax or validity and are used directly in the default

Re: RFR: 8320919: Clarify Locale related system properties [v3]

2023-12-12 Thread Stuart Marks
On Mon, 11 Dec 2023 23:20:25 GMT, Naoto Sato wrote: >> This is a doc change to clarify what the `Default Locale` is, and how it is >> established during the system startup using the system properties. Those >> locale-related system properties have existed since the early days of Java, >> but h

Re: RFR: JDK-8325189: Enable this-escape javac warning in java.base

2024-02-05 Thread Stuart Marks
On Fri, 2 Feb 2024 23:36:41 GMT, Joe Darcy wrote: > After the "this-escape" lint warning was added to javac (JDK-8015831), the > base module was not updated to be able to compile with this warning enabled. > This PR makes the necessary changes to allow the base module to build with > the warni

Re: RFR: 8344252: SM cleanup in java.util classes

2024-11-14 Thread Stuart Marks
On Thu, 14 Nov 2024 22:09:59 GMT, Roger Riggs wrote: > Remove use of doPrivileged and SecurityManager in java.util. Marked as reviewed by smarks (Reviewer). I mainly wanted to look at Arrays, and I looked briefly at the other stuff too. All looks good. - PR Review: https://git.op