From: David Caldwell <david@porkrind.org>
Date: Wed, 29 Jun 2011 17:11:59 GMT
Subject: Re: [PATCH] stash: Add --include-untracked option to stash and remove all untracked files
Message-ID: <BE416D9D73FF59D8584480F7@black.porkrind.org>

On 6/26/11 1:02 PM -0700 Junio C Hamano wrote:

> David Caldwell <david@porkrind.org> writes:
>
>>  create_stash () {
>>  	stash_msg="$1"
>> +	untracked="$2"
...
>> -	create_stash "$stash_msg"
>> +	create_stash "$stash_msg" $untracked
>
> Just a minor nit from internal API point of view, I would prefer to see
> something like
>
> 	create_stash --untracked=all "message"
>
> or even
>
> 	create_stash --untracked=all --message="message"
>
> once you start enriching these functions with optional behaviour.

I could do that, but doesn't that mean create_stash() would have to have a 
whole while loop and case statement just to parse the two options? Because 
that seems a little overkill for something that is only called from 2 
places. Is there some nice compact way of doing the parsing I'm missing?

I could also remove parameter passing part and just leave it as a global 
like the --patch option does. ;-)

-David