[Checkers] @DefaultQualifier
Michael Ernst
mernst at csail.mit.edu
Tue Jun 24 22:21:13 EDT 2008
Mahmood-
My suggestion for refactoring @DefaultQualifier was a bad one.
Please don't change @DefaultQualifier in that way.
Instead, please add a new @DefaultQualifiers annotation that takes as its
argument an array of @DefaultQualifier annotations. The syntax should be
something like this:
@DefaultQualifiers({
@DefaultQualifier("NonNull"),
@DefaultQualifier("Interned", ALL_EXCEPT_LOCALS),
@DefaultQualifier("Tainted")})
Sorry for the design change, but I think this is much clearer for users
than passing in two arrays. It does require that the framework handle both
kinds of annotations (@DefaultQualifier and @DefaultQualifiers), but I
think that's an acceptable tradeoff.
Thanks,
-Mike
More information about the checkers
mailing list