Remove Strings2.replaceAll(String, Pattern, String) Replace with direct calls to Pattern.matcher(String).replaceAll(String).
Project: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/commit/d113b0ba Tree: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/tree/d113b0ba Diff: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/diff/d113b0ba Branch: refs/heads/master Commit: d113b0ba63e1335ca4aa007036e0b0678934841f Parents: 64e9a4e Author: Andrew Gaul <[email protected]> Authored: Wed May 29 10:20:46 2013 -0700 Committer: Andrew Gaul <[email protected]> Committed: Mon Jun 3 14:41:27 2013 -0700 ---------------------------------------------------------------------- .../org/jclouds/atmos/filters/SignRequest.java | 2 +- .../filters/SharedKeyLiteAuthentication.java | 2 +- .../BindAddInternetServiceToXmlPayload.java | 4 ++-- .../binders/BindAddNodeServiceToXmlPayload.java | 2 +- core/src/main/java/org/jclouds/util/Strings2.java | 8 +------- 5 files changed, 6 insertions(+), 12 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/d113b0ba/apis/atmos/src/main/java/org/jclouds/atmos/filters/SignRequest.java ---------------------------------------------------------------------- diff --git a/apis/atmos/src/main/java/org/jclouds/atmos/filters/SignRequest.java b/apis/atmos/src/main/java/org/jclouds/atmos/filters/SignRequest.java index 7380d80..8caff86 100644 --- a/apis/atmos/src/main/java/org/jclouds/atmos/filters/SignRequest.java +++ b/apis/atmos/src/main/java/org/jclouds/atmos/filters/SignRequest.java @@ -147,7 +147,7 @@ public class SignRequest implements HttpRequestFilter { // newline characters and extra embedded white spaces in the value. for (String value : request.getHeaders().get(header)) { value = value.replace(" ", " "); - value = Strings2.replaceAll(value, NEWLINE_PATTERN, ""); + value = NEWLINE_PATTERN.matcher(value).replaceAll(""); toSign.append(value).append(' '); } toSign.deleteCharAt(toSign.lastIndexOf(" ")); http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/d113b0ba/common/azure/src/main/java/org/jclouds/azure/storage/filters/SharedKeyLiteAuthentication.java ---------------------------------------------------------------------- diff --git a/common/azure/src/main/java/org/jclouds/azure/storage/filters/SharedKeyLiteAuthentication.java b/common/azure/src/main/java/org/jclouds/azure/storage/filters/SharedKeyLiteAuthentication.java index 2c6188e..f6d61e1 100644 --- a/common/azure/src/main/java/org/jclouds/azure/storage/filters/SharedKeyLiteAuthentication.java +++ b/common/azure/src/main/java/org/jclouds/azure/storage/filters/SharedKeyLiteAuthentication.java @@ -158,7 +158,7 @@ public class SharedKeyLiteAuthentication implements HttpRequestFilter { if (header.startsWith("x-ms-")) { toSign.append(header.toLowerCase()).append(":"); for (String value : request.getHeaders().get(header)) { - toSign.append(Strings2.replaceAll(value, NEWLINE_PATTERN, "")).append(","); + toSign.append(NEWLINE_PATTERN.matcher(value).replaceAll("")).append(","); } toSign.deleteCharAt(toSign.lastIndexOf(",")); toSign.append("\n"); http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/d113b0ba/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddInternetServiceToXmlPayload.java ---------------------------------------------------------------------- diff --git a/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddInternetServiceToXmlPayload.java b/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddInternetServiceToXmlPayload.java index cffe5f2..30fc2ce 100644 --- a/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddInternetServiceToXmlPayload.java +++ b/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddInternetServiceToXmlPayload.java @@ -62,9 +62,9 @@ public class BindAddInternetServiceToXmlPayload implements MapBinder { String payload = Strings2.replaceTokens(xmlTemplate, ImmutableMap.of("name", name, "protocol", protocol, "port", port, "enabled", enabled, "ns", ns)); try { - payload = Strings2.replaceAll(payload, Patterns.TOKEN_TO_PATTERN.get("description"), description == null ? "" + payload = Patterns.TOKEN_TO_PATTERN.get("description").matcher(payload).replaceAll(description == null ? "" : String.format("\n\t<Description>%s</Description>", description)); - payload = Strings2.replaceAll(payload, Patterns.TOKEN_TO_PATTERN.get("monitor"), getMonitorString(postParams)); + payload = Patterns.TOKEN_TO_PATTERN.get("monitor").matcher(payload).replaceAll(getMonitorString(postParams)); } catch (ExecutionException e) { Throwables.propagate(e); } http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/d113b0ba/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddNodeServiceToXmlPayload.java ---------------------------------------------------------------------- diff --git a/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddNodeServiceToXmlPayload.java b/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddNodeServiceToXmlPayload.java index dd80d2d..03477d1 100644 --- a/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddNodeServiceToXmlPayload.java +++ b/common/trmk/src/main/java/org/jclouds/trmk/vcloud_0_8/binders/BindAddNodeServiceToXmlPayload.java @@ -63,7 +63,7 @@ public class BindAddNodeServiceToXmlPayload implements MapBinder { String payload = Strings2.replaceTokens(xmlTemplate, ImmutableMap.of("name", name, "ipAddress", ipAddress, "port", port, "enabled", enabled, "ns", ns)); try { - payload = Strings2.replaceAll(payload, Patterns.TOKEN_TO_PATTERN.get("description"), description == null ? "" + payload = Patterns.TOKEN_TO_PATTERN.get("description").matcher(payload).replaceAll(description == null ? "" : String.format("\n <Description>%s</Description>", description)); } catch (ExecutionException e) { Throwables.propagate(e); http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/d113b0ba/core/src/main/java/org/jclouds/util/Strings2.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/jclouds/util/Strings2.java b/core/src/main/java/org/jclouds/util/Strings2.java index 3b3b09e..8b85b53 100644 --- a/core/src/main/java/org/jclouds/util/Strings2.java +++ b/core/src/main/java/org/jclouds/util/Strings2.java @@ -113,12 +113,6 @@ public class Strings2 { } } - public static String replaceAll(String returnVal, Pattern pattern, String replace) { - Matcher m = pattern.matcher(returnVal); - returnVal = m.replaceAll(replace); - return returnVal; - } - public static String toString(InputSupplier<? extends InputStream> supplier) throws IOException { return CharStreams.toString(CharStreams.newReaderSupplier(supplier, @@ -173,7 +167,7 @@ public class Strings2 { public static String replaceTokens(String input, Multimap<String, ?> tokenValues) { for (Entry<String, ?> tokenValue : tokenValues.entries()) { Pattern pattern = TOKEN_TO_PATTERN.getUnchecked(tokenValue.getKey()); - input = replaceAll(input, pattern, tokenValue.getValue().toString()); + input = pattern.matcher(input).replaceAll(tokenValue.getValue().toString()); } return input; }
