Skip to content

Stream Parser Support - pt4 - Adding stream parsing support #37

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.

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

Merged
merged 2 commits into from
Aug 27, 2020
Merged

Stream Parser Support - pt4 - Adding stream parsing support #37

merged 2 commits into from
Aug 27, 2020

Conversation

jf-tech
Copy link

@jf-tech jf-tech commented Aug 26, 2020

See more technical discussion of the implementation in #32.

For stream parser, we require caller to give us one requirement XPath: streamElementXPath and one optional XPath: streamElementFilter. In general, streamElementXPath is for "pointing out" what element nodes we want to stream parse in from an XML document, and it should not have any filtering expression on the target element node. streamElementFilter, optional, provides caller an advanced option to do further target element filtering on its text or its children's. See README.md and test cases for more detailed usage.

Verified

This commit was signed with the committer’s verified signature.
mcous Michael Cousins
See more technical discussion of the implementation in #32.

For stream parser, we require caller to give us one requirement XPath: `streamElementXPath` and one
optional XPath: `streamElementFilter`. In general, `streamElementXPath` is for "pointing out" what
element nodes we want to stream parse in from an XML document, and it should not have any filtering
expression on the target element node. `streamElementFilter`, optional, provides caller an advanced
option to do further target element filtering on its text or its children's. See README.md and test
cases for more detailed usage.
@coveralls
Copy link

coveralls commented Aug 26, 2020

Coverage Status

Coverage increased (+3.2%) to 90.503% when pulling 81676ed on jf-tech:jf-tech/parse5 into fa5754e on antchfx:master.

@zhengchun zhengchun merged commit 8049e7d into antchfx:master Aug 27, 2020
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.

None yet

3 participants