Re: [PATCH v5 02/12] sha1-file: provide functions to look up hash algorithms
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Nov 13, 2018, 18:45 UTC
- Message-ID
- <CACsJy8AtWG=3nJuqG07seczqeb8taimzT4fPbFHc-_3Y8FE_5Q@mail.gmail.com>
- In-Reply-To
- <06e5bc95-b901-163a-5088-888878187ca2@gmail.com>
On Tue, Nov 13, 2018 at 7:42 PM Derrick Stolee <stolee@gmail.com> wrote:
Show 19 quoted lines
>
> On 11/4/2018 6:44 PM, brian m. carlson wrote:
> > +int hash_algo_by_name(const char *name)
> > +{
> > + int i;
> > + if (!name)
> > + return GIT_HASH_UNKNOWN;
> > + for (i = 1; i < GIT_HASH_NALGOS; i++)
> > + if (!strcmp(name, hash_algos[i].name))
> > + return i;
> > + return GIT_HASH_UNKNOWN;
> > +}
> > +
>
> Today's test coverage report [1] shows this method is not covered in the
> test suite. Looking at 'pu', it doesn't have any callers.
>
> Do you have a work in progress series that will use this? Could we add a
> test-tool to exercise this somehow?Going by the function name, it should be used when hash-object or other commands learn about --object-format=<name>.
-- Duy