[jbpm-commits] JBoss JBPM SVN: r6097 - in jbpm3/branches/jbpm-3.2-soa/modules/core/src/main/java/org/jbpm: util and 1 other directory.
do-not-reply at jboss.org
do-not-reply at jboss.org
Tue Jan 19 19:35:59 EST 2010
Author: alex.guizar at jboss.com
Date: 2010-01-19 19:35:58 -0500 (Tue, 19 Jan 2010)
New Revision: 6097
Modified:
jbpm3/branches/jbpm-3.2-soa/modules/core/src/main/java/org/jbpm/file/def/FileDefinition.java
jbpm3/branches/jbpm-3.2-soa/modules/core/src/main/java/org/jbpm/util/CustomLoaderObjectInputStream.java
Log:
improve a couple naughty uses of Arrays.asList
Modified: jbpm3/branches/jbpm-3.2-soa/modules/core/src/main/java/org/jbpm/file/def/FileDefinition.java
===================================================================
--- jbpm3/branches/jbpm-3.2-soa/modules/core/src/main/java/org/jbpm/file/def/FileDefinition.java 2010-01-19 17:14:51 UTC (rev 6096)
+++ jbpm3/branches/jbpm-3.2-soa/modules/core/src/main/java/org/jbpm/file/def/FileDefinition.java 2010-01-20 00:35:58 UTC (rev 6097)
@@ -28,11 +28,9 @@
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
-import java.util.ArrayList;
import java.util.Arrays;
import java.util.HashMap;
import java.util.Iterator;
-import java.util.List;
import java.util.Map;
import org.apache.commons.logging.Log;
@@ -256,26 +254,24 @@
}
String findNewDirName() {
- String newDirName = "files-1";
+ String dirName = "files-1";
File parentFile = new File(getRootDir());
- if (parentFile.exists()) {
+ if (parentFile.isDirectory()) {
// get the current contents of the directory
- String[] children = parentFile.list();
- List fileNames = new ArrayList();
- if (children != null) {
- fileNames = new ArrayList(Arrays.asList(children));
- }
+ String[] fileNames = parentFile.list();
+ Arrays.sort(fileNames);
// find an unused name for the directory to be created
- int seqNr = 1;
- while (fileNames.contains(newDirName)) {
- seqNr++;
- newDirName = "files-" + seqNr;
+ StringBuffer nameBuilder = new StringBuffer("files-");
+ for (int seqNr = 2; Arrays.binarySearch(fileNames, dirName) >= 0; seqNr++) {
+ dirName = nameBuilder.append(seqNr).toString();
+ // remove appended number
+ nameBuilder.setLength(6);
}
}
- return newDirName;
+ return dirName;
}
File getFilePath(String name) {
Modified: jbpm3/branches/jbpm-3.2-soa/modules/core/src/main/java/org/jbpm/util/CustomLoaderObjectInputStream.java
===================================================================
--- jbpm3/branches/jbpm-3.2-soa/modules/core/src/main/java/org/jbpm/util/CustomLoaderObjectInputStream.java 2010-01-19 17:14:51 UTC (rev 6096)
+++ jbpm3/branches/jbpm-3.2-soa/modules/core/src/main/java/org/jbpm/util/CustomLoaderObjectInputStream.java 2010-01-20 00:35:58 UTC (rev 6097)
@@ -27,7 +27,6 @@
import java.io.ObjectStreamClass;
import java.lang.reflect.Modifier;
import java.lang.reflect.Proxy;
-import java.util.Arrays;
/**
* Specialized object input stream that allows classes to be fetched through a
@@ -123,7 +122,7 @@
}
catch (IllegalArgumentException iae) {
throw new ClassNotFoundException("could not get proxy class for interfaces: "
- + Arrays.asList(classes), e);
+ + ArrayUtil.toString(classes), e);
}
}
}
More information about the jbpm-commits
mailing list