Re: [O] [BUG] org-table-beginning/end-of-field

2014-09-08 Thread Nicolas Goaziou
Hello,

Florian Beck  writes:

> The argument of `org-table-beginning-of-field' and
> `org-table-end-of-field' is in fact not optional.

Thanks for the patch. Though, wouldn't it make more sense to properly
handle a missing argument instead?


Regards,

-- 
Nicolas Goaziou



[O] [BUG] org-table-beginning/end-of-field

2014-09-07 Thread Florian Beck
Hi,

The argument of `org-table-beginning-of-field' and
`org-table-end-of-field' is in fact not optional.

-- 
Florian Beck
>From d1d12380a1c260bef7a2137831434614f7d9ec1f Mon Sep 17 00:00:00 2001
From: Florian Beck 
Date: Mon, 8 Sep 2014 07:34:56 +0200
Subject: [PATCH] Remove &optional argument spec

* lisp/org-table.el (org-table-beginning-of-field): fix argument
(org-table-end-of-field): fix argument
---
 lisp/org-table.el | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lisp/org-table.el b/lisp/org-table.el
index 547f933..a17e95a 100644
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -1061,7 +1061,7 @@ Before doing so, re-align the table if necessary."
   (if (looking-at "| ?")
   (goto-char (match-end 0
 
-(defun org-table-beginning-of-field (&optional n)
+(defun org-table-beginning-of-field (n)
   "Move to the end of the current table field.
 If already at or after the end, move to the end of the next table field.
 With numeric argument N, move N-1 fields forward first."
@@ -1076,7 +1076,7 @@ With numeric argument N, move N-1 fields forward first."
   (and (looking-at " ") (forward-char 1)))
 (if (>= (point) pos) (org-table-beginning-of-field 2
 
-(defun org-table-end-of-field (&optional n)
+(defun org-table-end-of-field (n)
   "Move to the beginning of the current table field.
 If already at or before the beginning, move to the beginning of the
 previous field.
-- 
1.9.1