-
Notifications
You must be signed in to change notification settings - Fork 347
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
Remove the static annotationNameCache from AnnotationUtils #2997
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
It's a bit odd that the ATFs have AnnotationMirrors for the qualifiers, but then use class literals for equality comparisons. Eventually, we should look into cleaning this up some more.
/** Size of the annotationClassNames cache. */ | ||
private static final int ANNOTATION_CACHE_SIZE = 500; | ||
|
||
/** Maps classes representing AnnotationMirrors to their names. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Canonical names, to make the value description more precise?
framework/src/main/java/org/checkerframework/framework/type/AnnotatedTypeFactory.java
Show resolved
Hide resolved
# Conflicts: # checker/src/main/java/org/checkerframework/checker/units/UnitsAnnotatedTypeFactory.java
… into areSameByClass
I didn't change the Value Checker because #2987 replaced most of those calls with calls to areSameByName. So, that PR should be merged first.