refactor: guard log statements

cleydyr · Mar 31, 2024 · fd24076 · fd24076
1 parent f80c6ec
commit fd24076
Showing 1 changed file with 15 additions and 5 deletions.
diff --git a/src/main/java/com/github/cleydyr/dart/net/NetworkingGithubLatestVersionProvider.java b/src/main/java/com/github/cleydyr/dart/net/NetworkingGithubLatestVersionProvider.java
@@ -55,27 +55,37 @@ public void onError(GitHubConnectorResponse ghcr) throws IOException {
             PagedIterable<GHRelease> ghReleases = repository.listReleases();
 
             for (GHRelease ghRelease : ghReleases) {
-                logger.debug("Checking release " + ghRelease.getName());
+                if (logger.isDebugEnabled()) {
+                    logger.debug("Checking release " + ghRelease.getName());
+                }
 
                 String version = ghRelease.getTagName();
 
                 DartSassReleaseParameter dartSassReleaseParameter = new DartSassReleaseParameter(os, arch, version);
 
                 for (GHAsset ghAsset : ghRelease.listAssets()) {
-                    logger.debug("Checking asset " + ghAsset.getName());
+                    if (logger.isDebugEnabled()) {
+                        logger.debug("Checking asset " + ghAsset.getName());
+                    }
 
                     if (ghAsset.getName().equals(dartSassReleaseParameter.getArtifactName())) {
                         return version;
                     }
                 }
 
-                logger.info("Skipping version " + version + " because it doesn't have a matching asset");
+                if (logger.isInfoEnabled()) {
+                    logger.info("Skipping version " + version + " because it doesn't have a matching asset");
+                }
             }
         } catch (IOException e) {
-            logger.warn("Error while getting latest version from GitHub API", e);
+            if (logger.isWarnEnabled()) {
+                logger.warn("Error while getting latest version from GitHub API", e);
+            }
         }
 
-        logger.warn("Falling back to latest known release (" + fallbackVersionProvider.get(os, arch) + ")");
+        if (logger.isWarnEnabled()) {
+            logger.warn("Falling back to latest known release (" + fallbackVersionProvider.get(os, arch) + ")");
+        }
 
         return fallbackVersionProvider.get(os, arch);
     }