From 6fcc4a33796196026b325e410dbdee68d5aa757d Mon Sep 17 00:00:00 2001
From: rfeng <rfeng@13f79535-47bb-0310-9956-ffa450edef68>
Date: Sat, 3 Apr 2010 15:14:18 +0000
Subject: One more place to close files

git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@930529 13f79535-47bb-0310-9956-ffa450edef68
---
 .../maven/compiler/ClassLoaderNameEnvironment.java     | 18 ++++++------------
 .../tuscany/maven/compiler/osgi/BundleResolver.java    |  8 +++-----
 2 files changed, 9 insertions(+), 17 deletions(-)

(limited to 'maven-plugins/trunk')

diff --git a/maven-plugins/trunk/maven-eclipse-compiler/src/main/java/org/apache/tuscany/maven/compiler/ClassLoaderNameEnvironment.java b/maven-plugins/trunk/maven-eclipse-compiler/src/main/java/org/apache/tuscany/maven/compiler/ClassLoaderNameEnvironment.java
index bb38abad0d..2f238a8a4d 100644
--- a/maven-plugins/trunk/maven-eclipse-compiler/src/main/java/org/apache/tuscany/maven/compiler/ClassLoaderNameEnvironment.java
+++ b/maven-plugins/trunk/maven-eclipse-compiler/src/main/java/org/apache/tuscany/maven/compiler/ClassLoaderNameEnvironment.java
@@ -23,6 +23,7 @@ import java.io.ByteArrayOutputStream;
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
+import java.net.URL;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -39,7 +40,6 @@ import org.eclipse.jdt.internal.compiler.env.NameEnvironmentAnswer;
  * @version $Rev$ $Date$
  */
 class ClassLoaderNameEnvironment implements INameEnvironment {
-    private final static char fileSeparator = System.getProperty("file.separator").charAt(0);
     
     private ClassLoader classLoader;
     private List<String> sourceLocations;
@@ -108,7 +108,7 @@ class ClassLoaderNameEnvironment implements INameEnvironment {
         if (sourceFile != null) {
             return sourceFile;
         }
-        String sourceName = className.replace('.', fileSeparator) + ".java";
+        String sourceName = className.replace('.', File.separatorChar) + ".java";
         sourceFile = sourceFileInSourceLocations(sourceName);
         sourceFiles.put(className, sourceFile);
         return sourceFile;
@@ -141,15 +141,8 @@ class ClassLoaderNameEnvironment implements INameEnvironment {
             return false;
         }
         String resourceName = '/' + name.replace('.', '/') + ".class";
-        InputStream is = classLoader.getResourceAsStream(resourceName);
-        if (is == null) {
-            return true;
-        } else {
-            try {
-                is.close();
-            } catch (IOException e) {}
-            return false;
-        }
+        URL url = classLoader.getResource(resourceName);
+        return url == null;
     }
 
     /**
@@ -172,7 +165,7 @@ class ClassLoaderNameEnvironment implements INameEnvironment {
                 return null;
             }
             ByteArrayOutputStream bos = new ByteArrayOutputStream();
-            byte[] b = new byte[2048];
+            byte[] b = new byte[4096];
             for (;;) {
                 int n = is.read(b);
                 if (n <= 0) {
@@ -180,6 +173,7 @@ class ClassLoaderNameEnvironment implements INameEnvironment {
                 }
                 bos.write(b, 0, n);
             }
+            is.close();
             byte[] classBytes = bos.toByteArray();
             
             ClassFileReader classFileReader = new ClassFileReader(classBytes, className.toCharArray(), true);
diff --git a/maven-plugins/trunk/maven-eclipse-compiler/src/main/java/org/apache/tuscany/maven/compiler/osgi/BundleResolver.java b/maven-plugins/trunk/maven-eclipse-compiler/src/main/java/org/apache/tuscany/maven/compiler/osgi/BundleResolver.java
index e31c29ead4..bc52252ba1 100644
--- a/maven-plugins/trunk/maven-eclipse-compiler/src/main/java/org/apache/tuscany/maven/compiler/osgi/BundleResolver.java
+++ b/maven-plugins/trunk/maven-eclipse-compiler/src/main/java/org/apache/tuscany/maven/compiler/osgi/BundleResolver.java
@@ -32,7 +32,6 @@ import java.util.Arrays;
 import java.util.Dictionary;
 import java.util.HashMap;
 import java.util.HashSet;
-import java.util.Iterator;
 import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Map;
@@ -159,11 +158,10 @@ public class BundleResolver {
     }
 
     private Properties manifestToProperties(Attributes d) {
-        Iterator iter = d.keySet().iterator();
         Properties result = new Properties();
-        while (iter.hasNext()) {
-            Attributes.Name key = (Attributes.Name)iter.next();
-            result.put(key.toString(), d.get(key));
+        for(Map.Entry<Object, Object> e: d.entrySet()) {
+            Attributes.Name key = (Attributes.Name)e.getKey();
+            result.put(key.toString(), e.getValue());
         }
         return result;
     }
-- 
cgit v1.2.3