You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by or...@apache.org on 2017/09/04 12:03:01 UTC

qpid-broker-j git commit: QPID-7892: [Java Broker] Use Pattern#quote instead of own regexp escaping

Repository: qpid-broker-j
Updated Branches:
  refs/heads/master 7d2263560 -> 11a74cabc


QPID-7892: [Java Broker] Use Pattern#quote instead of own regexp escaping


Project: http://git-wip-us.apache.org/repos/asf/qpid-broker-j/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-broker-j/commit/11a74cab
Tree: http://git-wip-us.apache.org/repos/asf/qpid-broker-j/tree/11a74cab
Diff: http://git-wip-us.apache.org/repos/asf/qpid-broker-j/diff/11a74cab

Branch: refs/heads/master
Commit: 11a74cabcc3a8d5a5d122b4c87edebdbf2a67fd8
Parents: 7d22635
Author: Alex Rudyy <or...@apache.org>
Authored: Mon Sep 4 13:02:43 2017 +0100
Committer: Alex Rudyy <or...@apache.org>
Committed: Mon Sep 4 13:02:43 2017 +0100

----------------------------------------------------------------------
 .../qpid/server/logging/logback/RollingPolicyDecorator.java  | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-broker-j/blob/11a74cab/broker-plugins/logging-logback/src/main/java/org/apache/qpid/server/logging/logback/RollingPolicyDecorator.java
----------------------------------------------------------------------
diff --git a/broker-plugins/logging-logback/src/main/java/org/apache/qpid/server/logging/logback/RollingPolicyDecorator.java b/broker-plugins/logging-logback/src/main/java/org/apache/qpid/server/logging/logback/RollingPolicyDecorator.java
index 0d20a22..2d56156 100644
--- a/broker-plugins/logging-logback/src/main/java/org/apache/qpid/server/logging/logback/RollingPolicyDecorator.java
+++ b/broker-plugins/logging-logback/src/main/java/org/apache/qpid/server/logging/logback/RollingPolicyDecorator.java
@@ -295,7 +295,6 @@ public class RollingPolicyDecorator implements RollingPolicy
 
     private static class FilePathBaseFolderAndPatternPair
     {
-        private static Pattern REGEX_SPECIAL_CHARACTERS = Pattern.compile("[{}()\\[\\].+*?^$\\\\|]");
         private final Path _baseFolder;
         private final Pattern _pattern;
 
@@ -318,12 +317,7 @@ public class RollingPolicyDecorator implements RollingPolicy
                 path = System.getProperty("user.dir");
             }
             _baseFolder = new File(path).toPath().toAbsolutePath();
-            _pattern = Pattern.compile(escape(path) + "/" + fileNamePattern);
-        }
-
-        private String escape(String string)
-        {
-            return REGEX_SPECIAL_CHARACTERS.matcher(string).replaceAll("\\\\$0");
+            _pattern = Pattern.compile(Pattern.quote(path) + "/" + fileNamePattern);
         }
 
         public Path getBaseFolder()


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org