Re: [PATCH v2] rev-list-options: clarify the usage of --reverse

2016-09-27 Thread Pranit Bauva
Hey Junio, On Wed, Sep 28, 2016 at 3:42 AM, Junio C Hamano wrote: > "Philip Oakley" writes: > >> micro-nit: 'first' and 'last' can be tricky to distinguish for lists... > > Let's do this then. > > -- >8 -- > From: Pranit Bauva >

Re: [PATCH v2] rev-list-options: clarify the usage of --reverse

2016-09-27 Thread Philip Oakley
From: "Junio C Hamano" "Philip Oakley" writes: micro-nit: 'first' and 'last' can be tricky to distinguish for lists... Let's do this then. Looks good to me. Thanks. -- Philip -- >8 -- From: Pranit Bauva Date: Tue, 27

Re: [PATCH v2] rev-list-options: clarify the usage of --reverse

2016-09-27 Thread Junio C Hamano
"Philip Oakley" writes: > micro-nit: 'first' and 'last' can be tricky to distinguish for lists... Let's do this then. -- >8 -- From: Pranit Bauva Date: Tue, 27 Sep 2016 20:44:09 + Subject: [PATCH] rev-list-options: clarify the usage of

Re: [PATCH v2] rev-list-options: clarify the usage of --reverse

2016-09-27 Thread Philip Oakley
From: "Pranit Bauva" Specify even more clearly that --reverse works only with the commits which are chosen to be shown so as to eliminate the confusion as to whether the first n or the last n commits with be shown when used hi Pranit, micro-nit: 'first' and 'last' can

[PATCH v2] rev-list-options: clarify the usage of --reverse

2016-09-27 Thread Pranit Bauva
Specify even more clearly that --reverse works only with the commits which are chosen to be shown so as to eliminate the confusion as to whether the first n or the last n commits with be shown when used with `-n --reverse`. Reported-by: Ruediger Meier Signed-off-by: Pranit