On Wed, Feb 16 2022, Glen Choo wrote:
> - commits = submodule_commits(changed, name);
> - oid_array_append(commits, &p->two->oid);
> + item = string_list_insert(changed, name);
> + if (!item->util)
> + /* NEEDSWORK: should we have oid_array_init()? */
> + item->util = xcalloc(1, sizeof(struct oid_array));
> + oid_array_append(item->util, &p->two->oid);
> }
> }
Yes, just adding it while we're at it seems worthwhile, and if not
defining this in terms of the macro would be better, as the two are
guaranteed not to drift apart. I.e. the pattern seen in:
git grep -W 'memcpy.*&blank'
