From mboxrd@z Thu Jan 1 00:00:00 1970 From: Carsten Dominik Subject: Re: [PATCH] org-table: reference to free variable `line' Date: Thu, 25 Mar 2010 12:47:59 +0100 Message-ID: References: <87d3ysd4w9.fsf@eee.lan> Mime-Version: 1.0 (Apple Message framework v936) Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes Content-Transfer-Encoding: 7bit Return-path: Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1NulXh-0004I3-9I for emacs-orgmode@gnu.org; Thu, 25 Mar 2010 07:48:05 -0400 Received: from [140.186.70.92] (port=50773 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NulXf-0004DS-I2 for emacs-orgmode@gnu.org; Thu, 25 Mar 2010 07:48:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1NulXe-0008It-A9 for emacs-orgmode@gnu.org; Thu, 25 Mar 2010 07:48:03 -0400 Received: from mail-ew0-f224.google.com ([209.85.219.224]:61067) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1NulXe-0008FT-5n for emacs-orgmode@gnu.org; Thu, 25 Mar 2010 07:48:02 -0400 Received: by mail-ew0-f224.google.com with SMTP id 24so3299356ewy.15 for ; Thu, 25 Mar 2010 04:48:01 -0700 (PDT) In-Reply-To: <87d3ysd4w9.fsf@eee.lan> 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: Mikael Fornius Cc: emacs-orgmode@gnu.org Applied, thanks. - Carsten On Mar 25, 2010, at 12:20 PM, Mikael Fornius wrote: > > As I compiled org-mode today I noticed a free variable and I think > this > change fixes it. > > diff --git a/lisp/org-table.el b/lisp/org-table.el > index 670edcb..9614608 100644 > --- a/lisp/org-table.el > +++ b/lisp/org-table.el > @@ -1043,8 +1043,8 @@ Return t when the line exists, nil if it does > not exist." > (goto-char (org-table-begin)) > (let ((end (org-table-end)) (cnt 0)) > (while (and (re-search-forward org-table-dataline-regexp end t) > - (< (setq cnt (1+ cnt)) line))) > - (= cnt line))) > + (< (setq cnt (1+ cnt)) N))) > + (= cnt N))) > > (defun org-table-blank-field () > "Blank the current table field or active region." > > -- > Mikael Fornius > _______________________________________________ > Emacs-orgmode mailing list > Please use `Reply All' to send replies to the list. > Emacs-orgmode@gnu.org > http://lists.gnu.org/mailman/listinfo/emacs-orgmode - Carsten