[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape fo
From: |
Kyle Meyer |
Subject: |
Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby |
Date: |
Wed, 12 Aug 2015 13:21:41 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Matthew MacLean <address@hidden> writes:
> Alright, done. Is this acceptable? (Provided that tests don't count towards
> line count, of course)
Thanks. A few minor comments on the commit message.
> Subject: [PATCH] ob-ruby: Fix double-escaping
>
> * lisp/ob-ruby.el: Remove second call to
> `org-babel-ruby-table-or-string' in `org-babel-ruby-evaluate'.
Please add the name of the changed function in parentheses after the
file name rather than putting it in the description body.
> * testing/lisp/test-ob-ruby.el: Add test to verify
> `org-babel-execute:ruby' can evaluate Ruby code. (What the
> double-escape prevented)
Same here for the test name. "Add test." for description would do.
> I removed the escaping from `org-babel-ruby-evaluate', because the only
> place `org-babel-ruby-evaluate' is ever called is in
> `org-babel-execute:ruby'.
>
> In this function, its result either escaped (Where the double escape
> previously occurred) or passed in as the "scalar-form" of
> `org-babel-result-cond', which handles the "pp" and "code" parameters.
> (A place that doesn't need escaping.)
I think the above two paragraphs could be replaced by a link to this ML
post.
Thanks for working on this.
--
Kyle
- [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Matthew MacLean, 2015/08/11
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Kyle Meyer, 2015/08/11
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Kyle Meyer, 2015/08/11
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Matthew MacLean, 2015/08/11
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Kyle Meyer, 2015/08/11
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Matthew MacLean, 2015/08/11
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Kyle Meyer, 2015/08/12
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Matthew MacLean, 2015/08/12
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby,
Kyle Meyer <=
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Matthew MacLean, 2015/08/12
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Kyle Meyer, 2015/08/12
- Re: [O] fa5fd6351605912ec75e783cb62649 breaks org-babel-script-escape for ob-ruby, Matthew MacLean, 2015/08/13