Re: [babel] double quotes in tables

["Eric Schulte" <schulte.eric@gmail.com>]

Hi Francesco,

Thanks for bringing this up.  I'm thinking that a more aggressive
version of your patch should be applied here.  It seems that org-babel
should not change the values of tables which pass through it.  I'm
leaning towards applying the following lines of your patch

-			     '(:fmt (lambda (cell) (format "%S" cell)))) "\n"))
+			     '(:fmt (lambda (cell) (format outputformat cell)))) "\n"))

That change doesn't seem to break any of our unit tests.

Can anyone think of a reason not to make this change?

Thanks -- Eric

Francesco Pizzolante <fpz@missioncriticalit.com> writes:

> Hi,
>
> Often, I find useful that babel could generate result tables without the
> double quotes (") enclosing all the text values in the cells.
>
> In order to do that, I added a 'noquotes` option to the :results header
> argument.
>
> Here's my patch. As I'm not an elisp expert, please tell me if there's a
> better/safer way to do that or maybe that it already exists an option to do
> that...
>
> diff --git a/contrib/babel/lisp/org-babel.el b/contrib/babel/lisp/org-babel.el
> index 4c9bff5..13c8237 100644
> --- a/contrib/babel/lisp/org-babel.el
> +++ b/contrib/babel/lisp/org-babel.el
> @@ -706,6 +706,9 @@ code ---- the results are extracted in the syntax of the source
>            code of the language being evaluated and are added
>            inside of a #+BEGIN_SRC block with the source-code
>            language set appropriately."
> +  (setq outputformat "%S")
> +  (if (member "noquotes" result-params)
> +      (setq outputformat "%s"))
>    (if (stringp result)
>        (progn
>          (setq result (org-babel-clean-text-properties result))
> @@ -739,7 +742,7 @@ code ---- the results are extracted in the syntax of the source
>  			     (if (and (listp (car result))
>                                        (listp (cdr (car result))))
>  				 result (list result))
> -			     '(:fmt (lambda (cell) (format "%S" cell)))) "\n"))
> +			     '(:fmt (lambda (cell) (format outputformat cell)))) "\n"))
>  	    (forward-line -1) (org-cycle))
>  	   ((member "file" result-params)
>  	    (insert result))
> @@ -827,7 +830,8 @@ parameters when merging lists."
>    (let ((results-exclusive-groups
>  	 '(("file" "vector" "table" "scalar" "raw" "org" "html" "latex" "code" "pp")
>  	   ("replace" "silent")
> -	   ("output" "value")))
> +	   ("output" "value")
> +           ("noquotes")))
>  	(exports-exclusive-groups
>  	 '(("code" "results" "both" "none")))
>  	params results exports tangle cache vars var ref)
>
> Thanks,
> Francesco
>
>
> _______________________________________________
> 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