Subject: Re: [PATCH 02/17] odb: introduce "files" source

From: Patrick Steinhardt <[email-masked]>
Date: Thu, 5 Mar 2026 14:23:16 +0100

On Wed, Mar 04, 2026 at 10:57:26AM -0600, Justin Tobler wrote:
> On 26/02/23 05:17PM, Patrick Steinhardt wrote:
> > diff --git a/odb/source.h b/odb/source.h
> > index 391d6d1e38..1c34265189 100644
> > --- a/odb/source.h
> > +++ b/odb/source.h
> > @@ -19,11 +21,8 @@ struct odb_source {
> >  	/* Object database that owns this object source. */
> >  	struct object_database *odb;
> >  
> > -	/* Private state for loose objects. */
> > -	struct odb_source_loose *loose;
> > -
> > -	/* Should only be accessed directly by packfile.c and midx.c. */
> 
> Is there any value to keeping this comment around?

I don't think so. With this series it becomes clear that all of the info
in the sources become private implementation details, and future patch
series will double down on that even further.

Patrick