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

Generated xml is broken #625

Closed

mrpi opened this issue Mar 30, 2016 · 8 comments

Labels

Bug

mrpi commented Mar 30, 2016

The xml writer uses the escape sequence "&#xXX;" incorrectly. Further details are described here #588.
I am just creating this bug report, so that the problem is not being overlooked.

ianmacs mentioned this issue

Open Tickets in Catch Bug Tracker #675

Closed

gatopeich commented Aug 23, 2016

Breaking my builds, too

Collaborator

philsquared commented Oct 14, 2016

I believe this one is fixed now? (as of Build 1.5.7). Please confirm and/ or close.

Author

mrpi commented Oct 14, 2016

Still broken. Char to Int overflows and creates sequences like &#xFFFFFFE5;

gatopeich commented Dec 5, 2016

I am working with a "works for me" patch...

Collaborator

philsquared commented Jan 16, 2017

Could you give a more complete example of what is going wrong for you (assuming it still is in 1.6.0).
The example given in #588 seems to work for me at this point.

Collaborator

philsquared commented Jan 16, 2017

Sorry ignore that - I was thinking of a different issue.
Looks like #588 may still have some outstanding parts.

horenmar added the Bug label

Collaborator

philsquared commented Mar 13, 2017

The encoding issues in the referenced ticket, #588, were fixed in February and that ticket was closed.
Is there any more to this ticket or can we close this one, too?
Or, if not, could you give any more details?

philsquared added the Resolved - pending review label

Member

horenmar commented May 16, 2017

Since there hasn't been any follow-up in two months, I am gonna close this.

horenmar closed this as completed

horenmar removed the Resolved - pending review label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment