zsh-workers
 help / color / mirror / code / Atom feed
From: Peter Stephenson <pws@csr.com>
To: zsh-workers@sunsite.dk
Subject: Re: need help debugging cvs completion problem
Date: Tue, 07 Jul 2009 16:11:47 +0100	[thread overview]
Message-ID: <200907071511.n67FBl9J021661@news01.csr.com> (raw)
In-Reply-To: <m363e4lgus.fsf@klanderman.net>

Greg Klanderman wrote:
> Unfortunately, it seems that when the new completion system was
> designed, it did not abstract stuff like handling parameter
> references, quoting, etc like the compctl stuff did.  So if you're
> doing file completion, _path_files has all that logic (at the expense
> of being completely incomprehensible) and it mostly works, but other
> sorts of completion either have to reimplement that logic (sometimes
> partially and/or buggily), or just not handle these complications.

Right, it's the same problem with generating the files for git
completion---if you're not using _path_files, you're starting from
scratch, and you don't have the useful stuff for handling directories,
hence you're doing too much work.

It would be really nice to have this all generic---pass in a directory
to a context-defined function, pass back a list of files, however
generated---even if it was in the first instance separate from
_path_files.  The combination of _path_files, compfiles, and the basic
completion system is as obfuscated as anything I know.

compctl was no better---but as the whole thing was hidden from you from
start to end it didn't matter.

-- 
Peter Stephenson <pws@csr.com>                  Software Engineer
CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
Cambridge, CB4 0WZ, UK                          Tel: +44 (0)1223 692070


      reply	other threads:[~2009-07-07 15:16 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-26 22:19 Greg Klanderman
2009-06-27 20:24 ` Peter Stephenson
2009-06-28  0:48   ` Greg Klanderman
2009-06-28  1:11     ` Greg Klanderman
2009-06-28 11:38       ` Geoff Wing
2009-06-28 17:49         ` Greg Klanderman
2009-06-28 18:52           ` Greg Klanderman
2009-06-28 19:54             ` Peter Stephenson
2009-06-28 20:16               ` Peter Stephenson
2009-07-07 14:15               ` Greg Klanderman
2009-07-07 15:11                 ` Peter Stephenson [this message]

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=200907071511.n67FBl9J021661@news01.csr.com \
    --to=pws@csr.com \
    --cc=zsh-workers@sunsite.dk \
    /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).