From: Josh Micich <josh@apache.org>
Date: Wed, 23 Dec 2009 03:43:33 +0000 (+0000)
Subject: replaced commented code with 'if (false)' block
X-Git-Tag: REL_3_7_BETA1~191
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=74409960b57424a3970d743a9adcdce82da1891a;p=poi.git

replaced commented code with 'if (false)' block

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@893395 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/src/java/org/apache/poi/ss/usermodel/DateUtil.java b/src/java/org/apache/poi/ss/usermodel/DateUtil.java
index e69225a854..238eefb920 100644
--- a/src/java/org/apache/poi/ss/usermodel/DateUtil.java
+++ b/src/java/org/apache/poi/ss/usermodel/DateUtil.java
@@ -170,8 +170,8 @@ public class DateUtil {
         Calendar calendar = new GregorianCalendar(); // using default time-zone
         setCalendar(calendar, wholeDays, millisecondsInDay, use1904windowing);
         return calendar.getTime();
-    }        
-    public static void setCalendar(Calendar calendar, int wholeDays, 
+    }
+    public static void setCalendar(Calendar calendar, int wholeDays,
             int millisecondsInDay, boolean use1904windowing) {
         int startYear = 1900;
         int dayAdjust = -1; // Excel thinks 2/29/1900 is a valid date, which it isn't
@@ -214,9 +214,9 @@ public class DateUtil {
         }
 
         String fs = formatString;
-        /*
-            Normalize the format string. The code below is equivalent
-            to the following consecutive regexp replacements:
+        if (false) {
+            // Normalize the format string. The code below is equivalent
+            // to the following consecutive regexp replacements:
 
              // Translate \- into just -, before matching
              fs = fs.replaceAll("\\\\-","-");
@@ -231,27 +231,27 @@ public class DateUtil {
              //  switching stuff, which we can ignore
              fs = fs.replaceAll(";@", "");
 
-             The code above was reworked as suggested in bug 48425:
-             simple loop is more efficient than consecutive regexp replacements.
-         */
-        StringBuilder sb = new StringBuilder();
-        for(int i = 0; i < fs.length(); i++){
+             // The code above was reworked as suggested in bug 48425:
+             // simple loop is more efficient than consecutive regexp replacements.
+        }
+        StringBuilder sb = new StringBuilder(fs.length());
+        for (int i = 0; i < fs.length(); i++) {
             char c = fs.charAt(i);
-            if(i < fs.length() - 1){
+            if (i < fs.length() - 1) {
                 char nc = fs.charAt(i + 1);
-                if(c == '\\'){
-                    switch (nc){
+                if (c == '\\') {
+                    switch (nc) {
                         case '-':
                         case ',':
                         case '.':
                         case ' ':
                         case '\\':
-                            //skip current '\' and continue to the next char
+                            // skip current '\' and continue to the next char
                             continue;
                     }
-                } else if (c == ';' && nc == '@'){
+                } else if (c == ';' && nc == '@') {
                     i++;
-                    //skip ";@" duplets
+                    // skip ";@" duplets
                     continue;
                 }
             }