On Wed, Mar 02, 2022 at 03:19:57PM -0500, Derrick Stolee wrote: > Nit: just realized this include could be replaced by a struct > declaration: > > > struct progress; > > struct rev_info; > > Like these. 'struct object;' should be enough for the typedef. Good catch. We would need one for the packed_git struct, too. I don't have a strong opinion about including object.h or not, though needing two stubs pushes me slightly in the direction of leaving the include alone. Thanks, Taylor
