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

Enable StrictNullChecks by default #886

Closed
k163377 opened this issue Jan 19, 2025 · 3 comments
Closed

Enable StrictNullChecks by default #886

k163377 opened this issue Jan 19, 2025 · 3 comments
Labels
3.x Issue affecting/planned for Jackson 3.x

Comments

@k163377
Copy link
Contributor

k163377 commented Jan 19, 2025

Previously, the StrictNullChecks option was disabled by default, which compromised null safety.
Therefore, I propose to enable this option by default in 3.0.

As a side note, since 2.19, improvements in #885 have significantly reduced the throughput degradation when the StrictNullChecks option is enabled, and the deserialization performance is almost the same as when it is disabled.
Thus, performance concerns have at least been addressed.

Please vote with 👍 or 👎.

@k163377 k163377 added the 3.x Issue affecting/planned for Jackson 3.x label Jan 19, 2025
@cowtowncoder
Copy link
Member

@k163377 I'll update https://github.com/FasterXML/jackson-future-ideas/wiki/JSTEP-2 and add an entry or two from here. But going forward maybe you can add links from there?

@k163377
Copy link
Contributor Author

k163377 commented Jan 20, 2025

@cowtowncoder
Thank you, I will do so.

@k163377
Copy link
Contributor Author

k163377 commented Mar 22, 2025

The change have been reflected (sorry for not making it to RC1 in time).

@k163377 k163377 closed this as completed Mar 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x Issue affecting/planned for Jackson 3.x
Projects
None yet
Development

No branches or pull requests

2 participants