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;