From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: Emacs 22.2.90 pretest Date: Sat, 16 Aug 2008 11:41:51 +0300 Message-ID: References: <87y72ymcig.fsf@stupidchicken.com> Reply-To: Eli Zaretskii NNTP-Posting-Host: lo.gmane.org X-Trace: ger.gmane.org 1218876234 8727 80.91.229.12 (16 Aug 2008 08:43:54 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sat, 16 Aug 2008 08:43:54 +0000 (UTC) To: cyd@stupidchicken.com, emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Aug 16 10:44:46 2008 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.50) id 1KUHOp-0005fz-Iq for ged-emacs-devel@m.gmane.org; Sat, 16 Aug 2008 10:44:39 +0200 Original-Received: from localhost ([127.0.0.1]:58946 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KUHNs-0001js-2d for ged-emacs-devel@m.gmane.org; Sat, 16 Aug 2008 04:43:40 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KUHMv-0001PM-T8 for emacs-devel@gnu.org; Sat, 16 Aug 2008 04:42:42 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KUHMt-0001OL-T8 for emacs-devel@gnu.org; Sat, 16 Aug 2008 04:42:40 -0400 Original-Received: from [199.232.76.173] (port=42362 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KUHMt-0001OA-4U for emacs-devel@gnu.org; Sat, 16 Aug 2008 04:42:39 -0400 Original-Received: from mtaout6.012.net.il ([84.95.2.16]:46418) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1KUHMs-0003JE-CW for emacs-devel@gnu.org; Sat, 16 Aug 2008 04:42:38 -0400 Original-Received: from HOME-C4E4A596F7 ([84.229.211.50]) by i-mtaout6.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0K5O00EA9QUEV350@i-mtaout6.012.net.il> for emacs-devel@gnu.org; Sat, 16 Aug 2008 11:42:15 +0300 (IDT) In-reply-to: X-012-Sender: halo1@inter.net.il X-detected-kernel: by monty-python.gnu.org: Solaris 10 (1203?) 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:102516 Archived-At: > Date: Sat, 16 Aug 2008 10:57:26 +0300 > From: Eli Zaretskii > Cc: emacs-devel@gnu.org > > > From: Chong Yidong > > Date: Fri, 15 Aug 2008 13:08:55 -0400 > > > > Emacs pretest version 22.2.90 is now available. This is the first > > pretest for Emacs 22.3, which will be a bugfix release. > > > > The source tarball is available here: > > > > http://alpha.gnu.org/gnu/emacs/pretest/emacs-22.2.90.tar.gz > > > > The xdelta against Emacs 22.2 is here: > > > > http://alpha.gnu.org/gnu/emacs/pretest/emacs-22.2-22.2.90.xdelta > > > > The CVS tag is EMACS_PRETEST_22_2_90. > > Thanks! I notice that in this pretest, moving point with C-f and C-b or inserting a single character is very sluggish: e.g., if I continuously press C-f, Emacs cannot keep up(!), although this is a 3.2 GHz machine. It almost feels like working on a remote machine. This recent change seems to be a likely suspect: 2008-07-28 Chong Yidong * textmodes/flyspell.el (flyspell-word, flyspell-large-region) (flyspell-region): Call ispell-maybe-find-aspell-dictionaries. It seems that its effect is to call ispell-maybe-find-aspell-dictionaries on every editing command, which is silly, IMO. Even if we do need to do that on every command (and I'd like to hear a reason why), the call should only be made if Ispell is actually Aspell, which in my case it isn't. On top of that, ispell-maybe-find-aspell-dictionaries obviously was not designed to be called frequently: it calls ispell-check-version, which is expensive and is supposed to be run once in an Ispell session (it makes unnecessary destructive changes to the Ispell buffer, invokes another Ispell process, etc.). Why was this change made?