Merge branch '1980-standardise-arrays' of https://github.com/sopa301/…

…RepoSense into 1980-standardise-arrays
reposense · Jan 20, 2024 · 1fc42aa · 1fc42aa
2 parents 26fa03f + 3a1d388
commit 1fc42aa
Showing 1 changed file with 9 additions and 10 deletions.
diff --git a/src/main/java/reposense/parser/RepoConfigCsvParser.java b/src/main/java/reposense/parser/RepoConfigCsvParser.java
@@ -114,20 +114,19 @@ protected void processLine(List<RepoConfiguration> results, CSVRecord record) th
 
         // If file diff limit is specified
         if (fileSizeLimitStringList.size() > 0) {
+            String fileSizeLimitString = fileSizeLimitStringList.get(0).trim();
+            int parseValue;
+
             if (isFileSizeLimitIgnored) {
                 logger.warning("Ignoring file size limit column since file size limit is ignored");
                 isFileSizeLimitOverriding = false;
+            } else if (!StringsUtil.isNumeric(fileSizeLimitString)
+                    || (parseValue = Integer.parseInt(fileSizeLimitString)) <= 0) {
+                logger.warning(String.format("Values in \"%s\" column should be positive integers.",
+                        FILESIZE_LIMIT_HEADER[0]));
+                isFileSizeLimitOverriding = false;
             } else {
-                String fileSizeLimitString = fileSizeLimitStringList.get(0).trim();
-                int parseValue;
-                if (!StringsUtil.isNumeric(fileSizeLimitString)
-                        || (parseValue = Integer.parseInt(fileSizeLimitString)) <= 0) {
-                    logger.warning(String.format("Values in \"%s\" column should be positive integers.",
-                            FILESIZE_LIMIT_HEADER[0]));
-                    isFileSizeLimitOverriding = false;
-                } else {
-                    fileSizeLimit = parseValue;
-                }
+                fileSizeLimit = parseValue;
             }
         }