Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix couple of bugs in serde deserializer #523

Merged
merged 5 commits into from Dec 12, 2022
Merged

Fix couple of bugs in serde deserializer #523

merged 5 commits into from Dec 12, 2022

Commits on Dec 11, 2022

  1. Add tests for excess elements before the struct 

    failures (2):
    struct_::excess_data_before::cdata
    struct_::excess_data_before::text_non_spaces
    @Mingun
    Mingun committed Dec 11, 2022
    Copy the full SHA
    54fc1db View commit details
    Browse the repository at this point in the history

  2. Fix incorrect skipping text and CDATA content before any map-like str… 

    …uctures

in serde deserializer, like

  ```xml
  unwanted text<struct>...</struct>
  ```
    @Mingun
    Mingun committed Dec 11, 2022
    Copy the full SHA
    d1601d9 View commit details
    Browse the repository at this point in the history

  3. Rename next_text to read_string because new name are better descr… 

    …ibes the purpose of method
    @Mingun
    Mingun committed Dec 11, 2022
    Copy the full SHA
    1373eb1 View commit details
    Browse the repository at this point in the history

Commits on Dec 12, 2022

  1. Fix incorrect handling of xs:lists with encoded spaces: they still … 

    …act as delimiters

Co-authored-by: Daniel Alley <dalley@redhat.com>
    @Mingun @dralley
    Mingun and dralley committed Dec 12, 2022
    Copy the full SHA
    daa6526 View commit details
    Browse the repository at this point in the history

  2. Add some comments about deserializing sequences and give a more descr… 

    …iptive name to a struct
    @Mingun
    Mingun committed Dec 12, 2022
    Copy the full SHA
    8283121 View commit details
    Browse the repository at this point in the history