[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Set page breaking properties in the System grob (issue 581470047 by addr
nine . fierce . ballads
Set page breaking properties in the System grob (issue 581470047 by address@hidden)
Wed, 22 Jan 2020 08:20:21 -0800
What's the purpose of this change? Is it preparation for a new feature?
File lily/page-breaking.cc (right):
lily/page-breaking.cc:624: for (SCM l = lines; scm_is_pair (l); l =
Veto! I've had more than one bad experience with lowercase ell.
In one case--I think it was elsewhere in LilyPond--it was purely a
readability issue, but reason enough to avoid it.
In the other, one person *thought* that he was using it as a loop
variable (i, j, k, l, ...), though he was actually using it outside its
loop. His code compiled anyway because another person had defined a
global of the same name. Thankfully, I don't work at that company
I suggest "SCM line = lines;" as an alternative.
- Set page breaking properties in the System grob (issue 581470047 by address@hidden),
nine . fierce . ballads <=