Refined check for cache turnoff threshold

jhoeller · jhoeller · commit 9cbac985fbe2 · 2013-10-30T19:28:08.000+01:00
Issue: SPR-10803
diff --git a/spring-core/src/main/java/org/springframework/util/AntPathMatcher.java b/spring-core/src/main/java/org/springframework/util/AntPathMatcher.java
@@ -279,7 +279,7 @@ protected AntPathStringMatcher getStringMatcher(String pattern) {
 		}
 		if (matcher == null) {
 			matcher = new AntPathStringMatcher(pattern);
-			if (cachePatterns == null && this.stringMatcherCache.size() == CACHE_TURNOFF_THRESHOLD) {
+			if (cachePatterns == null && this.stringMatcherCache.size() >= CACHE_TURNOFF_THRESHOLD) {
 				// Try to adapt to the runtime situation that we're encountering:
 				// There are obviously too many different paths coming in here...
 				// So let's turn off the cache since the patterns are unlikely to be reoccurring.

Original file line number	Diff line number	Diff line change
`@@ -279,7 +279,7 @@ protected AntPathStringMatcher getStringMatcher(String pattern) {`
`279`	`279`	`}`
`280`	`280`	`if (matcher == null) {`
`281`	`281`	`matcher = new AntPathStringMatcher(pattern);`
`282`		`- if (cachePatterns == null && this.stringMatcherCache.size() == CACHE_TURNOFF_THRESHOLD) {`
	`282`	`+ if (cachePatterns == null && this.stringMatcherCache.size() >= CACHE_TURNOFF_THRESHOLD) {`
`283`	`283`	`// Try to adapt to the runtime situation that we're encountering:`
`284`	`284`	`// There are obviously too many different paths coming in here...`
`285`	`285`	`// So let's turn off the cache since the patterns are unlikely to be reoccurring.`