-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Option to store null value in JSONObject when parsing a map #961
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
Conversation
|
What problem does this code solve? Does the code still compile with Java6? Risks Changes to the API? Will this require a new release? Should the documentation be updated? Does it break the unit tests? Was any code refactored in this commit? Review status Starting 3-day comment window. @effad Thanks for the PR. If you get a chance to make the code updates, they will be included in this PR. |
On it. |
This PR provides a new JSONParserConfiguration option to allow a Java
Map.of("foo", null)to be converted into{"foo": null}instead of{}.See also Issue #296 and PR #338, which was rejected due to API concerns. Since now we have a proper configuration object these concerns are addressed.