Re: [O] [PATCH] Noweb expansion doesn't duplicate prefix across lines

2014-04-03 Thread Pontus Michael
Hello Eric. I produced a list of arguments in the support of this change, which can be reviewed here: http://lists.gnu.org/archive/html/emacs-orgmode/2014-03/msg01543.html Please let me know if your question still stands after taking those concerns into account.

[O] [PATCH] Noweb expansion doesn't duplicate prefix across lines

2014-03-31 Thread Pontus Michael
* lisp/ob-core.el (org-babel-expand-noweb-references): Change the the behavior of noweb expansion by replacing the prefix duplication with whitespace. Fix handling of multiple noweb references on the same line by using temporary buffer to replace noweb references in place instead of

Re: [O] Proposal to replace the prefix repetition with whitespace during expansion of noweb references

2014-03-30 Thread Pontus Michael
Hello, Charles. Thank you for your reply. I appreciate your suggestion, however the example which I used is not a limiting factor which I seek help in overcoming. Instead I'm trying to address a potential design flaw. You make a valid point that change of this nature will inevitably break

[O] Proposal to replace the prefix repetition with whitespace during expansion of noweb references

2014-03-28 Thread Pontus Michael
I would like to propose a change to the behavior of noweb expansion in relation to prefix handling. Current behavior intends to insert the text preceding noweb reference before every successive line of referenced code. For instance the expansion of following source block: #+begin_src emacs-lisp

Re: [O] [PATCH] Source block fontification handling indentation

2014-03-25 Thread Pontus Michael
examples of both correct and incorrect behavior. On 21 March 2014 12:23, Bastien b...@gnu.org wrote: Hi Michael, thanks for the patch. Pontus Michael m.pon...@gmail.com writes: Primary reason for this change is to fix the problem which I describe as follows: This function

Re: [O] [PATCH] Source block fontification handling indentation

2014-03-25 Thread Pontus Michael
Sorry, the code in last source block in my attachment got a little bit messed up. I corrected it in this attachment. On 21 March 2014 12:23, Bastien b...@gnu.org wrote: Hi Michael, thanks for the patch. Pontus Michael m.pon...@gmail.com writes: Primary reason for this change is to fix

[O] [PATCH] Source block fontification handling indentation

2014-02-28 Thread Pontus Michael
I make a change to function which copies contents of source code block to temporary buffer, applies fontification through the means of relevant major mode and transcribes text-properties back to org-buffer so that they are applied on top of code in affected source code block. Primary reason for