From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Chong Yidong Newsgroups: gmane.emacs.devel Subject: Re: blink-cursor-end sometimes fails and disrupts pre-command-hook Date: Sat, 19 Aug 2006 11:16:39 -0400 Message-ID: <87irko92nc.fsf@furball.mit.edu> References: <2cd46e7f0608181530l1c1a18f7k1923f610edf73f59@mail.gmail.com> <87veoo95zy.fsf@furball.mit.edu> <2cd46e7f0608190731j6247e8bbr7f7f8d1ecb6ac3ce@mail.gmail.com> <87r6zc9475.fsf@furball.mit.edu> <2cd46e7f0608190805x70bd8715n6f1b552d57a80a5c@mail.gmail.com> NNTP-Posting-Host: main.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1156001080 23036 80.91.229.2 (19 Aug 2006 15:24:40 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Sat, 19 Aug 2006 15:24:40 +0000 (UTC) Cc: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Aug 19 17:24:38 2006 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by ciao.gmane.org with esmtp (Exim 4.43) id 1GESgc-0008Ap-TK for ged-emacs-devel@m.gmane.org; Sat, 19 Aug 2006 17:24:35 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1GESgc-0003Wx-5D for ged-emacs-devel@m.gmane.org; Sat, 19 Aug 2006 11:24:34 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1GESfn-0002x3-FY for emacs-devel@gnu.org; Sat, 19 Aug 2006 11:23:43 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1GESfn-0002we-1t for emacs-devel@gnu.org; Sat, 19 Aug 2006 11:23:43 -0400 Original-Received: from [199.232.41.67] (helo=mx20.gnu.org) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA:32) (Exim 4.52) id 1GESmk-00050Q-Fn for emacs-devel@gnu.org; Sat, 19 Aug 2006 11:30:54 -0400 Original-Received: from [18.72.1.2] (helo=south-station-annex.mit.edu) by mx20.gnu.org with esmtp (Exim 4.52) id 1GESfm-0000CZ-By for emacs-devel@gnu.org; Sat, 19 Aug 2006 11:23:42 -0400 Original-Received: from central-city-carrier-station.mit.edu (CENTRAL-CITY-CARRIER-STATION.MIT.EDU [18.7.7.72]) by south-station-annex.mit.edu (8.13.6/8.9.2) with ESMTP id k7JFNZUM028280; Sat, 19 Aug 2006 11:23:37 -0400 (EDT) Original-Received: from outgoing-legacy.mit.edu (OUTGOING-LEGACY.MIT.EDU [18.7.22.104]) by central-city-carrier-station.mit.edu (8.13.6/8.9.2) with ESMTP id k7JFGbhf029889; Sat, 19 Aug 2006 11:16:37 -0400 (EDT) Original-Received: from furball.mit.edu (SYDNEYPACIFIC-THREE-NINETY-NINE.MIT.EDU [18.95.6.144]) ) by outgoing-legacy.mit.edu (8.13.6/8.12.4) with ESMTP id k7JFGZfP011210; Sat, 19 Aug 2006 11:16:35 -0400 (EDT) Original-Received: from cyd by furball.mit.edu with local (Exim 3.36 #1 (Debian)) id 1GESYx-0000o7-00; Sat, 19 Aug 2006 11:16:39 -0400 Original-To: "Ken Manheimer" In-Reply-To: <2cd46e7f0608190805x70bd8715n6f1b552d57a80a5c@mail.gmail.com> (Ken Manheimer's message of "Sat, 19 Aug 2006 11:05:35 -0400") User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.0.50 (gnu/linux) X-Spam-Score: 1.217 X-Scanned-By: MIMEDefang 2.42 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:58518 Archived-At: "Ken Manheimer" writes: > i see now - blink-cursor-start is called by the timer, which sets > blink-cursor-end on the pre-command-hook, does some cursor visibility > business, and then reestablishes the blink-cursor-timer. > > seems to me it would be more reliable to set the blink-cursor-timer > before adding blink-cursor-end to the pre-command-hook! > > i just tried making this change and removing the change from > blink-cursor-end (by putting revised copies of both functions in my > emacs rc), and the problem is not happening. i would suggest going > with both changes - blink-cursor-end with the condition i suggested > adding, and blink-cursor-start looking like: Bingo --- I think that's it. Something in your customizations must be causing the run-with-timer call in blink-cursor-start to signal an error. One possibility is an invalid `blink-cursor-interval' variable. Since blink-cursor-start is run from an (idle) timer, quitting is disabled. The question is, why did this surface only with the latest Emacs update?