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 OOM for ConstructingParser #209

Merged
merged 4 commits into from
Jun 26, 2018
Merged

Fix OOM for ConstructingParser #209

merged 4 commits into from
Jun 26, 2018

Commits on Jun 25, 2018

  1. SI-4520 Tests for OOE in XML parser

    @terma @ashawley
    terma authored and ashawley committed Jun 25, 2018
    Configuration menu
    Copy the full SHA
    4940b6a View commit details
    Browse the repository at this point in the history

Commits on Jun 26, 2018

  1. Refactor Artem Stasiuk's OOE parser tests 

    Shorten test names and minimize string literals.
    @ashawley
    ashawley committed Jun 26, 2018
    Configuration menu
    Copy the full SHA
    6f76dd1 View commit details
    Browse the repository at this point in the history

  2. Fix OOM in parser 

    Add eof checks to:

- MarkupParser.xEntityValue
- MarkupParser.xComment
- MarkupParser.systemLiteral
- MarkupParser.pubidLiteral
- MarkupParser.notationDecl
- MarkupParserCommon.xAttributeValue
- MarkupParserCommon.xTakeUntil
    @ashawley
    ashawley committed Jun 26, 2018
    Configuration menu
    Copy the full SHA
    239e3ed View commit details
    Browse the repository at this point in the history

  3. Silence test output ConstructingParser

    @ashawley
    ashawley committed Jun 26, 2018
    Configuration menu
    Copy the full SHA
    e696ee6 View commit details
    Browse the repository at this point in the history