On Wed, Apr 5, 2023 at 10:28 AM Calvin Wan <calvinwan@xxxxxxxxxx> wrote:
>
> Instead of moving these declarations, can we move
> strbuf_repo_add_unique_abbrev() and strbuf_add_unique_abbrev() to
> object-name.[ch]? These functions are related to both strbuf and
> object-name, but object-name should be a higher level API than strbuf
> so it seems more natural to belong in there.
I like that suggestion; that would be better overall. However, should
it be in this (already lengthy) series? I'm guessing you likely
already have such a change in the series you're including, and if I
were to add it to mine (and risk doing it slightly differently), that
might increase the conflicts you need to deal with. Would it perhaps
be easier to keep this small change for this series, or even drop this
particular patch, and then let you address this improved direction
with your strbuf work?
> "Elijah Newren via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes:
> > From: Elijah Newren <newren@xxxxxxxxx>
> >
> > Signed-off-by: Elijah Newren <newren@xxxxxxxxx>
> > ---
> > strbuf.h | 9 ++-------
> > 1 file changed, 2 insertions(+), 7 deletions(-)
> >
> > diff --git a/strbuf.h b/strbuf.h
> > index 3dfeadb44c2..547696fb233 100644
> > --- a/strbuf.h
> > +++ b/strbuf.h
> > @@ -1,6 +1,8 @@
> > #ifndef STRBUF_H
> > #define STRBUF_H
> >
> > +struct object_id;
> > +struct repository;
> > struct string_list;
> >
> > /**
> > @@ -72,12 +74,6 @@ struct strbuf {
> > extern char strbuf_slopbuf[];
> > #define STRBUF_INIT { .buf = strbuf_slopbuf }
> >
> > -/*
> > - * Predeclare this here, since cache.h includes this file before it defines the
> > - * struct.
> > - */
> > -struct object_id;
> > -
> > /**
> > * Life Cycle Functions
> > * --------------------
> > @@ -634,7 +630,6 @@ void strbuf_list_free(struct strbuf **list);
> > * Add the abbreviation, as generated by repo_find_unique_abbrev(), of `sha1` to
> > * the strbuf `sb`.
> > */
> > -struct repository;
> > void strbuf_repo_add_unique_abbrev(struct strbuf *sb, struct repository *repo,
> > const struct object_id *oid, int abbrev_len);
> > void strbuf_add_unique_abbrev(struct strbuf *sb, const struct object_id *oid,
> > --
> > gitgitgadget
