From 684e8419fb4fa0f38fe9a1fd3b474a853a274484 Mon Sep 17 00:00:00 2001
From: Bananeweizen <bananeweizen@gmx.de>
Date: Sun, 24 Feb 2013 10:25:43 +0100
Subject: code cleanup

* rename several lower case classes
* more strict visibility modifiers
* Lint warnings
* style warnings
---
 .../cgeo/geocaching/test/RegExPerformanceTest.java | 24 ++++++++++------------
 1 file changed, 11 insertions(+), 13 deletions(-)

(limited to 'tests/src/cgeo/geocaching/test/RegExPerformanceTest.java')

diff --git a/tests/src/cgeo/geocaching/test/RegExPerformanceTest.java b/tests/src/cgeo/geocaching/test/RegExPerformanceTest.java
index bd30532..ec84824 100644
--- a/tests/src/cgeo/geocaching/test/RegExPerformanceTest.java
+++ b/tests/src/cgeo/geocaching/test/RegExPerformanceTest.java
@@ -76,7 +76,7 @@ public class RegExPerformanceTest extends TestCase {
     }
 
     public static void testRegEx() {
-        List<String> output = doTheTests(10);
+        final List<String> output = doTheTests(10);
 
         for (String s : output) {
             System.out.println(s);
@@ -85,7 +85,7 @@ public class RegExPerformanceTest extends TestCase {
 
     public static List<String> doTheTests(final int iterations) {
 
-        List<String> output = new ArrayList<String>();
+        final List<String> output = new ArrayList<String>();
 
         output.addAll(measure(iterations, "description", PATTERN_DESCRIPTION_OLD, PATTERN_DESCRIPTION));
 
@@ -94,7 +94,7 @@ public class RegExPerformanceTest extends TestCase {
 
     private static List<String> measure(int iterations, String fieldName, Pattern p1, Pattern p2) {
 
-        List<String> output = new ArrayList<String>();
+        final List<String> output = new ArrayList<String>();
         output.add(fieldName + ":");
 
         for (MockedCache cache : MOCKED_CACHES) {
@@ -106,15 +106,13 @@ public class RegExPerformanceTest extends TestCase {
             long diff1, diff2;
 
             output.add("Parsing " + cache.getGeocode() + " " + cache.getName());
-            {
-                diff1 = parse(page, p1, iterations);
-                output.add("Time pattern 1:\t" + diff1 + " ms");
-            }
-
-            {
-                diff2 = parse(page, p2, iterations);
-                output.add("Time pattern 2:\t" + diff2 + " ms");
-            }
+
+            diff1 = parse(page, p1, iterations);
+            output.add("Time pattern 1:\t" + diff1 + " ms");
+
+            diff2 = parse(page, p2, iterations);
+            output.add("Time pattern 2:\t" + diff2 + " ms");
+
             float reduction = (float) diff2 * 100 / diff1;
             output.add("New runtime:\t" + String.format("%.1f", reduction) + "%\n");
         }
@@ -124,7 +122,7 @@ public class RegExPerformanceTest extends TestCase {
     }
 
     private static long parse(String page, Pattern pattern, int iterations) {
-        long start = System.currentTimeMillis();
+        final long start = System.currentTimeMillis();
         for (int j = 0; j < iterations; j++) {
             BaseUtils.getMatch(page, pattern, true, "");
         }
-- 
cgit v1.1