Re: RFR: 8332749: Broken link in MemorySegment.Scope.html

2024-05-23 Thread Iris Clark
On Thu, 23 May 2024 11:39:11 GMT, Per Minborg wrote: > This PR proposes to fix a broken link in the `MemorySegment.Scope` class. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/19366#pullrequestreview-2074232921

Re: RFR: 8330182: Start of release updates for JDK 24 [v5]

2024-05-20 Thread Iris Clark
On Mon, 20 May 2024 22:42:20 GMT, Joe Darcy wrote: >> Get JDK 24 underway. > > Joe Darcy has updated the pull request with a new target base due to a merge > or a rebase. The pull request now contains 11 commits: > > - Fix-up test. > - Merge branch 'master' into JDK-8330188 > - Adjust test

Re: RFR: 8332424: Update IANA Language Subtag Registry to Version 2024-05-16

2024-05-17 Thread Iris Clark
On Fri, 17 May 2024 18:04:02 GMT, Justin Lu wrote: > Please review this PR which is a trivial update to the IANA sub tag registry > to version 2024-05-16. Locale tests pass as expected after update. > > Associated announcement -> >

Re: RFR: 8331957: Release Note: System Property for Java SE Specification Maintenance Version

2024-05-08 Thread Iris Clark
On Thu, 9 May 2024 00:49:43 GMT, xiaotaonan wrote: > …ersion > > Release Note: System Property for Java SE Specification Maintenance Version This system property should not be set for a feature release. It should only be set if there has been a JCP Maintenance Release. Perhaps this PR

Re: RFR: 8330182: Start of release updates for JDK 24 [v3]

2024-04-30 Thread Iris Clark
On Mon, 29 Apr 2024 22:50:26 GMT, Joe Darcy wrote: >> Get JDK 24 underway. > > Joe Darcy has updated the pull request with a new target base due to a merge > or a rebase. The incremental webrev excludes the unrelated changes brought in > by the merge/rebase. The pull request contains seven

Re: RFR: 8330686: Fix typos in the DatabaseMetaData javadoc [v3]

2024-04-24 Thread Iris Clark
On Wed, 24 Apr 2024 08:54:53 GMT, Jin Kwon wrote: >> Fix typos within the `DatabaseMetaData.java`. > > Jin Kwon has updated the pull request incrementally with one additional > commit since the last revision: > > 8330686: Fix typos in the DatabaseMetaData javadoc > > Reviewed-by: jpai

Re: RFR: 8330178: Clean up non-standard use of /** comments in `java.base` [v2]

2024-04-22 Thread Iris Clark
On Fri, 19 Apr 2024 19:21:13 GMT, Jonathan Gibbons wrote: >> Please review a set of updates to clean up use of `/**` comments in the >> vicinity of declarations. >> >> There are various categories of update: >> >> * "Box comments" beginning with `/**` >> * Misplaced doc comments before

Re: RFR: 8330178: Clean up non-standard use of /** comments in `java.base`

2024-04-18 Thread Iris Clark
On Thu, 18 Apr 2024 20:44:00 GMT, Jonathan Gibbons wrote: > Please review a set of updates to clean up use of `/**` comments in the > vicinity of declarations. > > There are various categories of update: > > * "Box comments" beginning with `/**` > * Misplaced doc comments before package or

Re: RFR: 8330458: Add missing @since tag to ClassFile.JAVA_23_VERSION

2024-04-17 Thread Iris Clark
On Wed, 17 Apr 2024 20:46:26 GMT, Joe Darcy wrote: > 8330458: Add missing @since tag to ClassFile.JAVA_23_VERSION Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/18828#pullrequestreview-2007547483

Re: RFR: 8330182: Start of release updates for JDK 24 [v2]

2024-04-17 Thread Iris Clark
On Wed, 17 Apr 2024 05:43:12 GMT, Joe Darcy wrote: >> Get JDK 24 underway. > > Joe Darcy has updated the pull request incrementally with two additional > commits since the last revision: > > - Correct release date as observed in review feedback. > - Improve javadoc of class file update.

Re: RFR: 8329805: Deprecate for removal ObjectOutputStream.PutField.write

2024-04-16 Thread Iris Clark
On Tue, 16 Apr 2024 19:44:36 GMT, Roger Riggs wrote: > The method `java.io.ObjectOutputStream.PutField.write` has been deprecated > since 1.4 and should be deprecated for removal. The Deprecation annotation is > updated to indicate the intention to remov the method. Marked as reviewed by iris

Re: RFR: 8330279: Typo in `java.text.Bidi` class description

2024-04-15 Thread Iris Clark
On Mon, 15 Apr 2024 19:39:41 GMT, Naoto Sato wrote: > Fixing a typo in the javadoc Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/18788#pullrequestreview-2002041316

Re: RFR: 8330182: Start of release updates for JDK 24

2024-04-15 Thread Iris Clark
On Mon, 15 Apr 2024 19:01:08 GMT, Joe Darcy wrote: > Get JDK 24 underway. The copyright year was not updated in all files *14.java. I assume that's intentional. I've also Reviewed the associated CSRs. make/conf/version-numbers.conf line 36: > 34: DEFAULT_VERSION_EXTRA2=0 > 35:

Re: RFR: 8329787: Fix typo in CLDRConverter

2024-04-05 Thread Iris Clark
On Fri, 5 Apr 2024 17:22:36 GMT, Naoto Sato wrote: > Fix a file/class name in the `CLDRConverter` build tool, with some clean-up > for a switch statement. I like the new switch! - Marked as reviewed by iris (Reviewer). PR Review:

Re: RFR: 8328183: Minor mistakes in docs of PrintStream.append()

2024-04-03 Thread Iris Clark
On Wed, 3 Apr 2024 19:01:14 GMT, Brian Burkhalter wrote: > Clarify the behavior of `append` for a `null` `CharSequence` parameter and > clean up a couple of other typos. Marked as reviewed by iris (Reviewer). - PR Review:

Re: RFR: JDK-8329557: Fix statement around MathContext.DECIMAL128 rounding

2024-04-02 Thread Iris Clark
On Tue, 2 Apr 2024 23:43:24 GMT, Joe Darcy wrote: > Happened to notice a semantic typo in the description of > MathContext.DECIMAL128, use of "decimal64" where "decimal128" was intended, > and added some additional information to make the related descriptions more > informative. Marked as

Re: RFR: 8326627: Document Double/Float.valueOf(String) behaviour for numeric strings with non-ASCII digits

2024-03-27 Thread Iris Clark
On Wed, 27 Mar 2024 19:58:17 GMT, Naoto Sato wrote: > Clarifying the behavior for non-ASCII digits in Double/Float.valueOf(String) > method descriptions. A draft CSR has also been created. Associated CSR also Reviewed. I like the relocation of existing text to the more appropriate

Re: RFR: 8328700: Unused import and variable should be deleted in regex package

2024-03-22 Thread Iris Clark
On Fri, 22 Mar 2024 17:58:25 GMT, Raffaello Giulietti wrote: > Make use of an unused local variable probably intended to replace later casts. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/18460#pullrequestreview-1955535154

Re: RFR: 8327180: Failed: java/io/ObjectStreamClass/ObjectStreamClassCaching.java#G1 [v3]

2024-03-13 Thread Iris Clark
On Wed, 13 Mar 2024 20:10:25 GMT, Roger Riggs wrote: >> The intermittent failure of ObjectStreamClassCaching is due to an incorrect >> assumption about GC behavior and a race condition. >> >> Removed test based on incorrect assumptions about simultaneous clearing of >> WeakReferences. >>

Re: RFR: 8327180: Failed: java/io/ObjectStreamClass/ObjectStreamClassCaching.java#G1

2024-03-13 Thread Iris Clark
On Wed, 13 Mar 2024 18:01:21 GMT, Roger Riggs wrote: > The intermittent failure of ObjectStreamClassCaching is due to an incorrect > assumption about GC behavior and a race condition. > > Removed test based on incorrect assumptions about simultaneous clearing of > WeakReferences. > Added a

Re: RFR: 8327242: Document supported CLDR versions in the javadoc [v4]

2024-03-13 Thread Iris Clark
On Wed, 13 Mar 2024 16:06:26 GMT, Naoto Sato wrote: >> Adding a table that maps JDK versions and corresponding CLDR releases as an >> implNote. A draft CSR has also been created. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: >

Re: RFR: 8327242: Document supported CLDR versions in the javadoc [v2]

2024-03-12 Thread Iris Clark
On Tue, 12 Mar 2024 18:50:38 GMT, Naoto Sato wrote: >> Adding a table that maps JDK versions and corresponding CLDR releases as an >> implNote. A draft CSR has also been created. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: >

Re: RFR: 8327705: Remove mention of "applet" from java.text package description [v2]

2024-03-11 Thread Iris Clark
On Mon, 11 Mar 2024 16:32:31 GMT, Naoto Sato wrote: >> Removing left over "applet" mention in the package-info doc. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > Addressing review comments Marked as reviewed by iris

Re: RFR: JDK-8327631: Update IANA Language Subtag Registry to Version 2024-03-07

2024-03-08 Thread Iris Clark
On Thu, 7 Mar 2024 23:31:46 GMT, Justin Lu wrote: > Please review this PR which is a trivial update to the IANA sub tag registry > to version 2024-03-07. Tests pass as expected after update. > > Associated announcement -> >

Re: RFR: 8327705: Remove mention of "applet" from java.text package description

2024-03-08 Thread Iris Clark
On Fri, 8 Mar 2024 19:54:31 GMT, Naoto Sato wrote: > Removing left over "applet" mention in the package-info doc. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/18173#pullrequestreview-1925813600

Re: RFR: 8327167: Clarify the handling of Leap year by Calendar

2024-03-07 Thread Iris Clark
On Thu, 7 Mar 2024 19:28:13 GMT, Naoto Sato wrote: > A simple doc update to include a leap day example in the `Calendar` class. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/18158#pullrequestreview-1923843891

Re: RFR: 8327434: Test java/util/PluggableLocale/TimeZoneNameProviderTest.java timed out

2024-03-06 Thread Iris Clark
On Wed, 6 Mar 2024 21:32:11 GMT, Naoto Sato wrote: > Fixing timeout failures in the test case. Time zone names that are missing > (chiefly for minor locales) are now calculated at runtime after the fix to > JDK-8174269. This extra calculation time was multiplied in the test case as > it

Re: RFR: 8327208: Remove unused method java.util.jar.Manifest.make72Safe

2024-03-04 Thread Iris Clark
On Mon, 4 Mar 2024 09:55:09 GMT, Eirik Bjørsnøs wrote: > Please review this cleanup PR which suggests we remove the package-private, > unused and deprecated method `java.util.jar.Manifest.make72Safe`. > > This method was marked deprecated after a cleanup/refactoring in >

Re: RFR: 8326351: Update the Zlib version in open/src/java.base/share/legal/zlib.md to 1.3.1

2024-02-20 Thread Iris Clark
On Tue, 20 Feb 2024 19:56:53 GMT, Lance Andersen wrote: > Please review the updates to open/src/java.base/share/legal/zlib.md to update > the file from zlib 1.2.13 to zlib 1.3.1 which was missed as part of the PR > for [JDK-8324632](https://bugs.openjdk.org/browse/JDK-8324632) Marked as

Re: RFR: 8326158: Javadoc for java.time.DayOfWeek#minus(long)

2024-02-20 Thread Iris Clark
On Tue, 20 Feb 2024 18:34:59 GMT, Naoto Sato wrote: > Fixing a typo in the said method. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/17933#pullrequestreview-1891250872

Re: CFV: New Core Libraries Group Member: Raffaello Giulietti

2024-02-13 Thread Iris Clark
Vote: yes Iris

Re: RFR: 8325150: (tz) Update Timezone Data to 2024a

2024-02-07 Thread Iris Clark
On Wed, 7 Feb 2024 08:45:40 GMT, Johny Jose wrote: > Timezone data 2024a changes Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/17743#pullrequestreview-1868554271

Re: [jdk22] RFR: 8322066: Update troff manpages in JDK 22 before RC

2024-02-05 Thread Iris Clark
On Sun, 4 Feb 2024 22:43:28 GMT, David Holmes wrote: > This update drops the "ea" from the version string. > > We also propagate the following fixes from the markdown sources to the troff > manpage file: > > JDK-8322478: Update java manpage for multi-file source-code launcher > JDK-8302233:

Re: RFR: 8324998: Add test cases for String.regionMatches comparing Turkic dotted/dotless I with uppercase latin I

2024-01-30 Thread Iris Clark
On Tue, 30 Jan 2024 19:57:01 GMT, Eirik Bjørsnøs wrote: > Please review this test-only PR which improves test coverage of > `String.regionMatches` when comparing the Turkic "dotted I" and "dotless i" > characters with their latin cousins. > > The test `CompactStrings/RegionMatches.java`

Re: Integrated: 8271147: java/nio/file/Path.java javadoc typo

2024-01-30 Thread Iris Clark
On Fri, 23 Jul 2021 03:37:44 GMT, Jaikiran Pai wrote: > Can I please get a review for this change which fixes the typo noted in > https://bugs.openjdk.java.net/browse/JDK-8271147? `make docs-image` worked > fine and the generated javadoc looks fine. Marked as reviewed by iris (Reviewer).

Re: RFR: 8324632: Update Zlib Data Compression Library to Version 1.3.1

2024-01-29 Thread Iris Clark
On Mon, 29 Jan 2024 16:57:00 GMT, Lance Andersen wrote: > Hi all, > > Please review this PR which updates zlib from 1.3 to 1.3.1 in OpenJDK > > The [Zlib Data Compression Library](https://github.com/madler/zlib) has > released Zlib 1.3.1 on January 24, 2024. > > There are a [small number of

Re: RFR: 8324657: Intermittent OOME on exception message create

2024-01-24 Thread Iris Clark
On Mon, 22 Jan 2024 20:52:32 GMT, Roger Riggs wrote: > When an exception handler for an OutOfMemoryError uses string concatenation > to compose an exception message, the invoke dynamic string format > implementation may itself exhaust memory, preventing the exception from being > handled. >

Re: RFR: JDK-6503196: API doc for DecimalFormat::getMaximumIntegerDigits is unclear

2024-01-23 Thread Iris Clark
On Tue, 23 Jan 2024 18:40:42 GMT, Justin Lu wrote: > Please review this PR which clarifies some confusion for the digit getter and > setter methods of DecimalFormat. > > When formatting non `BigInteger` and `BigDecimal` values, DecimalFormat uses > 309/340 as hard limits for integer and

Re: RFR: 8324065: Daylight saving information for `Africa/Casablanca` are incorrect [v2]

2024-01-18 Thread Iris Clark
On Thu, 18 Jan 2024 21:02:26 GMT, Naoto Sato wrote: >> Fixing incorrect std/dst transitions for time zones that have rules beyond >> 2037. The year 2037 restriction seems to come from the old `zic` command >> implementation and thus can be expanded in the JDK. Arbitrary I chose 2100 >> which

Re: RFR: 8324065: Daylight saving information for `Africa/Casablanca` are incorrect

2024-01-18 Thread Iris Clark
On Thu, 18 Jan 2024 19:37:33 GMT, Naoto Sato wrote: > Fixing incorrect std/dst transitions for time zones that have rules beyond > 2037. The year 2037 restriction seems to come from the old `zic` command > implementation and thus can be expanded in the JDK. Arbitrary I chose 2100 > which is

Re: [jdk22] RFR: 8322235: Split up and improve LocaleProvidersRun

2024-01-12 Thread Iris Clark
On Fri, 12 Jan 2024 18:25:25 GMT, Justin Lu wrote: > Please review this PR which is a backport of commit > [4ea7b364](https://github.com/openjdk/jdk/commit/4ea7b36447ea96d62b1ca164c34e2b2b74a16579) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The original commit was a

Re: [jdk22] RFR: 8323571: Regression in source resolution process

2024-01-11 Thread Iris Clark
On Thu, 11 Jan 2024 22:56:38 GMT, Joe Wang wrote: > Backport to fix the regression introduced in JDK 22. Verified changes same as those in main-line. - Marked as reviewed by iris (Reviewer). PR Review: https://git.openjdk.org/jdk22/pull/63#pullrequestreview-1817021441

Re: [jdk22] RFR: 8320788: The system properties page is missing some properties

2024-01-10 Thread Iris Clark
On Wed, 10 Jan 2024 20:56:14 GMT, Naoto Sato wrote: > Backporting the document only fix to jdk22 Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk22/pull/59#pullrequestreview-1814223539

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

2024-01-09 Thread Iris Clark
On Tue, 9 Jan 2024 23:55:30 GMT, Naoto Sato wrote: > Backporting the document clarification to JDK22 Confirmed that this change corresponds to the changes in main-line and the previously approved CSR. - Marked as reviewed by iris (Reviewer). PR Review:

Re: RFR: 8320788: The system properties page is missing some properties [v2]

2024-01-09 Thread Iris Clark
On Tue, 9 Jan 2024 19:23:53 GMT, Naoto Sato wrote: >> Adding an explanation of the locale-related system properties in the >> `System.getProperties()` method. Corresponding CSR has also been drafted. > > Naoto Sato has updated the pull request incrementally with one additional > commit since

Re: [jdk22] RFR: 8321480: ISO 4217 Amendment 176 Update

2024-01-09 Thread Iris Clark
On Tue, 9 Jan 2024 19:27:12 GMT, Justin Lu wrote: > Please review this PR which is the backport of the ISO 4217 Amendment 176 > Update. Commit > [8b24851b](https://github.com/openjdk/jdk/commit/8b24851b9d3619c41c7a6cdb9193ed26a9b732dc) > from the [openjdk/jdk](https://git.openjdk.org/jdk)

Re: RFR: 8320788: The system properties page is missing some properties

2024-01-09 Thread Iris Clark
On Tue, 9 Jan 2024 00:51:28 GMT, Naoto Sato wrote: > Adding an explanation of the locale-related system properties in the > `System.getProperties()` method. Corresponding CSR has also been drafted. src/java.base/share/classes/java/lang/System.java line 818: > 816: * {@link

Re: [jdk22] RFR: 8322725: (tz) Update Timezone Data to 2023d

2024-01-09 Thread Iris Clark
On Fri, 5 Jan 2024 17:05:04 GMT, Dan Lutker wrote: > Clean backport tzdata 2023d. > `make test TEST="test/jdk/java/util/TimeZone test/jdk/java/time/test > test/jdk/sun/util/resources test/jdk/sun/text/resources > test/jdk/sun/util/calendar"` is all passing Marked as reviewed by iris

Re: RFR: 8310995: missing @since tags in 36 jdk.dynalink classes

2024-01-08 Thread Iris Clark
On Mon, 8 Jan 2024 13:24:55 GMT, Athijegannathan Sundararajan wrote: > Adding missing "@ since 9" in javadoc comment of the public classes, > interfaces and packages of the jdk.dynalink module. Marked as reviewed by iris (Reviewer). - PR Review:

Re: [jdk22] RFR: 8322214: Return value of XMLInputFactory.getProperty() changed from boolean to String in JDK 22 early access builds

2024-01-04 Thread Iris Clark
On Thu, 4 Jan 2024 19:09:34 GMT, Joe Wang wrote: > backport of PR: https://github.com/openjdk/jdk/pull/17252 Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk22/pull/32#pullrequestreview-1804849821

Re: RFR: 8322647: Short name for the `Europe/Lisbon` time zone is incorrect

2023-12-22 Thread Iris Clark
On Fri, 22 Dec 2023 18:59:20 GMT, Naoto Sato wrote: > This is a regression caused by the fix to > [JDK-8317979](https://bugs.openjdk.org/browse/JDK-8317979), where the parsing > of TZDB files was incorrect. With this fix, `TimeZoneNames_en.java` which is > generated during the build time has

Re: [jdk22] RFR: 8321514: UTF16 string gets constructed incorrectly from codepoints if CompactStrings is not enabled

2023-12-19 Thread Iris Clark
On Mon, 18 Dec 2023 11:56:50 GMT, Aleksei Voitylov wrote: > Hi all, > > This pull request contains a backport of commit > [fde5b168](https://github.com/openjdk/jdk/commit/fde5b16817c3263236993f2e8c2d2469610d99bd) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The

Re: RFR: 8320279: Link issues in java.xml module-info.java

2023-12-13 Thread Iris Clark
On Wed, 13 Dec 2023 20:54:16 GMT, Joe Wang wrote: > Doc-only change: fix incorrect links in module-info.java and StAX factories. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/17093#pullrequestreview-1780486241

Re: RFR: 8321406: Null IDs should be resolved as before catalogs are added

2023-12-05 Thread Iris Clark
On Tue, 5 Dec 2023 19:34:58 GMT, Joe Wang wrote: > Improve null handling. When both the public and system IDs are null, this > change lets the resolution process continue as usual, before the catalogs > were added. This is consistent with the process on the other part of the > program where a

Re: RFR: JDK-8319413: Start of release updates for JDK 23 [v5]

2023-12-04 Thread Iris Clark
On Tue, 5 Dec 2023 01:48:14 GMT, Joe Darcy wrote: >> Time to start making preparations for JDK 23. > > Joe Darcy has updated the pull request with a new target base due to a merge > or a rebase. The pull request now contains 15 commits: > > - Merge branch 'master' into JDK-8319413 > - Add

Re: RFR: JDK-8319413: Start of release updates for JDK 23 [v4]

2023-12-04 Thread Iris Clark
On Sat, 2 Dec 2023 20:40:57 GMT, Andrey Turbanov wrote: >> Joe Darcy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Update symbol files to JDK 22 b26. > > test/langtools/tools/javac/versions/Versions.java line 97: > >> 95:

Re: RFR: 8321131: Console read line with zero out should zero out underlying buffer in JLine [v2]

2023-12-01 Thread Iris Clark
On Fri, 1 Dec 2023 21:24:47 GMT, Naoto Sato wrote: >> This is the fix to JLine, which makes it on par with the built-in Console >> fix ([JDK-8320798](https://bugs.openjdk.org/browse/JDK-8320798)). > > Naoto Sato has updated the pull request incrementally with one additional > commit since the

Re: RFR: 8321131: Console read line with zero out should zero out underlying buffer in JLine

2023-12-01 Thread Iris Clark
On Fri, 1 Dec 2023 18:24:48 GMT, Naoto Sato wrote: > This is the fix to JLine, which makes it on par with the built-in Console fix > ([JDK-8320798](https://bugs.openjdk.org/browse/JDK-8320798)). Marked as reviewed by iris (Reviewer).

Re: RFR: JDK-8319413: Start of release updates for JDK 23 [v3]

2023-11-30 Thread Iris Clark
On Thu, 30 Nov 2023 20:48:21 GMT, Joe Darcy wrote: >> Time to start making preparations for JDK 23. > > Joe Darcy has updated the pull request with a new target base due to a merge > or a rebase. The pull request now contains eight commits: > > - Merge branch 'master' into JDK-8319413 > -

Re: RFR: JDK-8320940: Fix typo in java.lang.Double

2023-11-28 Thread Iris Clark
On Wed, 29 Nov 2023 02:00:14 GMT, Joe Darcy wrote: > Typo fix to to the new text added in JDK-8295391. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/16872#pullrequestreview-1754401831

Re: RFR: 8320918: Fix errors in the built-in Catalog implementation [v3]

2023-11-28 Thread Iris Clark
On Tue, 28 Nov 2023 21:36:20 GMT, Joe Wang wrote: >> Fix errors in the built-in Catalog implementation, specifically: >> -- incorrect case in the property name as defined in the CSR >> (https://bugs.openjdk.org/browse/JDK-8306056) >> -- tests needed to be updated accordingly >> --

Re: RFR: JDK-8319413: Start of release updates for JDK 23

2023-11-27 Thread Iris Clark
On Fri, 3 Nov 2023 23:42:03 GMT, Joe Darcy wrote: > Time to start making preparations for JDK 23. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/16505#pullrequestreview-1751551968

Re: RFR: 8320602: Lock contention in SchemaDVFactory.getInstance()

2023-11-27 Thread Iris Clark
On Mon, 27 Nov 2023 17:24:35 GMT, Joe Wang wrote: > Applying an update from the upstream source, quote: "Removing synchronized > from the getInstance() methods". As the reporter noted, this change was made > 16 years ago. > > Tests: SQE & unit tests passed Verified `synchronized` keyword

Re: RFR: 8320440: Implementation of Structured Concurrency (Second Preview)

2023-11-24 Thread Iris Clark
On Mon, 20 Nov 2023 19:48:20 GMT, Alan Bateman wrote: > This API is sitting out JDK 22, meaning no API/implementation changes in this > PR. We expect this API to do a third preview in JDK 23 with some API changes. > > So for now, we just need to bump JEP number/title that shows up in the >

Re: RFR: 8320199: Fix HTML 5 errors in java.math.BigInteger

2023-11-15 Thread Iris Clark
On Wed, 15 Nov 2023 21:59:56 GMT, Brian Burkhalter wrote: > Clean up HTML error due to nested anchor (``) elements. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/16679#pullrequestreview-1733201186

Re: RFR: 8319986: Invalid/inconsistent description and example for DateFormat [v4]

2023-11-14 Thread Iris Clark
On Tue, 14 Nov 2023 19:32:57 GMT, Naoto Sato wrote: >> Correcting the explanation of the `DateFormat.SHORT` constant. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > abbreviated/full clarification Marked as reviewed by iris

Re: RFR: 8319640: ClassicFormat::parseObject (from DateTimeFormatter) does not conform to the javadoc and may leak DateTimeException

2023-11-09 Thread Iris Clark
On Thu, 9 Nov 2023 17:44:44 GMT, Naoto Sato wrote: > Fixing the `Format::parseObject(String, ParsePosition)` implementation in > `DateTimeFormatter` class, to not throw `DateTimeException` but to return > null. Marked as reviewed by iris (Reviewer). - PR Review:

Re: RFR: 8305814: Update Xalan Java to 2.7.3

2023-11-06 Thread Iris Clark
On Mon, 6 Nov 2023 22:52:47 GMT, Joe Wang wrote: > Xalan 2.7.3: merge minor changes from the upstream project. > > Test: existing XML tests pass Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/16532#pullrequestreview-1716553613

Re: RFR: 8303920: Avoid calling out to python in DataDescriptorSignatureMissing test [v9]

2023-11-03 Thread Iris Clark
On Mon, 30 Oct 2023 17:48:48 GMT, Eirik Bjorsnos wrote: >> Please review this PR which brings the DataDescriptorSignatureMissing test >> back to life. >> >> This test currently calls out to Python to create a test vector ZIP with a >> Data Descriptor without the recommended but optional

Re: RFR: 8318466: Improve spec of NumberFormat's methods with unsupported operations

2023-11-02 Thread Iris Clark
On Wed, 1 Nov 2023 21:27:57 GMT, Justin Lu wrote: > Please review this simple change which refines the specification of some > NumberFormat methods (with unsupported operations) to separate the API and > implementation specification. Marked as reviewed by iris (Reviewer). - PR

Re: RFR: 8318487: Specification of the ListFormat.equals() method can be improved [v3]

2023-10-24 Thread Iris Clark
On Tue, 24 Oct 2023 17:58:48 GMT, Naoto Sato wrote: >> Spec clarification of `ListFormat.equals()` method, utilizing newly >> introduced `getLocale()` and `getPatterns()`. A corresponding CSR has also >> been drafted. > > Naoto Sato has updated the pull request incrementally with one

Re: RFR: 8318569: Add getter methods for Locale and Patterns in ListFormat

2023-10-20 Thread Iris Clark
On Fri, 20 Oct 2023 17:57:46 GMT, Naoto Sato wrote: > Proposing adding getter methods for `locale` and `patterns` fields in the > `ListFormat` instances. Those fields are used for `equals()`, but without the > public getter methods, users cannot tell the reasoning of the equality of two >

Re: RFR: 8318458: Update javac and java manpages with restricted method options

2023-10-18 Thread Iris Clark
On Wed, 18 Oct 2023 15:32:34 GMT, Maurizio Cimadamore wrote: > This PR updates the man pages for `javac` and `java` to document: > * the new `javac` warning for restricted methods (`-Xlint:restricted`) > * the `java` option to allow access to restricted methods without warnings >

Re: RFR: 8318322: Update IANA Language Subtag Registry to Version 2023-10-16

2023-10-17 Thread Iris Clark
On Tue, 17 Oct 2023 20:06:03 GMT, Justin Lu wrote: > This change updates the IANA subtag registry to the update released on > 2023-10-16. > > Announcement -> > https://mm.icann.org/pipermail/ietf-languages-announcements/2023-October/89.html Marked as reviewed by iris (Reviewer).

Re: RFR: 8317979: Use TZ database style abbreviations in the CLDR locale provider [v3]

2023-10-17 Thread Iris Clark
On Tue, 17 Oct 2023 16:52:12 GMT, Naoto Sato wrote: >> CLDR provides very few short names for time zones, such as PST/PDT. This >> will typically end up substituting names from the COMPAT provider. Once the >> COMPAT is removed, they will be displayed in the GMT format, i.e., >> GMT+XX:YY.

Re: RFR: 8318126: Refresh manpages

2023-10-16 Thread Iris Clark
On Mon, 16 Oct 2023 01:18:52 GMT, David Holmes wrote: > The open nroff manpage files are out of sync with their closed markdown > sources and need to be refreshed. > > The changes to apply are coming from: > > - [JDK-8288158](https://bugs.openjdk.org/browse/JDK-8288158): Add an anchor > to

Re: RFR: 8316704: Regex-free parsing of Formatter and FormatProcessor specifiers

2023-10-16 Thread Iris Clark
On Sun, 17 Sep 2023 16:01:33 GMT, Shaojin Wen wrote: > @cl4es made performance optimizations for the simple specifiers of > String.format in PR https://github.com/openjdk/jdk/pull/2830. Based on the > same idea, I continued to make improvements. I made patterns like %2d %02d > also be

Re: RFR: 8317873: Add @sealedGraph to IllegalFormatException

2023-10-11 Thread Iris Clark
On Wed, 11 Oct 2023 07:59:24 GMT, Per Minborg wrote: > This PR proposes to add @sealedGraph to `IllegalFormatException` Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/16140#pullrequestreview-1672411158

Re: RFR: 8316704: Regex-free parsing of Formatter and FormatProcessor specifiers [v16]

2023-10-09 Thread Iris Clark
On Thu, 5 Oct 2023 20:02:44 GMT, 温绍锦 wrote: >> @cl4es made performance optimizations for the simple specifiers of >> String.format in PR https://github.com/openjdk/jdk/pull/2830. Based on the >> same idea, I continued to make improvements. I made patterns like %2d %02d >> also be optimized.

Re: RFR: 8317471: ListFormat::parseObject() spec can be improved on parsePosition valid values

2023-10-05 Thread Iris Clark
On Thu, 5 Oct 2023 17:57:00 GMT, Naoto Sato wrote: > Adding IOOBE clause to clarify the behavior on an invalid `parsePos` argument > on calling `ListFormat::parseObject()`. A corresponding CSR has also been > drafted. Documents existing behaviour. Associated CSR also Reviewed. -

Re: RFR: 8317126: Redundant entries in Windows `tzmappings` file

2023-09-28 Thread Iris Clark
On Thu, 28 Sep 2023 17:37:00 GMT, Naoto Sato wrote: > Removing redundant entries in `lib/tzmappings` file on Windows. The file maps > Windows time zones to Java time zones according to the region. Since `001` > means world, no region-specific entries are needed if those time zones are > the

Re: RFR: 8317119: Unused imports in the java.util.stream package [v2]

2023-09-27 Thread Iris Clark
On Wed, 27 Sep 2023 22:44:52 GMT, Mourad Abbay wrote: >> Remove unused imports in the java.util.stream package. > > Mourad Abbay has updated the pull request incrementally with one additional > commit since the last revision: > > Update copyright year. Marked as reviewed by iris (Reviewer).

Re: RFR: 8316970: Add internal annotation to mark restricted methods [v2]

2023-09-27 Thread Iris Clark
On Wed, 27 Sep 2023 17:29:29 GMT, Maurizio Cimadamore wrote: >> This patch adds a new internal annotation that is used to mark all >> restricted me >> thods in the FFM API. The new annotation is similar to the one we used for >> preview API methods. >> >> We plan to use the new annotation

Re: RFR: 8316038: Fix doc typos in java.io.Console and java.util.Scanner

2023-09-11 Thread Iris Clark
On Mon, 11 Sep 2023 17:58:20 GMT, Pavel Rappo wrote: > Please review this trivial PR. Marked as reviewed by iris (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/15667#pullrequestreview-1620730522

Re: RFR: 8315938: Deprecate for removal Unsafe methods that have standard APIs for many releases

2023-09-08 Thread Iris Clark
On Fri, 8 Sep 2023 15:54:05 GMT, Alan Bateman wrote: > There are several methods defined by sun.misc.Unsafe that have standard API > equivalents for many years and releases. The change proposed here is to > deprecate, for removal, the park, unpark, getLoadAverage, loadFence, > storeFence, and

Re: RFR: 8267174: Many test files have the wrong Copyright header

2023-09-06 Thread Iris Clark
On Tue, 5 Sep 2023 22:49:41 GMT, Erik Joelsson wrote: > There are a number of files in the `test` directory that have an incorrect > copyright header, which includes the "classpath" exception text. This patch > removes that text from all test files that I could find it in. I did this > using

Re: RFR: 8314483: Optionally override copyright header in generated source [v2]

2023-08-24 Thread Iris Clark
On Thu, 24 Aug 2023 17:36:46 GMT, Erik Joelsson wrote: >> In the JDK build we have various build tools that generate source code from >> data files. For most of these tools, the source files are based on template >> files, which already have copyright headers, but for some, the complete >>

Re: RFR: 8314495: Update to use jtreg 7.3.1

2023-08-18 Thread Iris Clark
On Thu, 17 Aug 2023 07:24:14 GMT, Christian Stein wrote: > Please review the change to update to using jtreg 7.3,1. > > The primary change is to the `jib-profiles.js` file, which specifies the > version of jtreg to use, for those systems that rely on this file. In > addition, the

Re: RFR: 8313949: Missing word in GPLv2 license text in StackMapTableAttribute.java

2023-08-14 Thread Iris Clark
On Mon, 14 Aug 2023 12:47:46 GMT, Dmitry Cherepanov wrote: > The PR suggests updating copyright headers in two files so that they are the > same as in other files. The updated license appears to match the expected license:

Re: RFR: 8313702: Update IANA Language Subtag Registry to Version 2023-08-02

2023-08-03 Thread Iris Clark
On Thu, 3 Aug 2023 16:31:01 GMT, Justin Lu wrote: > Please review this PR which updates the IANA data from version 05-11-2023 to > version > [08-02-2023](https://mm.icann.org/pipermail/ietf-languages-announcements/2023-August/88.html). > > The contents of the update added variant subtag

Re: RFR: 8311006: missing @since info in jdk.xml.dom

2023-08-02 Thread Iris Clark
On Wed, 2 Aug 2023 22:26:37 GMT, Joe Wang wrote: > Please review a patch that add `@since` tags following existing convention > for XML APIs in java.xml and jdk.xml.dom modules. Marked as reviewed by iris (Reviewer). - PR Review:

Re: [jdk21] RFR: 8312572: JDK 21 RDP2 L10n resource files update

2023-08-02 Thread Iris Clark
On Wed, 2 Aug 2023 17:03:40 GMT, Justin Lu wrote: > Please review this PR (L10N translation update), which is a backport of > commit > [9b55e9a7](https://github.com/openjdk/jdk/commit/9b55e9a706de9893b1a71c7a6a4e23c4b8842f18) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. >

Re: RFR: 8310991: missing @since tags in java.xml

2023-08-01 Thread Iris Clark
On Tue, 1 Aug 2023 19:32:40 GMT, Joe Wang wrote: > Please review a patch that adds the @since tag to DOM package. Added @since tags follow existing convention for XML APIs in java.xml and jdk.xml.dom modules. - Marked as reviewed by iris (Reviewer). PR Review:

Re: [jdk21] RFR: 8039165: [Doc] MessageFormat null locale generates NullPointerException

2023-07-31 Thread Iris Clark
On Mon, 31 Jul 2023 21:13:16 GMT, Justin Lu wrote: > Please review this change which contains a backport of commit > [c6396dce](https://github.com/openjdk/jdk/commit/c6396dceb9a64578d5b335af27ad1d968190a1fa) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > This PR is a

Re: CFV: New Core Libraries Group Member: Joe Wang

2023-07-31 Thread Iris Clark
Vote: yes Thanks, Iris

Re: CFV: New Core Libraries Group Member: Brent Christian

2023-07-31 Thread Iris Clark
Vote: yes Thanks, Iris

Re: CFV: New Core Libraries Group Member: Brian Burkhalter

2023-07-31 Thread Iris Clark
Vote: yes Thanks, Iris

Re: [jdk21] RFR: 8300937: Update nroff pages in JDK 21 before RC

2023-07-31 Thread Iris Clark
On Mon, 31 Jul 2023 08:33:07 GMT, David Holmes wrote: > Main changes are to use 21 instead of 21-ea. In addition the following files > contain additional updates from the closed sources: > > - src/java.base/share/man/java.1 > > [JDK-8273131](https://bugs.openjdk.org/browse/JDK-8273131):

Re: [jdk21] RFR: 6960866: [Fmt-Ch] ChoiceFormat claims impossible and unimplemented functionality

2023-07-13 Thread Iris Clark
On Thu, 13 Jul 2023 18:28:38 GMT, Justin Lu wrote: > Please review this PR which backports > https://github.com/openjdk/jdk/commit/6cb9ec32a68634f0b6f7461c9dc7ef0c0e59861d > > Thank you Marked as reviewed by iris (Reviewer). - PR Review:

Re: [jdk21] RFR: 8310892: ScopedValue throwing StructureViolationException should be clearer

2023-07-11 Thread Iris Clark
On Tue, 11 Jul 2023 17:21:13 GMT, Alan Bateman wrote: > Clean backport of https://git.openjdk.org/jdk/pull/14679 Same as 22. - Marked as reviewed by iris (Reviewer). PR Review: https://git.openjdk.org/jdk21/pull/113#pullrequestreview-1524839981

  1   2   3   >