Skip to content

Update xml parser#3

Open
tukanos wants to merge 90 commits intoGsDevKit:gemstonefrom
tukanos:updateXMLParser
Open

Update xml parser#3
tukanos wants to merge 90 commits intoGsDevKit:gemstonefrom
tukanos:updateXMLParser

Conversation

@tukanos
Copy link
Copy Markdown

@tukanos tukanos commented Oct 10, 2025

This is an update of XMLParser for GemStone. All tests pass. I had to adjust some tests to reflect some prohibited characters behaviour.

Ducasse and others added 30 commits September 7, 2021 13:19
For Pharo 90 loading (no GT Tools in Pharo anymore)
…erValuesAtAdd tests by not adding to the addedHeaders collection the second time

Listen to some code critics
…stHeaderValuesAdd when Pharo version is before 10 (needs the Zinc patch to ZnMultiValueDictionary that make #at:add idempotent for identical key/values)
Fix XMLHTTPMessageTest (Request|Response)
use vX.X.X style instead of X.X.X to reference version
Patrik added 30 commits October 7, 2025 11:57
To prevent useless code changes
The Unicode range of codePoints from 16rD800-16rDFFF is reserved for
encoding leading/trailing surrogate pairs for UTF-16 encoding. These
can never be legal Unicode characters, and as such, it is an error to
attempt to create a Character in this range.
- Added convenience methods to TestAsserter
Testing invalid characters is different in Pharo and GemStone.  GemStone
throws OutOfRange exception directly when trying to create Array with
forbidden characters.  Refactoring tests to reflect this reality.
Signed-off-by: Patrik <development@finworks.biz>
Signed-off-by: Patrik <development@finworks.biz>
Signed-off-by: Patrik <development@finworks.biz>
Signed-off-by: Patrik <development@finworks.biz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants