RFR: 8294858: XMLStreamReader does not respect jdk.xml.maxXMLNameLimit=0 for namespace names

2022-11-02 Thread Joe Wang
Fixed an issue where the condition check for limit=0 was missed. - Commit messages: - 8294858: XMLStreamReader does not respect jdk.xml.maxXMLNameLimit=0 for namespace names Changes: https://git.openjdk.org/jdk/pull/10958/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=1095

Re: RFR: 8294858: XMLStreamReader does not respect jdk.xml.maxXMLNameLimit=0 for namespace names

2022-11-02 Thread Lance Andersen
On Wed, 2 Nov 2022 19:32:47 GMT, Joe Wang wrote: > Fixed an issue where the condition check for limit=0 was missed. Marked as reviewed by lancea (Reviewer). - PR: https://git.openjdk.org/jdk/pull/10958

Re: RFR: 8294858: XMLStreamReader does not respect jdk.xml.maxXMLNameLimit=0 for namespace names [v2]

2022-11-02 Thread Joe Wang
> Fixed an issue where the condition check for limit=0 was missed. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: source changes - Changes: - all: https://git.openjdk.org/jdk/pull/10958/files - new: https://git.openjdk.

Re: RFR: 8294858: XMLStreamReader does not respect jdk.xml.maxXMLNameLimit=0 for namespace names [v2]

2022-11-02 Thread Lance Andersen
On Wed, 2 Nov 2022 20:00:55 GMT, Joe Wang wrote: >> Fixed an issue where the condition check for limit=0 was missed. > > Joe Wang has updated the pull request incrementally with one additional > commit since the last revision: > > source changes Looks fine as discussed - Marked

Re: RFR: 8294858: XMLStreamReader does not respect jdk.xml.maxXMLNameLimit=0 for namespace names [v2]

2022-11-02 Thread Naoto Sato
On Wed, 2 Nov 2022 20:00:55 GMT, Joe Wang wrote: >> Fixed an issue where the condition check for limit=0 was missed. > > Joe Wang has updated the pull request incrementally with one additional > commit since the last revision: > > source changes Marked as reviewed by naoto (Reviewer). -

Re: RFR: 8294858: XMLStreamReader does not respect jdk.xml.maxXMLNameLimit=0 for namespace names [v2]

2022-11-03 Thread Joe Wang
On Wed, 2 Nov 2022 20:00:55 GMT, Joe Wang wrote: >> Fixed an issue where the condition check for limit=0 was missed. > > Joe Wang has updated the pull request incrementally with one additional > commit since the last revision: > > source changes Thanks Lance, Naoto! - PR: https