From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 26 Feb 2026 19:56:41 GMT
Subject: Re: [PATCH v2 3/5] builtin/repo: add OID annotations to table output
Message-ID: <xmqqpl5r1dom.fsf@gitster.g>
In-Reply-To: <20260223174120.2356504-4-jltobler@gmail.com>

Justin Tobler <jltobler@gmail.com> writes:

> +	if (table->annotations.nr)
> +		printf("\n");
> +	for_each_string_list_item(item, &table->annotations)
> +		printf("%s\n", item->string);
> +

It is minor, but I suspect

	if (table->annotations.nr) {
		printf("\n");
		for_each_string_list_item(...)
			printf("%s\n", item->string);
	}

would be easier to reason about.