On Tue, Sep 25, 2018 at 12:26:44PM -0700, Stefan Beller wrote:
> On Sat, Sep 22, 2018 at 5:58 AM Ævar Arnfjörð Bjarmason
> <ava...@gmail.com> wrote:
> >
> >
> > On Fri, Sep 21 2018, Stefan Beller wrote:
> >
> > > +/*
> > > + * Apply want to each entry in array, retaining only the entries for
> > > + * which the function returns true. Preserve the order of the entries
> > > + * that are retained.
> > > + */
> > > +void oid_array_filter(struct oid_array *array,
> > > + for_each_oid_fn want,
> > > + void *cbdata);
> > > +
> > > #endif /* SHA1_ARRAY_H */
> >
> > The code LGTM, but this comment should instead be an update to the API
> > docs, see my recent 5cc044e025 ("get_short_oid: sort ambiguous objects
> > by type, then SHA-1", 2018-05-10) for an addition of a new function to
> > this API where I added some new docs.
>
> ok will fix for consistency (this whole API is there).
>
> Longer term (I thought) we were trying to migrate API docs
> to headers instead?
Yes, please. I think it prevents exactly this sort of confusion. :)
-Peff
