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

Annotations #434

Merged
merged 11 commits into from Jan 22, 2014
Merged

Annotations #434

merged 11 commits into from Jan 22, 2014

Conversation

sharwell
Copy link
Member

  • Move @NotNull and @Nullable to their own artifact
  • Add NullUsageProcessor annotation processor to validate correct usage (at compile time) of the annotations in many scenarios

The following is a list of the scenarios analyzed and reported by this processor during the build process.

  • Error: an element is annotated with both @NotNull and @Nullable.
  • Error: an method which returns void is annotated with @NotNull or @Nullable.
  • Error: an element with a primitive type is annotated with @Nullable.
  • Error: a parameter is annotated with @NotNull, but the method overrides or implements a method where the parameter is annotated @Nullable.
  • Error: a method is annotated with @Nullable, but the method overrides or implements a method that is annotated with @NotNull.
  • Warning: an element with a primitive type is annotated with @NotNull.
  • Warning: a parameter is annotated with @NotNull, but the method overrides or implements a method where the parameter is not annotated.
  • Warning: a method is annotated with @Nullable, but the method overrides or implements a method that is not annotated.

parrt added a commit that referenced this pull request Jan 22, 2014
@parrt parrt merged commit 97b5d77 into antlr:master Jan 22, 2014
@sharwell sharwell deleted the annotations branch January 22, 2014 03:08
@ghost ghost assigned sharwell Jan 22, 2014
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.

None yet

2 participants