Stefan Beller <sbeller@xxxxxxxxxx> writes: > On Thu, Jan 25, 2018 at 6:02 AM, Derrick Stolee <stolee@xxxxxxxxx> 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