You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by gn...@apache.org on 2010/04/28 16:26:25 UTC
svn commit: r938984 -
/felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/Scanner.java
Author: gnodet
Date: Wed Apr 28 14:26:24 2010
New Revision: 938984
URL: http://svn.apache.org/viewvc?rev=938984&view=rev
Log:
FELIX-1776: minimize the number of calls to File#getCanonicalFile()
Modified:
felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/Scanner.java
Modified: felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/Scanner.java
URL: http://svn.apache.org/viewvc/felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/Scanner.java?rev=938984&r1=938983&r2=938984&view=diff
==============================================================================
--- felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/Scanner.java (original)
+++ felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/Scanner.java Wed Apr 28 14:26:24 2010
@@ -69,7 +69,7 @@ public class Scanner {
*/
public Scanner(File directory, FilenameFilter filter)
{
- this.directory = directory;
+ this.directory = canon(directory);
this.filter = filter;
}
@@ -107,7 +107,7 @@ public class Scanner {
Set/*<File>*/ removed = new HashSet/*<File>*/(storedChecksums.keySet());
for (int i = 0; i < list.length; i++)
{
- File file = canon(list[i]);
+ File file = list[i];
long lastChecksum = lastChecksums.get(file) != null ? ((Long) lastChecksums.get(file)).longValue() : 0;
long storedChecksum = storedChecksums.get(file) != null ? ((Long) storedChecksums.get(file)).longValue() : 0;
long newChecksum = checksum(file);