Add support for remove subrange

backtrace-labs · Jan 22, 2024 · b6adad3 · b6adad3
1 parent 1e9f7af
commit b6adad3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/Sources/Features/Breadcrumb/BacktraceBreadcrumbFile.swift b/Sources/Features/Breadcrumb/BacktraceBreadcrumbFile.swift
@@ -53,10 +53,7 @@ enum BacktraceBreadcrumbFileError: Error {
                 let breadcrumbSize = queueBreadcrumb.size
                 // Pop last element if size is greater than maxQueueFileSizeBytes
                 if size + breadcrumbSize > maxQueueFileSizeBytes && !queue.isEmpty {
-                    for breadcrumbsToRemove in (0...index).reversed() {
-                       _ = queue.pop(at: breadcrumbsToRemove)
-                    }
-
+                    queue.removeSubrange(range: (0...index))
                     break
                 }
                 let breadcrumbJsonData = queueBreadcrumb.json

diff --git a/Sources/Features/Breadcrumb/Queue.swift b/Sources/Features/Breadcrumb/Queue.swift
@@ -20,6 +20,10 @@ public class Queue<T>: NSObject {
         return elements.first
     }
 
+    func removeSubrange(range: ClosedRange<Int>) {
+        elements.removeSubrange(range);
+    }
+
     func remove(at index: Int) -> T? {
         guard index < elements.count else {
             return nil
@@ -28,7 +32,6 @@ public class Queue<T>: NSObject {
         if (index < 0) {
             return nil
         }
-
         return elements.remove(at: index)
     }