lilypond-devel
[Top][All Lists]
Advanced

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

Re: Revert "Load only the default font for System_start_delimiter" (issu


From: lemzwerg
Subject: Re: Revert "Load only the default font for System_start_delimiter" (issue 557670043 by address@hidden)
Date: Mon, 13 Apr 2020 04:58:53 -0700

some nits


https://codereview.appspot.com/557670043/diff/569610043/input/regression/system-start-brace-style.ly
File input/regression/system-start-brace-style.ly (right):

https://codereview.appspot.com/557670043/diff/569610043/input/regression/system-start-brace-style.ly#newcode4
input/regression/system-start-brace-style.ly:4: SystemStartGrob's style
of @code{StaffGroup} to @code{'brace}, always prints a
@code{SystemStart@var{Grob}} – and no comma after 'brace

https://codereview.appspot.com/557670043/diff/569610043/input/regression/system-start-brace-style.ly#newcode6
input/regression/system-start-brace-style.ly:6: Every @code{StaffGroup}
should start with a @code{SystemStartBrace}.
If you want a new paragraph, please add an empty line before the
sentence.

https://codereview.appspot.com/557670043/diff/569610043/lily/system-start-delimiter.cc
File lily/system-start-delimiter.cc (right):

https://codereview.appspot.com/557670043/diff/569610043/lily/system-start-delimiter.cc#newcode151
lily/system-start-delimiter.cc:151: esp. because that triggers mktextfm
for non-existent
mktextfm is no longer relevant to LilyPond since we don't have a TeX
back-end anymore.  Maybe the comment should be just

  We use the style sheet to look up the font file
  name.  This is better than using 'find_font' directly.

https://codereview.appspot.com/557670043/



reply via email to

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