Daniel Mensinger

Any feedback? Will this get merged or did I miss something?

The last commit is from Jul 20, 2018, so this project looks abandoned to me.

> While I understand it could be used to retain information Retaining information is an absolute must-have in my use case (serializing and deserializing data). > I don't think I...

If you also don't like something like base32 tags, would it be OK to have the default being replacing invalid chars with `_` and then having an option that either...

I have refactored the PR a bit, and now there is support for multiple strategies for escaping tags: | Key | Description | Why | |--------------------|---------------------------------------------------------------------------------------------------------------|------------------------------------------------------| | `NONE` | No...

Anything that else that is still needed? Is it OK to include multiple strategies for escaping tag names or do you only want one that can be toggled with a...

So, for this PR specifically, do you want me to just drop the `ATTRIBUTE_ESCAPE` in this case?

> So: I don't think I will accept approach as a whole as defined here. Fine by me, bit (I know we have different priorities here) in the end I...

@cowtowncoder I have updated the PR to use a new extension point (`XmlTagProcessor`) as requested.

ping :)