Re: AW: [PATCH 1/2] stash--helper: implement "git stash--helper"

2016-02-01 Thread Junio C Hamano
Michael Blume writes: > Maybe this isn't important given that it looks like the patch is going > to be rewritten, but I have > > stash.c:43:18: warning: incompatible pointer types assigning to 'const > char *const *' from 'const char *'; take the address with & > [-Wincompatible-pointer-types] >

Re: AW: [PATCH 1/2] stash--helper: implement "git stash--helper"

2016-02-01 Thread Michael Blume
On Fri, Jan 29, 2016 at 11:58 AM, Junio C Hamano wrote: > Matthias Aßhauer writes: > > [administrivia: please wrap your lines to reasonable lengths] > >>> 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

Re: AW: [PATCH 1/2] stash--helper: implement "git stash--helper"

2016-01-29 Thread Junio C Hamano
Matthias Aßhauer writes: [administrivia: please wrap your lines to reasonable lengths] >> 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 >>

AW: [PATCH 1/2] stash--helper: implement "git stash--helper"

2016-01-29 Thread Matthias Aßhauer
ng on a v2 that incorporates the feedback from you, Thomas Gummerer and Stefan Beller then. Further feedback is of course welcome. -Ursprüngliche Nachricht- Von: Junio C Hamano [mailto:gits...@pobox.com] Gesendet: Freitag, 29. Januar 2016 00:06 An: Matthias Asshauer Cc: git@vger.kernel.o

Re: [PATCH 1/2] stash--helper: implement "git stash--helper"

2016-01-28 Thread Junio C Hamano
Matthias Asshauer writes: > From: Matthias Aßhauer > > This patch implements a new "git stash--helper" builtin plumbing > command that will be used to migrate "git-stash.sh" to C. > > We start by implementing only the "--non-patch" option that will > handle the core part of the non-patch stashin

[PATCH 1/2] stash--helper: implement "git stash--helper"

2016-01-28 Thread Matthias Asshauer
From: Matthias Aßhauer This patch implements a new "git stash--helper" builtin plumbing command that will be used to migrate "git-stash.sh" to C. We start by implementing only the "--non-patch" option that will handle the core part of the non-patch stashing. Signed-off-by: Matthias Aßhauer ---