Build: un-nodify markup-*-commands.tely

Build: un-nodify markup-*-commands.tely

[Graham Percival (old account), 2010-09-03 17:17:22]

This isn't a miracle of clean scheme, but I don't think it's completely
horrible, gets the job done, and makes the doc-editor's lives easier.

[Carl, 2010-09-03 17:41:37]

Looks acceptable to me, with one slight change if it's possible.

THanks,

Carl

http://codereview.appspot.com/2140041/diff/2001/3006
File scm/documentation-generate.scm (right):

http://codereview.appspot.com/2140041/diff/2001/3006#newcode56
scm/documentation-generate.scm:56: ;; magic number to remove the initial part
Can we have something in the comment that says what would have to change in
order to change the magic number?

[Graham Percival (old account), 2010-09-03 18:01:28]

On 2010/09/03 17:41:37, Carl wrote:
> scm/documentation-generate.scm:56: ;; magic number to remove the initial part
> Can we have something in the comment that says what would have to change in
> order to change the magic number?

Thanks for the comment; done.  I should have done this math to begin with,
instead of determining it via experimental error.  :)

Cheers,
- Graham

[c_sorensen, 2010-09-03 18:36:10]

LGTM.

Carl


On 9/3/10 12:01 PM, "percival.music.ca@gmail.com"
<percival.music.ca@gmail.com> wrote:

> On 2010/09/03 17:41:37, Carl wrote:
>> scm/documentation-generate.scm:56: ;; magic number to remove the
> initial part
>> Can we have something in the comment that says what would have to
> change in
>> order to change the magic number?
> 
> Thanks for the comment; done.  I should have done this math to begin
> with, instead of determining it via experimental error.  :)
> 
> Cheers,
> - Graham
> 
> http://codereview.appspot.com/2140041/