From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Bastien Newsgroups: gmane.emacs.devel Subject: Re: Timer scheduling and cancel-timer Date: Tue, 22 Jan 2013 14:57:49 +0100 Message-ID: <87ehhdxqua.fsf@bzg.ath.cx> References: <20130121005802.c7383f4994ead287e738f96f@cx4a.org> <50FC43DB.6070505@cs.ucla.edu> <0C917BD4690341A181CF2BE23E754FC7@us.oracle.com> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-Trace: ger.gmane.org 1358863088 10831 80.91.229.3 (22 Jan 2013 13:58:08 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 22 Jan 2013 13:58:08 +0000 (UTC) Cc: 'Tomohiro Matsuyama' , 'Paul Eggert' , Drew Adams , emacs-devel@gnu.org To: Stefan Monnier Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Jan 22 14:58:26 2013 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 1TxeMp-00041J-HN for ged-emacs-devel@m.gmane.org; Tue, 22 Jan 2013 14:58:23 +0100 Original-Received: from localhost ([::1]:36646 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TxeMY-0007X7-CE for ged-emacs-devel@m.gmane.org; Tue, 22 Jan 2013 08:58:06 -0500 Original-Received: from eggs.gnu.org ([208.118.235.92]:43222) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TxeMR-0007Wm-6f for emacs-devel@gnu.org; Tue, 22 Jan 2013 08:58:04 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TxeMN-00082I-8S for emacs-devel@gnu.org; Tue, 22 Jan 2013 08:57:59 -0500 Original-Received: from mail-wg0-f47.google.com ([74.125.82.47]:54714) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TxeMN-00082A-17 for emacs-devel@gnu.org; Tue, 22 Jan 2013 08:57:55 -0500 Original-Received: by mail-wg0-f47.google.com with SMTP id dq11so1444175wgb.26 for ; Tue, 22 Jan 2013 05:57:53 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:sender:from:to:cc:subject:in-reply-to:references :user-agent:date:message-id:mime-version:content-type; bh=Da970qd1T8tDgH7aW84iRY0gFPJJ2GfGqpRarYaQww8=; b=nn0/71CoSUsfU/69Irw0qSn3PPcIwyLqHbMluoAGm0fb80yzVZZDSzV4m8SkQBPQgm BUyPLMZBNIxofbJ0D6V/sjGdmE67Eh0giZYQATT1PV4GRL6MWHciIWdlic4+UlXKbSnV wDZbu1UWsS1u0S8PvkeThiOpdYfGAdBkhy8DJQEZlHZq8CxMBXoRAIwWvMold68zYUrI FCFTJWR1tNV/e74s8qC/Znc0sbOP78OuUUYDnG1uYtvWFJbtV4ohlwqvoNbnSW25/dMW w+Naqm+zvbcvUWA148HatgeBzHyWCMus+hDb/DFEYVA7kd0OD6+deFMrUGbeUUyFV0ve kJrg== X-Received: by 10.180.92.36 with SMTP id cj4mr21377241wib.23.1358863073796; Tue, 22 Jan 2013 05:57:53 -0800 (PST) Original-Received: from bzg.localdomain (62-193-54-78.stella-net.net. [62.193.54.78]) by mx.google.com with ESMTPS id e6sm23976813wiz.1.2013.01.22.05.57.52 (version=TLSv1.1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Tue, 22 Jan 2013 05:57:53 -0800 (PST) Original-Received: by bzg.localdomain (Postfix, from userid 1000) id 9997D1C205A4; Tue, 22 Jan 2013 14:57:49 +0100 (CET) In-Reply-To: (Stefan Monnier's message of "Tue, 22 Jan 2013 08:34:04 -0500") User-Agent: Gnus/5.130006 (Ma Gnus v0.6) Emacs/24.3.50 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 74.125.82.47 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:156579 Archived-At: --=-=-= Content-Type: text/plain Stefan Monnier writes: >> Perhaps it is worth reverting the code change you made that makes this happen >> now, until you find the bug? > > No, the change is there specifically to avoid hiding bugs under the > rug. The attached patch fixes the bug. Can anyone confirm this? --=-=-= Content-Type: text/x-patch Content-Disposition: inline; filename=paren.el.patch === modified file 'lisp/paren.el' --- lisp/paren.el 2013-01-01 09:11:05 +0000 +++ lisp/paren.el 2013-01-22 13:55:42 +0000 @@ -243,23 +243,23 @@ ;; ;; Turn on highlighting for the matching paren, if found. ;; If it's an unmatched paren, turn off any such highlighting. - (unless (integerp pos) - (delete-overlay show-paren-overlay)) - (let ((to (if (or (eq show-paren-style 'expression) - (and (eq show-paren-style 'mixed) - (not (pos-visible-in-window-p pos)))) - (point) - pos)) - (from (if (or (eq show-paren-style 'expression) + (if (not (integerp pos)) + (when show-paren-overlay (delete-overlay show-paren-overlay)) + (let ((to (if (or (eq show-paren-style 'expression) (and (eq show-paren-style 'mixed) (not (pos-visible-in-window-p pos)))) - pos - (save-excursion - (goto-char pos) - (- (point) dir))))) - (if show-paren-overlay - (move-overlay show-paren-overlay from to (current-buffer)) - (setq show-paren-overlay (make-overlay from to nil t)))) + (point) + pos)) + (from (if (or (eq show-paren-style 'expression) + (and (eq show-paren-style 'mixed) + (not (pos-visible-in-window-p pos)))) + pos + (save-excursion + (goto-char pos) + (- (point) dir))))) + (if show-paren-overlay + (move-overlay show-paren-overlay from to (current-buffer)) + (setq show-paren-overlay (make-overlay from to nil t))))) ;; ;; Always set the overlay face, since it varies. (overlay-put show-paren-overlay 'priority show-paren-priority) --=-=-= Content-Type: text/plain -- Bastien --=-=-=--