From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 26 Jan 2018 20:47:31 GMT
Subject: Re: [PATCH 05/14] packed-graph: implement construct_graph()
Message-ID: <xmqqbmhg73fg.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <CAGZ79kaNj-+oBNmbTKtOE8P-ZFcBVZSaBtwzrB4RQTWnYXcoqA@mail.gmail.com>

Stefan Beller <sbeller@google.com> writes:

> On Thu, Jan 25, 2018 at 6:02 AM, Derrick Stolee <stolee@gmail.com> 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