org-babel-parse-src-block-match params parse bug

org-babel-parse-src-block-match params parse bug

[Karl Berg]

[-- Attachment #1: Type: text/plain, Size: 404 bytes --]


Hi, 

Seems like the org-babel-parse-src-block-match function does something
wrong when parsing the block parameters. In my case I am trying to use the :file param with plantuml but it does not get down to the execute function.
To me it seems like the (string-match "-i\\>" switches) call messes up the (match-string
4) call further down.

Attaching a patch
Best regards,
Karl Berg 		 	   		  

[-- Attachment #2: org-babel.patch --]
[-- Type: application/octet-stream, Size: 798 bytes --]

--- ob.el.orig	2012-05-16 11:15:25.000000000 +0200
+++ ob.el	2012-05-16 11:14:58.000000000 +0200
@@ -1148,6 +1148,7 @@
 	 (lang (org-babel-clean-text-properties (match-string 2)))
          (lang-headers (intern (concat "org-babel-default-header-args:" lang)))
 	 (switches (match-string 3))
+     (params (match-string 4))
          (body (org-babel-clean-text-properties
 		(let* ((body (match-string 5))
 		       (sub-length (- (length body) 1)))
@@ -1169,7 +1170,7 @@
            (org-babel-params-from-properties lang)
 	   (if (boundp lang-headers) (eval lang-headers) nil)
 	   (org-babel-parse-header-arguments
-            (org-babel-clean-text-properties (or (match-string 4) ""))))
+            (org-babel-clean-text-properties (or params ""))))
 	  switches
 	  block-indentation)))
 

Re: org-babel-parse-src-block-match params parse bug

[Eric Schulte]

Karl Berg <kalle465@hotmail.com> writes:

> Hi, 
>
> Seems like the org-babel-parse-src-block-match function does something
> wrong when parsing the block parameters. In my case I am trying to use the :file param with plantuml but it does not get down to the execute function.
> To me it seems like the (string-match "-i\\>" switches) call messes up the (match-string
> 4) call further down.
>
> Attaching a patch
> Best regards,
> Karl Berg 		 	   		  
>

Please ignore my previous request.  I've just pushed up a fix for this
issue.  Please let me know if the problem persists, and thanks for
pointing this out.

Best,

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