Code cleanup

Author: lalinsky

src/Index.zig

@@ -260,9 +260,13 @@ fn stopCheckpointThread(self: *Self) void {
 }
 
 fn updateManifestFile(self: *Self, segments: *FileSegmentList) !void {
-    const infos = try segments.getInfos(self.allocator);
+    const infos = try self.allocator.alloc(SegmentInfo, segments.nodes.items.len);
     defer self.allocator.free(infos);
 
+    for (segments.nodes.items, 0..) |node, i| {
+        infos[i] = node.value.info;
+    }
+
     try filefmt.writeManifestFile(self.dir, infos);
 }
 

src/segment_list.zig

@@ -100,14 +100,6 @@ pub fn SegmentList(Segment: type) type {
             }
         }
 
-        pub fn getInfos(self: Self, allocator: Allocator) Allocator.Error![]SegmentInfo {
-            var infos = try allocator.alloc(SegmentInfo, self.nodes.items.len);
-            for (self.nodes.items, 0..) |node, i| {
-                infos[i] = node.value.info;
-            }
-            return infos;
-        }
-
         pub fn search(self: Self, hashes: []const u32, results: *SearchResults, deadline: Deadline) !void {
             for (self.nodes.items) |node| {
                 if (deadline.isExpired()) {
@@ -118,14 +110,6 @@ pub fn SegmentList(Segment: type) type {
             results.removeOutdatedResults(self);
         }
 
-        pub fn getMaxCommitId(self: Self) u64 {
-            var max_commit_id: u64 = 0;
-            for (self.nodes.items) |node| {
-                max_commit_id = @max(max_commit_id, node.value.max_commit_id);
-            }
-            return max_commit_id;
-        }
-
         fn compareByVersion(_: void, lhs: u32, rhs: Node) bool {
             return lhs < rhs.value.id.version;
         }
@@ -146,10 +130,6 @@ pub fn SegmentList(Segment: type) type {
             return false;
         }
 
-        pub fn count(self: Self) usize {
-            return self.nodes.items.len;
-        }
-
         pub fn getFirst(self: Self) ?Node {
             return if (self.nodes.items.len > 0) self.nodes.items[0] else null;
         }