lilypond-devel
[Top][All Lists]
Advanced

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

Re: Issue 3330: redo much of the stencil stacking/spacing/empty-check (i


From: dak
Subject: Re: Issue 3330: redo much of the stencil stacking/spacing/empty-check (issue 8869044)
Date: Tue, 30 Apr 2013 05:04:24 +0000


https://codereview.appspot.com/8869044/diff/22012/scm/stencil.scm
File scm/stencil.scm (right):

https://codereview.appspot.com/8869044/diff/22012/scm/stencil.scm#newcode46
scm/stencil.scm:46: (lambda (tail head)
On 2013/04/30 03:10:52, Keith wrote:
Usually, the head is fixed-length and the tail grows, so maybe
'accumulated_stencil' and 'next_stencil'

Don't like those.  Perhaps front and next or so.

https://codereview.appspot.com/8869044/diff/22012/scm/stencil.scm#newcode56
scm/stencil.scm:56: (let loop ((head (car stils)) (rest (cdr stils)))
On 2013/04/30 03:10:52, Keith wrote:
Somewhere you need to step through the list of 'padding', in order to
do what
the old code did.

Huh.  And I've not been using "reduce" here exactly because it does not
allow for several list variables.  Embarrassing.

https://codereview.appspot.com/8869044/



reply via email to

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