Use early return

1 files changed, 6 insertions, 7 deletions

diff --git a/bundle-plugin/src/main/java/com/yahoo/container/plugin/mojo/GenerateOsgiManifestMojo.java b/bundle-plugin/src/main/java/com/yahoo/container/plugin/mojo/GenerateOsgiManifestMojo.java
index fdb2cae2187..7ea1b11b935 100644
--- a/bundle-plugin/src/main/java/com/yahoo/container/plugin/mojo/GenerateOsgiManifestMojo.java
+++ b/bundle-plugin/src/main/java/com/yahoo/container/plugin/mojo/GenerateOsgiManifestMojo.java
@@ -254,15 +254,14 @@ public class GenerateOsgiManifestMojo extends AbstractGenerateOsgiManifestMojo {
 
         Set<ProvidedArtifact> included = includedArtifacts.stream().map(ProvidedArtifact::new).collect(Collectors.toSet());
         Set<ProvidedArtifact> providedIncluded = Sets.intersection(included, new HashSet<>(providedArtifacts));
+        if (providedIncluded.isEmpty()) return;
 
-        if (! providedIncluded.isEmpty()) {
-            List<String> sorted = providedIncluded.stream()
-                    .map(ProvidedArtifact::stringValue)
-                    .sorted().toList();
+        List<String> sorted = providedIncluded.stream()
+                .map(ProvidedArtifact::stringValue)
+                .sorted().toList();
 
-            // TODO: improve error message
-            warnOrThrow("Artifacts provided from Jdisc runtime are included in compile scope: " + sorted);
-        }
+        // TODO: improve error message
+        warnOrThrow("Artifacts provided from Jdisc runtime are included in compile scope: " + sorted);
     }
 
     private static String trimWhitespace(Optional<String> lines) {