svn commit: r1154391 - /cocoon/cocoon3/trunk/cocoon-monitoring/src/main/java/org/apache/cocoon/monitoring/cache/CacheBurstActions.java

[an...@apache.org]

Author: anathaniel
Date: Fri Aug  5 22:05:22 2011
New Revision: 1154391

URL: http://svn.apache.org/viewvc?rev=1154391&view=rev
Log:
Method can be static

Modified:
    cocoon/cocoon3/trunk/cocoon-monitoring/src/main/java/org/apache/cocoon/monitoring/cache/CacheBurstActions.java

Modified: cocoon/cocoon3/trunk/cocoon-monitoring/src/main/java/org/apache/cocoon/monitoring/cache/CacheBurstActions.java
URL: http://svn.apache.org/viewvc/cocoon/cocoon3/trunk/cocoon-monitoring/src/main/java/org/apache/cocoon/monitoring/cache/CacheBurstActions.java?rev=1154391&r1=1154390&r2=1154391&view=diff
==============================================================================
--- cocoon/cocoon3/trunk/cocoon-monitoring/src/main/java/org/apache/cocoon/monitoring/cache/CacheBurstActions.java (original)
+++ cocoon/cocoon3/trunk/cocoon-monitoring/src/main/java/org/apache/cocoon/monitoring/cache/CacheBurstActions.java Fri Aug  5 22:05:22 2011
@@ -251,11 +251,11 @@ public class CacheBurstActions {
     private void performActionOnCaches(long minSize, long maxSize, String minLastModifyDate, String maxLastModifyDate,
             String minExpiresDate, String maxExpiresDate, CacheAction action) {
 
-        Date minExpires = this.parseDate(minExpiresDate);
-        Date maxExpires = this.parseDate(maxExpiresDate);
-        Date minLastModify = this.parseDate(minLastModifyDate);
-        Date maxLastModify = this.parseDate(maxLastModifyDate);
-        int points = this.countPoints(minSize, maxSize, minLastModify, maxLastModify, minExpires, maxExpires);
+        Date minExpires = parseDate(minExpiresDate);
+        Date maxExpires = parseDate(maxExpiresDate);
+        Date minLastModify = parseDate(minLastModifyDate);
+        Date maxLastModify = parseDate(maxLastModifyDate);
+        int points = countPoints(minSize, maxSize, minLastModify, maxLastModify, minExpires, maxExpires);
 
         for (Cache cache : this.caches) {
             for (CacheKey cacheKey : cache.keySet()) {
@@ -312,7 +312,7 @@ public class CacheBurstActions {
      * @param date string in format that matches regular expression ^\d+[smhd]$
      * @return
      */
-    private Date parseDate(String date) {
+    private static Date parseDate(String date) {
         if (StringUtils.isEmpty(date)) {
             return null;
         }
@@ -356,7 +356,7 @@ public class CacheBurstActions {
      * @param maxExpiresDate If this parameter is null or of point wouldn't increased.
      * @return value of point's
      */
-    private int countPoints(long minSize, long maxSize, Date minLastModifyDate, Date maxLastModifiDate,
+    private static int countPoints(long minSize, long maxSize, Date minLastModifyDate, Date maxLastModifiDate,
             Date minExpiresDate, Date maxExpiresDate) {
 
         int points = 0;