Re: RFR: 8355215: Add @spec tags to Emoji related methods

2025-04-21 Thread Joe Wang
On Mon, 21 Apr 2025 20:11:47 GMT, Naoto Sato wrote: > Adding @spec tags to Emoji related methods in the Character class. A CSR has > also been drafted. Marked as reviewed by joehw (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/24779#pullrequestreview-2782236067

Integrated: 8354774: DocumentBuilderFactory getAttribute throws NPE

2025-04-21 Thread Joe Wang
On Thu, 17 Apr 2025 17:41:56 GMT, Joe Wang wrote: > Fix a NPE on calling DocumentBuilderFactory::getAttribute, refer to the bug > report. > > Also in this patch: consolidates get and set properties to use the same Util > methods to reduce potential errors when code cha

Re: RFR: 8354774: DocumentBuilderFactory getAttribute throws NPE [v2]

2025-04-21 Thread Joe Wang
On Mon, 21 Apr 2025 08:11:33 GMT, Andrey Turbanov wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> remove unused variable pName; remove unused imports > > src/java.xml/share/classes/jdk

Re: RFR: 8354774: DocumentBuilderFactory getAttribute throws NPE [v3]

2025-04-21 Thread Joe Wang
> Fix a NPE on calling DocumentBuilderFactory::getAttribute, refer to the bug > report. > > Also in this patch: consolidates get and set properties to use the same Util > methods to reduce potential errors when code changes. > > Test: > Tier1 - 3 passed > JC

Re: RFR: 8354774: DocumentBuilderFactory getAttribute throws NPE [v2]

2025-04-19 Thread Joe Wang
On Sun, 20 Apr 2025 03:05:20 GMT, SendaoYan wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> remove unused variable pName; remove unused imports > > test/jaxp/javax/xml/jaxp/unittest/comm

Re: RFR: 8259540: MissingResourceException for key cvc-complex-type.2.4.d.1 [v3]

2025-04-18 Thread Joe Wang
On Fri, 18 Apr 2025 22:15:10 GMT, Alexey Bakhtin wrote: >> Please find a trivial fix for the missing `cvc-complex-type.2.4.d.1` >> resource in the main XMLSchemaMessages.properties file. > > Alexey Bakhtin has updated the pull request incrementally with one additional > commit since the last re

Re: RFR: 8259540: MissingResourceException for key cvc-complex-type.2.4.d.1

2025-04-18 Thread Joe Wang
On Fri, 18 Apr 2025 17:40:10 GMT, Joe Wang wrote: >> Added missed localized strings for _de, _ja and _zh_CN localities based on >> the JDK11 localizations > >> Added missed localized strings for _de, _ja and _zh_CN localities based on >> the JDK11 localizations >

Re: RFR: 8354774: DocumentBuilderFactory getAttribute throws NPE [v2]

2025-04-18 Thread Joe Wang
On Fri, 18 Apr 2025 20:06:01 GMT, Naoto Sato wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> remove unused variable pName; remove unused imports > > src/java.xml/share/classes/com/sun/o

Re: RFR: 8354774: DocumentBuilderFactory getAttribute throws NPE [v2]

2025-04-18 Thread Joe Wang
> Fix a NPE on calling DocumentBuilderFactory::getAttribute, refer to the bug > report. > > Also in this patch: consolidates get and set properties to use the same Util > methods to reduce potential errors when code changes. > > Test: > Tier1 - 3 passed > JC

Re: RFR: 8259540: MissingResourceException for key cvc-complex-type.2.4.d.1

2025-04-18 Thread Joe Wang
On Fri, 18 Apr 2025 17:09:40 GMT, Alexey Bakhtin wrote: > Added missed localized strings for _de, _ja and _zh_CN localities based on > the JDK11 localizations Probably not necessary as they will be covered by L10n resource update. - PR Comment: https://git.openjdk.org/jdk/pull/247

Re: RFR: 8259540: MissingResourceException for key cvc-complex-type.2.4.d.1

2025-04-18 Thread Joe Wang
On Fri, 18 Apr 2025 04:08:56 GMT, Alexey Bakhtin wrote: > Please find a trivial fix for the missing `cvc-complex-type.2.4.d.1` resource > in the main XMLSchemaMessages.properties file. @alexeybakhtin also, I assigned the issue to you. Thanks. - PR Comment: https://git.openjdk.org/

Re: RFR: 8259540: MissingResourceException for key cvc-complex-type.2.4.d.1

2025-04-17 Thread Joe Wang
On Fri, 18 Apr 2025 04:08:56 GMT, Alexey Bakhtin wrote: > Please find a trivial fix for the missing `cvc-complex-type.2.4.d.1` resource > in the main XMLSchemaMessages.properties file. LGTM. Added a link to JDK-8355006. - Marked as reviewed by joehw (Reviewer). PR Review: https:/

RFR: 8354774: DocumentBuilderFactory getAttribute throws NPE

2025-04-17 Thread Joe Wang
Fix a NPE on calling DocumentBuilderFactory::getAttribute. This issue was found during the previous JCK test (JCK-7322355). The JCK failure was from a different method call, but it appears this is the root cause. Also in this patch: consolidates get and set properties to use the same Util metho

Re: RFR: 8354084: Streamline XPath API's extension function control [v4]

2025-04-16 Thread Joe Wang
On Wed, 16 Apr 2025 13:47:28 GMT, Roger Riggs wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> update javadoc for the jdk.xml.enableExtensionFunctions property > > src/java.xml/share/cla

Re: RFR: 8354084: Streamline XPath API's extension function control [v5]

2025-04-16 Thread Joe Wang
. There were unrelated/known > failures (e.g. bug8329757). > JCK passed except the previously reported issue (JCK-7322355). Joe Wang 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/reba

Re: RFR: 8354084: Streamline XPath API's extension function control [v4]

2025-04-16 Thread Joe Wang
. There were unrelated/known > failures (e.g. bug8329757). > JCK passed except the previously reported issue (JCK-7322355). Joe Wang has updated the pull request incrementally with one additional commit since the last revision: update javadoc for the jdk.xml.enableExtensionFunctions propert

Re: RFR: 8354084: Streamline XPath API's extension function control [v3]

2025-04-15 Thread Joe Wang
. There were unrelated/known > failures (e.g. bug8329757). > JCK passed except the previously reported issue (JCK-7322355). Joe Wang has updated the pull request incrementally with one additional commit since the last revision: rewind resource files, leave them to L10n resource files update --

Re: RFR: 8346230: [perf] scalability issue for the specjvm2008::xml.transform workload [v6]

2025-04-14 Thread Joe Wang
On Mon, 14 Apr 2025 15:22:21 GMT, Vladimir Ivanov wrote: >> The HashMap for caching was deleted. Now it use only ThreadLocal variable >> without synchronization. >> According to the specjvm2008::xml.transform workload the performance for low >> threads counts was not affected and improved for h

Re: RFR: 8354084: Streamline XPath API's extension function control [v2]

2025-04-11 Thread Joe Wang
. There were unrelated/known > failures (e.g. bug8329757). > JCK passed except the previously reported issue (JCK-7322355). Joe Wang has updated the pull request incrementally with one additional commit since the last revision: add copyright and docs - Changes: - a

RFR: 8354084: Streamline XPath API's extension function control

2025-04-11 Thread Joe Wang
Streamline the extension function control in the XPath API by relying solely on [XPathFunctionResolver](https://docs.oracle.com/en/java/javase/24/docs/api/java.xml/javax/xml/xpath/XPathFunctionResolver.html), which provides a secure and developer-defined mechanism. Remove the feature FEATURE_SEC

Re: RFR: 8353234: Refactor XMLSecurityPropertyManager

2025-04-04 Thread Joe Wang
On Mon, 31 Mar 2025 19:18:58 GMT, Joe Wang wrote: >> LGTM. Technically it is still March in PDT though > >> LGTM. Technically it is still March in PDT though > > Thanks. It is, tricky. Thought by the time it's out, it would be April. > @JoeWang-Java, > > Can

Integrated: 8353234: Refactor XMLSecurityPropertyManager

2025-04-04 Thread Joe Wang
On Mon, 31 Mar 2025 17:42:20 GMT, Joe Wang wrote: > Refactor XMLSecurityPropertyManager. > The Xerces and Xalan components each had its own XMLSecurityManager and > XMLSecurityPropertyManager. In a previous fix, the former had been unified as > part of that changeset. This chang

Re: RFR: 8353234: Refactor XMLSecurityPropertyManager

2025-04-02 Thread Joe Wang
On Mon, 31 Mar 2025 17:42:20 GMT, Joe Wang wrote: > Refactor XMLSecurityPropertyManager. > The Xerces and Xalan components each had its own XMLSecurityManager and > XMLSecurityPropertyManager. In a previous fix, the former had been unified as > part of that changeset. This chang

Re: RFR: 8353234: Refactor XMLSecurityPropertyManager

2025-04-02 Thread Joe Wang
On Mon, 31 Mar 2025 17:42:20 GMT, Joe Wang wrote: > Refactor XMLSecurityPropertyManager. > The Xerces and Xalan components each had its own XMLSecurityManager and > XMLSecurityPropertyManager. In a previous fix, the former had been unified as > part of that changeset. This chang

Re: RFR: 8353234: Refactor XMLSecurityPropertyManager

2025-04-01 Thread Joe Wang
On Mon, 31 Mar 2025 17:42:20 GMT, Joe Wang wrote: > Refactor XMLSecurityPropertyManager. > The Xerces and Xalan components each had its own XMLSecurityManager and > XMLSecurityPropertyManager. In a previous fix, the former had been unified as > part of that changeset. This chang

RFR: 8353234: Refactor XMLSecurityPropertyManager

2025-03-31 Thread Joe Wang
Refactor XMLSecurityPropertyManager. The Xerces and Xalan components each had its own XMLSecurityManager and XMLSecurityPropertyManager. In a previous fix, the former had been unified as part of that changeset. This change will allow future changes to handle XMLSecurityManager and XMLSecurityPro

Re: RFR: 8353234: Refactor XMLSecurityPropertyManager

2025-03-31 Thread Joe Wang
On Mon, 31 Mar 2025 19:11:03 GMT, Naoto Sato wrote: > LGTM. Technically it is still March in PDT though Thanks. It is, tricky. Thought by the time it's out, it would be April. - PR Comment: https://git.openjdk.org/jdk/pull/24334#issuecomment-2767177971

Re: RFR: 8346230: [perf] scalability issue for the specjvm2008::xml.transform workload [v4]

2025-03-25 Thread Joe Wang
On Mon, 20 Jan 2025 13:51:22 GMT, Alan Bateman wrote: >> Vladimir Ivanov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8346230 [perf] scalability issue for the specjvm2008::xml.transform >> workload > > src/java.xml/share/classes/com/

Re: RFR: 8352716: (tz) Update Timezone Data to 2025b

2025-03-25 Thread Joe Wang
On Tue, 25 Mar 2025 16:52:28 GMT, Naoto Sato wrote: > Incorporating the latest IANA Time Zone Database (2025b). Manually confirmed > the newly introduced time zone stays at the same offset (-03) on/after > 2025-04-06: > > jshell> > ZoneId.of("America/Coyhaique").getRules().getValidOffsets(Loc

Re: RFR: 8346230: [perf] scalability issue for the specjvm2008::xml.transform workload [v4]

2025-03-25 Thread Joe Wang
On Fri, 14 Mar 2025 01:28:34 GMT, Vladimir Ivanov wrote: >> The HashMap for caching was deleted. Now it use only ThreadLocal variable >> without synchronization. >> According to the specjvm2008::xml.transform workload the performance for low >> threads counts was not affected and improved for h

Re: RFR: 8352628: Refine Grapheme test

2025-03-21 Thread Joe Wang
On Fri, 21 Mar 2025 20:17:31 GMT, Naoto Sato wrote: > Refining regex tests for Grapheme break so that every test case will run even > with some failing ones. Marked as reviewed by joehw (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/24168#pullrequestreview-2707275607

Re: RFR: 8346948: Update CLDR to Version 47.0 [v2]

2025-03-20 Thread Joe Wang
On Wed, 19 Mar 2025 17:48:20 GMT, Naoto Sato wrote: >> Upgrading the CLDR to version 47.0. A corresponding CSR has also been >> drafted. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > Copyright year/bugid update Marked as

Integrated: 8351969: Add Public Identifiers to the JDK built-in Catalog

2025-03-19 Thread Joe Wang
On Thu, 13 Mar 2025 19:01:03 GMT, Joe Wang wrote: > Add public identifiers to the JDK built-in Catalog; Replace the incorrect > Schema 1.1 DTD files (note the Public Identifier at line 2) with the correct > Schema 1.0 DTDs. The JDK built-in Catalog contains Schema 1.0 files only.

Re: RFR: 8351969: Add Public Identifiers to the JDK built-in Catalog [v3]

2025-03-19 Thread Joe Wang
On Wed, 19 Mar 2025 00:43:28 GMT, Joe Wang wrote: >> Add public identifiers to the JDK built-in Catalog; Replace the incorrect >> Schema 1.1 DTD files (note the Public Identifier at line 2) with the correct >> Schema 1.0 DTDs. The JDK built-in Catalog contains Schema 1.0 file

Re: RFR: 8351969: Add Public Identifiers to the JDK built-in Catalog [v3]

2025-03-18 Thread Joe Wang
LSchema.dtd and datatypes.dtd, were > removed and replaced with those of Schema 1.0. These files were downloaded > from w3.org without modification. The file names were unfortunately the same, > which (with the Diffs) made it look like they were modified though they were > not. Joe Wang h

Re: RFR: 8351969: Add Public Identifiers to the JDK built-in Catalog [v2]

2025-03-18 Thread Joe Wang
On Tue, 18 Mar 2025 19:05:32 GMT, Roger Riggs wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> update the test: change variables and etc. > > test/jaxp/javax/xml/jaxp/unittest/common/jdkca

Re: RFR: 8351969: Add Public Identifiers to the JDK built-in Catalog [v2]

2025-03-18 Thread Joe Wang
On Tue, 18 Mar 2025 18:26:09 GMT, Roger Riggs wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> update the test: change variables and etc. > > src/java.xml/share/classes/jdk/xml/intern

Re: RFR: 8351969: Add Public Identifiers to the JDK built-in Catalog [v2]

2025-03-18 Thread Joe Wang
On Tue, 18 Mar 2025 18:50:24 GMT, Roger Riggs wrote: > The XMLSchema.dtd changes the referenced version from the 2009 version to the > 2001 version. Is that intentional. Yes. Note the Public Identifier: "-//W3C//DTD XSD 1.1//EN" vs "-//W3C//DTD XMLSCHEMA 200102//EN" at line 2. The 2009 version

Re: RFR: 8351969: Add Public Identifiers to the JDK built-in Catalog [v2]

2025-03-15 Thread Joe Wang
> Add public identifiers to the JDK built-in Catalog; Replace the incorrect > Schema 1.1 DTD files (note the Public Identifier at line 2) with the correct > Shema 1.0 DTDs. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: u

Re: RFR: 8351969: Add Public Identifiers to the JDK built-in Catalog

2025-03-14 Thread Joe Wang
On Thu, 13 Mar 2025 19:01:03 GMT, Joe Wang wrote: > Add public identifiers to the JDK built-in Catalog; Replace the incorrect > Schema 1.1 DTD files (note the Public Identifier at line 2) with the correct > Shema 1.0 DTDs. Thanks for review. Updated the test. - PR Comme

RFR: 8351969: Add Public Identifiers to the JDK built-in Catalog

2025-03-13 Thread Joe Wang
Add public identifiers to the JDK built-in Catalog; Replace the incorrect Schema 1.1 DTD files (note the Public Identifier at line 2) with the correct Shema 1.0 DTDs. - Commit messages: - 8351969: Add Public Identifiers to the JDK built-in Catalog Changes: https://git.openjdk.org/

Re: adding Xalan's XSL 3 implementation within jdk

2025-03-03 Thread Joe Wang
On 3/1/25 10:19 PM, Mukul Gandhi wrote: Hi Joe,     Thanks for nice thoughts. On Fri, 28 Feb, 2025, 00:06 Joe Wang, wrote: What's your assessment on the readiness for a formal release (or how much additional work is needed)? What are the conformance test results? The link

Integrated: 8349516: StAXStream2SAX.handleCharacters() fails on empty CDATA

2025-03-03 Thread Joe Wang
On Fri, 28 Feb 2025 17:42:24 GMT, Joe Wang wrote: > Fix handleCharacters by adding a guard to zero length and letting the rest of > the process continue (e.g. closing tags and etc). This pull request has now been integrated. Changeset: 96613cc5 Author: Joe Wang URL:

Re: RFR: 8349516: StAXStream2SAX.handleCharacters() fails on empty CDATA [v2]

2025-03-03 Thread Joe Wang
On Mon, 3 Mar 2025 18:01:48 GMT, Naoto Sato wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> remove commented section > > test/jaxp/javax/xml/jaxp/unittest/validation/Validatio

Re: RFR: 8349516: StAXStream2SAX.handleCharacters() fails on empty CDATA [v3]

2025-03-03 Thread Joe Wang
> Fix handleCharacters by adding a guard to zero length and letting the rest of > the process continue (e.g. closing tags and etc). Joe Wang has updated the pull request incrementally with one additional commit since the last revision: improve the javadoc for the test - C

Re: RFR: 8349516: StAXStream2SAX.handleCharacters() fails on empty CDATA

2025-03-01 Thread Joe Wang
On Fri, 28 Feb 2025 22:33:14 GMT, Naoto Sato wrote: >> Fix handleCharacters by adding a guard to zero length and letting the rest >> of the process continue (e.g. closing tags and etc). > > src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xsltc/trax/StAXStream2SAX.java > line 286: >

Re: RFR: 8349516: StAXStream2SAX.handleCharacters() fails on empty CDATA [v2]

2025-02-28 Thread Joe Wang
On Fri, 28 Feb 2025 23:02:28 GMT, Naoto Sato wrote: >> Hi Naoto, >> >> Yes, I did look through that code and bug 5046319. Unfortunately, it >> happened when JAXP was standalone, the history of the change was lost. >> The variable "buf" seems to be an instance variable that serves to cache the

Re: RFR: 8349516: StAXStream2SAX.handleCharacters() fails on empty CDATA [v2]

2025-02-28 Thread Joe Wang
> Fix handleCharacters by adding a guard to zero length and letting the rest of > the process continue (e.g. closing tags and etc). Joe Wang has updated the pull request incrementally with one additional commit since the last revision: remove commented section - C

RFR: 8349516: StAXStream2SAX.handleCharacters() fails on empty CDATA

2025-02-28 Thread Joe Wang
Fix handleCharacters by adding a guard to zero length and letting the rest of the process continue (e.g. closing tags and etc). - Commit messages: - 8349516: StAXStream2SAX.handleCharacters() fails on empty CDATA Changes: https://git.openjdk.org/jdk/pull/23847/files Webrev: https

Re: adding Xalan's XSL 3 implementation within jdk

2025-02-27 Thread Joe Wang
Hi Mukul, Xalan-J's XSL 3 implementation has not been integrated into any Xalan releases. I understand it implemented most of the XSLT 3.0 specification, but not all of them, esp. the Streaming feature. Was the incompleteness the reason why it was not included in an Xalan release? What's your

Re: RFR: 8345213: JVM Prefers /etc/timezone Over /etc/localtime on Debian 12

2025-02-24 Thread Joe Wang
On Mon, 24 Feb 2025 19:46:52 GMT, Naoto Sato wrote: > Removing the support for /etc/timezone file that is Debian distro specific. > The file is no longer considered authoritative > (https://wiki.debian.org/TimeZoneChanges#Check_Configured_Timezone), and in > fact, it is now out of sync as the

Re: RFR: 8349959: Test CR6740048.java passes unexpectedly missing CR6740048.xsd [v2]

2025-02-21 Thread Joe Wang
On Fri, 21 Feb 2025 01:38:34 GMT, SendaoYan wrote: >> Hi all, >> >> Test test/jaxp/javax/xml/jaxp/unittest/validation/CR6740048.java run passes >> unexpectedly when missing the depentdent file >> test/jaxp/javax/xml/jaxp/unittest/validation/CR6740048.xsd. This PR add a >> NULL check after cal

Re: RFR: 8349959: Test CR6740048.java passes unexpectedly missing CR6740048.xsd

2025-02-20 Thread Joe Wang
On Fri, 14 Feb 2025 07:44:26 GMT, SendaoYan wrote: > Hi all, > > Test test/jaxp/javax/xml/jaxp/unittest/validation/CR6740048.java run passes > unexpectedly when missing the depentdent file > test/jaxp/javax/xml/jaxp/unittest/validation/CR6740048.xsd. This PR add a > NULL check after call `get

Re: RFR: 8349959: Test CR6740048.java passes unexpectedly missing CR6740048.xsd

2025-02-19 Thread Joe Wang
On Fri, 14 Feb 2025 07:44:26 GMT, SendaoYan wrote: > Hi all, > > Test test/jaxp/javax/xml/jaxp/unittest/validation/CR6740048.java run passes > unexpectedly when missing the depentdent file > test/jaxp/javax/xml/jaxp/unittest/validation/CR6740048.xsd. This PR add a > NULL check after call `get

Integrated: 8349699: XSL transform fails with certain UTF-8 characters on 1024 byte boundaries

2025-02-19 Thread Joe Wang
On Fri, 14 Feb 2025 00:56:03 GMT, Joe Wang wrote: > Fix an edge case in the patch for JDK-8207760. This pull request has now been integrated. Changeset: 4e60c2d9 Author: Joe Wang URL: https://git.openjdk.org/jdk/commit/4e60c2d937fca8170b356f36e72b271104130c40 Stats: 127 lines

Re: RFR: 8349699: XSL transform fails with certain UTF-8 characters on 1024 byte boundaries [v3]

2025-02-18 Thread Joe Wang
> Fix an edge case in the patch for JDK-8207760. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: add tests verifying invalid sequence; also add tests to verify cases where the pairs are at the edges or anywhere in the t

Re: RFR: 8349699: XSL transform fails with certain UTF-8 characters on 1024 byte boundaries [v2]

2025-02-18 Thread Joe Wang
On Fri, 14 Feb 2025 19:55:05 GMT, Naoto Sato wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> un-needed property removed > > test/jaxp/javax/xml/jaxp/unittest/transform/JDK8207760.java l

Re: RFR: 8349699: XSL transform fails with certain UTF-8 characters on 1024 byte boundaries [v2]

2025-02-14 Thread Joe Wang
> Fix an edge case in the patch for JDK-8207760. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: un-needed property removed - Changes: - all: https://git.openjdk.org/jdk/pull/23623/files - new: https://git.openjdk.

Re: RFR: 8349699: XSL transform fails with certain UTF-8 characters on 1024 byte boundaries [v2]

2025-02-14 Thread Joe Wang
On Fri, 14 Feb 2025 18:25:30 GMT, Lance Andersen wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> un-needed property removed > > test/jaxp/javax/xml/jaxp/unittest/transform/JDK82

Re: RFR: 8349699: XSL transform fails with certain UTF-8 characters on 1024 byte boundaries

2025-02-14 Thread Joe Wang
On Fri, 14 Feb 2025 00:56:03 GMT, Joe Wang wrote: > Fix an edge case in the patch for JDK-8207760. Hello Skara (hopefully this will trigger the notification email). - PR Comment: https://git.openjdk.org/jdk/pull/23623#issuecomment-2659974504

Re: RFR: 8349873: StackOverflowError after JDK-8342550 if -Duser.timezone= is set to a deprecated zone id [v2]

2025-02-13 Thread Joe Wang
On Wed, 12 Feb 2025 23:07:26 GMT, Naoto Sato wrote: >> Fixing the regression caused by the JDK-8342550 fix. Since the logging >> facility depends on TimeZone class, it should not use the logging facility. >> Replacing the logging with simple System.err.printf() will fix the issue. > > Naoto Sat

Re: RFR: 8349873: StackOverflowError after JDK-8342550 if -Duser.timezone= is set to a deprecated zone id

2025-02-12 Thread Joe Wang
On Wed, 12 Feb 2025 19:35:18 GMT, Naoto Sato wrote: > Fixing the regression caused by the JDK-8342550 fix. Since the logging > facility depends on TimeZone class, it should not use the logging facility. > Replacing the logging with simple System.err.printf() will fix the issue. Marked as revie

Re: RFR: 8344925: translet-name ignored, when package-name is also set via TransformerFactory.setAttribute

2025-02-07 Thread Joe Wang
On Thu, 28 Nov 2024 01:09:47 GMT, Zheng Feng wrote: > …String) > > This bug seems from the improper handling of package and class name > initialization in `XSLTC` since there is a default value of `_packageName` > with `die.verwandlung`. This fix is just adjusting to call `setPackageName` >

Integrated: 8327378: XMLStreamReader throws EOFException instead of XMLStreamException

2025-02-07 Thread Joe Wang
On Fri, 7 Feb 2025 20:03:21 GMT, Joe Wang wrote: > Fix an error handling where the XMLStreamReader impl throws EOFException > instead of XMLStreamException as defined. This pull request has now been integrated. Changeset: 5395ffa0 Author: Joe Wang URL: https://git.openjdk.o

Re: RFR: 8327378: XMLStreamReader throws EOFException instead of XMLStreamException

2025-02-07 Thread Joe Wang
On Fri, 7 Feb 2025 20:03:21 GMT, Joe Wang wrote: > Fix an error handling where the XMLStreamReader impl throws EOFException > instead of XMLStreamException as defined. RN added. Thanks all! - PR Comment: https://git.openjdk.org/jdk/pull/23524#issuecomment-267199

RFR: 8327378: XMLStreamReader throws EOFException instead of XMLStreamException

2025-02-07 Thread Joe Wang
Fix an error handling where the XMLStreamReader impl throws EOFException instead of XMLStreamException as defined. - Commit messages: - 8327378: XMLStreamReader throws EOFException instead of XMLStreamException Changes: https://git.openjdk.org/jdk/pull/23524/files Webrev: https:/

Integrated: 8344925: translet-name ignored when package-name is also set

2025-02-06 Thread Joe Wang
On Tue, 4 Feb 2025 18:30:34 GMT, Joe Wang wrote: > Fix an issue where the translet-name is incorrectly set when the package-name > is also specified. This pull request has now been integrated. Changeset: 3989a199 Author: Joe Wang URL: https://git.openjdk.org/jdk/

Re: RFR: 8344925: translet-name ignored when package-name is also set [v2]

2025-02-06 Thread Joe Wang
> Fix an issue where the translet-name is incorrectly set when the package-name > is also specified. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: update test - Changes: - all: https://git.openjdk.org/jdk/pull

Re: RFR: 8344925: translet-name ignored when package-name is also set [v2]

2025-02-06 Thread Joe Wang
On Thu, 6 Feb 2025 17:58:08 GMT, Severin Gehwolf wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> update test > > test/jaxp/javax/xml/jaxp/unittest/transform/PropertiesTest.java line 94

Re: RFR: 8344925: translet-name ignored when package-name is also set [v2]

2025-02-06 Thread Joe Wang
On Thu, 6 Feb 2025 18:04:47 GMT, Naoto Sato wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> update test > > test/jaxp/javax/xml/jaxp/libs/jaxp/library/JUnitTestUtil.java line 33: >

Re: RFR: 8344925: translet-name ignored when package-name is also set

2025-02-05 Thread Joe Wang
On Wed, 5 Feb 2025 10:57:14 GMT, Severin Gehwolf wrote: > > @JoeWang-Java Have you seen https://git.openjdk.org/jdk/pull/22425? > > Just mentioning since it looks like a duplicate proposal. Indeed, I missed that. The right procedure to take over an issue is to ask the person assigned to the bu

Re: RFR: 8344925: translet-name ignored, when package-name is also set via TransformerFactory.setAttribute

2025-02-05 Thread Joe Wang
On Thu, 28 Nov 2024 01:09:47 GMT, Zheng Feng wrote: > …String) > > This bug seems from the improper handling of package and class name > initialization in `XSLTC` since there is a default value of `_packageName` > with `die.verwandlung`. This fix is just adjusting to call `setPackageName` >

RFR: 8344925: translet-name ignored when package-name is also set

2025-02-04 Thread Joe Wang
Fix an issue where the translet-name is incorrectly set when the package-name is also specified. - Commit messages: - update copyright - 8344925: translet-name ignored when package-name is also set Changes: https://git.openjdk.org/jdk/pull/23446/files Webrev: https://webrevs.ope

Re: RFR: 8301875: java.util.TimeZone.getSystemTimeZoneID uses C library default file mode

2025-01-29 Thread Joe Wang
On Wed, 29 Jan 2025 20:42:30 GMT, Naoto Sato wrote: > Force opening "tzmappings" file in text mode. Confirmed the fix with customer > provided test case. Also replaced the file open function with the safer one. Marked as reviewed by joehw (Reviewer). - PR Review: https://git.openj

Re: RFR: 8343609: Broken links in java.xml [v3]

2025-01-23 Thread Joe Wang
On Thu, 23 Jan 2025 00:46:07 GMT, Joe Wang wrote: >> Fix broken links in java.xml: >> >> Catalog: contacted Oasis. The standard page >> (https://www.oasis-open.org/standard/xmlcatalogs/) now links to the PDF >> version. That is what I'm using now, replacing

Integrated: 8343609: Broken links in java.xml

2025-01-23 Thread Joe Wang
On Wed, 22 Jan 2025 18:55:10 GMT, Joe Wang wrote: > Fix broken links in java.xml: > > Catalog: contacted Oasis. The standard page > (https://www.oasis-open.org/standard/xmlcatalogs/) now links to the PDF > version. That is what I'm using now, replacing the html pages.

Re: RFR: 8343609: Broken links in java.xml [v2]

2025-01-22 Thread Joe Wang
On Thu, 23 Jan 2025 00:35:25 GMT, Joe Wang wrote: >> Fix broken links in java.xml: >> >> Catalog: contacted Oasis. The standard page >> (https://www.oasis-open.org/standard/xmlcatalogs/) now links to the PDF >> version. That is what I'm using now, replacing

Re: RFR: 8343609: Broken links in java.xml [v3]

2025-01-22 Thread Joe Wang
thin the browser rather than downloaded as the html > version did. > > QName: rewrote the javadocs. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: s/javax.xml.XMLConstants/XMLConstants - Changes: - all: https:/

Re: RFR: 8343609: Broken links in java.xml [v2]

2025-01-22 Thread Joe Wang
thin the browser rather than downloaded as the html > version did. > > QName: rewrote the javadocs. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: use return construct; s/http/https - Changes: - all: https://git.ope

RFR: 8343609: Broken links in java.xml

2025-01-22 Thread Joe Wang
Fix broken links in java.xml: Catalog: contacted Oasis. The standard page (https://www.oasis-open.org/standard/xmlcatalogs/) now links to the PDF version. That is what I'm using now, replacing the html pages. Not ideal, but at least it's displayed within the browser rather than downloaded as th

Re: RFR: 8347498: JDK 24 RDP2 L10n resource files update

2025-01-21 Thread Joe Wang
On Fri, 17 Jan 2025 22:29:15 GMT, Justin Lu wrote: > Please review this PR which contains the l10n translations for between RDP1 > and RDP2 for the JDK24 stabilization branch. > > Note that these translations are only associated with changes made to the > stabilization branch. This PR will not

Re: RFR: 8347841: Test fixes that use deprecated time zone IDs [v2]

2025-01-17 Thread Joe Wang
On Fri, 17 Jan 2025 00:45:54 GMT, Naoto Sato wrote: >> This fix is a follow on for >> [JDK-8342550](https://bugs.openjdk.org/browse/JDK-8342550). Replaces/Removes >> usages of those deprecated time zone ids in tests. > > Naoto Sato has updated the pull request incrementally with one additional

Re: RFR: 8346142: [perf] scalability issue for the specjvm2008::xml.validation workload [v2]

2025-01-13 Thread Joe Wang
On Fri, 13 Dec 2024 02:18:35 GMT, Vladimir Ivanov wrote: >> The double check needs less time vs enter to 'synchronize' block when it not >> needed. This patch for the xml.validation workload for runs with 112 threads >> reports ~2.5% improvement on the server with Xeon 8480: >> orig: 1T - 433.6

Re: RFR: 8323740: java.lang.ExceptionInInitializerError when trying to load XML classes in wrong order [v2]

2025-01-13 Thread Joe Wang
On Fri, 20 Dec 2024 19:27:00 GMT, Liam Miller-Cushon wrote: >> Please consider this fix for >> [JDK-8323740](https://bugs.openjdk.org/browse/JDK-8323740), which avoids an >> `ExceptionInInitializerError` in the class initializer of >> `XMLDocumentFragmentScannerImpl` and other classes that hav

Re: RFR: 8342550: Log warning for using JDK1.1 compatible time zone IDs for future removal

2025-01-13 Thread Joe Wang
On Fri, 10 Jan 2025 23:05:13 GMT, Naoto Sato wrote: > The use of [three-letter time zone > IDs](https://download.java.net/java/early_access/jdk24/docs/api/java.base/java/util/TimeZone.html#three-letter-time-zone-ids-heading) > has long been deprecated. For their actual removal in the future, ma

Re: RFR: 8347146: Convert IncludeLocalesPluginTest to use JUnit [v2]

2025-01-13 Thread Joe Wang
On Thu, 9 Jan 2025 22:44:15 GMT, Naoto Sato wrote: >> A simple refinement to the test makes diagnosing failures easier. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > Addresses review comments Marked as reviewed by joehw (R

Re: RFR: 8343478: Remove unnecessary @SuppressWarnings annotations (core-libs) [v11]

2025-01-02 Thread Joe Wang
On Thu, 2 Jan 2025 21:11:04 GMT, Archie Cobbs wrote: >> Please review this patch which removes unnecessary `@SuppressWarnings` >> annotations. > > Archie Cobbs has updated the pull request incrementally with one additional > commit since the last revision: > > Update @LastModified tags. Mar

Re: RFR: 8343478: Remove unnecessary @SuppressWarnings annotations (core-libs) [v10]

2025-01-02 Thread Joe Wang
On Thu, 2 Jan 2025 19:56:16 GMT, Archie Cobbs wrote: >> Please review this patch which removes unnecessary `@SuppressWarnings` >> annotations. > > Archie Cobbs has updated the pull request incrementally with one additional > commit since the last revision: > > Revert changes under src/java.d

Re: RFR: 8345799: Update copyright year to 2024 for core-libs in files where it was missed [v3]

2024-12-09 Thread Joe Wang
On Mon, 9 Dec 2024 15:42:57 GMT, Magnus Ihse Bursie wrote: >> Some files have been modified in 2024, but the copyright year has not been >> properly updated. This should be fixed. >> >> I have located these modified files using: >> >> git log --since="Jan 1" --name-only --pretty=format: | sor

Re: RFR: 8345799: Update copyright year to 2024 for core-libs in files where it was missed [v3]

2024-12-09 Thread Joe Wang
On Mon, 9 Dec 2024 15:42:57 GMT, Magnus Ihse Bursie wrote: >> Some files have been modified in 2024, but the copyright year has not been >> properly updated. This should be fixed. >> >> I have located these modified files using: >> >> git log --since="Jan 1" --name-only --pretty=format: | sor

Re: RFR: 8345327: JDK 24 RDP1 L10n resource files update

2024-12-06 Thread Joe Wang
On Thu, 5 Dec 2024 22:36:12 GMT, Justin Lu wrote: > Please review this PR which contains the open L10n drop changes for RDP1. > > I recommend viewing the improved diffs which are built out by Jon's tool > here: https://cr.openjdk.org/~jlu/output/. As always, I can not confirm the > correctnes

Integrated: 8345353: Test for JDK-8344800 W3C DTDs and XSDs in the built-in Catalog

2024-12-03 Thread Joe Wang
On Tue, 3 Dec 2024 02:22:15 GMT, Joe Wang wrote: > Add test for JDK-8344800 W3C DTDs and XSDs in the built-in Catalog This pull request has now been integrated. Changeset: 447f8d49 Author: Joe Wang URL: https://git.openjdk.org/jdk/commit/447f8d49963fef59d36c464c4b31bff79ef02

Integrated: 8343001: Adjust XSLT and XPath Extension Function Property

2024-12-03 Thread Joe Wang
On Tue, 3 Dec 2024 01:04:30 GMT, Joe Wang wrote: > Disables XSLT and XPath Extension Functions by default, setting > jdk.xml.enableExtensionFunctions to false. > > Adjusted tests accordingly. This pull request has now been integrated. Changeset: 7ec36bb7 Author:J

Re: RFR: 8345353: Test for JDK-8344800 W3C DTDs and XSDs in the built-in Catalog [v2]

2024-12-03 Thread Joe Wang
> Add test for JDK-8344800 W3C DTDs and XSDs in the built-in Catalog Joe Wang has updated the pull request incrementally with one additional commit since the last revision: remove unnecessary checks - Changes: - all: https://git.openjdk.org/jdk/pull/22506/files - new: ht

Re: RFR: 8343001: Adjust XSLT and XPath Extension Function Property [v3]

2024-12-03 Thread Joe Wang
On Tue, 3 Dec 2024 17:23:40 GMT, Roger Riggs wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> fix error message and inline windows check > > src/java.xml/share/classes/com/sun/org/apach

Re: RFR: 8343001: Adjust XSLT and XPath Extension Function Property [v3]

2024-12-03 Thread Joe Wang
> Disables XSLT and XPath Extension Functions by default, setting > jdk.xml.enableExtensionFunctions to false. > > Adjusted tests accordingly. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: fix error message and inline w

Re: RFR: 8343001: Adjust XSLT and XPath Extension Function Property [v2]

2024-12-03 Thread Joe Wang
On Tue, 3 Dec 2024 11:42:41 GMT, Lance Andersen wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> uncomment a test in XSLTFunctionsTest > > test/jaxp/javax/xml/jaxp/unittest/transform/X

Re: RFR: 8343001: Adjust XSLT and XPath Extension Function Property [v2]

2024-12-03 Thread Joe Wang
> Disables XSLT and XPath Extension Functions by default, setting > jdk.xml.enableExtensionFunctions to false. > > Adjusted tests accordingly. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: uncomment a test in XSLTF

RFR: 8345353: Test for JDK-8344800 W3C DTDs and XSDs in the built-in Catalog

2024-12-02 Thread Joe Wang
Add test for JDK-8344800 W3C DTDs and XSDs in the built-in Catalog - Commit messages: - 8345353: Test for JDK-8344800 W3C DTDs and XSDs in the built-in Catalog Changes: https://git.openjdk.org/jdk/pull/22506/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22506&range=00 I

RFR: 8343001: Adjust XSLT and XPath Extension Function Property

2024-12-02 Thread Joe Wang
Disables XSLT and XPath Extension Functions by default, setting jdk.xml.enableExtensionFunctions to false. Adjusted tests accordingly. - Commit messages: - 8343001: Adjust XSLT and XPath Extension Function Property Changes: https://git.openjdk.org/jdk/pull/22504/files Webrev: ht

  1   2   3   4   5   >