Skip to content

Commit 659ac56

Browse files
authored
Cleanup constructors in ExpressionEvaluatingMessageProcessor
The `IllegalArgumentException` is really not thrown in some of those ctors. Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
1 parent d8e2c89 commit 659ac56

1 file changed

Lines changed: 4 additions & 14 deletions

File tree

spring-integration-core/src/main/java/org/springframework/integration/handler/ExpressionEvaluatingMessageProcessor.java

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
* @author Mark Fisher
3333
* @author Artem Bilan
3434
* @author Gary Russell
35+
* @author Jiandong Ma
3536
*
3637
* @since 2.0
3738
*/
@@ -57,13 +58,8 @@ public ExpressionEvaluatingMessageProcessor(Expression expression) {
5758
*/
5859
public ExpressionEvaluatingMessageProcessor(Expression expression, @Nullable Class<T> expectedType) {
5960
Assert.notNull(expression, "The expression must not be null");
60-
try {
61-
this.expression = expression;
62-
this.expectedType = expectedType;
63-
}
64-
catch (ParseException e) {
65-
throw new IllegalArgumentException("Failed to parse expression.", e);
66-
}
61+
this.expression = expression;
62+
this.expectedType = expectedType;
6763
}
6864

6965
/**
@@ -72,13 +68,7 @@ public ExpressionEvaluatingMessageProcessor(Expression expression, @Nullable Cla
7268
* @since 5.0
7369
*/
7470
public ExpressionEvaluatingMessageProcessor(String expression) {
75-
try {
76-
this.expression = EXPRESSION_PARSER.parseExpression(expression);
77-
this.expectedType = null;
78-
}
79-
catch (ParseException e) {
80-
throw new IllegalArgumentException("Failed to parse expression.", e);
81-
}
71+
this(expression, null);
8272
}
8373

8474
/**

0 commit comments

Comments
 (0)