[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] Have SRC_BLOCK :padline accept numbers
From: |
Nicolas Goaziou |
Subject: |
Re: [O] Have SRC_BLOCK :padline accept numbers |
Date: |
Tue, 18 Apr 2017 10:37:53 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Hello,
Daniel P Gomez <address@hidden> 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
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [O] Have SRC_BLOCK :padline accept numbers,
Nicolas Goaziou <=