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 8a755afaa5aec63946003a19017be335f0c39b80..93ebea4c96cfaf75db438a1cbcd83b675a3f2dc9 100644 (file)
--- 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