From: Derrick Stolee <stolee@gmail.com>
Date: Fri, 14 Aug 2020 12:24:38 GMT
Subject: Re: [PATCH v2 07/10] commit-graph: implement corrected commit date
Message-ID: <2c7ee14e-1c80-9860-7bcc-633ac43910a6@gmail.com>
In-Reply-To: <20200814045957.GA1380@Abhishek-Arch>

On 8/14/2020 12:59 AM, Abhishek Kumar wrote:
> I solved the issue by adding a new member to struct commit_graph
> `read_generation_data` to maintain the "global" state of the entire
> commit-graph chain instead.
> 
> The relevant changes are in validate_mixed_generation_chain(),
> read_commit_graph_one() and fill_commit_graph_info().

I think this is a good way to go. Adding that restriction about
the tip commit-graph was short-sighted of me and was likely to
break in the future.

I think your solution here to store extra state from the entire
chain into each layer makes a lot of sense.

Thanks!
-Stolee