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.

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

Fix parsing of hexadecimal encoded entities. #636

Merged

Conversation

@gijsbert802
Copy link
Contributor

@gijsbert802 gijsbert802 commented Apr 28, 2017

I noticed this code was probably copied from another repo at http://gforge.hl7.org. Is this the correct place to fix this? Or should this (also) be fixed there?

@coveralls
Copy link

@coveralls coveralls commented Apr 28, 2017

Coverage Status

Coverage increased (+0.009%) to 77.295% when pulling 1fc1590 on gijsbert802:fix-hex-encoded-xml-entity-parsing into f21ab8b on jamesagnew:master.

@jamesagnew
Copy link
Owner

@jamesagnew jamesagnew commented May 20, 2017

Looks good, thanks! Merging now.

@jamesagnew jamesagnew merged commit 77109cb into jamesagnew:master May 20, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.009%) to 77.295%
Details
jamesagnew added a commit that referenced this pull request May 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.