This is an automated email from the ASF dual-hosted git repository. mgrigorov pushed a commit to branch wicket-8.x in repository https://gitbox.apache.org/repos/asf/wicket.git
commit 532cedc7f6b135b1faba7251475a0e82307cf92b Author: Thomas Heigl <[email protected]> AuthorDate: Tue Sep 29 17:20:19 2020 +0200 WICKET-6835 Improve performance of `AbstractMapper.getPlaceholder` (cherry picked from commit 3e29178981b1d42cd75fdeccdba620b8ce89e37f) --- .../main/java/org/apache/wicket/request/mapper/AbstractMapper.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/wicket-request/src/main/java/org/apache/wicket/request/mapper/AbstractMapper.java b/wicket-request/src/main/java/org/apache/wicket/request/mapper/AbstractMapper.java index 1fdf955..14408c0 100644 --- a/wicket-request/src/main/java/org/apache/wicket/request/mapper/AbstractMapper.java +++ b/wicket-request/src/main/java/org/apache/wicket/request/mapper/AbstractMapper.java @@ -68,7 +68,11 @@ public abstract class AbstractMapper implements IRequestMapper */ protected String getPlaceholder(final String s, char startChar) { - if ((s == null) || (s.length() < 4) || !s.startsWith(startChar + "{") || !s.endsWith("}")) + if (s == null || s.length() < 4) + { + return null; + } + else if (s.charAt(0) != startChar || s.charAt(1) != '{' || s.charAt(s.length() - 1) != '}') { return null; }
