From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Subject: Re: [ANN] Exporter for taskjuggler Date: Tue, 08 Jun 2010 17:59:09 +0100 Message-ID: References: <87ljdarxpw.fsf@saadawi.sbszh.ch> <874ojbgmi8.fsf@stats.ox.ac.uk> <87wrw3dag8.fsf@saadawi.sbszh.ch> <87k4s21r6m.fsf@saadawi.sbszh.ch> <871vchubgh.fsf@saadawi.sbszh.ch> <8EB5419B-3737-401F-978F-50FB69580BB4@gmail.com> <87wru9slcj.fsf@saadawi.sbszh.ch> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from [140.186.70.92] (port=51576 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OM296-0003dq-RG for emacs-orgmode@gnu.org; Tue, 08 Jun 2010 12:59:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OM295-0005RX-NI for emacs-orgmode@gnu.org; Tue, 08 Jun 2010 12:59:24 -0400 Received: from lo.gmane.org ([80.91.229.12]:39129) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OM295-0005RI-BX for emacs-orgmode@gnu.org; Tue, 08 Jun 2010 12:59:23 -0400 Received: from list by lo.gmane.org with local (Exim 4.69) (envelope-from ) id 1OM291-0007M2-SE for emacs-orgmode@gnu.org; Tue, 08 Jun 2010 18:59:19 +0200 Received: from cpc1-cmbg13-0-0-cust596.cmbg.cable.ntl.com ([86.9.122.85]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Tue, 08 Jun 2010 18:59:19 +0200 Received: from sdl.web by cpc1-cmbg13-0-0-cust596.cmbg.cable.ntl.com with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Tue, 08 Jun 2010 18:59:19 +0200 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 On 2010-06-08 14:30 +0100, Christian Egli wrote: >> In org-taskjuggler-close-maybe: >> org-taskjuggler.el:633:13:Warning: reference to free variable `old- >> level' >> org-taskjuggler.el:635:25:Warning: assignment to free variable `old- >> level' > > I just pushed a patch to the taskjuggler-export branch that should fix > this. I suppose old-level is a let-bound dynamic variable. I have found code giving a seemingly global variable name to such variables confusing at times. Would be better just make it global or insert a form at the beginning of the body of those functions like this: (declare (special dynvar1 dynvar2 ...)) I usually use the latter approach. Cheers. Leo -- CCL-USER> (if you fail to plan (plan to fail))