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

feat(jdk17): enhance exception handling for Java 14+ NPE #34

Open
wants to merge 1 commit into
base: arch/arm64
Choose a base branch
from

Conversation

halibobo1205
Copy link
Owner

@halibobo1205 halibobo1205 commented Oct 21, 2024

What does this PR do?

  • update condition to check for NullPointerException instances
  • address JEP 358 changes to NullPointerException behavior

Refs: JEP 358, JDK-8220715

Why are these changes required?

  • maintain compatibility with both pre-Java 14 and Java 14+ versions
  • ensure correct handling of non-empty NPE messages in Java 14+

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@halibobo1205 halibobo1205 changed the title feat(jvm): enhance exception handling for Java 14+ NullPointerException feat(jdk17): enhance exception handling for Java 14+ NullPointerException Oct 23, 2024
@halibobo1205 halibobo1205 changed the title feat(jdk17): enhance exception handling for Java 14+ NullPointerException feat(jdk17): enhance exception handling for Java 14+ NPE Oct 23, 2024
@halibobo1205 halibobo1205 changed the base branch from develop to arch/arm64 November 1, 2024 08:11
    - update condition to check for NullPointerException instances
    - maintain compatibility with both pre-Java 14 and Java 14+ versions
    - address JEP 358 changes to NullPointerException behavior
    - ensure correct handling of non-empty NPE messages in Java 14+

    Refs: JEP 358, JDK-8220715
@halibobo1205 halibobo1205 force-pushed the feat/JEP-358-Helpful-NullPointerExceptions branch from 0602b05 to 164c21b Compare November 22, 2024 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant