Subject: Re: [PATCH 05/14] packed-graph: implement construct_graph()

From: Junio C Hamano <[email-masked]>
Date: Fri, 26 Jan 2018 12:47:31 -0800

Stefan Beller <[email-masked]> writes:

> On Thu, Jan 25, 2018 at 6:02 AM, Derrick Stolee <[email-masked]> wrote:
>
>> +struct object_id *construct_graph(const char *pack_dir)
>> +{
>> +       // Find a list of oids, adding the pointer to a list.
>
> Comment style.
> Here is where the bike shedding begins. ;)

I also saw some violations of the usual formatting conventions, e.g.

	if (condition) {
		block

should not be

	if (condition)
	{
		block