"m...@apollinemike.com" <m...@apollinemike.com> writes:

> But I just pushed the actual cure.
>
> The problem was that positions stores two Y positions - a left and a
> right.  If the left is higher than right, than the interval is read as
> empty.  So, the center needs to be calculated manually.
>
> Interval is not really the right thing to use for positions if one
> wants to use the instance methods of interval (like is_empty), but I
> use it because it is already used in tuplet-bracket.cc for the print
> function (in a way that doesn't risk an assertion error).  It could
> also be put in a real drul array, but there is currently no
> robust_scm2realdrul.  This would definitely be a welcome addition to
> avoid confusion in the code base.

This sounds really really icky.  The sort of code I hope never to need
to deal with.  Could you file a suitable "Enhancement" issue so that
redoing this in a less hackish manner does not get forgotten?  That way
I have a better chance for my hopes becoming true.

-- 
David Kastrup


_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to