Re: Table Of Contents crash with negative first-page-number (issue 258740043 by address@hidden)

[dak]

https://codereview.appspot.com/258740043/diff/1/scm/define-markup-commands.scm
File scm/define-markup-commands.scm (right):

https://codereview.appspot.com/258740043/diff/1/scm/define-markup-commands.scm#newcode542
scm/define-markup-commands.scm:542: (+ table-page-number (+ 1 (* -1
first-page-number))))
Uh, (- table-page-number (1- first-page-number)) ?

Do the links actually work when first-page-number is, say, 10?  If so,
did they not work before?

https://codereview.appspot.com/258740043/diff/1/scm/define-markup-commands.scm#newcode548
scm/define-markup-commands.scm:548: (ly:stencil-add
What's the reason for delaying this part of the stencil expression as
well?

https://codereview.appspot.com/258740043/