We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Mal4J/src/main/java/dev/katsute/mal4j/Json.java
Lines 276 to 299 in bf3dcd6
Return null instead of throwing a NPE.
Use .valueOf rather than .parse, try catch on NFE.
.valueOf
.parse
Lines 301 to 303 in bf3dcd6
Return an empty JsonObject if null, this will protect against chained nulls.
JsonObject
Lines 313 to 319 in bf3dcd6
Return an empty array if null, this will protect against null loops.
Various as methods need to be rewritten to return null if null object.
as
These changes should remove the need for requireNonNull.
requireNonNull
The text was updated successfully, but these errors were encountered:
Katsute
Successfully merging a pull request may close this issue.
Mal4J/src/main/java/dev/katsute/mal4j/Json.java
Lines 276 to 299 in bf3dcd6
Return null instead of throwing a NPE.
Use
.valueOf
rather than.parse
, try catch on NFE.Mal4J/src/main/java/dev/katsute/mal4j/Json.java
Lines 301 to 303 in bf3dcd6
Return an empty
JsonObject
if null, this will protect against chained nulls.Mal4J/src/main/java/dev/katsute/mal4j/Json.java
Lines 313 to 319 in bf3dcd6
Return an empty array if null, this will protect against null loops.
Various
as
methods need to be rewritten to return null if null object.These changes should remove the need for
requireNonNull
.The text was updated successfully, but these errors were encountered: