-
Type:
Improvement
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 3.0.0-beta-2
-
Component/s: None
-
Labels:
The type of classCache can not represent its true responsibility:
protected final Map<String, Class> classCache = new UnlimitedConcurrentCache<String, Class>();
So I propose to change the type to EvictableCache<String, Class>
Similarly, protected final Map<String, Class> sourceCache = new StampedCommonCache<String, Class>(); has the same bad smell
It will be a breaking change, but it is time to clean the historical bad smell in a big version like Groovy 3.0.0
- links to