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 <f...@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
_______________________________________________
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
