[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [patch][babel] `org-babel-result-end' bug fix and regression tes
Re: [O] [patch][babel] `org-babel-result-end' bug fix and regression tests
Fri, 06 Jan 2012 18:37:06 +0000
Gnus/5.13 (Gnus v5.13) Emacs/24.0.92 (darwin)
Eric Schulte <address@hidden> writes:
> 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.
Yes I appreciate that would be a problem if there were empty lines. I just
thought that most of the time is better than none of the time, but I
understand your choice. Certainly wrap works fine.
I have noticed one small issue regarding :results wrap, which is that an
extra newline is appended to the end of the result each time
`org-babel-execute-src-block' is executed. I imagine this can be safely
There are a few other minor issues also with `org-babel-remove-results'
that probably should be fixed at some time (regarding append and
> I think this patch should not be applied (although maybe some of the
> test cases could still be useful).
All the tests supplied with the exception of
`test-ob/org-babel-remove-result--results-raw' will still pass without