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()) {