unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Kangas <stefan@marxist.se>
To: "Andreas Röhler" <andreas.roehler@easy-emacs.de>
Cc: 29323@debbugs.gnu.org
Subject: bug#29323: kill-do-not-save-duplicate, FR
Date: Mon, 11 Nov 2019 03:14:38 +0100	[thread overview]
Message-ID: <875zjrrwfl.fsf@marxist.se> (raw)
In-Reply-To: <0c96aedc-36e2-2bbd-8672-0008dbbfd7d5@easy-emacs.de> ("Andreas Röhler"'s message of "Sun, 19 Nov 2017 19:42:27 +0100")

Andreas Röhler <andreas.roehler@easy-emacs.de> writes:

> On 16.11.2017 20:51, Andreas Rvhler wrote:
>
>  Feature request:
>
>  Currently variable kill-do-not-save-duplicates checks only the (car kill-ring) as docu explains: 
>
>  Do not add a new string to kill-ring if it duplicates the last one.
>  The comparison is done using equal-including-properties.
>
>  AFAIU it would be trivial replace this check by a call of "member", thus checking the whole kill-ring.
>
>  Thanks,
>
>  Andreas
>
> This variant of kill-new should do it. Diff basically at comment " ;; delete string from kill-ring" 

I personally can't imagine when I would want the behaviour you ask
for, but I think it could make sense to have it as an optional
behaviour.

Drew suggested that this variable should have three possible values,
where 't' and 'nil' stays as they are, but we introduce a new symbol
which means to have the behaviour you want.  I think this is a better
proposal than changing the existing default.

Best regards,
Stefan Kangas





  reply	other threads:[~2019-11-11  2:14 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-16 19:51 bug#29323: kill-do-not-save-duplicate, FR Andreas Röhler
2017-11-16 22:34 ` Drew Adams
2017-11-18 19:06   ` Andreas Röhler
2017-11-18 19:48     ` Drew Adams
2017-11-19 18:42 ` Andreas Röhler
2019-11-11  2:14   ` Stefan Kangas [this message]
2021-11-30 14:48     ` Lars Ingebrigtsen

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=875zjrrwfl.fsf@marxist.se \
    --to=stefan@marxist.se \
    --cc=29323@debbugs.gnu.org \
    --cc=andreas.roehler@easy-emacs.de \
    /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.savannah.gnu.org/cgit/emacs.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).