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

Address remaining JSR-305 annotations #3198

Closed
epkugelmass opened this issue Jul 3, 2018 · 1 comment
Closed

Address remaining JSR-305 annotations #3198

epkugelmass opened this issue Jul 3, 2018 · 1 comment

Comments

@epkugelmass
Copy link

The (partial) transition from "JSR-305" nullability annotations to Checker Framework annotations in 23.6 [1] was a welcome change. However, the remaining JSR-305 annotation usages like "package-level" nullability default [2] make it difficult for FindBugs users to use Guava >= 23.6. FindBugs does not recognize the new Checker Qual annotations and so reasons that null-tolerant methods like ToStringHelper::add(String, @org.checkerframework.checker.nullness.qual.Nullable Object) are actually null-intolerant.

Given that null-intolerance is the default the Checker type system, and also given the open task [3] to remove the JSR-305 dependency entirely, would Guava consider removing usages of javax.annotation.ParametersAreNonnullByDefault and friends in the next release?

[1] https://github.com/google/guava/releases/tag/v23.6
[2] https://github.com/google/guava/blob/master/guava/src/com/google/common/base/package-info.java#L59
[3] #2960

@cgdecker
Copy link
Member

cgdecker commented Jul 3, 2018

Marking as a duplicate of #2960 which is still open for tracking this issue. AFAIK we still need Jetbrains to do something so that we won't break Kotlin users by migrating away from @ParametersAreNonNullByDefault.

@cpovirk in case there's something here you want to note in the other issue.

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

No branches or pull requests

2 participants