From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: phillip.lord@russet.org.uk (Phillip Lord) Newsgroups: gmane.emacs.bugs Subject: bug#21968: Apparent regression on the undo command Date: Mon, 23 Nov 2015 17:27:37 +0000 Message-ID: <87wpt8aanq.fsf__1762.25722902945$1448299713$gmane$org@russet.org.uk> References: <87bnarsdfy.fsf@russet.org.uk> <871tbm9v83.fsf@russet.org.uk> <87wpte6slz.fsf@russet.org.uk> <87io4xpio3.fsf@red-bean.com> <87y4drllg8.fsf@russet.org.uk> <1512E415-81ED-4C4D-B03E-4ED0EE29D34C@gmail.com> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1448299714 8057 80.91.229.3 (23 Nov 2015 17:28:34 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 23 Nov 2015 17:28:34 +0000 (UTC) Cc: 21968@debbugs.gnu.org, emacs-devel To: David Reitter Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Mon Nov 23 18:28:15 2015 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1a0uuT-0008JQ-HU for geb-bug-gnu-emacs@m.gmane.org; Mon, 23 Nov 2015 18:28:13 +0100 Original-Received: from localhost ([::1]:33709 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a0uuU-0005zY-7H for geb-bug-gnu-emacs@m.gmane.org; Mon, 23 Nov 2015 12:28:14 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:45866) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a0uuN-0005sf-Ss for bug-gnu-emacs@gnu.org; Mon, 23 Nov 2015 12:28:10 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a0uuI-0004bH-4K for bug-gnu-emacs@gnu.org; Mon, 23 Nov 2015 12:28:07 -0500 Original-Received: from debbugs.gnu.org ([208.118.235.43]:60069) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a0uuI-0004bB-1v for bug-gnu-emacs@gnu.org; Mon, 23 Nov 2015 12:28:02 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.80) (envelope-from ) id 1a0uuH-0006Fx-Tg for bug-gnu-emacs@gnu.org; Mon, 23 Nov 2015 12:28:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: phillip.lord@russet.org.uk (Phillip Lord) Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 23 Nov 2015 17:28:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 21968 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 21968-submit@debbugs.gnu.org id=B21968.144829966224009 (code B ref 21968); Mon, 23 Nov 2015 17:28:01 +0000 Original-Received: (at 21968) by debbugs.gnu.org; 23 Nov 2015 17:27:42 +0000 Original-Received: from localhost ([127.0.0.1]:49777 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1a0utx-0006F9-B8 for submit@debbugs.gnu.org; Mon, 23 Nov 2015 12:27:41 -0500 Original-Received: from cheviot22.ncl.ac.uk ([128.240.234.22]:33855) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1a0utu-0006Ew-FM for 21968@debbugs.gnu.org; Mon, 23 Nov 2015 12:27:40 -0500 Original-Received: from smtpauth-vm.ncl.ac.uk ([10.8.233.129] helo=smtpauth.ncl.ac.uk) by cheviot22.ncl.ac.uk with esmtp (Exim 4.63) (envelope-from ) id 1a0utt-0000cV-Ey; Mon, 23 Nov 2015 17:27:37 +0000 Original-Received: from jangai.ncl.ac.uk ([10.66.67.223] helo=localhost) by smtpauth.ncl.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from ) id 1a0utt-0002PG-Gu; Mon, 23 Nov 2015 17:27:37 +0000 In-Reply-To: <1512E415-81ED-4C4D-B03E-4ED0EE29D34C@gmail.com> (David Reitter's message of "Sat, 21 Nov 2015 19:55:22 -0500") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 208.118.235.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:109131 Archived-At: Okay, am working on this now! David Reitter writes: > Phil, > > I tried your fix. It works for backward-kill-word, but for kill-word > (forward), the cursor ends up in the wrong place (after the word, not where it > was initially). > > - David > > >> On Nov 21, 2015, at 6:18 AM, Artur Malabarba wrote: >> >> I can't speak for the code, but I just tested your fix and looks like it works. >> >> Thanks Phil. >> >> 2015-11-21 10:02 GMT+00:00 Phillip Lord : >> >> I have a possible fix for this now, currently on the >> branch fix/undo-point-in-wrong-place. I waiting for some other feedback >> on my code before I squash and merge it, but if you have the chance it >> would be good to know if it works. >> >> Phil >> >> >> Artur Malabarba writes: >> >> > No problem Phil. >> > Thanks for looking into this. >> > On 20 Nov 2015 7:51 am, "Phillip Lord" wrote: >> > >> >> >> >> It is my commit after all. I will work on it as soon as I can. Had a nasty >> >> cold for the last few days. >> >> >> >> Phil >> >> ________________________________________ >> >> From: emacs-devel-bounces+phillip.lord=newcastle.ac.uk@gnu.org >> >> [emacs-devel-bounces+phillip.lord=newcastle.ac.uk@gnu.org] on behalf of >> >> Karl Fogel [kfogel@red-bean.com] >> >> Sent: 20 November 2015 01:24 >> >> To: Phillip Lord >> >> Cc: Artur Malabarba; emacs-devel >> >> Subject: Re: Apparent regression on the undo command >> >> >> >> phillip.lord@russet.org.uk (Phillip Lord) writes: >> >> >Artur Malabarba writes: >> >> >> 2015-11-19 9:47 GMT+00:00 Phillip Lord : >> >> >>> I'll try and see if I can automate the problem and >> >> >>> run a bisect. >> >> >> >> >> >> The following works for me. When called on the *scratch* buffer it >> >> >> returns non-nil if the problem occurs. >> >> >> >> >> >> (progn >> >> >> (kmacro-call-macro nil nil nil [134217788 11 14 14 11 67108911]) >> >> >> (equal (point-min) (point))) >> >> >> >> >> >> That macro simply calls M-< C-k C-n C-n C-k C-/ >> >> > >> >> > >> >> >I tried various combinations -- works interactively but fails in batch >> >> >either because *scratch* hasn't been initialized, or undo appears to >> >> >work wrongly. >> >> > >> >> >No worries, it's easy enough to test by hand, and am doing so. >> >> >> >> Any luck? FWIW I'm noticing the bug too, in emacs-25 branch built from >> >> commit c210b8b128c. >> >> >> >> Best regards, >> >> -Karl >> >> >> >> >>