Repository: karaf Updated Branches: refs/heads/karaf-2.x 8d377fa63 -> 9d10031a8
[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/9d10031a Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/9d10031a Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/9d10031a Branch: refs/heads/karaf-2.x Commit: 9d10031a8235d1e87173986d121ab4af990f725c Parents: 8d377fa Author: Jean-Baptiste Onofré <[email protected]> Authored: Thu Jul 10 21:27:11 2014 +0200 Committer: Jean-Baptiste Onofré <[email protected]> Committed: Thu Jul 10 21:27:11 2014 +0200 ---------------------------------------------------------------------- .../mbeans/bundles/internal/BundlesSelector.java | 10 +++++----- .../java/org/apache/karaf/shell/osgi/BundleSelector.java | 10 +++++----- 2 files changed, 10 insertions(+), 10 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/karaf/blob/9d10031a/management/mbeans/bundles/src/main/java/org/apache/karaf/management/mbeans/bundles/internal/BundlesSelector.java ---------------------------------------------------------------------- diff --git a/management/mbeans/bundles/src/main/java/org/apache/karaf/management/mbeans/bundles/internal/BundlesSelector.java b/management/mbeans/bundles/src/main/java/org/apache/karaf/management/mbeans/bundles/internal/BundlesSelector.java index 91b222e..b8d5903 100644 --- a/management/mbeans/bundles/src/main/java/org/apache/karaf/management/mbeans/bundles/internal/BundlesSelector.java +++ b/management/mbeans/bundles/src/main/java/org/apache/karaf/management/mbeans/bundles/internal/BundlesSelector.java @@ -50,7 +50,7 @@ public class BundlesSelector { // bundle ID is a number Pattern pattern = Pattern.compile("^\\d+$"); Matcher matcher = pattern.matcher(bundleId); - if (matcher.find()) { + if (matcher.matches()) { Bundle bundle = this.getBundleById(bundleId); bundles.add(bundle); return bundles; @@ -59,7 +59,7 @@ public class BundlesSelector { // bundle is an ID range pattern = Pattern.compile("^(\\d+)-(\\d+)$"); matcher = pattern.matcher(bundleId); - if (matcher.find()) { + if (matcher.matches()) { int index = bundleId.indexOf('-'); long startId = Long.parseLong(bundleId.substring(0, index)); long stopId = Long.parseLong(bundleId.substring(index + 1)); @@ -147,17 +147,17 @@ public class BundlesSelector { 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]); } http://git-wip-us.apache.org/repos/asf/karaf/blob/9d10031a/shell/osgi/src/main/java/org/apache/karaf/shell/osgi/BundleSelector.java ---------------------------------------------------------------------- diff --git a/shell/osgi/src/main/java/org/apache/karaf/shell/osgi/BundleSelector.java b/shell/osgi/src/main/java/org/apache/karaf/shell/osgi/BundleSelector.java index 16852ec..d905dcd 100644 --- a/shell/osgi/src/main/java/org/apache/karaf/shell/osgi/BundleSelector.java +++ b/shell/osgi/src/main/java/org/apache/karaf/shell/osgi/BundleSelector.java @@ -46,7 +46,7 @@ public class BundleSelector { // id is a number Pattern pattern = Pattern.compile("^\\d+$"); Matcher matcher = pattern.matcher(id); - if (matcher.find()) { + if (matcher.matches()) { Bundle bundle = this.getBundleById(id); addBundle(bundle, id, force, bundles); continue; @@ -55,7 +55,7 @@ public class BundleSelector { // 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)); @@ -176,17 +176,17 @@ public class BundleSelector { 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]); }
