From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Dmitry Gutov Newsgroups: gmane.emacs.bugs Subject: bug#12447: 24.1.50; Stuck in garbage collection on OS X Date: Wed, 19 Sep 2012 20:21:32 +0400 Message-ID: <5059F10C.6070806@yandex.ru> References: <505598C8.8070904@yandex.ru> <834nmys1ht.fsf@gnu.org> <5055AD8E.5020309@yandex.ru> <83zk4qqj4d.fsf@gnu.org> <5055C0EB.3040908@yandex.ru> <83wqzuqgzr.fsf@gnu.org> <5055D34F.1040800@yandex.ru> <83txuyqdtv.fsf@gnu.org> <5055E16E.1070604@yandex.ru> <83sjaiqaqb.fsf@gnu.org> <5055F69B.4020004@yandex.ru> <837grr1idg.fsf@gnu.org> <5059115B.8010407@yandex.ru> <83txuuzpqo.fsf@gnu.org> <5059965F.5030708@yandex.ru> <83obl2yr19.fsf@gnu.org> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Trace: ger.gmane.org 1348071776 12595 80.91.229.3 (19 Sep 2012 16:22:56 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Wed, 19 Sep 2012 16:22:56 +0000 (UTC) Cc: hanche@math.ntnu.no, 12447@debbugs.gnu.org To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Wed Sep 19 18:22:58 2012 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 1TEN34-0005G0-Nl for geb-bug-gnu-emacs@m.gmane.org; Wed, 19 Sep 2012 18:22:50 +0200 Original-Received: from localhost ([::1]:35709 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TEN30-0006JM-CE for geb-bug-gnu-emacs@m.gmane.org; Wed, 19 Sep 2012 12:22:46 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:33241) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TEN2x-0006J5-JX for bug-gnu-emacs@gnu.org; Wed, 19 Sep 2012 12:22:44 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TEN2q-0002hf-5e for bug-gnu-emacs@gnu.org; Wed, 19 Sep 2012 12:22:43 -0400 Original-Received: from debbugs.gnu.org ([140.186.70.43]:34317) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TEN2q-0002hV-29 for bug-gnu-emacs@gnu.org; Wed, 19 Sep 2012 12:22:36 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.72) (envelope-from ) id 1TEN4E-000486-7S for bug-gnu-emacs@gnu.org; Wed, 19 Sep 2012 12:24:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Dmitry Gutov Original-Sender: debbugs-submit-bounces@debbugs.gnu.org Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 19 Sep 2012 16:24:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 12447 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 12447-submit@debbugs.gnu.org id=B12447.134807178715800 (code B ref 12447); Wed, 19 Sep 2012 16:24:02 +0000 Original-Received: (at 12447) by debbugs.gnu.org; 19 Sep 2012 16:23:07 +0000 Original-Received: from localhost ([127.0.0.1]:43863 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1TEN3K-00046n-MB for submit@debbugs.gnu.org; Wed, 19 Sep 2012 12:23:06 -0400 Original-Received: from forward11.mail.yandex.net ([95.108.130.93]:39378) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1TEN3G-00046M-PL for 12447@debbugs.gnu.org; Wed, 19 Sep 2012 12:23:05 -0400 Original-Received: from smtp13.mail.yandex.net (smtp13.mail.yandex.net [95.108.130.68]) by forward11.mail.yandex.net (Yandex) with ESMTP id 6138CE82D94; Wed, 19 Sep 2012 20:21:31 +0400 (MSK) Original-Received: from smtp13.mail.yandex.net (localhost [127.0.0.1]) by smtp13.mail.yandex.net (Yandex) with ESMTP id 19E8BE40517; Wed, 19 Sep 2012 20:21:31 +0400 (MSK) Original-Received: from 5x166x246x245.dynamic.spb.ertelecom.ru (5x166x246x245.dynamic.spb.ertelecom.ru [5.166.246.245]) by smtp13.mail.yandex.net (nwsmtp/Yandex) with ESMTP id LUW4uXQT-LUWGSsSi; Wed, 19 Sep 2012 20:21:30 +0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yandex.ru; s=mail; t=1348071691; bh=5qMU3zvRm8sXIu2XFtG+1ooGZUfqXVpPwn956OmqAWQ=; h=Message-ID:Date:From:User-Agent:MIME-Version:To:CC:Subject: References:In-Reply-To:Content-Type:Content-Transfer-Encoding; b=OkDTLCT1m7hHriuBkXHy23pVkMmRAEcwFoM6dsGwarcYzEvepNO09NY5vBNzFGn0N n3CFfOGHA1dVV1ewq9GIy+LzDd90Cx5I77lEGoy9gfi9buk/K8VWX+biD0Al1pbxJY KJGKKLyQn6dZ51Q710lwrhmBivbbeWIfOVy1URO4= User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/20120907 Thunderbird/15.0.1 In-Reply-To: <83obl2yr19.fsf@gnu.org> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.13 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 2) X-Received-From: 140.186.70.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:64605 Archived-At: On 19.09.2012 19:24, Eli Zaretskii wrote: >> Date: Wed, 19 Sep 2012 13:54:39 +0400 >> From: Dmitry Gutov >> CC: jan.h.d@swipnet.se, 12447@debbugs.gnu.org, hanche@math.ntnu.no >> >> On 19.09.2012 6:54, Eli Zaretskii wrote: >>>> I'll keep the workaround, though, as it reportedly also fixes the >>>> long-standing OS X freeze bug. >>> >>> What workaround is that? >> >> The one I posted in #12326: calling timer-activate-when-idle with nil >> DONT-WAIT argument. > > I don't think that's a workaround. I think that's what js2 should do, > if (AFAIU) it wants the timer handler be invoked once, after Emacs has > been idle for more than 1 sec. Maybe, but run-with-idle-timer calls it with DONT-WAIT t, and js2-mode-reset-timer is only supposed to be called in response to some user action. So with your patch, it works effectively the same either way. By the way, here's what run-with-idle-timer docstring says: "Perform an action the next time Emacs is idle for SECS seconds." Shouldn't this mean that it should also pass DONT-WAIT nil? Below that, the docstring mentions what will happen if Emacs has been idle for N seconds (N < SECS), but doesn't clarify that if N => SECS, it will fire immediately. >> https://github.com/mooz/js2-mode/commit/b02f4a0d72d0e2087038fe891e2580c4505415ef > > Btw, in this line: > > (timer-activate-when-idle timer nil))) > > I think you can lose the last argument, since it is optional anyway. Sure, I wrote it that way just to be explicit.