Re: Have SRC_BLOCK :padline accept numbers

[Nicolas Goaziou <mail@nicolasgoaziou.fr>]

Hello,

Daniel P Gomez <gomez.danp@gmail.com> writes:

> I've amended the commit including the changes you've mentioned.
> I did not change the following, though:
>
> (numberp (string-to-int padlines)) -> (string-match-p "\\`[0-9]+\\'" padlines)
>
> because that changed the behaviour of org-babel-tangle. Having a
> string-match-p as a
> match condition broke the default of adding no padlines in the
> first tangled SRC block (when passed with no arguments).

`string-to-int' will fail the same way as `string-match-p' if `padlines'
is nil. So I do not see how the behaviour could differ. Could you
elaborate on the issue?

> I also noticed that adding this functionality to ob-tangle breaks eldoc
> for me. I guess all org babel header arguments have to be strings?

I didn't check, but that seems a reasonable assumption.

BTW, you didn't include the patch, did you?

Regards,

-- 
Nicolas Goaziou                                                0x80A93738