| commit | 8c14319c84b9f885bf7a9319cfabeb1056c80434 | [log] [tgz] |
|---|---|---|
| author | Torsten Curdt <tcurdt@apache.org> | Fri May 13 08:29:28 2011 +0000 |
| committer | Torsten Curdt <tcurdt@apache.org> | Fri May 13 08:29:28 2011 +0000 |
| tree | cb912062c7015684e0eecf9ac02fce1b7fbb99d4 | |
| parent | 451babb0ceaf4e68a0a3d67ca61b5ed1501c99c8 [diff] |
skip NA tests, fixed resource names, don't leak input stream git-svn-id: https://svn.apache.org/repos/asf/commons/proper/jci/trunk@1102604 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/compilers/eclipse/src/main/java/org/apache/commons/jci/compilers/EclipseJavaCompiler.java b/compilers/eclipse/src/main/java/org/apache/commons/jci/compilers/EclipseJavaCompiler.java index 7204eec..81764d5 100644 --- a/compilers/eclipse/src/main/java/org/apache/commons/jci/compilers/EclipseJavaCompiler.java +++ b/compilers/eclipse/src/main/java/org/apache/commons/jci/compilers/EclipseJavaCompiler.java
@@ -236,7 +236,7 @@ final String resourceName = ConversionUtils.convertClassToResourcePath(pClazzName); - final byte[] clazzBytes = pStore.read(pClazzName); + final byte[] clazzBytes = pStore.read(resourceName); if (clazzBytes != null) { log.debug("loading from store " + pClazzName); @@ -294,6 +294,11 @@ final InputStream is = pClassLoader.getResourceAsStream(ConversionUtils.convertClassToResourcePath(pClazzName)); if (is != null) { log.debug("found the class for " + pClazzName + "- no package"); + try { + is.close(); + } catch (final IOException ie) { + log.error("could not close input stream", ie); + } return false; }
diff --git a/compilers/rhino/src/test/java/org/apache/commons/jci/compilers/RhinoJavaCompilerTestCase.java b/compilers/rhino/src/test/java/org/apache/commons/jci/compilers/RhinoJavaCompilerTestCase.java index fd64b05..c21cf47 100644 --- a/compilers/rhino/src/test/java/org/apache/commons/jci/compilers/RhinoJavaCompilerTestCase.java +++ b/compilers/rhino/src/test/java/org/apache/commons/jci/compilers/RhinoJavaCompilerTestCase.java
@@ -118,8 +118,11 @@ } public void testCrossReferenceCompilation() throws Exception { - + // NA } + public void testAdditionalTopLevelClassCompile() throws Exception { + // NA + } }