fixing TODOs
This commit is contained in:
parent
f82561ba53
commit
a23311c4ba
|
|
@ -15,7 +15,7 @@
|
||||||
*/
|
*/
|
||||||
package org.springframework.expression;
|
package org.springframework.expression;
|
||||||
|
|
||||||
// TODO (asc) Is the resolver/executor model too pervasive in this package?
|
// TODO Is the resolver/executor model too pervasive in this package?
|
||||||
/**
|
/**
|
||||||
* Executors are built by resolvers and can be cached by the infrastructure to repeat an operation quickly without going
|
* Executors are built by resolvers and can be cached by the infrastructure to repeat an operation quickly without going
|
||||||
* back to the resolvers. For example, the particular constructor to run on a class may be discovered by the reflection
|
* back to the resolvers. For example, the particular constructor to run on a class may be discovered by the reflection
|
||||||
|
|
|
||||||
|
|
@ -58,7 +58,7 @@ public interface EvaluationContext {
|
||||||
*/
|
*/
|
||||||
void setVariable(String name, Object value);
|
void setVariable(String name, Object value);
|
||||||
|
|
||||||
// TODO (asc) lookupReference() - is it too expensive to return all objects within a context?
|
// TODO lookupReference() - is it too expensive to return all objects within a context?
|
||||||
/**
|
/**
|
||||||
* Look up an object reference in a particular context. If no contextName is specified (null), assume the default
|
* Look up an object reference in a particular context. If no contextName is specified (null), assume the default
|
||||||
* context. If no objectName is specified (null), return all objects in the specified context (List<Object>).
|
* context. If no objectName is specified (null), return all objects in the specified context (List<Object>).
|
||||||
|
|
|
||||||
|
|
@ -19,15 +19,16 @@ import org.springframework.expression.spel.standard.StandardIndividualTypeConver
|
||||||
import org.springframework.expression.spel.standard.StandardTypeConverter;
|
import org.springframework.expression.spel.standard.StandardTypeConverter;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* A type converter can convert values between different types. There is a default implementation called {@link StandardTypeConverter}
|
* A type converter can convert values between different types. There is a default implementation called
|
||||||
* that supports some basic conversions. That default implementation can be extended through subclassing or
|
* {@link StandardTypeConverter} that supports some basic conversions. That default implementation can be extended
|
||||||
* via registration of new {@link StandardIndividualTypeConverter} instances with the StandardTypeConverter.
|
* through subclassing or via registration of new {@link StandardIndividualTypeConverter} instances with the
|
||||||
|
* StandardTypeConverter.
|
||||||
*
|
*
|
||||||
* @author Andy Clement
|
* @author Andy Clement
|
||||||
*/
|
*/
|
||||||
public interface TypeConverter {
|
public interface TypeConverter {
|
||||||
// TODO (asc) replace this stuff with Keiths spring-binding conversion code
|
// TODO replace this stuff with Keiths spring-binding conversion code
|
||||||
// TODO (asc) should ExpressionException be thrown for lost precision in the case of coercion?
|
// TODO should ExpressionException be thrown for lost precision in the case of coercion?
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Convert (may coerce) a value from one type to another, for example from a boolean to a string.
|
* Convert (may coerce) a value from one type to another, for example from a boolean to a string.
|
||||||
|
|
@ -39,7 +40,6 @@ public interface TypeConverter {
|
||||||
*/
|
*/
|
||||||
Object convertValue(Object value, Class<?> targetType) throws EvaluationException;
|
Object convertValue(Object value, Class<?> targetType) throws EvaluationException;
|
||||||
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Return true if the type converter can convert the specified type to the desired target type.
|
* Return true if the type converter can convert the specified type to the desired target type.
|
||||||
*
|
*
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue