You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by fm...@apache.org on 2008/02/15 11:44:08 UTC

svn commit: r628003 - /incubator/sling/trunk/scripting/javascript/src/main/java/org/apache/sling/scripting/javascript/RhinoJavaScriptEngine.java

Author: fmeschbe
Date: Fri Feb 15 02:44:07 2008
New Revision: 628003

URL: http://svn.apache.org/viewvc?rev=628003&view=rev
Log:
eval() method must return the result of the evaluation

Modified:
    incubator/sling/trunk/scripting/javascript/src/main/java/org/apache/sling/scripting/javascript/RhinoJavaScriptEngine.java

Modified: incubator/sling/trunk/scripting/javascript/src/main/java/org/apache/sling/scripting/javascript/RhinoJavaScriptEngine.java
URL: http://svn.apache.org/viewvc/incubator/sling/trunk/scripting/javascript/src/main/java/org/apache/sling/scripting/javascript/RhinoJavaScriptEngine.java?rev=628003&r1=628002&r2=628003&view=diff
==============================================================================
--- incubator/sling/trunk/scripting/javascript/src/main/java/org/apache/sling/scripting/javascript/RhinoJavaScriptEngine.java (original)
+++ incubator/sling/trunk/scripting/javascript/src/main/java/org/apache/sling/scripting/javascript/RhinoJavaScriptEngine.java Fri Feb 15 02:44:07 2008
@@ -91,7 +91,7 @@
             final int lineNumber = 1;
             final Object securityDomain = null;
 
-            rhinoContext.evaluateReader(scope, scriptReader, scriptName,
+            return rhinoContext.evaluateReader(scope, scriptReader, scriptName,
                 lineNumber, securityDomain);
         } catch (Throwable t) {
             final ScriptException se = new ScriptException("Failure running script " + scriptName
@@ -101,7 +101,6 @@
         } finally {
             Context.exit();
         }
-        return null;
     }
 
 }