[PATCH v3 01/13] reftable/pq: use `size_t` to track iterator index

From: Patrick Steinhardt <ps@pks.im>
Date: Mar 4, 2024, 10:48 UTC
Message-ID: <c998039333739d07ed2c68c5bbedd6129efbd1fd.1709548907.git.ps@pks.im>
In-Reply-To: <cover.1709548907.git.ps@pks.im>

The reftable priority queue is used by the merged iterator to yield records from its sub-iterators in the expected order. Each entry has a record corresponding to such a sub-iterator as well as an index that indicates which sub-iterator the record belongs to. But while the sub-iterators are tracked with a `size_t`, we store the index as an `int` in the entry.

Fix this and use `size_t` consistently.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
---
 reftable/pq.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/reftable/pq.h b/reftable/pq.h
index e85bac9b52..9e25a43a36 100644
--- a/reftable/pq.h
+++ b/reftable/pq.h
@@ -12,7 +12,7 @@ license that can be found in the LICENSE file or at
 #include "record.h"
 
 struct pq_entry {
-	int index;
+	size_t index;
 	struct reftable_record rec;
 };

-- 
2.44.0