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

From: Karthik Nayak <[email-masked]>
Date: Tue, 09 Jun 2015 01:49:23 +0530

On 06/08/2015 10:51 PM, Matthieu Moy wrote:
> 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.
>

But that would break the flow, wouldn't it? I wanted ref-filter to be 
introduced together, hence right after ref-filter.h we move code to
ref-filter.c

-- 
Regards,
Karthik