From: Mike Solomon <mike@apollinemike.com>
Date: Sat, 17 Sep 2011 11:45:34 +0000 (+0200)
Subject: Fixes issue 1881: cyclic dependency in Stem #'Y-extent.
X-Git-Tag: release/2.15.12-1~32
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=47f5cb2e80eca506ff4897b7628129597a60f5cb;p=lilypond.git

Fixes issue 1881: cyclic dependency in Stem #'Y-extent.

The fix does not trigger Y-extent calculations in Stem::set_stem_positions,
as this function may itself be called in a Y-extent calculation.  Now,
this function sets the begin position and the length, which are required
by Stem::height in order to report the Y-extent.
---

diff --git a/lily/stem.cc b/lily/stem.cc
index 8a755afaa5..93ebea4c96 100644
--- a/lily/stem.cc
+++ b/lily/stem.cc
@@ -160,7 +160,6 @@ Stem::set_stem_positions (Grob *me, Real se)
 
   me->set_property ("stem-begin-position", scm_from_double (height[-d] * 2 / staff_space));
   me->set_property ("length", scm_from_double (height.length () * 2 / staff_space));
-  (void) me->extent (me, Y_AXIS);
 }
 
 /* Note head that determines hshift for upstems