Re: [RFC PATCH v2] revision: new rev^-n shorthand for rev^n..rev

2016-09-26 Thread Philip Oakley
From: "Junio C Hamano" "Philip Oakley" writes: From: "Vegard Nossum" I use rev^..rev daily, and I'm surely not the only one. Not everyone knows the 'trick' and may not use it daily. Consider stating what it is useful for

Re: [RFC PATCH v2] revision: new rev^-n shorthand for rev^n..rev

2016-09-25 Thread Junio C Hamano
"Philip Oakley" writes: > From: "Vegard Nossum" >>I use rev^..rev daily, and I'm surely not the only one. > > Not everyone knows the 'trick' and may not use it daily. > > Consider stating what it is useful for (e.g. "useful to get the > commits

Re: [RFC PATCH v2] revision: new rev^-n shorthand for rev^n..rev

2016-09-25 Thread Philip Oakley
From: "Vegard Nossum" I use rev^..rev daily, and I'm surely not the only one. Not everyone knows the 'trick' and may not use it daily. Consider stating what it is useful for (e.g. "useful to get the commits and all commits in the branches that were merged into

Re: [RFC PATCH v2] revision: new rev^-n shorthand for rev^n..rev

2016-09-25 Thread Jakub Narębski
W dniu 25.09.2016 o 10:55, Vegard Nossum pisze: > I use rev^..rev daily, and I'm surely not the only one. To save typing > (or copy-pasting, if the rev is long -- like a full SHA-1 or branch name) > we can make rev^- a shorthand for that. > > The existing syntax rev^! seems like it should do the

Re: [RFC PATCH v2] revision: new rev^-n shorthand for rev^n..rev

2016-09-25 Thread Ramsay Jones
On 25/09/16 09:55, Vegard Nossum wrote: > I use rev^..rev daily, and I'm surely not the only one. To save typing > (or copy-pasting, if the rev is long -- like a full SHA-1 or branch name) > we can make rev^- a shorthand for that. > > The existing syntax rev^! seems like it should do the same,

Re: [RFC PATCH v2] revision: new rev^-n shorthand for rev^n..rev

2016-09-25 Thread Matthieu Moy
Vegard Nossum writes: > Documentation/revisions.txt | 14 +++ > builtin/rev-parse.c | 28 ++ > revision.c | 91 > + > revision.h | 1 + This would obviously need

[RFC PATCH v2] revision: new rev^-n shorthand for rev^n..rev

2016-09-25 Thread Vegard Nossum
I use rev^..rev daily, and I'm surely not the only one. To save typing (or copy-pasting, if the rev is long -- like a full SHA-1 or branch name) we can make rev^- a shorthand for that. The existing syntax rev^! seems like it should do the same, but it doesn't really do the right thing for merge