moparisthebest/poi
1
0
Fork 0
Code Issues Pull Requests Releases 1 Wiki Activity

replaced commented code with 'if (false)' block

Browse Source 
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@893395 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Josh Micich 2009-12-23 03:43:33 +00:00
parent 3bbde4880c
commit 56c40e3a0f
1 changed files with 16 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/java/org/apache/poi/ss/usermodel/DateUtil.java
View File

@ -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;
}
}