Repository: karaf Updated Branches: refs/heads/karaf-3.0.x b2d3d0d52 -> 65e1457f7
[KARAF-2749] Use Matcher.matches() instead of find() in the BundleSelector Project: http://git-wip-us.apache.org/repos/asf/karaf/repo Commit: http://git-wip-us.apache.org/repos/asf/karaf/commit/65e1457f Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/65e1457f Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/65e1457f Branch: refs/heads/karaf-3.0.x Commit: 65e1457f7222f28a209099b0bc2fb01636248523 Parents: b2d3d0d Author: Jean-Baptiste Onofré <[email protected]> Authored: Thu Jul 10 21:24:49 2014 +0200 Committer: Jean-Baptiste Onofré <[email protected]> Committed: Thu Jul 10 21:24:49 2014 +0200 ---------------------------------------------------------------------- .../karaf/bundle/core/internal/BundleSelectorImpl.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/karaf/blob/65e1457f/bundle/core/src/main/java/org/apache/karaf/bundle/core/internal/BundleSelectorImpl.java ---------------------------------------------------------------------- diff --git a/bundle/core/src/main/java/org/apache/karaf/bundle/core/internal/BundleSelectorImpl.java b/bundle/core/src/main/java/org/apache/karaf/bundle/core/internal/BundleSelectorImpl.java index 5806f39..82abd01 100644 --- a/bundle/core/src/main/java/org/apache/karaf/bundle/core/internal/BundleSelectorImpl.java +++ b/bundle/core/src/main/java/org/apache/karaf/bundle/core/internal/BundleSelectorImpl.java @@ -54,7 +54,7 @@ public class BundleSelectorImpl { Pattern pattern = Pattern.compile("^\\d+$"); Matcher matcher = pattern.matcher(id); - if (matcher.find()) { + if (matcher.matches()) { Bundle bundle = this.getBundleById(id); addBundle(bundle, id, bundles); return; @@ -63,7 +63,7 @@ public class BundleSelectorImpl { // id as a number range pattern = Pattern.compile("^(\\d+)-(\\d+)$"); matcher = pattern.matcher(id); - if (matcher.find()) { + if (matcher.matches()) { int index = id.indexOf('-'); long startId = Long.parseLong(id.substring(0, index)); long endId = Long.parseLong(id.substring(index + 1)); @@ -159,17 +159,17 @@ public class BundleSelectorImpl { if (version != null) { String bundleVersion = (String) bundles[i].getHeaders().get(Constants.BUNDLE_VERSION); if (bundleVersion != null) { - boolean nameMatch = (nameMatcher != null && nameMatcher.find()) || symbolicNameMatcher.find(); + boolean nameMatch = (nameMatcher != null && nameMatcher.matches()) || symbolicNameMatcher.matches(); if (nameMatch) { Pattern versionPattern = Pattern.compile(version); Matcher versionMatcher = versionPattern.matcher(bundleVersion); - if (versionMatcher.find()) { + if (versionMatcher.matches()) { result.add(bundles[i]); } } } } else { - boolean nameMatch = (nameMatcher != null && nameMatcher.find()) || symbolicNameMatcher.find(); + boolean nameMatch = (nameMatcher != null && nameMatcher.matches()) || symbolicNameMatcher.matches(); if (nameMatch) { result.add(bundles[i]); }
