You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by tc...@apache.org on 2005/11/07 21:02:53 UTC

svn commit: r331592 - in /jakarta/commons/sandbox/jci/trunk: TODO project.xml src/java/org/apache/commons/jci/compilers/groovy/GroovyJavaCompiler.java src/java/org/apache/commons/jci/listeners/CompilingListener.java

Author: tcurdt
Date: Mon Nov  7 12:02:44 2005
New Revision: 331592

URL: http://svn.apache.org/viewcvs?rev=331592&view=rev
Log:
updated TODO, fixed a spelling mistake, look for the .groovy file extension

Modified:
    jakarta/commons/sandbox/jci/trunk/TODO
    jakarta/commons/sandbox/jci/trunk/project.xml
    jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/compilers/groovy/GroovyJavaCompiler.java
    jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/listeners/CompilingListener.java

Modified: jakarta/commons/sandbox/jci/trunk/TODO
URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/jci/trunk/TODO?rev=331592&r1=331591&r2=331592&view=diff
==============================================================================
--- jakarta/commons/sandbox/jci/trunk/TODO (original)
+++ jakarta/commons/sandbox/jci/trunk/TODO Mon Nov  7 12:02:44 2005
@@ -1,11 +1,14 @@
-o integrate just4log into the build system
+o integrate just4log into the build system (or a different logging solution?)
 o compiler factory
 o compiler implementations
-  o embedded java compiler
   o javac 
   o jikes
+  o pizza
+  o jruby
+  o jpython
 o make suffixes matching (*.java, *.class) configurable
 o turn the JavaCompilerFactory into an interface and maybe provide a simple default impl
 o documentation
 o dependency analysis for proper re-try after errors
 o store resources as org/apache/my.class instead of org.apache.my
+o removing of anonymous classes if parent class is being removed

Modified: jakarta/commons/sandbox/jci/trunk/project.xml
URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/jci/trunk/project.xml?rev=331592&r1=331591&r2=331592&view=diff
==============================================================================
--- jakarta/commons/sandbox/jci/trunk/project.xml (original)
+++ jakarta/commons/sandbox/jci/trunk/project.xml Mon Nov  7 12:02:44 2005
@@ -68,13 +68,13 @@
     </developer>
   </developers>
 
-  <contributers>
-    <contributer>
+  <contributors>
+    <contributor>
       <name>Joerk Heinicke</name>
       <id>jheinicke</id>
       <email>joerg.heinicke at gmx.de</email>
-    </contributer>
-  </contributers>
+    </contributor>
+  </contributors>
 
   <dependencies>
 

Modified: jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/compilers/groovy/GroovyJavaCompiler.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/compilers/groovy/GroovyJavaCompiler.java?rev=331592&r1=331591&r2=331592&view=diff
==============================================================================
--- jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/compilers/groovy/GroovyJavaCompiler.java (original)
+++ jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/compilers/groovy/GroovyJavaCompiler.java Mon Nov  7 12:02:44 2005
@@ -38,7 +38,7 @@
         final CompilationUnit unit = new CompilationUnit(configuration);
         final SourceUnit[] source = new SourceUnit[clazzNames.length];
         for (int i = 0; i < source.length; i++) {
-            final String filename = clazzNames[i].replace('.','/') + ".java";
+            final String filename = clazzNames[i].replace('.','/') + ".groovy";
             log.debug("adding source unit " + filename);
             source[i] = new SourceUnit(
                     filename,

Modified: jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/listeners/CompilingListener.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/listeners/CompilingListener.java?rev=331592&r1=331591&r2=331592&view=diff
==============================================================================
--- jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/listeners/CompilingListener.java (original)
+++ jakarta/commons/sandbox/jci/trunk/src/java/org/apache/commons/jci/listeners/CompilingListener.java Mon Nov  7 12:02:44 2005
@@ -87,6 +87,7 @@
         if (deleted.size() > 0) {
             for (Iterator it = deleted.iterator(); it.hasNext();) {
                 final File file = (File) it.next();
+                // FIXME: does not remove nested classes
                 transactionalStore.remove(ReloadingClassLoader.clazzName(repository, file));
             }
             reload = true;



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org