LGTM, apart from a query https://codereview.appspot.com/12945044/diff/1/lily/ledger-line-spanner.cc File lily/ledger-line-spanner.cc (right): https://codereview.appspot.com/12945044/diff/1/lily/ledger-line-spanner.cc#newcode331 lily/ledger-line-spanner.cc:331: " property of the ...
10 years, 8 months ago
(2013-08-14 21:35:16 UTC)
#1
On 2013/08/15 06:59:58, Mark Polesky wrote: > On 2013/08/14 21:35:16, Trevor Daniels wrote: > > ...
10 years, 8 months ago
(2013-08-15 07:24:48 UTC)
#3
On 2013/08/15 06:59:58, Mark Polesky wrote:
> On 2013/08/14 21:35:16, Trevor Daniels wrote:
> > Should this not be @ref{...}?
>
> No. @rinternals{...}. See
>
http://lilypond.org/doc/v2.17/Documentation/contributor/syntax-survey#cross-r...
>
> - Mark
That documentation states as first entry:
@ref{…} — link within current manual.
On 2013/08/15 07:24:48, dak wrote: > That documentation states as first entry: > > @ref{…} ...
10 years, 8 months ago
(2013-08-15 09:43:33 UTC)
#4
On 2013/08/15 07:24:48, dak wrote:
> That documentation states as first entry:
>
> @ref{…} — link within current manual.
Well, shoot. That's why we have a countdown. Sorry for the
mistake. Just, um, asserting my humanity, I guess...
Anyway, I made the change.
- Mark
Issue 12945044: Remove 'thickness from LedgerLineSpanner interface.
(Closed)
Created 10 years, 8 months ago by Mark Polesky
Modified 10 years, 8 months ago
Reviewers: Trevor Daniels, dak, janek
Base URL:
Comments: 1