From: Derrick Stolee <stolee@gmail.com>
Date: Fri, 22 Jun 2018 00:17:37 GMT
Subject: Re: [PATCH 2/5] commit-graph: add missing forward declaration
Message-ID: <316b6a43-a5d0-d10f-f995-94b3165ac325@gmail.com>
In-Reply-To: <20180621223942.81451-1-jonathantanmy@google.com>

On 6/21/2018 6:39 PM, Jonathan Tan wrote:
>> Both this and the previous patch look good to me;
>> you seem to have better (stricter) checking for
>> missing includes/forward declarations, am I missing
>> a compile option? (I have DEVELOPER=1 in config.mak)
> Thanks. No I don't - I discovered that these were missing as I was
> including these header files in other files.

I discovered similar missing headers in object-store.h while reworking 
the MIDX patch series. Thanks for fixing them here!