You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by he...@apache.org on 2011/05/23 11:59:52 UTC

svn commit: r1126413 - in /commons/proper/jexl/trunk: ./ pom.xml src/main/java/org/apache/commons/jexl2/internal/introspection/IntrospectorBase.java

Author: henrib
Date: Mon May 23 09:59:51 2011
New Revision: 1126413

URL: http://svn.apache.org/viewvc?rev=1126413&view=rev
Log:
Reverted modification in IntrospectorBase (previous commit used 1.6 feature, JEXL needs 1.5);
Updated POM to use commons parent 21;
Added .idea to .ignore

Modified:
    commons/proper/jexl/trunk/   (props changed)
    commons/proper/jexl/trunk/pom.xml
    commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl2/internal/introspection/IntrospectorBase.java

Propchange: commons/proper/jexl/trunk/
------------------------------------------------------------------------------
--- svn:ignore (original)
+++ svn:ignore Mon May 23 09:59:51 2011
@@ -14,3 +14,4 @@ bin
 *.iml
 jcoverage.ser
 nb-configuration.xml
+.idea

Modified: commons/proper/jexl/trunk/pom.xml
URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/pom.xml?rev=1126413&r1=1126412&r2=1126413&view=diff
==============================================================================
--- commons/proper/jexl/trunk/pom.xml (original)
+++ commons/proper/jexl/trunk/pom.xml Mon May 23 09:59:51 2011
@@ -19,7 +19,7 @@
     <parent>
         <groupId>org.apache.commons</groupId>
         <artifactId>commons-parent</artifactId>
-        <version>20</version>
+        <version>21</version>
     </parent>
     <modelVersion>4.0.0</modelVersion>
     <groupId>org.apache.commons</groupId>
@@ -154,7 +154,7 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-compiler-plugin</artifactId>
-             </plugin>
+            </plugin>
             <!-- workaround MRELEASE-424 when publishing -->
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>

Modified: commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl2/internal/introspection/IntrospectorBase.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl2/internal/introspection/IntrospectorBase.java?rev=1126413&r1=1126412&r2=1126413&view=diff
==============================================================================
--- commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl2/internal/introspection/IntrospectorBase.java (original)
+++ commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl2/internal/introspection/IntrospectorBase.java Mon May 23 09:59:51 2011
@@ -253,7 +253,10 @@ public class IntrospectorBase {
                         // add it to list of known loaded classes
                         constructibleClasses.put(cname, clazz);
                     }
-                    List<Constructor<?>> l = new LinkedList<Constructor<?>>(Arrays.asList(clazz.getConstructors()));
+                    List<Constructor<?>> l = new LinkedList<Constructor<?>>();
+                    for(Constructor<?> ictor : clazz.getConstructors()) {
+                        l.add(ictor);
+                    }
                     // try to find one
                     ctor = key.getMostSpecificConstructor(l);
                     if (ctor != null) {