Subject: bug#4981 closed by Juri Linkov <[email-masked]> (Re: bug#4981: C-l during query-replace)

From: [email-masked] (Emacs bug Tracking System)
Date: Mon, 30 Nov 2009 16:30:14 +0000

This is an automatic notification regarding your bug report
which was filed against the emacs package:

#4981: C-l during query-replace

It has been closed by Juri Linkov <[email-masked]>.

Their explanation is attached below along with your original report.
If this explanation is unsatisfactory and you have not received a
better one in a separate message then please contact Juri Linkov <[email-masked]> by
replying to this email.

-- 
4981: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=4981
Emacs Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#4981: C-l during query-replace
From: Juri Linkov <[email-masked]>
To: Stefan Monnier <[email-masked]>
Cc: [email-masked], Dan Nicolaescu <[email-masked]>
Date: Mon, 30 Nov 2009 18:06:33 +0200

> Why not:
>
>> Index: lisp/replace.el
>> ===================================================================
>> RCS file: /sources/emacs/emacs/lisp/replace.el,v
>> retrieving revision 1.287
>> diff -u -r1.287 replace.el
>> --- lisp/replace.el	12 Nov 2009 06:55:43 -0000	1.287
>> +++ lisp/replace.el	29 Nov 2009 23:43:28 -0000
>> @@ -1785,7 +1788,9 @@
>>  			((eq def 'skip)
>>  			 (setq done t))
>>  			((eq def 'recenter)
>> -			 (recenter nil))
>> +                      (let ((this-command 'recenter-top-bottom))
>> +			 (recenter-top-bottom)))
>>  			((eq def 'edit)
>>  			 (let ((opos (point-marker)))
>>  			   (setq real-match-data (replace-match-data

Thanks, this helped.  Fixed with more changes to keep the cycling order.

-- 
Juri Linkov
http://www.jurta.org/emacs/

Subject: C-l during query-replace
From: Dan Nicolaescu <[email-masked]>
To: bug-gnu-emacs <[email-masked]>
Date: Thu, 19 Nov 2009 16:16:59 -0800 (PST)

C-l during query-replace should run `recenter-top-bottom', not
`recenter' for consistency with what C-l normally does nowadays.