quick-xml icon indicating copy to clipboard operation
quick-xml copied to clipboard

Resolve "xs:list deserialization does not split on all whitespace"

Open JGM01 opened this issue 10 months ago • 3 comments

Changed DELIMITER (u8) into an array DELIMITERS (u8; 4) that holds 4 possible delimiters.

This was implemented with issue #839 in mind, so I covered the following whitespaces: , \t, \r, \n. It is fairly easy to include more if necessary.

JGM01 avatar Feb 01 '25 22:02 JGM01

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 60.56%. Comparing base (a9391f3) to head (b86f89a). Report is 26 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #843      +/-   ##
==========================================
+ Coverage   60.21%   60.56%   +0.35%     
==========================================
  Files          41       41              
  Lines       16021    15951      -70     
==========================================
+ Hits         9647     9661      +14     
+ Misses       6374     6290      -84     
Flag Coverage Δ
unittests 60.56% <100.00%> (+0.35%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Feb 02 '25 13:02 codecov-commenter

Sorry for necro but has this been looked at? Was just looking through my old repos and saw the fork.

JGM01 avatar Jul 28 '25 15:07 JGM01

It in my short list. Probably I'll implement other solution based on #379, but until that, let it remain open.

Mingun avatar Jul 28 '25 15:07 Mingun