[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Introduce markup-list-command table (issue 289980043 by address@hidd

From: dak
Subject: Re: Introduce markup-list-command table (issue 289980043 by address@hidden)
Date: Sat, 30 Jan 2016 23:32:17 +0000
File scm/define-markup-commands.scm (right):
scm/define-markup-commands.scm:4767: ;; -> (list 0 1 3 6 10)
It's not clear why rl is even a list if it always has to have exactly
one member.  And the (reverse rl) indicates that this function does
something else.

Also the comment does not mention the function of "padding".  So the
function description tries to be sort-of abstract by avoiding to mention
what the function is actually being used for, but there are lots of
missing pieces which can only be guessed if you actually know what is
being done here.

Bad combination.  Try being more complete.  I suspect that this will
yield to a call of fold, but it's hard to say with the documentation not
really matching the code.
scm/define-markup-commands.scm:4783: (split-lst lst columns '())))
We've had this a few times.  A markup-list? argument is not necessarily
a list of markups.  It can also be a markup list command call.  The only
thing you can reliably do before starting list processing is to call
interpret-markup-list on the _whole_ markup-list argument.  Only _then_
can you start splitting the resulting list of stencils.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]