From 611bc678a04d23b9dc151937301a73123f814463 Mon Sep 17 00:00:00 2001
From: Russell Howe <rhowe@siksai.co.uk>
Date: Wed, 10 Aug 2022 08:10:54 +0100
Subject: [PATCH] Simplify hasClassifier

We can remove all branching from this method and return the check
directly.
---
 .../org/apache/maven/plugins/jar/AbstractJarMojo.java     | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/main/java/org/apache/maven/plugins/jar/AbstractJarMojo.java b/src/main/java/org/apache/maven/plugins/jar/AbstractJarMojo.java
index 9285e6e..d54386b 100644
--- a/src/main/java/org/apache/maven/plugins/jar/AbstractJarMojo.java
+++ b/src/main/java/org/apache/maven/plugins/jar/AbstractJarMojo.java
@@ -335,13 +335,7 @@ private boolean projectHasAlreadySetAnArtifact()
      */
     protected boolean hasClassifier()
     {
-        boolean result = false;
-        if ( getClassifier() != null && getClassifier().trim().length() > 0 )
-        {
-            result = true;
-        }
-
-        return result;
+        return getClassifier() != null && getClassifier().trim().length() > 0;
     }
 
     private String[] getIncludes()