Re: [PATCH v2 08/27] revisions API users: use release_revisions() needing "{ 0 }" init
Junio C Hamano <gitster@pobox.com> writes:
> Without such a mechanism to clearly say "here is what we initialize
"what" -> "where and how".
Show 6 quoted lines
> a rev_info", the first call to repo_init_revisions() looks like the
> place that initializes a rev_info, and call to release_revisions()
> on a rev_info that did not go through repo_init_revisions() looks
> like a call to free() of a pointer that hasn't been assigned the
> result from an allocation from the heap. That is where my "iffy
> from the API cleanliness POV" comes from.
Junio C Hamano <gitster@pobox.com>