You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2017/08/14 14:12:20 UTC
svn commit: r1804998 -
/sling/whiteboard/cziegeler/feature-analyser/src/main/java/org/apache/sling/feature/analyser/BundleDescriptor.java
Author: cziegeler
Date: Mon Aug 14 14:12:20 2017
New Revision: 1804998
URL: http://svn.apache.org/viewvc?rev=1804998&view=rev
Log:
Remove unnecessary null checks
Modified:
sling/whiteboard/cziegeler/feature-analyser/src/main/java/org/apache/sling/feature/analyser/BundleDescriptor.java
Modified: sling/whiteboard/cziegeler/feature-analyser/src/main/java/org/apache/sling/feature/analyser/BundleDescriptor.java
URL: http://svn.apache.org/viewvc/sling/whiteboard/cziegeler/feature-analyser/src/main/java/org/apache/sling/feature/analyser/BundleDescriptor.java?rev=1804998&r1=1804997&r2=1804998&view=diff
==============================================================================
--- sling/whiteboard/cziegeler/feature-analyser/src/main/java/org/apache/sling/feature/analyser/BundleDescriptor.java (original)
+++ sling/whiteboard/cziegeler/feature-analyser/src/main/java/org/apache/sling/feature/analyser/BundleDescriptor.java Mon Aug 14 14:12:20 2017
@@ -51,23 +51,19 @@ public abstract class BundleDescriptor e
public abstract Manifest getManifest();
public boolean isExportingPackage(final String packageName) {
- if ( getExportedPackages() != null ) {
- for(final PackageInfo i : getExportedPackages()) {
- if ( i.getName().equals(packageName) ) {
- return true;
- }
+ for(final PackageInfo i : getExportedPackages()) {
+ if ( i.getName().equals(packageName) ) {
+ return true;
}
}
return false;
}
public boolean isExportingPackage(final PackageInfo info) {
- if ( getExportedPackages() != null ) {
- for(final PackageInfo i : getExportedPackages()) {
- if ( i.getName().equals(info.getName())
- && (info.getVersion() == null || info.getPackageVersionRange().includes(i.getPackageVersion()))) {
- return true;
- }
+ for(final PackageInfo i : getExportedPackages()) {
+ if ( i.getName().equals(info.getName())
+ && (info.getVersion() == null || info.getPackageVersionRange().includes(i.getPackageVersion()))) {
+ return true;
}
}
return false;