diff options
| author | Andreas Gampe <agampe@google.com> | 2015-03-06 15:53:06 -0800 |
|---|---|---|
| committer | Andreas Gampe <agampe@google.com> | 2015-03-07 00:19:01 +0000 |
| commit | 055678b5fe8f892ca64a384e839564b6de4d89cc (patch) | |
| tree | 816beaf7f613174b76fd596acf0624691aa19ee4 | |
| parent | e59b7ce2564a916240e561b36f4476f2d72a65c5 (diff) | |
| download | frameworks_base-055678b5fe8f892ca64a384e839564b6de4d89cc.zip frameworks_base-055678b5fe8f892ca64a384e839564b6de4d89cc.tar.gz frameworks_base-055678b5fe8f892ca64a384e839564b6de4d89cc.tar.bz2 | |
Frameworks/base: Remove unnecessary Pattern instance
Using a static Pattern in UriMatcher prevents compile-time
initialization.
It is also not efficient, as String.split has a fast path for simple
splits.
Bug: 19542228
Change-Id: Ie9e5bfe6da04c6d05ec10b1426d0cd136ef46ef2
| -rw-r--r-- | core/java/android/content/UriMatcher.java | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/core/java/android/content/UriMatcher.java b/core/java/android/content/UriMatcher.java index 8487dae..71a035e 100644 --- a/core/java/android/content/UriMatcher.java +++ b/core/java/android/content/UriMatcher.java @@ -20,7 +20,6 @@ import android.net.Uri; import java.util.ArrayList; import java.util.List; -import java.util.regex.Pattern; /** Utility class to aid in matching URIs in content providers. @@ -171,7 +170,7 @@ public class UriMatcher if (path.length() > 0 && path.charAt(0) == '/') { newPath = path.substring(1); } - tokens = PATH_SPLIT_PATTERN.split(newPath); + tokens = newPath.split("/"); } int numTokens = tokens != null ? tokens.length : 0; @@ -207,8 +206,6 @@ public class UriMatcher node.mCode = code; } - static final Pattern PATH_SPLIT_PATTERN = Pattern.compile("/"); - /** * Try to match against the path in a url. * |