You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bsf-dev@jakarta.apache.org by se...@apache.org on 2009/08/01 05:22:49 UTC
svn commit: r799793 -
/jakarta/bsf/trunk/bsf3/bsf-api/src/main/java/javax/script/SimpleScriptContext.java
Author: sebb
Date: Sat Aug 1 03:22:48 2009
New Revision: 799793
URL: http://svn.apache.org/viewvc?rev=799793&view=rev
Log:
More efficient:
- don't fetch value twice
- no need to check for null when we would just return null anyway
Modified:
jakarta/bsf/trunk/bsf3/bsf-api/src/main/java/javax/script/SimpleScriptContext.java
Modified: jakarta/bsf/trunk/bsf3/bsf-api/src/main/java/javax/script/SimpleScriptContext.java
URL: http://svn.apache.org/viewvc/jakarta/bsf/trunk/bsf3/bsf-api/src/main/java/javax/script/SimpleScriptContext.java?rev=799793&r1=799792&r2=799793&view=diff
==============================================================================
--- jakarta/bsf/trunk/bsf3/bsf-api/src/main/java/javax/script/SimpleScriptContext.java (original)
+++ jakarta/bsf/trunk/bsf3/bsf-api/src/main/java/javax/script/SimpleScriptContext.java Sat Aug 1 03:22:48 2009
@@ -96,9 +96,10 @@
checkName(name);
- if (engineScope.get(name) != null) {
- return engineScope.get(name);
- } else if (globalScope != null && globalScope.get(name) != null) {
+ final Object engineObject = engineScope.get(name);
+ if (engineObject != null) {
+ return engineObject;
+ } else if (globalScope != null) {
return globalScope.get(name);
} else {
return null;
---------------------------------------------------------------------
To unsubscribe, e-mail: bsf-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: bsf-dev-help@jakarta.apache.org