From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bernt Hansen Subject: Re: [PATCH] Add org-clock-clocking-in for hooks Date: Thu, 20 May 2010 11:43:01 -0400 Message-ID: <871vd64m0q.fsf@gollum.intra.norang.ca> References: <1274368441-4727-1-git-send-email-bernt@norang.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from [140.186.70.92] (port=49966 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OF7tu-0004Ez-Lu for emacs-orgmode@gnu.org; Thu, 20 May 2010 11:43:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OF7tp-0007go-47 for emacs-orgmode@gnu.org; Thu, 20 May 2010 11:43:10 -0400 Received: from mho-01-ewr.mailhop.org ([204.13.248.71]:62589) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OF7to-0007gb-Rv for emacs-orgmode@gnu.org; Thu, 20 May 2010 11:43:05 -0400 Received: from cpe000102d0fe75-cm0012256ecbde.cpe.net.cable.rogers.com ([99.239.148.180] helo=mail.norang.ca) by mho-01-ewr.mailhop.org with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.68) (envelope-from ) id 1OF7to-0002At-18 for emacs-orgmode@gnu.org; Thu, 20 May 2010 15:43:04 +0000 In-Reply-To: <1274368441-4727-1-git-send-email-bernt@norang.ca> (Bernt Hansen's message of "Thu\, 20 May 2010 11\:14\:01 -0400") List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: emacs-orgmode@gnu.org Bernt Hansen writes: > org-clock-out-hook can now query org-clock-clocking-in to see if org-clock-out is > being called inside org-clock-in. This allows the hook to selectively clock > in another task without leaving clocks open. OOPS. Please don't apply this just yet. I originally called my new var org-clock-in-active but found there's a org-clock-is-active already and thought it was too similar. So I changed it to org-clock-clocking-in... and it turns out we have one of those already. New patch to follow. Thanks, Bernt