Updates value error messages to be consistent. by johnjaylward · Pull Request #675 · stleary/JSON-java

Fixes #649

What problem does this code solve?

Updates value error messages to be consistent. Some errors would output value information, but others not.

Provide both the type and value that failed conversion. Tries not to
"toString" large value types like Arrays or Maps. For those types it
will just output the type and not a value.

Risks

Low

Changes to the API?

No

Will this require a new release?

Yes

Should the documentation be updated?

No

Does it break the unit tests?

Yes. Some tests that verify error messages have been updated.

Was any code refactored in this commit?

Private methods were removed that allowed generation of the error messages with no value information.

Review status
APPROVED