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

2012-01-06 Thread Eric Schulte
Applied, Thanks, Martyn Jago writes: > Hi Eric > > Eric Schulte writes: > > > [...] > >>> >>> All the tests supplied with the exception of >>> `test-ob/org-babel-remove-result--results-raw' will still pass without >>> the change. >>> >> >> Alright, would you be willing to resubmit the patch inc

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

2012-01-06 Thread Martyn Jago
Hi Eric Eric Schulte writes: [...] >> >> All the tests supplied with the exception of >> `test-ob/org-babel-remove-result--results-raw' will still pass without >> the change. >> > > Alright, would you be willing to resubmit the patch including only those > tests which should still apply? This

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

2012-01-06 Thread Eric Schulte
Martyn Jago writes: > Eric Schulte writes: > Hi Eric > >> 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

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

2012-01-06 Thread Martyn Jago
Eric Schulte writes: Hi Eric > 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

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

2012-01-06 Thread 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

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

2012-01-06 Thread Martyn Jago
`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