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

Single quotes in xml are printed as ' #4

Closed
sjudd opened this issue Dec 29, 2017 · 4 comments
Closed

Single quotes in xml are printed as ' #4

sjudd opened this issue Dec 29, 2017 · 4 comments

Comments

@sjudd
Copy link
Contributor

sjudd commented Dec 29, 2017

Here's an example from the checkstyle plugin:

╔════════════╤═════════════════════════════════════════════════════════════════╤══════════╤══════╤═════════════════════════════════════════════════════════════════════╗
  ║ Reporter   │ Rule                                                            │ Severity │ Line │ Message                                                             ║
  ╠════════════╪═════════════════════════════════════════════════════════════════╪══════════╪══════╪═════════════════════════════════════════════════════════════════════╣
  ║ Checkstyle │ com.puppycrawl.tools.checkstyle.checks.imports.ImportOrderCheck │ ERROR    │ 6    │ Wrong order for 'android.content.ContentResolver' import. ║
  ╚════════════╧═════════════════════════════════════════════════════════════════╧══════════╧══════╧═════════════════════════════════════════════════════════════════════╝

It's probably possible to render them as expected, if nothing else by just replacing the few special characters xml seems to allow: http://xml.silmaril.ie/specials.html.

I'll also try and look in to it, just filing this as a reminder.

@sjudd sjudd changed the title Single quotes in xml are printed as ' Single quotes in xml are printed as ' Dec 29, 2017
@tomasbjerre
Copy link
Owner

I'd really like to keep the pom of the lib free from dependencies.

But there is also the argument of not re-implementing a bunch of utilities...

So perhaps it is best to add Guava to the lib, and relocate it in the jar.

@sjudd
Copy link
Contributor Author

sjudd commented Dec 30, 2017

I think I missed something, do you think Guava has a utility that would interpret these symbols?

@tomasbjerre
Copy link
Owner

@sjudd
Copy link
Contributor Author

sjudd commented Dec 31, 2017

Closing this in favor of tomasbjerre/violations-lib#28.

@sjudd sjudd closed this as completed Dec 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants