Re: [O] Release Org 8.2.8

2014-10-03 Thread Pascal Wittmann
On 10/03/2014 10:18 AM, Bastien wrote:
 I've released Org 8.2.8, a bugfix release.
 Thanks to everyone who worked on it.

Thanks! I've already updated the package on NixOS.



signature.asc
Description: OpenPGP digital signature


[O] orgtbl-insert-radio-tableĀ forbid blanks in the table name

2012-10-09 Thread Pascal Wittmann
Hi,

I noticed that its possible to enter a name containing blanks when
calling orgtbl-insert-radio-table. I don't think that this is intended,
because blanks are used as separators in the #+ORGTBL line.

Entering a the name first table will result in the following error
message:

No such transformation function table

For me (as a novice) it was at first not obvious what I did wrong. My
suggestion is to chose an other input method that forbids blanks.

I fixed this locally with the attached patch, but I'm sure there are
better solutions (I find it unintuitive that blanks terminate the input).

Best Regards
Pascal
diff --git a/lisp/org-table.el b/lisp/org-table.el
index 0555041..a8d2826 100644
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -4437,7 +4437,7 @@ First element has index 0, or I0 if given.
 	 (txt (nth 1 e))
 	 name pos)
 (unless e (error No radio table setup defined for %s major-mode))
-(setq name (read-string Table name: ))
+(setq name (read-no-blanks-input Table name: ))
 (while (string-match %n txt)
   (setq txt (replace-match name t t txt)))
 (or (bolp) (insert \n))


signature.asc
Description: OpenPGP digital signature