On 06/08/2015 10:51 PM, Matthieu Moy wrote:
Junio C Hamano <gitster@xxxxxxxxx> writes:
Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> writes:
Karthik Nayak <karthik.188@xxxxxxxxx> writes:
On 06/08/2015 08:23 PM, Matthieu Moy wrote:
Karthik Nayak <karthik.188@xxxxxxxxx> 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
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html