Re: [PATCH v6 7/7] t-reftable-pq: add tests for merged_iter_pqueue_top()

[Patrick Steinhardt <ps@pks.im>]

[-- Attachment #1: Type: text/plain, Size: 824 bytes --]

On Thu, Jul 25, 2024 at 02:55:59PM +0530, Chandra Pratap wrote:
> @@ -95,10 +104,51 @@ static void t_pq_index(void)
>  	merged_iter_pqueue_release(&pq);
>  }
>  
> +static void t_merged_iter_pqueue_top(void)
> +{
> +	struct merged_iter_pqueue pq = { 0 };
> +	struct reftable_record recs[13];
> +	size_t N = ARRAY_SIZE(recs), i;
> +
> +	for (i = 0; i < N; i++) {
> +		reftable_record_init(&recs[i], BLOCK_TYPE_REF);
> +		recs[i].u.ref.refname = (char *) "refs/heads/master";
> +	}
> +
> +	i = 1;
> +	do {
> +		struct pq_entry e = {
> +			.rec = &recs[i],
> +			.index = i,
> +		};
> +
> +		merged_iter_pqueue_add(&pq, &e);
> +		merged_iter_pqueue_check(&pq);
> +		i = (i * 7) % N;
> +	}
> +	while (i != 1);

Same nit here. Other than that this patch series looks good to me,
thanks!

Patrick

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]