From: Stefan Beller Date: Thu, 21 Jun 2018 21:43:52 GMT Subject: Re: [PATCH 2/5] commit-graph: add missing forward declaration Message-ID: In-Reply-To: <82c9edb9108f49468031bfd7c65a9eb8db2cb671.1529616356.git.jonathantanmy@google.com> Hi Jonathan, On Thu, Jun 21, 2018 at 2:29 PM Jonathan Tan wrote: > > Signed-off-by: Jonathan Tan 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)