zsh-workers
 help / color / mirror / code / Atom feed
From: Nikolai Weibull <now@bitwi.se>
To: Felipe Contreras <felipe.contreras@gmail.com>
Cc: Mikael Magnusson <mikachu@gmail.com>,
	Frank Terbeck <ft@bewatermyfriend.org>,
	zsh-workers@zsh.org
Subject: Re: Slowness issue with git completion
Date: Wed, 27 Apr 2011 12:15:43 +0200	[thread overview]
Message-ID: <BANLkTi=DjdOa-wS7070AVZfTD+aU_O0Uug@mail.gmail.com> (raw)
In-Reply-To: <BANLkTikODTsXkB82KhVhWBaAs9GcYnqBgg@mail.gmail.com>

On Wed, Apr 27, 2011 at 10:30, Felipe Contreras
<felipe.contreras@gmail.com> wrote:
> On Wed, Apr 27, 2011 at 9:11 AM, Nikolai Weibull <now@bitwi.se> wrote:
>> Felie Contreras wrote:
>>> Are you interested in fixing this use-case even if it means to make
>>> some compromises in correctness or not?
>>
>> No, I’m not.
>
> I am not used to zsh development, so is Nikolai's opinion shared by
> the rest?

I am primarily not interested in fixing it when you pose it to me in
the manner that you do.  You may not be aware of it, but your way of
expressing yourself is rather inflammatory.

There’s a big difference between writing

  Would you consider rewriting the code so that it would be a lot
faster, while still being mostly correct?

and what you wrote, which is a commandment hidden inside a question.

If we can find a solution that’s a lot quicker and still maintains
some of the nice features that we currently have, preferably correct
tags and descriptions, then I’m certainly interested.  There have,
however, not been any suggestions made in this area until Frank posted
his suggestion yesterday and perhaps Benjamin’s suggestion has some
merits to it as well (posted today).  Oliver also mentioned rewriting
it in the way that he did for the perforce completion (or was it
Subversion?).  I did, as I’ve already said, try rewriting it along
those lines, but it didn’t help.  I’m glad that we’re having this
discussion, because, believe it or not, I’m not happy with the delays
either.  I will, however, not be told what to do.

> Even if I provide a fix that make things slightly less
> correct but usable, you wouldn't take the patch?

You haven’t offered to do so until now, so how could we take a stance
on it?  What you’ve been saying so far is that /we/ should rewrite it
in a way that /you/ want it to work.


  parent reply	other threads:[~2011-04-27 10:15 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-26 18:26 Felipe Contreras
2011-04-26 18:43 ` Frank Terbeck
2011-04-26 19:06   ` Nikolai Weibull
2011-04-26 20:10     ` Frank Terbeck
2011-04-26 20:23   ` Felipe Contreras
2011-04-26 20:34     ` Mikael Magnusson
2011-04-26 20:57       ` Felipe Contreras
2011-04-26 20:59         ` Mikael Magnusson
2011-04-26 21:10           ` Felipe Contreras
2011-04-26 21:13             ` Mikael Magnusson
2011-04-26 21:35               ` Felipe Contreras
2011-04-26 22:03                 ` Nikolai Weibull
2011-04-26 22:25                   ` Felipe Contreras
2011-04-26 23:14                     ` Benjamin R. Haskell
2011-04-27  7:01                       ` Benjamin R. Haskell
2011-04-27  1:52                     ` gi1242+zsh
2011-04-27  6:11                     ` Nikolai Weibull
2011-04-27  8:30                       ` Felipe Contreras
2011-04-27  8:47                         ` Frank Terbeck
2011-04-27  9:06                           ` Felipe Contreras
2011-04-27 10:15                         ` Nikolai Weibull [this message]
2011-04-27 10:42                           ` Felipe Contreras
2011-04-27 11:14                             ` Nikolai Weibull
2011-04-27  4:19                 ` Bart Schaefer
2011-04-27  6:13                   ` Nikolai Weibull
2011-04-27 12:51                   ` Nikolai Weibull
2011-04-27 14:55                     ` Bart Schaefer
2011-04-27 18:36                       ` Nikolai Weibull
2011-04-30 14:37                     ` Simon Ruderich
2011-04-30 15:00                       ` Simon Ruderich
2011-05-02  9:59                       ` Nikolai Weibull
2011-05-03 13:59                         ` Nikolai Weibull
2011-04-26 21:52         ` Benjamin R. Haskell
2011-04-26 22:04           ` Felipe Contreras
2011-04-26 22:35             ` Benjamin R. Haskell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='BANLkTi=DjdOa-wS7070AVZfTD+aU_O0Uug@mail.gmail.com' \
    --to=now@bitwi.se \
    --cc=felipe.contreras@gmail.com \
    --cc=ft@bewatermyfriend.org \
    --cc=mikachu@gmail.com \
    --cc=zsh-workers@zsh.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).