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 files only. >> >> DTD

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

2025-03-19 Thread Roger Riggs
On Tue, 18 Mar 2025 18:27:56 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/internal/jdkcatalog/w3c/dtd/schema10/XMLSc

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

2025-03-19 Thread Roger Riggs
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 files only. >> >> DTD

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

2025-03-19 Thread Roger Riggs
On Tue, 18 Mar 2025 19:55:39 GMT, Joe Wang wrote: >> src/java.xml/share/classes/jdk/xml/internal/jdkcatalog/w3c/dtd/schema10/XMLSchema.dtd >> line 8: >> >>> 6: >>> >>> 7: >>> >>> 8: >>> >> >> The other XML comments use the multi-line format with a single begin

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

2025-03-18 Thread Roger Riggs
On Fri, 14 Mar 2025 18:35:22 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. > > Joe Wang has updated the pull request incrementally with one addi

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

2025-03-18 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 > Schema 1.0 DTDs. The JDK built-in Catalog contains Schema 1.0 files only. > > DTD files: the Schema 1.1 files, XMLSchema.dtd and datatypes.

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/jdkcatalog/JDKCatalogTest.java line

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/internal/jdkcatalog/w3c/dtd/schema10/XMLSc

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-18 Thread Roger Riggs
On Fri, 14 Mar 2025 18:35:22 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. > > Joe Wang has updated the pull request incrementally with one addi

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

2025-03-18 Thread Iris Clark
On Fri, 14 Mar 2025 18:35:22 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. > > Joe Wang has updated the pull request incrementally with one addi

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

2025-03-18 Thread Lance Andersen
On Fri, 14 Mar 2025 18:35:22 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. > > Joe Wang has updated the pull request incrementally with one addi

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

2025-03-17 Thread Mikhail Yankelevich
On Fri, 14 Mar 2025 18:35:12 GMT, Joe Wang wrote: > Thanks for review. Updated the test. Thank you! - PR Comment: https://git.openjdk.org/jdk/pull/24039#issuecomment-2729299732

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

2025-03-15 Thread Mikhail Yankelevich
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. Looks good to me, just a few very minor questions. Thank you test/jaxp

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: update the test

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 Comment: https:

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/