fix: fix ArrayIndexOutOfBoundsException issue when need-remove-commit…

…-id count larger than 20
lingochamp · Oct 18, 2018 · 9b6e587 · 9b6e587
1 parent 1b69f5d
commit 9b6e587
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/main/groovy/com/liulishuo/okcheck/util/ChangeFile.groovy b/src/main/groovy/com/liulishuo/okcheck/util/ChangeFile.groovy
@@ -187,8 +187,10 @@ class ChangeFile {
 
         backupFile.createNewFile()
 
-        if (backupCommitIdList.size() + newCommitIds.size() >= 20) {
-            int needRemoveSize = backupCommitIdList.size() + newCommitIds.size() - 20
+        int maxRemainCount = Math.max(30, newCommitIds.size())
+
+        if (backupCommitIdList.size() + newCommitIds.size() >= maxRemainCount) {
+            int needRemoveSize = backupCommitIdList.size() + newCommitIds.size() - maxRemainCount
 
             Util.printLog("will delete $needRemoveSize from origin count ${backupCommitIdList.size()} because count of new insert ${newCommitIds.size()}")