Re: [O] [patch][babel] `org-babel-result-end' bug fix and regression tests

[Eric Schulte]

Hi Martyn,

Unfortunately there is no way to remove raw results because there is no
way to know where the results end.  While your patch will certainly work
most of the time, it will not work in cases where the results includes
an empty line, and ultimately I think any attempt to remove raw results
will result in confusion.

If removable raw results are desired then the ":results wrap" option may
be used.  I believe this is mentioned in the manual (if not it should
be).

I think this patch should not be applied (although maybe some of the
test cases could still be useful).

Thanks,

Martyn Jago <address@hidden> writes:

> `org-babel-result-end' bug fix and `org-babel-remove-result' regression tests.
>
> * lisp/ob.el:
>
> The code block below will currently act as though :results prepend 
> is set. This is due to `org-babel-result-end' being unable to
> find the correct end of a raw result. This patch fixes that.
>
> #+begin_src emacs-lisp :results raw
> "a line"
> #+end_src
>
> #+results:
> a line
> a line
>
> * testing/lisp/test-ob.el:
>
> Several regression tests that test the correct (multiple) execution of
> code blocks in the various results formats. The tests also test that
> 'org-babel-remove-result' correctly removes the result.
>
> Best, Martyn
>
>

-- 
Eric Schulte
http://cs.unm.edu/~eschulte/