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

Update onXExample_pre.jpage #3807

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jevanlingen
Copy link

@jevanlingen jevanlingen commented Jan 8, 2025

Update onX example, to make the JDK 8+ onConstructor syntax more clear.

Notice, the written documentation is already on point:

On javac7, to use any of the 3 onX features, you must wrap the annotations to be applied to the constructor / method / parameter in @__(@AnnotationGoesHere). On javac8 and up, you add an underscore after onMethod, onParam, or onConstructor.

Update `onX` example, to make the JDK 8 `onConstructor` syntax more clear.
@jevanlingen
Copy link
Author

The documentation also mentioned:

Possibly java 9 will offer (much) better ways of supporting this feature.

I don't know if java 9 did support this better, but at least that line should be updated (or removed) as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant