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 (e.g. "useful to get the commits and all commits in the branches t

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 and all commits in the branches that were merged i

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 commit" - paraphrased from th

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 s

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, b

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 tests before you can drop the

[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 co