On Tue, Aug 19, 2025 at 11:56:54AM -0700, Junio C Hamano wrote:
> Patrick Steinhardt <ps@xxxxxx> writes:
> > diff --git a/odb.h b/odb.h
> > index 2dc3bdc79d..f1736b067c 100644
> > --- a/odb.h
> > +++ b/odb.h
> > @@ -124,17 +124,10 @@ struct object_database {
> > unsigned commit_graph_attempted : 1; /* if loading has been attempted */
> >
> > /*
> > - * private data
> > - *
> > - * should only be accessed directly by packfile.c
> > + * Should only be accessed directly by packfile.c
> > */
>
> Hmph, would this be better done in the step [01/16]? Or did the
> removal of kept_pack_cache make the last piece of "private data"
> disappear with this step?
Yeah, the latter. All packfile-related private data is now encapsulated
in `struct packfile_store`, so I felt like the comment became redundant
with this commit here.
I'll mention this in the commit message.
Patrick
