Re: AW: [PATCH 1/2] stash--helper: implement "git stash--helper"
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 29, 2016, 19:58 UTC
- Message-ID
- <xmqqlh78ximf.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <BLU436-SMTP10996033F3EBFE2E8639F96A5DB0@phx.gbl>
Matthias Aßhauer <mha1993@live.de> writes:
[administrivia: please wrap your lines to reasonable lengths]
Show 7 quoted lines
>> Honestly, I had high hopes after seeing the "we are rewriting it >> in C" but I am not enthused after seeing this. I was hoping that >> the rewritten version would do this all in-core, by calling these >> functions that we already have: > > These functions might be obvious to you, but I'm new to git's > source code, ...
Ahh, I didn't realize I was talking with somebody unfamiliar with the codebase. Apologies.
Nevertheless, the list of functions I gave are a good starting point; they are widely used building blocks in the codebase.
> I'll be working on a v2 that incorporates the feedback from you, > Thomas Gummerer and Stefan Beller then. Further feedback is of > course welcome.
Thanks.