[Schmitzm-commits] r2044 - trunk/schmitzm-core/src/main/java/de/schmitzm/regex

[scm-commit at wald.intevation.org]

Author: alfonx
Date: 2012-05-28 02:08:33 +0200 (Mon, 28 May 2012)
New Revision: 2044

Modified:
   trunk/schmitzm-core/src/main/java/de/schmitzm/regex/RegexCache.java
Log:


Modified: trunk/schmitzm-core/src/main/java/de/schmitzm/regex/RegexCache.java
===================================================================
--- trunk/schmitzm-core/src/main/java/de/schmitzm/regex/RegexCache.java	2012-05-27 23:46:03 UTC (rev 2043)
+++ trunk/schmitzm-core/src/main/java/de/schmitzm/regex/RegexCache.java	2012-05-28 00:08:33 UTC (rev 2044)
@@ -21,16 +21,16 @@
 
 	final static Logger log = Logger.getLogger(RegexCache.class);
 
-	/**
-	 * Matcher-Cache für Regex+Value verwenden? Durch die methode mather.reset wird pro Thread ein Matcher
-	 * wiederverwendet
-	 */
-	public static boolean matcherCacheEnabled = false;
+	// /**
+	// * Matcher-Cache für Regex+Value verwenden? Durch die methode mather.reset wird pro Thread ein Matcher
+	// * wiederverwendet
+	// */
+	// public static boolean matcherCacheEnabled = false;
 
 	/**
 	 * Regex-Auswertungen komplett cachen
 	 */
-	public static boolean resultsCacheEnabled = true;
+	public static boolean resultsCacheEnabled = false;
 
 	/**
 	 * Ein String der länger ist als die hier angegebene Anzahl Zeichen wir nicht gecached. Die Wahrscheinlichkeit, dass
@@ -52,8 +52,9 @@
 	 */
 	private static final int CACHE_VALUES_TO_RESULT_MAX_REGEX_LENGTH = 10000;
 
-	private final ConcurrentLRUCache<String, ThreadLocal<Matcher>> matchersCache = new ConcurrentLRUCache<String, ThreadLocal<Matcher>>(
-			60000, 40000);
+	// private final ConcurrentLRUCache<String, ThreadLocal<Matcher>> matchersCache = new ConcurrentLRUCache<String,
+	// ThreadLocal<Matcher>>(
+	// 60000, 40000);
 
 	private static RegexCache singletonInstance;
 
@@ -80,27 +81,27 @@
 	 */
 	public final Matcher getMatcher(final String regex, final String text) {
 
-		if (!matcherCacheEnabled) {
-			return getPattern(regex).matcher(text);
-		}
-
-		ThreadLocal<Matcher> threadLocal = matchersCache.get(regex);
-
-		if (threadLocal == null) {
-
-			threadLocal = new ThreadLocal<Matcher>() {
-
-				@Override
-				protected Matcher initialValue() {
-					return getPattern(regex).matcher("");
-				}
-			};
-			matchersCache.put(regex, threadLocal);
-
-			return threadLocal.get();
-		}
-
-		return threadLocal.get().reset(text);
+		// if (!matcherCacheEnabled) {
+		return getPattern(regex).matcher(text);
+		// }
+		//
+		// ThreadLocal<Matcher> threadLocal = matchersCache.get(regex);
+		//
+		// if (threadLocal == null) {
+		//
+		// threadLocal = new ThreadLocal<Matcher>() {
+		//
+		// @Override
+		// protected Matcher initialValue() {
+		// return getPattern(regex).matcher("");
+		// }
+		// };
+		// matchersCache.put(regex, threadLocal);
+		//
+		// return threadLocal.get();
+		// }
+		//
+		// return threadLocal.get().reset(text);
 	}
 
 	/**
@@ -186,12 +187,12 @@
 		return resultsCache.getStats();
 	}
 
-	/**
-	 * Liefert die aktuellen Statistiken des ResultsCache
-	 */
-	public Stats getMatcherCacheStats() {
-		return matchersCache.getStats();
-	}
+	// /**
+	// * Liefert die aktuellen Statistiken des ResultsCache
+	// */
+	// public Stats getMatcherCacheStats() {
+	// return matchersCache.getStats();
+	// }
 
 	/**
 	 * Liefert die aktuellen Statistiken des PatternCache
@@ -237,10 +238,10 @@
 			patternCache.destroy();
 		}
 
-		if (matchersCache != null) {
-			matchersCache.clear();
-			matchersCache.destroy();
-		}
+		// if (matchersCache != null) {
+		// matchersCache.clear();
+		// matchersCache.destroy();
+		// }
 	}
 
 	public void report(PrintStream out) {
@@ -249,10 +250,10 @@
 				+ cStats.getCumulativeLookups() + " Misses=" + cStats.getCumulativeMisses() + " currentSize="
 				+ cStats.getCurrentSize());
 
-		cStats = getMatcherCacheStats();
-		out.println("MatcherCacheStats: Hits=" + cStats.getCumulativeHits() + " Lookups="
-				+ cStats.getCumulativeLookups() + " Misses=" + cStats.getCumulativeMisses() + " currentSize="
-				+ cStats.getCurrentSize());
+		// cStats = getMatcherCacheStats();
+		// out.println("MatcherCacheStats: Hits=" + cStats.getCumulativeHits() + " Lookups="
+		// + cStats.getCumulativeLookups() + " Misses=" + cStats.getCumulativeMisses() + " currentSize="
+		// + cStats.getCurrentSize());
 
 		cStats = getResultsCacheStats();
 		out.println("ResultsCacheStats: Hits=" + cStats.getCumulativeHits() + " Lookups="