You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by ra...@apache.org on 2005/08/27 04:40:50 UTC
svn commit: r240378 -
/jakarta/commons/sandbox/scxml/trunk/src/main/java/org/apache/commons/scxml/env/ELEvaluator.java
Author: rahul
Date: Fri Aug 26 19:40:47 2005
New Revision: 240378
URL: http://svn.apache.org/viewcvs?rev=240378&view=rev
Log:
Found by FindBugs:
SIC: Should org.apache.commons.scxml.env.ELEvaluator$CtxWrapper be a _static_ inner class?
SIC: Should org.apache.commons.scxml.env.ELEvaluator$FunctWrapper be a _static_ inner class?
Yes to both.
Modified:
jakarta/commons/sandbox/scxml/trunk/src/main/java/org/apache/commons/scxml/env/ELEvaluator.java
Modified: jakarta/commons/sandbox/scxml/trunk/src/main/java/org/apache/commons/scxml/env/ELEvaluator.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/scxml/trunk/src/main/java/org/apache/commons/scxml/env/ELEvaluator.java?rev=240378&r1=240377&r2=240378&view=diff
==============================================================================
--- jakarta/commons/sandbox/scxml/trunk/src/main/java/org/apache/commons/scxml/env/ELEvaluator.java (original)
+++ jakarta/commons/sandbox/scxml/trunk/src/main/java/org/apache/commons/scxml/env/ELEvaluator.java Fri Aug 26 19:40:47 2005
@@ -133,7 +133,7 @@
/**
* A Context wrapper that implements VariableResolver.
*/
- class CtxWrapper implements VariableResolver {
+ static class CtxWrapper implements VariableResolver {
/** Context to be wrapped. */
private Context ctx = null;
/**
@@ -156,7 +156,7 @@
/**
* A simple function mapper for SCXML defined functions.
*/
- class FunctWrapper implements FunctionMapper {
+ static class FunctWrapper implements FunctionMapper {
/**
* @see FunctionMapper#resolveFunction(String, String)
---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org