Re: [PATCH] lisp/ob-screen.el: Support ~:var~ header args for babel blocks

[Max Nikulin]

On 25/02/2023 01:33, Ken Mankoff wrote:
> Subject: [PATCH] lisp/ob-screen.el: Support ~:var~ header args for babel
>  blocks

Thank you for the patch. I suppose, suggestion to send patches as 
attachments is added mostly to prevent patches inside HTML message body. 
If a message could be feed to "git am" then it should be OK.

I am unsure if Org markup is suitable for commit messages (~:var~) since 
it may appear in Emacs commit logs.

> +++ b/lisp/ob-screen.el
...
> +(defun org-babel-variable-assignments:screen (params)
> +  "Return list of shell statements assigning the block's variables."
> +  ;; From ob-shell but change function name from ":shell" to ":screen"
...
> +       (if (string-suffix-p "bash" shell-file-name)
> +          (org-babel--variable-assignments:bash

The code still depends on on ob-shell. Is there a reason why calling 
`org-babel-variable-assignments:shell' is a worse variant than copy of 
the whole function body?