From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Stefan Monnier Newsgroups: gmane.emacs.devel Subject: Re: disabling undo boundaries Date: Sat, 08 Aug 2015 17:28:28 -0400 Message-ID: References: <87fv746rd5.fsf@newcastle.ac.uk> <871tiiowjw.fsf@newcastle.ac.uk> <87siaxk4dl.fsf@newcastle.ac.uk> <87iobsr6m2.fsf@newcastle.ac.uk> <87oalgaicm.fsf@newcastle.ac.uk> <87y4kkkzlo.fsf@newcastle.ac.uk> <87mw0zk7yp.fsf@newcastle.ac.uk> <87617mgp1b.fsf@newcastle.ac.uk> <871ti9ammc.fsf@newcastle.ac.uk> <878ucaqm3g.fsf@newcastle.ac.uk> <87y4hrm911.fsf@newcastle.ac.uk> <87wpx79pz6.fsf@newcastle.ac.uk> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1439069327 13005 80.91.229.3 (8 Aug 2015 21:28:47 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sat, 8 Aug 2015 21:28:47 +0000 (UTC) Cc: Emacs-Devel devel To: phillip.lord@newcastle.ac.uk (Phillip Lord) Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Aug 08 23:28:39 2015 Return-path: Envelope-to: ged-emacs-devel@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 1ZOBfS-0001Wd-Jn for ged-emacs-devel@m.gmane.org; Sat, 08 Aug 2015 23:28:38 +0200 Original-Received: from localhost ([::1]:53830 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZOBfR-0002gE-SF for ged-emacs-devel@m.gmane.org; Sat, 08 Aug 2015 17:28:37 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:35508) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZOBfO-0002fv-My for emacs-devel@gnu.org; Sat, 08 Aug 2015 17:28:35 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZOBfL-0002EW-6x for emacs-devel@gnu.org; Sat, 08 Aug 2015 17:28:34 -0400 Original-Received: from pruche.dit.umontreal.ca ([132.204.246.22]:53063) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZOBfL-0002EO-1x for emacs-devel@gnu.org; Sat, 08 Aug 2015 17:28:31 -0400 Original-Received: from fmsmemgm.homelinux.net (lechon.iro.umontreal.ca [132.204.27.242]) by pruche.DIT.UMontreal.CA (8.14.1/8.14.1) with ESMTP id t78LSPb9026634; Sat, 8 Aug 2015 17:28:26 -0400 Original-Received: by fmsmemgm.homelinux.net (Postfix, from userid 20848) id D2FE1AE1C4; Sat, 8 Aug 2015 17:28:28 -0400 (EDT) In-Reply-To: (Stefan Monnier's message of "Fri, 07 Aug 2015 13:10:51 -0400") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) X-NAI-Spam-Flag: NO X-NAI-Spam-Threshold: 5 X-NAI-Spam-Score: 0 X-NAI-Spam-Rules: 1 Rules triggered RV5392=0 X-NAI-Spam-Version: 2.3.0.9393 : core <5392> : inlines <3584> : streams <1485266> : uri <2009151> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 132.204.246.22 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:188628 Archived-At: >> Anyway, the patch is there. Any chance? If not, anything (further) that >> I can add (or remove!) to the branch to make it happen? > I'll try and look at it soon, IIUC this only removes the code which adds a boundary when we change buffer. IOW it switches from the conservative option of adding a boundary even if it might not be needed, to the risky option of hoping that someone else will add the needed boundary. As explained, this will most likely introduce problems with ever-growing undo-logs in buffers filled by process output and timers. It's an acceptable first step, but it needs to be compensated by a second step which keeps track of all buffers modified during the execution of a command and change the "add a boundary in current-buffer at the end of a command" to "add boundaries in all modified buffers at the end of a command". Stefan