unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: bug#8183: isearch yanking
       [not found] <AANLkTikku0rP_Cd4Y9LpKE2GF2-UbhuCog2AOMR4i=Gq@mail.gmail.com>
@ 2011-03-06  0:23 ` Chong Yidong
  2011-03-06  0:27   ` Lennart Borgman
  2011-03-06  5:33   ` Stefan Monnier
  0 siblings, 2 replies; 4+ messages in thread
From: Chong Yidong @ 2011-03-06  0:23 UTC (permalink / raw)
  To: emacs-devel; +Cc: 8183, gnu.emacs.bug

gnu.emacs.bug@wongs.net writes:

> I just spent a while trying to track down a bug in isearch.el that
> turned out to be a "feature".  C-y has been hijacked to do some weird
> thing where it pastes the entire next line of the buffer instead of
> the kill ring. (Wacky!)

> Fortunately, the fix for this is easy. In isearch.el, just change the
> line that defines C-y as isearch-yank-line to isearch-yank-kill.

This has been bugging me, and has been discussed before, e.g.

  http://lists.gnu.org/archive/html/emacs-devel/2008-11/msg00485.html

Back in that 2008 thread, Juri Linkov suggested exactly this change, and
moving isearch-yank-line over to M-s C-e.

Nothing seemed to come of that suggestion, maybe due to simple inertia.
I now propose to make it so.  Any objections?



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: bug#8183: isearch yanking
  2011-03-06  0:23 ` bug#8183: isearch yanking Chong Yidong
@ 2011-03-06  0:27   ` Lennart Borgman
  2011-03-06  5:33   ` Stefan Monnier
  1 sibling, 0 replies; 4+ messages in thread
From: Lennart Borgman @ 2011-03-06  0:27 UTC (permalink / raw)
  To: Chong Yidong; +Cc: 8183, gnu.emacs.bug, emacs-devel

On Sun, Mar 6, 2011 at 1:23 AM, Chong Yidong <cyd@stupidchicken.com> wrote:
> gnu.emacs.bug@wongs.net writes:
>
>> I just spent a while trying to track down a bug in isearch.el that
>> turned out to be a "feature".  C-y has been hijacked to do some weird
>> thing where it pastes the entire next line of the buffer instead of
>> the kill ring. (Wacky!)
>
>> Fortunately, the fix for this is easy. In isearch.el, just change the
>> line that defines C-y as isearch-yank-line to isearch-yank-kill.
>
> This has been bugging me, and has been discussed before, e.g.
>
>  http://lists.gnu.org/archive/html/emacs-devel/2008-11/msg00485.html
>
> Back in that 2008 thread, Juri Linkov suggested exactly this change, and
> moving isearch-yank-line over to M-s C-e.
>
> Nothing seemed to come of that suggestion, maybe due to simple inertia.
> I now propose to make it so.  Any objections?

It might have stalled because we where discussing other changes to
isearch keymap, but I can see no reason to not make the proposed
change here.



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: bug#8183: isearch yanking
  2011-03-06  0:23 ` bug#8183: isearch yanking Chong Yidong
  2011-03-06  0:27   ` Lennart Borgman
@ 2011-03-06  5:33   ` Stefan Monnier
  2011-03-06 14:05     ` Dani Moncayo
  1 sibling, 1 reply; 4+ messages in thread
From: Stefan Monnier @ 2011-03-06  5:33 UTC (permalink / raw)
  To: Chong Yidong; +Cc: 8183, gnu.emacs.bug, emacs-devel

> Nothing seemed to come of that suggestion, maybe due to simple inertia.
> I now propose to make it so.  Any objections?

Please go ahead,


        Stefan



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: bug#8183: isearch yanking
  2011-03-06  5:33   ` Stefan Monnier
@ 2011-03-06 14:05     ` Dani Moncayo
  0 siblings, 0 replies; 4+ messages in thread
From: Dani Moncayo @ 2011-03-06 14:05 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Chong Yidong, 8183, gnu.emacs.bug, emacs-devel

On Sun, Mar 6, 2011 at 06:33, Stefan Monnier <monnier@iro.umontreal.ca> wrote:
>> Nothing seemed to come of that suggestion, maybe due to simple inertia.
>> I now propose to make it so.  Any objections?
>
> Please go ahead,
>

FWIW, this bug is a duplicate of #7700.

...and what about M-y? If we are looking for consistency, M-y should
also DTRT, i.e., replace the just-yanked (in the Isearch prompt) text
with an earlier entry from the kill-ring.


Regards,

-- 
Dani Moncayo



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-03-06 14:05 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <AANLkTikku0rP_Cd4Y9LpKE2GF2-UbhuCog2AOMR4i=Gq@mail.gmail.com>
2011-03-06  0:23 ` bug#8183: isearch yanking Chong Yidong
2011-03-06  0:27   ` Lennart Borgman
2011-03-06  5:33   ` Stefan Monnier
2011-03-06 14:05     ` Dani Moncayo

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).