lilypond-devel
[Top][All Lists]
Advanced

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

Re: Brings beamed rest closer to cross staff beam. (issue 5908043)


From: mike
Subject: Re: Brings beamed rest closer to cross staff beam. (issue 5908043)
Date: Wed, 25 Apr 2012 06:29:14 +0000


http://codereview.appspot.com/5908043/diff/1/lily/beam.cc
File lily/beam.cc (right):

http://codereview.appspot.com/5908043/diff/1/lily/beam.cc#newcode1326
lily/beam.cc:1326: || beam->get_property_data ("direction") ==
ly_symbol2scm ("calculation-in-progress"))
On 2012/04/25 06:21:54, Keith wrote:
Mike, any idea why you added this?
It just adds to the cases where we lie and say the beam is on the
centerline.
It causes this to go wrongly
<< {b''8[ r16. g''32] } \\ {r8 <f' g' a' b'>16. r } >>

Take a look at today's patchset at
http://codereview.appspot.com/6035053/

If this bit isn't there, a circular dependency could be called up later
on in the function while looking for the direction.  I forget the exact
reason for the circular dependency, but it crept up in several regtests.
 Like several other pure functions, I have it cowardly fail if a pure
estimation is not possible because of circular dependencies, which
usually has a minimal impact on the typeset result.

http://codereview.appspot.com/5908043/



reply via email to

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