[jboss-svn-commits] JBL Code SVN: r13738 - labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/mvel.

[jboss-svn-commits at lists.jboss.org]

Author: pombredanne
Date: 2007-07-23 18:24:06 -0400 (Mon, 23 Jul 2007)
New Revision: 13738

Modified:
   labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/mvel/MVELDialect.java
Log:
Added FIXME comment on potential NPE (caught by the caller though)

Modified: labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/mvel/MVELDialect.java
===================================================================
--- labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/mvel/MVELDialect.java	2007-07-23 21:32:05 UTC (rev 13737)
+++ labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/mvel/MVELDialect.java	2007-07-23 22:24:06 UTC (rev 13738)
@@ -348,7 +348,7 @@
         if ( interceptors != null ) {
             parserContext.setInterceptors( interceptors );
         }
-
+        //FIXME: analysis can be null, throws an NPE
         List list[] = analysis.getBoundIdentifiers();
         DeclarationScopeResolver resolver = context.getDeclarationResolver();
         for ( Iterator it = list[0].iterator(); it.hasNext(); ) {