Re: [PATCH v2] am --skip/--abort: merge HEAD/ORIG_HEAD tree into index
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Aug 19, 2015, 17:55 UTC
- Message-ID
- <xmqqy4h79n93.fsf@gitster.dls.corp.google.com>
- In-Reply-To
- <20150819082222.GA27685@potato.chippynet.com>
Paul Tan <pyokagan@gmail.com> writes:
> (In addition, I fixed a small mistake with the "struct tree_desc" array > size.)
Yeah, one thing I forgot to mention was that I suspect this kind of "oneway merge" already in other places in the code, and a future mini-project might be to inspect them all and try to see if their commonalities can be taken advantage of. If we had a small helper function to do the "do the equivalent of 'git reset $TREE'", this miscounting wouldn't have happened.
The change looks sensible. Thanks.