Re: [PATCH v2 2/4] stripspace: Use parse-options for command-line parsing
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Oct 20, 2015, 15:47 UTC
- Message-ID
- <xmqq37x5a6e1.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <20151020084823.GP2468@distanz.ch>
Tobias Klauser <tklauser@distanz.ch> writes:
Show 11 quoted lines
> On 2015-10-17 at 23:24:13 +0200, Junio C Hamano <gitster@pobox.com> wrote: >> Before starting v3, please fetch from me and check what is queued on >> 'pu'. It may turn out that the fix-ups I did while queuing this >> round is sufficient, in which case you can just say that instead ;-) > > Now that patches 3 and 4 will be dropped and no changes being necessary > for patches 1 and 2 (except for your changes that I see are already > folded into 'pu'), do you want me to submit a v3 of the series? Or is it > enough if I ask you to drop patches 3 (stripspace: implement > --count-lines option) and 4 (rebase -i: use "stripspace --count-lines" > when counting todo items)?
Yes, the latter is sufficient and preferrable (less work for both of us, without losing anything to help other people to review and discuss who may want to chime in).
Thanks.