You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bval.apache.org by mb...@apache.org on 2019/02/16 19:54:09 UTC
[bval] branch master updated: add comment to clarify the purpose of
the previous change
This is an automated email from the ASF dual-hosted git repository.
mbenson pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bval.git
The following commit(s) were added to refs/heads/master by this push:
new 28a176f add comment to clarify the purpose of the previous change
28a176f is described below
commit 28a176f09b4a67e9e6c98c9cab46ff99a635dbae
Author: Matt Benson <mb...@apache.org>
AuthorDate: Sat Feb 16 13:54:05 2019 -0600
add comment to clarify the purpose of the previous change
---
bval-jsr/src/main/java/org/apache/bval/el/ELFacade.java | 1 +
1 file changed, 1 insertion(+)
diff --git a/bval-jsr/src/main/java/org/apache/bval/el/ELFacade.java b/bval-jsr/src/main/java/org/apache/bval/el/ELFacade.java
index 40797de..e1c96c1 100644
--- a/bval-jsr/src/main/java/org/apache/bval/el/ELFacade.java
+++ b/bval-jsr/src/main/java/org/apache/bval/el/ELFacade.java
@@ -61,6 +61,7 @@ public final class ELFacade implements MessageEvaluator {
@Override
public String interpolate(final String message, final Map<String, Object> annotationParameters,
final Object validatedValue) {
+ // BVAL-170: simple pre-check to improve performance
if (message.contains("${") || message.contains("#{")) {
try {
if (EvaluationType.IMMEDIATE.regex.matcher(message).find()) {