Re: [PATCH] completion: avoid ls-remote in certain scenarios

2013-06-02 Thread Felipe Contreras
On Sun, Jun 2, 2013 at 5:08 PM, Junio C Hamano wrote: > Felipe Contreras writes: > >> It's _very_ slow in many cases, and there's really no point in fetching >> *everything* from the remote just for completion. In many cases it might >> be faster for the user to type the whole thing. > > Besides,

Re: [PATCH] completion: avoid ls-remote in certain scenarios

2013-06-02 Thread Junio C Hamano
Felipe Contreras writes: > It's _very_ slow in many cases, and there's really no point in fetching > *everything* from the remote just for completion. In many cases it might > be faster for the user to type the whole thing. Besides, if I understand the use of __git_refs correctly, it is primaril

Re: [PATCH] completion: avoid ls-remote in certain scenarios

2013-06-01 Thread Ramkumar Ramachandra
Felipe Contreras wrote: > diff --git a/contrib/completion/git-completion.bash > b/contrib/completion/git-completion.bash > index 1c35eef..2ce4f7d 100644 > --- a/contrib/completion/git-completion.bash > +++ b/contrib/completion/git-completion.bash > @@ -427,14 +427,8 @@ __git_refs () >

Re: [PATCH] completion: avoid ls-remote in certain scenarios

2013-06-01 Thread Felipe Contreras
On Wed, May 29, 2013 at 2:44 AM, SZEDER Gábor wrote: > On Tue, May 28, 2013 at 10:20:48PM -0500, Felipe Contreras wrote: >> It's _very_ slow in many cases, and there's really no point in fetching >> *everything* from the remote just for completion. In many cases it might >> be faster for the user

Re: [PATCH] completion: avoid ls-remote in certain scenarios

2013-05-29 Thread SZEDER Gábor
On Tue, May 28, 2013 at 10:20:48PM -0500, Felipe Contreras wrote: > It's _very_ slow in many cases, and there's really no point in fetching > *everything* from the remote just for completion. In many cases it might > be faster for the user to type the whole thing. > > If the user manually specifie

[PATCH] completion: avoid ls-remote in certain scenarios

2013-05-28 Thread Felipe Contreras
It's _very_ slow in many cases, and there's really no point in fetching *everything* from the remote just for completion. In many cases it might be faster for the user to type the whole thing. If the user manually specifies 'refs/*', then the full ls-remote completion is triggered. Signed-off-by: