all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Vitalie Spinu <spinuvit@gmail.com>
To: Le Wang <l26wang@gmail.com>
Cc: Leo Liu <sdl.web@gmail.com>, Drew Adams <drew.adams@oracle.com>,
	emacs-devel@gnu.org
Subject: Re: fix for bug 10994 breaks ido customizations in major way
Date: Tue, 07 May 2013 21:00:43 +0200	[thread overview]
Message-ID: <87y5bqty1w.fsf@gmail.com> (raw)
In-Reply-To: <CAM=K+iquN_x-stbe7RA_v+Op2PvkRGgz_qPhHX0GDQUNpDd2LA@mail.gmail.com> (Le Wang's message of "Tue, 7 May 2013 22:47:58 +0800")

 >> Le Wang <l26wang@gmail.com>
 >> on Tue, 7 May 2013 22:47:58 +0800 wrote:

 > Hi Drew,
 > On Tue, May 7, 2013 at 10:44 PM, Drew Adams <drew.adams@oracle.com> wrote:
 >> I don't care much about Ido, so I don't have an opinion about the question at
 >> hand.
 >> 
 >> However, I will say that it is NOT at all the case that there is nothing out
 >> there, in the wild, that takes advantage of completion candidates that have the
 >> same text (same string chars)

Indeed, it is not difficult to imagine an application when same names
have different faces.

 > We can say with certainty that there is "nothing in the wild" because
 > there is a bug in the latest released Emacs and ido that does not work
 > with duplicate strings.

This is your subjective opinion. The current state is a very reasonable
default, it does *work* (aka cycles) with duplicated strings, and call
this behavior a bug is at least inappropriate.

To have useless properties that are not needed by your application and
rely on IDO to delete those for you is a bad and lazy style. So the
applications that you mention are buggy, not IDO.

Don't want duplicated names, pass through delete-dups before calling
ido-completing-read, or even better, don't collect those properties in
the first place. Not a big deal.

What you propose will force *every* implementation out there to
explicitly deal with duplicated strings. As said before, if they have
duplicated strings they have them for a reason (even if it is simple
laziness). Don't you see that you propose to cut some of the
programmers' and users' flexibility?

    Vitalie



  reply	other threads:[~2013-05-07 19:00 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-02 17:57 fix for bug 10994 breaks ido customizations in major way Le Wang
2013-05-03  4:13 ` Leo Liu
2013-05-03 12:49   ` Le Wang
2013-05-03 20:33     ` Leo Liu
2013-05-04  7:00       ` Le Wang
2013-05-04  8:58         ` Óscar Fuentes
2013-05-04 13:00           ` Le Wang
2013-05-05 10:57             ` Óscar Fuentes
2013-05-05 11:39               ` Leo Liu
2013-05-05 12:20                 ` Óscar Fuentes
2013-05-05 12:58                   ` Leo Liu
2013-05-05 13:38                     ` Óscar Fuentes
2013-05-05 14:31                       ` Stephen J. Turnbull
2013-05-05 15:26                         ` Óscar Fuentes
2013-05-06 15:11                           ` Le Wang
2013-05-06 22:49         ` Vitalie Spinu
2013-05-07  1:01           ` Óscar Fuentes
2013-05-07  9:35           ` Le Wang
2013-05-07 10:26             ` Vitalie Spinu
2013-05-07 10:35               ` Óscar Fuentes
2013-05-07 14:49                 ` Le Wang
2013-05-07 21:18                   ` Stefan Monnier
2013-05-07 14:42               ` Le Wang
2013-05-07 14:44             ` Drew Adams
2013-05-07 14:47               ` Le Wang
2013-05-07 19:00                 ` Vitalie Spinu [this message]
2013-05-07 19:53                   ` Óscar Fuentes
2013-05-08  0:04                     ` Leo Liu
2013-05-08  0:35                       ` Le Wang
2013-05-08  3:10                         ` Leo Liu
2013-05-08  3:29                           ` Leo Liu
2013-05-08  4:49                             ` Leo Liu
2013-05-08  8:14                               ` Vitalie Spinu
2013-05-08  8:42                                 ` Leo Liu
2013-05-08 12:23                                   ` Le Wang
2013-05-08 14:29                                     ` Leo Liu
2013-05-08 20:56                                   ` Juri Linkov
2013-05-10  1:52                                     ` Leo Liu
2013-05-17  2:48                                     ` Leo Liu

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=87y5bqty1w.fsf@gmail.com \
    --to=spinuvit@gmail.com \
    --cc=drew.adams@oracle.com \
    --cc=emacs-devel@gnu.org \
    --cc=l26wang@gmail.com \
    --cc=sdl.web@gmail.com \
    /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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.