Subject: Re: [WIP/PATCH v5 05/10] for-each-ref: introduce 'ref_array_clear()'

From: Matthieu Moy <[email-masked]>
Date: Mon, 08 Jun 2015 19:21:02 +0200

Junio C Hamano <[email-masked]> writes:

> Matthieu Moy <[email-masked]> writes:
>
>> Karthik Nayak <[email-masked]> writes:
>>
>>> On 06/08/2015 08:23 PM, Matthieu Moy wrote:
>>>> Karthik Nayak <[email-masked]> writes:
>>>>
>>>> > +/* Free all memory allocated for ref_array */
>>>> > +void ref_array_clear(struct ref_array *array)
>>>>
>>>> Is this a private function? If so, then add static. If not, you probably
>>>> want to export it in a .h file.
>>>>
>>> It is in ref-filter.h.
>>
>> Ah, OK. It comes later in the series.
>
> Confused I am; if it comes later not in the same patch then it is
> not OK, is it?

We could introduce ref-filter.h earlier, indeed. To me, the current
solution is good enough, but introducing ref-filter.h early and adding
function definition there in the same commit as you drop the "static"
keyword for them would clearly be an improvement.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/