]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/beam.cc
patch::: 1.5.3.jcn1
[lilypond.git] / lily / beam.cc
index 23a62f4a5c4bc9f4a065c371b0554a28bbc0bd26..48c81d14cf764266d1db2665f99b643dbf370376 100644 (file)
@@ -3,69 +3,61 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997--2000 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c)  1997--2001 Han-Wen Nienhuys <hanwen@cs.uu.nl>
     Jan Nieuwenhuizen <janneke@gnu.org>
 
 */
 
 /*
   [TODO]
-    * less hairy code
-    * move paper vars to scm
 
-*/
+  -* shorter! (now +- 1000 lines)
+  
+  -* less hairy code
+  
+  */
 
 
-#include <math.h>              // tanh.
+#include <math.h> // tanh.
 
+#include "molecule.hh" 
 #include "directional-element-interface.hh"
 #include "beaming.hh"
-#include "dimensions.hh"
 #include "beam.hh"
 #include "misc.hh"
-#include "debug.hh"
 #include "least-squares.hh"
 #include "stem.hh"
 #include "paper-def.hh"
 #include "lookup.hh"
 #include "group-interface.hh"
 #include "staff-symbol-referencer.hh"
-#include "cross-staff.hh"
-#include "lily-guile.icc"
-
-Beam::Beam ()
-{
-  Group_interface g (this, "stems");
-  g.set_interface ();
-}
+#include "item.hh"
+#include "spanner.hh"
+#include "warn.hh"
 
 void
-Beam::add_stem (Stem*s)
+Beam::add_stem (Grob*me, Grob*s)
 {
-  Group_interface gi (this, "stems");
-  gi.add_element (s);
+  Pointer_group_interface:: add_element (me, "stems", s);
   
-  s->add_dependency (this);
+  s->add_dependency (me);
 
-  assert (!s->beam_l ());
-  s->set_elt_property ("beam", self_scm_);
+  assert (!Stem::beam_l (s));
+  s->set_grob_property ("beam", me->self_scm ());
 
-  if (!get_bound (LEFT))
-    set_bound (LEFT,s);
-  else
-    set_bound (RIGHT,s);
+  add_bound_item (dynamic_cast<Spanner*> (me), dynamic_cast<Item*> (s));
 }
 
 int
-Beam::get_multiplicity () const
+Beam::get_multiplicity (Grob*me) 
 {
   int m = 0;
-  for (SCM s = get_elt_property ("stems"); gh_pair_p (s); s = gh_cdr (s))
+  for (SCM s = me->get_grob_property ("stems"); gh_pair_p (s); s = gh_cdr (s))
     {
-      Score_element * sc = unsmob_element (gh_car (s));
+      Grob * sc = unsmob_grob (gh_car (s));
 
-      if (Stem * st = dynamic_cast<Stem*> (sc))
-       m = m >? st->beam_count (LEFT) >? st->beam_count (RIGHT);
+      if (Stem::has_interface (sc))
+       m = m >? Stem::beam_count (sc,LEFT) >? Stem::beam_count (sc,RIGHT);
     }
   return m;
 }
@@ -78,29 +70,35 @@ Beam::get_multiplicity () const
   [Alternatively, stems could set its own directions, according to
    their beam, during 'final-pre-processing'.]
  */
-void
-Beam::before_line_breaking ()
+MAKE_SCHEME_CALLBACK (Beam,before_line_breaking,1);
+SCM
+Beam::before_line_breaking (SCM smob)
 {
+  Grob * me =  unsmob_grob (smob);
+
   // Why?
-  if (visible_stem_count () < 2)
+  /*
+    Why what?  Why the warning (beams with less than 2 stems are
+    degenerate beams, should never happen), or why would this ever
+    happen (don't know). */
+  if (visible_stem_count (me) < 2)
     {
       warning (_ ("beam has less than two stems"));
-      //      set_elt_property ("transparent", SCM_BOOL_T);
     }
-
-  if (!directional_element (this).get ())
-    directional_element (this).set (get_default_dir ());
-
-  auto_knees ();
-  set_stem_directions ();
-  set_stem_shorten (); 
+  if (visible_stem_count (me) >= 1)
+    {
+      if (!Directional_element_interface::get (me))
+       Directional_element_interface::set (me, get_default_dir (me));
+      
+      consider_auto_knees (me);
+      set_stem_directions (me);
+      set_stem_shorten (me);
+    }
+  return SCM_EOL;
 }
 
-/*
- FIXME
- */
 Direction
-Beam::get_default_dir () const
+Beam::get_default_dir (Grob*me) 
 {
   Drul_array<int> total;
   total[UP]  = total[DOWN] = 0;
@@ -108,12 +106,15 @@ Beam::get_default_dir () const
   count[UP]  = count[DOWN] = 0;
   Direction d = DOWN;
 
-  for (int i=0; i <stem_count (); i++)
-    do { // HUH -- waar slaat dit op?
-      Stem *s = stem (i);
-      Direction sd = directional_element (s).get ();
+  Link_array<Item> stems=
+       Pointer_group_interface__extract_elements (me, (Item*)0, "stems");
+
+  for (int i=0; i <stems.size (); i++)
+    do {
+      Grob *s = stems[i];
+      Direction sd = Directional_element_interface::get (s);
       int current = sd ? (1 + d * sd)/2
-       : s->get_center_distance ((Direction)-d);
+       : Stem::get_center_distance (s, (Direction)-d);
 
       if (current)
        {
@@ -121,23 +122,22 @@ Beam::get_default_dir () const
          count[d] ++;
        }
 
-    } while (flip(&d) != DOWN);
+    } while (flip (&d) != DOWN);
   
+  SCM func = me->get_grob_property ("dir-function");
+  SCM s = gh_call2 (func,
+                   gh_cons (gh_int2scm (count[UP]),
+                            gh_int2scm (count[DOWN])),
+                   gh_cons (gh_int2scm (total[UP]),
+                            gh_int2scm (total[DOWN])));
 
-  SCM s = scm_eval (gh_list (ly_symbol2scm ("beam-dir-algorithm"),
-                            ly_quote_scm (gh_cons (gh_int2scm (count[UP]),
-                                                   gh_int2scm (count[DOWN]))),
-                            ly_quote_scm (gh_cons (gh_int2scm (total[UP]),
-                                                   gh_int2scm (total[DOWN]))),
-                            SCM_UNDEFINED));
   if (gh_number_p (s) && gh_scm2int (s))
     return to_dir (s);
   
   /*
-    If dir is not determined: get from paper
+    If dir is not determined: get default
   */
-  return (Direction)(int)
-    paper_l ()->get_var ("stem_default_neutral_direction");
+  return to_dir (me->get_grob_property ("neutral-direction"));
 }
 
 
@@ -147,72 +147,89 @@ Beam::get_default_dir () const
        once stem gets cleaned-up.
  */
 void
-Beam::set_stem_directions ()
+Beam::set_stem_directions (Grob*me)
 {
-  Direction d = directional_element (this).get ();
-  for (int i=0; i <stem_count (); i++)
+  Link_array<Item> stems
+    =Pointer_group_interface__extract_elements (me, (Item*) 0, "stems");
+  Direction d = Directional_element_interface::get (me);
+  
+  for (int i=0; i <stems.size (); i++)
     {
-      Stem *s = stem (i);
-      SCM force = s->remove_elt_property ("dir-forced");
+      Grob *s = stems[i];
+      SCM force = s->remove_grob_property ("dir-forced");
       if (!gh_boolean_p (force) || !gh_scm2bool (force))
-       directional_element (s).set (d);
+       Directional_element_interface ::set (s,d);
     }
 } 
 
-void
-Beam::auto_knees ()
-{
-  if (!auto_knee ("auto-interstaff-knee-gap", true))
-    auto_knee ("auto-knee-gap", false);
-}
-
 /*
   Simplistic auto-knees; only consider vertical gap between two
   adjacent chords.
 
   `Forced' stem directions are ignored.  If you don't want auto-knees,
-  don't set, or unset autoKneeGap/autoInterstaffKneeGap.
+  don't set, or unset auto-knee-gap.
  */
-bool
-Beam::auto_knee (String gap_str, bool interstaff_b)
+void
+Beam::consider_auto_knees (Grob *me)
 {
-  bool knee_b = false;
-  int knee_y = 0;
-  SCM gap = get_elt_property (gap_str);
-  Direction d = directional_element (this).get ();
-  
-  if (gh_number_p (gap))
+  SCM scm = me->get_grob_property ("auto-knee-gap");
+
+  if (gh_number_p (scm))
     {
-      int auto_gap_i = gh_scm2int (gap);
-      for (int i=1; i < stem_count (); i++)
+      bool knee_b = false;
+      Real knee_y = 0;
+      Real staff_space = Staff_symbol_referencer::staff_space (me);
+      Real gap = gh_scm2double (scm) / staff_space;
+
+      Direction d = Directional_element_interface::get (me);
+      Link_array<Item> stems=
+       Pointer_group_interface__extract_elements (me, (Item*)0, "stems");
+      
+      Grob *common = me->common_refpoint (stems[0], Y_AXIS);
+      for (int i=1; i < stems.size (); i++)
+       if (!Stem::invisible_b (stems[i]))
+         common = common->common_refpoint (stems[i], Y_AXIS);
+
+      int l = 0;
+      for (int i=1; i < stems.size (); i++)
         {
-         bool is_b = (bool)(calc_interstaff_dist (stem (i), this) 
-           - calc_interstaff_dist (stem (i-1), this));
-         int l_y = (int)(stem (i-1)->head_positions()[d])
-           + (int)calc_interstaff_dist (stem (i-1), this);
-         int r_y = (int)(stem (i)->head_positions()[d])
-           + (int)calc_interstaff_dist (stem (i), this);
-         int gap_i = r_y - l_y;
-
-         if ((abs (gap_i) >= auto_gap_i) && (!interstaff_b || is_b))
+         if (!Stem::invisible_b (stems[i-1]))
+           l = i - 1;
+         if (Stem::invisible_b (stems[l]))
+           continue;
+         if (Stem::invisible_b (stems[i]))
+           continue;
+         
+         Real left = Stem::extremal_heads (stems[l])[d]
+           ->relative_coordinate (common, Y_AXIS);
+         Real right = Stem::extremal_heads (stems[i])[-d]
+           ->relative_coordinate (common, Y_AXIS);
+
+         Real dy = right - left;
+
+         if (abs (dy) >= gap)
            {
-             knee_y = (r_y + l_y) / 2;
+             knee_y = (right + left) / 2;
              knee_b = true;
              break;
            }
        }
-    }
-  if (knee_b)
-    {
-      for (int i=0; i < stem_count (); i++)
-        {
-         int y = (int)(stem (i)->head_positions()[d])
-           + (int)calc_interstaff_dist (stem (i), this);
-         directional_element (stem (i)).set (y < knee_y ? UP : DOWN);
-         stem (i)->set_elt_property ("dir-forced", SCM_BOOL_T);
+      
+      if (knee_b)
+       {
+         for (int i=0; i < stems.size (); i++)
+           {
+             if (Stem::invisible_b (stems[i]))
+               continue;
+             Item *s = stems[i];         
+             Real y = Stem::extremal_heads (stems[i])[d]
+               ->relative_coordinate (common, Y_AXIS);
+
+             Directional_element_interface::set (s, y < knee_y ? UP : DOWN);
+             s->set_grob_property ("dir-forced", SCM_BOOL_T);
+           }
        }
     }
-  return knee_b;
 }
 
 /*
@@ -222,171 +239,165 @@ Beam::auto_knee (String gap_str, bool interstaff_b)
     scmify forced-fraction
  */
 void
-Beam::set_stem_shorten ()
+Beam::set_stem_shorten (Grob*m)
 {
-  if (!visible_stem_count ())
-    return;
+  Spanner*me = dynamic_cast<Spanner*> (m);
 
-  Real forced_fraction = forced_stem_count () / visible_stem_count ();
+  Real forced_fraction = forced_stem_count (me) / visible_stem_count (me);
   if (forced_fraction < 0.5)
     return;
 
-  int multiplicity = get_multiplicity ();
-  // grace stems?
-  SCM shorten = ly_eval_str ("beamed-stem-shorten");
+  int multiplicity = get_multiplicity (me);
 
-  Array<Real> a;
-  scm_to_array (shorten, &a);
-  if (!a.size ())
+  SCM shorten = me->get_grob_property ("beamed-stem-shorten");
+  if (shorten == SCM_EOL)
     return;
 
-  Staff_symbol_referencer_interface st (this);
-  Real staff_space = st.staff_space ();
-  Real shorten_f = a[multiplicity <? (a.size () - 1)] * staff_space;
+  int sz = scm_ilength (shorten);
+  
+  Real staff_space = Staff_symbol_referencer::staff_space (me);
+  SCM shorten_elt = scm_list_ref (shorten, gh_int2scm (multiplicity <? (sz - 1)));
+  Real shorten_f = gh_scm2double (shorten_elt) * staff_space;
 
-  /* cute, but who invented this -- how to customise ? */
+  /* cute, but who invented me -- how to customise ? */
   if (forced_fraction < 1)
     shorten_f /= 2;
 
-  for (int i=0; i < stem_count (); i++)
+  Link_array<Item> stems=
+    Pointer_group_interface__extract_elements (me, (Item*)0, "stems");
+
+  for (int i=0; i < stems.size (); i++)
     {
-      Stem* s = stem (i);
-      if (s->invisible_b ())
+      Item* s = stems[i];
+      if (Stem::invisible_b (s))
         continue;
-      if (gh_number_p (s->get_elt_property ("shorten")))
-       s->set_elt_property ("shorten", gh_double2scm (shorten_f));
+      if (gh_number_p (s->get_grob_property ("shorten")))
+       s->set_grob_property ("shorten", gh_double2scm (shorten_f));
     }
 }
 
 /*
-  Set elt properties height and y-position if not set.
-  Adjust stem lengths to reach beam.
+  Call list of y-dy-callbacks, that handle setting of
+  grob-properties y, dy.
+
+  User may set grob-properties: y-position-hs and height-hs
+ (to be fixed) that override the calculated y and dy.
+
+  Because y and dy cannot be calculated and quanted separately, we
+  always calculate both, then check for user override.
  */
-void
-Beam::after_line_breaking ()
+MAKE_SCHEME_CALLBACK (Beam, after_line_breaking, 1);
+SCM
+Beam::after_line_breaking (SCM smob)
 {
-  /* first, calculate y, dy */
-  Real y, dy;
-  calc_position_and_height (&y, &dy);
-  if (visible_stem_count ())
-    {
-      if (suspect_slope_b (y, dy))
-       dy = 0;
+  Grob * me =  unsmob_grob (smob);
+  
+  me->set_grob_property ("y", gh_double2scm (0));
+  me->set_grob_property ("dy", gh_double2scm (0));
 
-      Real damped_dy = calc_slope_damping_f (dy);
-      Real quantised_dy = quantise_dy_f (damped_dy);
+  /* Hmm, callbacks should be called by, a eh, callback mechanism
+    somewhere (?), I guess, not by looping here. */
+  
+  SCM list = me->get_grob_property ("y-dy-callbacks");
+  for (SCM i = list; gh_pair_p (i); i = gh_cdr (i))
+    gh_call1 (gh_car (i), smob);
 
-      y += (dy - quantised_dy) / 2;
-      dy = quantised_dy;
-    }
-  /*
-    until here, we used only stem_info, which acts as if dir=up
-   */
-  y *= directional_element (this).get ();
-  dy *= directional_element (this).get ();
+  // UGH. Y is not in staff position unit?
+  // Ik dacht datwe daar juist van weg wilden?
+  
+  // Hmm, nu hebben we 3 dimensies, want inmiddels zijn we daar
+  // weer terug, maar dan / 2
+  // (staff-space iso staff-position)
+  
+  set_stem_lengths (me);
 
-  Staff_symbol_referencer_interface st (this);
-  Real half_space = st.staff_space () / 2;
+  return SCM_UNSPECIFIED;
+}
 
-  /* check for user-override of dy */
-  SCM s = remove_elt_property ("height-hs");
-  if (gh_number_p (s))
-    {
-      dy = gh_scm2double (s) * half_space;
-    }
-  set_elt_property ("height", gh_double2scm (dy));
 
-  /* check for user-override of y */
-  s = remove_elt_property ("y-position-hs");
-  if (gh_number_p (s))
+MAKE_SCHEME_CALLBACK (Beam, least_squares, 1);
+SCM
+Beam::least_squares (SCM smob)
+{
+ Grob *me = unsmob_grob (smob);
+
+ if (visible_stem_count (me) <= 1)
+   return SCM_UNSPECIFIED;
+
+  Real y = 0;
+  Real dy = 0;
+
+  /* Stem_info, and thus y,dy in this function are corrected for beam-dir */
+  Real first_ideal = Stem::calc_stem_info (first_visible_stem (me)).idealy_f_;
+  if (first_ideal == Stem::calc_stem_info (last_visible_stem (me)).idealy_f_)
     {
-      y = gh_scm2double (s) * half_space;
+      y = first_ideal;
+      dy = 0;
     }
   else
-    { 
-      /* we can modify y, so we should quantise y */
-      Real y_shift = check_stem_length_f (y, dy);
-      y += y_shift;
-      y = quantise_y_f (y, dy, 0);
-      set_stem_length (y, dy);
-      y_shift = check_stem_length_f (y, dy);
+    {
+      Array<Offset> ideals;
+
+      // ugh -> use commonx
+      Real x0 = first_visible_stem (me)->relative_coordinate (0, X_AXIS);
+      Link_array<Item> stems=
+       Pointer_group_interface__extract_elements (me, (Item*)0, "stems");
 
-      if (y_shift > half_space / 4)
+      for (int i=0; i < stems.size (); i++)
        {
-         y += y_shift;
-
-         /*
-           for significantly lengthened or shortened stems,
-           request quanting the other way.
-         */
-         int quant_dir = 0;
-         if (abs (y_shift) > half_space / 2)
-           quant_dir = sign (y_shift) * directional_element (this).get ();
-         y = quantise_y_f (y, dy, quant_dir);
+         Item* s = stems[i];
+         if (Stem::invisible_b (s))
+           continue;
+         ideals.push (Offset (s->relative_coordinate (0, X_AXIS) - x0, 
+                              Stem::calc_stem_info (s).idealy_f_));
        }
-    }
-  // UGH. Y is not in staff position unit?
-  // Ik dacht datwe daar juist van weg wilden?
-  set_stem_length (y, dy);
-  set_elt_property ("y-position", gh_double2scm (y)); 
-}
-
-/*
-  See Documentation/tex/fonts.doc
- */
-void
-Beam::calc_position_and_height (Real* y, Real* dy) const
-{
-  *y = *dy = 0;
-  if (visible_stem_count () <= 1)
-    return;
+      Real dydx;
+      minimise_least_squares (&dydx, &y, ideals);
 
-  Real first_ideal = first_visible_stem ()->calc_stem_info ().idealy_f_;
-  if (first_ideal == last_visible_stem ()->calc_stem_info ().idealy_f_)
-    {
-      *dy = 0;
-      *y = first_ideal;
-      return;
-    }
-
-  Array<Offset> ideals;
-  Real x0 = first_visible_stem ()->relative_coordinate (0, X_AXIS);
-  for (int i=0; i < stem_count (); i++)
-    {
-      Stem* s = stem (i);
-      if (s->invisible_b ())
-        continue;
-      ideals.push (Offset (s->relative_coordinate (0, X_AXIS) - x0, 
-                          s->calc_stem_info ().idealy_f_));
+      Real dx = last_visible_stem (me)->relative_coordinate (0, X_AXIS) - x0;
+      dy = dydx * dx;
     }
-  Real dydx;
-  minimise_least_squares (&dydx, y, ideals); // duh, takes references
 
-  Real dx = last_visible_stem ()->relative_coordinate (0, X_AXIS) - x0;
-  *dy = dydx * dx;
+  /* Store true, not dir-corrected values */
+  Direction dir = Directional_element_interface::get (me);
+  me->set_grob_property ("y", gh_double2scm (y * dir));
+  me->set_grob_property ("dy", gh_double2scm (dy * dir));
+  return SCM_UNSPECIFIED;
 }
 
-bool
-Beam::suspect_slope_b (Real y, Real dy) const
+MAKE_SCHEME_CALLBACK (Beam, cancel_suspect_slope, 1);
+SCM
+Beam::cancel_suspect_slope (SCM smob)
 {
-  /* first, calculate y, dy */
-  /*
-    steep slope running against lengthened stem is suspect
-  */
-  Real first_ideal = first_visible_stem ()->calc_stem_info ().idealy_f_;
-  Real last_ideal = last_visible_stem ()->calc_stem_info ().idealy_f_;
-  Real lengthened = paper_l ()->get_var ("beam_lengthened");
-  Real steep = paper_l ()->get_var ("beam_steep_slope");
-
-  Real dx = last_visible_stem ()->relative_coordinate (0, X_AXIS) - first_visible_stem ()->relative_coordinate (0, X_AXIS);
+  Grob *me = unsmob_grob (smob);
+  
+  if (visible_stem_count (me) <= 1)
+    return SCM_UNSPECIFIED;
+  
+  /* Stem_info, and thus y,dy in this function are corrected for beam-dir */
+  Direction dir = Directional_element_interface::get (me);
+  Real y = gh_scm2double (me->get_grob_property ("y")) * dir;
+  Real dy = gh_scm2double (me->get_grob_property ("dy")) * dir;
+  
+ /* steep slope running against lengthened stem is suspect */
+  Real first_ideal = Stem::calc_stem_info (first_visible_stem (me)).idealy_f_;
+  Real last_ideal = Stem::calc_stem_info (last_visible_stem (me)).idealy_f_;
+  Real lengthened = gh_scm2double (me->get_grob_property ("outer-stem-length-limit"));
+  Real steep = gh_scm2double (me->get_grob_property ("slope-limit"));
+
+  // ugh -> use commonx
+  Real dx = last_visible_stem (me)->relative_coordinate (0, X_AXIS) - first_visible_stem (me)->relative_coordinate (0, X_AXIS);
   Real dydx = dy && dx ? dy/dx : 0;
 
-  if (((y - first_ideal > lengthened) && (dydx > steep))
+  if (( (y - first_ideal > lengthened) && (dydx > steep))
       || ((y + dy - last_ideal > lengthened) && (dydx < -steep)))
     {
-      return true;
+      Real adjusted_y = y + dy / 2;
+      /* Store true, not dir-corrected values */
+      me->set_grob_property ("y", gh_double2scm (adjusted_y * dir));
+      me->set_grob_property ("dy", gh_double2scm (0)); 
     }
-  return false;
+  return SCM_UNSPECIFIED;
 }
 
 /*
@@ -394,40 +405,188 @@ Beam::suspect_slope_b (Real y, Real dy) const
   damped = tanh (slope)
   corresponds with some tables in [Wanske]
 */
-Real
-Beam::calc_slope_damping_f (Real dy) const
+MAKE_SCHEME_CALLBACK (Beam, slope_damping, 1);
+SCM
+Beam::slope_damping (SCM smob)
 {
-  SCM damp = get_elt_property ("damping"); // remove?
-  int damping = 1;             // ugh.
-  if (gh_number_p (damp))
-    damping = gh_scm2int (damp);
+  Grob *me = unsmob_grob (smob);
+
+  if (visible_stem_count (me) <= 1)
+    return SCM_UNSPECIFIED;
+
+  SCM s = me->get_grob_property ("damping"); 
+  int damping = gh_scm2int (s);
 
   if (damping)
     {
-      Real dx = last_visible_stem ()->relative_coordinate (0, X_AXIS)
-       - first_visible_stem ()->relative_coordinate (0, X_AXIS);
+      /* y,dy in this function are corrected for beam-dir */
+      Direction dir = Directional_element_interface::get (me);
+      Real y = gh_scm2double (me->get_grob_property ("y")) * dir;
+      Real dy = gh_scm2double (me->get_grob_property ("dy")) * dir;
+      
+      // ugh -> use commonx
+      Real dx = last_visible_stem (me)->relative_coordinate (0, X_AXIS)
+       - first_visible_stem (me)->relative_coordinate (0, X_AXIS);
       Real dydx = dy && dx ? dy/dx : 0;
       dydx = 0.6 * tanh (dydx) / damping;
-      return dydx * dx;
+
+      Real damped_dy = dydx * dx;
+      Real adjusted_y = y + (dy - damped_dy) / 2;
+      /* Store true, not dir-corrected values */
+      me->set_grob_property ("y", gh_double2scm (adjusted_y * dir));
+      me->set_grob_property ("dy", gh_double2scm (damped_dy * dir));
+    }
+    return SCM_UNSPECIFIED;
+}
+
+/*
+  Quantise dy (height) of beam.
+  Generalisation of [Ross].
+  */
+MAKE_SCHEME_CALLBACK (Beam, quantise_dy, 1);
+SCM
+Beam::quantise_dy (SCM smob)
+{
+  Grob *me = unsmob_grob (smob);
+
+  if (visible_stem_count (me) <= 1)
+    return SCM_UNSPECIFIED;
+
+  Array<Real> a;
+  SCM proc = me->get_grob_property ("height-quants");
+  SCM quants = gh_call2 (proc, me->self_scm (),
+                        gh_double2scm (me->paper_l ()->get_var ("stafflinethickness")
+                                       / 1.0));
+  
+  for (SCM s = quants; gh_pair_p (s); s = gh_cdr (s))
+    a.push (gh_scm2double (gh_car (s)));
+  
+  if (a.size () > 1)
+    {
+      /* y,dy in this function are corrected for beam-dir */
+      Direction dir = Directional_element_interface::get (me);
+      Real y = gh_scm2double (me->get_grob_property ("y")) * dir;
+      Real dy = gh_scm2double (me->get_grob_property ("dy")) * dir;
+
+      Real staff_space = Staff_symbol_referencer::staff_space (me);
+      
+      Interval iv = quantise_iv (a, abs (dy)/staff_space) * staff_space;
+      Real q = (abs (dy) - iv[SMALLER] <= iv[BIGGER] - abs (dy))
+       ? iv[SMALLER]
+       : iv[BIGGER];
+      
+      Real quantised_dy = q * sign (dy);
+      Real adjusted_y = y + (dy - quantised_dy) / 2;
+      /* Store true, not dir-corrected values */
+      me->set_grob_property ("y", gh_double2scm (adjusted_y * dir));
+      me->set_grob_property ("dy", gh_double2scm (quantised_dy * dir));
+    }
+  return SCM_UNSPECIFIED;
+}
+
+/* It's tricky to have the user override y,dy directly, so we use this
+   translation func.  Also, if our staff_space != 1 (smaller staff, eg),
+   user will expect staff-position to be discrete values. */
+MAKE_SCHEME_CALLBACK (Beam, user_override, 1);
+SCM
+Beam::user_override (SCM smob)
+{
+  Grob *me = unsmob_grob (smob);
+  Real staff_space = Staff_symbol_referencer::staff_space (me);
+
+  SCM s = me->get_grob_property ("staff-position");
+  if (gh_number_p (s))
+    {
+      Real y = gh_scm2double (s) * staff_space * 0.5;
+      me->set_grob_property ("y", gh_double2scm (y));
+    }
+
+  /* Name suggestions? Tilt, slope, vertical-* ? */
+  s = me->get_grob_property ("height");
+  if (gh_number_p (s))
+    {
+      Real dy = gh_scm2double (s) * staff_space * 0.5;
+      me->set_grob_property ("dy", gh_double2scm (dy));
+    }
+  
+  return SCM_UNSPECIFIED;
+}
+
+/*
+  Ugh, this must be last, after user_override
+  Assumes directionised y/dy.
+ */
+MAKE_SCHEME_CALLBACK (Beam, do_quantise_y, 1);
+SCM
+Beam::do_quantise_y (SCM smob)
+{
+  Grob *me = unsmob_grob (smob);
+
+  /*
+    If the user set y-position, we shouldn't do quanting.
+   */
+  if (gh_number_p (me->get_grob_property ("y-position-hs")))
+    return SCM_UNSPECIFIED;
+
+  Real y = gh_scm2double (me->get_grob_property ("y"));
+  Real dy = gh_scm2double (me->get_grob_property ("dy"));
+      
+  /* we can modify y, so we should quantise y */
+  Real half_space = Staff_symbol_referencer::staff_space (me) / 2;
+  Real y_shift = check_stem_length_f (me, y, dy);
+  y += y_shift;
+  y = quantise_y_f (me, y, dy, 0);
+
+  /*
+    Hmm, this is a bit keyhole operation: we're passing `this' as a
+    parameter, and member vars as SCM properties.  We should decide on
+    SCM/C/C++ boundary */
+  me->set_grob_property ("y", gh_double2scm (y));
+  set_stem_lengths (me);
+  y = gh_scm2double (me->get_grob_property ("y"));
+  
+  y_shift = check_stem_length_f (me, y, dy);
+
+  if (y_shift > half_space / 4)
+    {
+      y += y_shift;
+
+      /*
+       for significantly lengthened or shortened stems,
+       request quanting the other way.
+      */
+      int quant_dir = 0;
+      if (abs (y_shift) > half_space / 2)
+       quant_dir = sign (y_shift) * Directional_element_interface::get (me);
+      y = quantise_y_f (me, y, dy, quant_dir);
     }
-  return dy;
+  
+  me->set_grob_property ("y", gh_double2scm (y));
+  // me->set_grob_property ("dy", gh_double2scm (dy));
+  return SCM_UNSPECIFIED;
 }
 
+
 Real
-Beam::calc_stem_y_f (Stem* s, Real y, Real dy) const
+Beam::calc_stem_y_f (Grob*me,Item* s, Real y, Real dy) 
 {
-  Real thick = gh_scm2double (get_elt_property ("beam-thickness"));
-  int beam_multiplicity = get_multiplicity ();
-  int stem_multiplicity = (s->flag_i () - 2) >? 0;
+  int beam_multiplicity = get_multiplicity (me);
+  int stem_multiplicity = (Stem::flag_i (s) - 2) >? 0;
+
+  SCM space_proc = me->get_grob_property ("space-function");
+  SCM space = gh_call1 (space_proc, gh_int2scm (beam_multiplicity));
+
+  Real thick = gh_scm2double (me->get_grob_property ("thickness")) ;
+  Real interbeam_f = gh_scm2double (space) ;
 
-  Real interbeam_f = paper_l ()->interbeam_f (beam_multiplicity);
-  Real x0 = first_visible_stem ()->relative_coordinate (0, X_AXIS);
-  Real dx = last_visible_stem ()->relative_coordinate (0, X_AXIS) - x0;
+  // ugh -> use commonx
+  Real x0 = first_visible_stem (me)->relative_coordinate (0, X_AXIS);
+  Real dx = last_visible_stem (me)->relative_coordinate (0, X_AXIS) - x0;
   Real stem_y = (dy && dx ? (s->relative_coordinate (0, X_AXIS) - x0) / dx * dy : 0) + y;
 
   /* knee */
-   Direction dir  = directional_element(this).get ();
-   Direction sdir = directional_element (s).get ();
+   Direction dir  = Directional_element_interface::get (me);
+   Direction sdir = Directional_element_interface::get (s);
    
     /* knee */
    if (dir!= sdir)
@@ -435,35 +594,38 @@ Beam::calc_stem_y_f (Stem* s, Real y, Real dy) const
        stem_y -= dir 
        * (thick / 2 + (beam_multiplicity - 1) * interbeam_f);
 
-      Staff_symbol_referencer_interface me (s);
-      Staff_symbol_referencer_interface last (last_visible_stem ());
+
       
       // huh, why not for first visible?
-      if (//(s != first_visible_stem ()) &&
-         me.staff_symbol_l () != last.staff_symbol_l ())
-       stem_y += directional_element (this).get ()
-         * (beam_multiplicity - stem_multiplicity) * interbeam_f;
-    }
+       if (Staff_symbol_referencer::staff_symbol_l (s)
+          != Staff_symbol_referencer::staff_symbol_l (last_visible_stem (me)))
+        stem_y += Directional_element_interface::get (me)
+          * (beam_multiplicity - stem_multiplicity) * interbeam_f;
+      }
+
   return stem_y;
 }
 
 Real
-Beam::check_stem_length_f (Real y, Real dy) const
+Beam::check_stem_length_f (Grob*me,Real y, Real dy) 
 {
   Real shorten = 0;
   Real lengthen = 0;
-  Direction dir = directional_element (this).get ();
-  
-  for (int i=0; i < stem_count (); i++)
+  Direction dir = Directional_element_interface::get (me);
+
+  Link_array<Item> stems=
+    Pointer_group_interface__extract_elements (me, (Item*)0, "stems");
+
+  for (int i=0; i < stems.size (); i++)
     {
-      Stem* s = stem (i);
-      if (s->invisible_b ())
+      Item* s = stems[i];
+      if (Stem::invisible_b (s))
        continue;
 
-      Real stem_y = calc_stem_y_f (s, y, dy);
+      Real stem_y = calc_stem_y_f (me, s, y, dy);
        
       stem_y *= dir;
-      Stem_info info = s->calc_stem_info ();
+      Stem_info info = Stem::calc_stem_info (s);
 
       // if (0 > info.maxy_f_ - stem_y)
       shorten = shorten <? info.maxy_f_ - stem_y;
@@ -483,97 +645,108 @@ Beam::check_stem_length_f (Real y, Real dy) const
   stem directions and length should set to relative to the chord's
   position of the beam.  */
 void
-Beam::set_stem_length (Real y, Real dy)
+Beam::set_stem_lengths (Grob *me)
 {
-  Staff_symbol_referencer_interface st (this);
-  Real half_space = st.staff_space ()/2;
-  for (int i=0; i < stem_count (); i++)
+  if (visible_stem_count (me) <= 1)
+    return;
+  
+  Real y = gh_scm2double (me->get_grob_property ("y"));
+  Real dy = gh_scm2double (me->get_grob_property ("dy"));
+
+  Real half_space = Staff_symbol_referencer::staff_space (me)/2;
+  Link_array<Item> stems=
+    Pointer_group_interface__extract_elements (me, (Item*)0, "stems");
+
+  Grob *common = me->common_refpoint (stems[0], Y_AXIS);
+  for (int i=1; i < stems.size (); i++)
+    if (!Stem::invisible_b (stems[i]))
+      common = common->common_refpoint (stems[i], Y_AXIS);
+
+  for (int i=0; i < stems.size (); i++)
     {
-      Stem* s = stem (i);
-      if (s->invisible_b ())
+      Item* s = stems[i];
+      if (Stem::invisible_b (s))
        continue;
 
-      Real stem_y = calc_stem_y_f (s, y, dy);
+      Real stem_y = calc_stem_y_f (me, s, y, dy);
 
+      // doesn't play well with dvips
+      if (scm_definedp (ly_symbol2scm ("ps-testing"), SCM_UNDEFINED)
+         == SCM_BOOL_T)
+       if (Stem::get_direction (s) == Directional_element_interface::get (me))
+         stem_y += Stem::get_direction (s)
+           * gh_scm2double (me->get_grob_property ("thickness")) / 2;
+      
       /* caution: stem measures in staff-positions */
-      s->set_stemend ((stem_y + calc_interstaff_dist (s, this)) / half_space);
+      Real id = me->relative_coordinate (common, Y_AXIS)
+       - stems[i]->relative_coordinate (common, Y_AXIS);
+      Stem::set_stemend (s, (stem_y + id) / half_space);
     }
 }
 
-/*
-  [Ross] (simplification of)
-  Set dy complying with:
-    - zero
-    - thick / 2 + staffline_f / 2
-    - thick + staffline_f
-  + n * staff_space
-*/
-Real
-Beam::quantise_dy_f (Real dy) const
-{
-  SCM quants = ly_eval_str ("beam-height-quants");
-
-  Array<Real> a;
-  scm_to_array (quants, &a);
-  if (a.size () <= 1)
-    return dy;
-
-  Staff_symbol_referencer_interface st (this);
-  Real staff_space = st.staff_space ();
-  
-  Interval iv = quantise_iv (a, abs (dy)/staff_space) * staff_space;
-  Real q = (abs (dy) - iv[SMALLER] <= iv[BIGGER] - abs (dy))
-    ? iv[SMALLER]
-    : iv[BIGGER];
-  
-  return q * sign (dy);
-}
-
 /*
   Prevent interference from stafflines and beams.
-  See Documentation/tex/fonts.doc
 
-  We only need to quantise the (left) y-position of the beam,
+  We only need to quantise the (left) y of the beam,
   since dy is quantised too.
   if extend_b then stems must *not* get shorter
  */
 Real
-Beam::quantise_y_f (Real y, Real dy, int quant_dir)
+Beam::quantise_y_f (Grob*me,Real y, Real dy, int quant_dir)
 {
-  int multiplicity = get_multiplicity ();
-  Staff_symbol_referencer_interface st (this);
-  Real staff_space = st.staff_space ();
-  SCM quants = scm_eval (gh_list (
-                                 ly_symbol2scm ("beam-vertical-position-quants"),
-                                 gh_int2scm (multiplicity),
-                                 gh_double2scm (dy/staff_space),
-                                 SCM_UNDEFINED));
+  int multiplicity = get_multiplicity (me);
+
+  Real staff_space = Staff_symbol_referencer::staff_space (me);
+  Real thick = me->paper_l ()->get_var ("stafflinethickness");
+
+
+  SCM proc = me->get_grob_property ("vertical-position-quant-function");
+  SCM quants = scm_apply (proc,
+                         me->self_scm (),
+                         gh_list (gh_int2scm (multiplicity),
+                                  gh_double2scm (dy/staff_space),
+                                  gh_double2scm (thick/staff_space),
+                                  SCM_EOL, SCM_UNDEFINED));
+  
   Array<Real> a;
-  scm_to_array (quants, &a);
+
+  for (; gh_pair_p (quants); quants = gh_cdr (quants))
+    a.push (gh_scm2double (gh_car (quants)));
+
   if (a.size () <= 1)
     return y;
 
-  Real up_y = directional_element (this).get () * y;
+  Real up_y = Directional_element_interface::get (me) * y;
   Interval iv = quantise_iv (a, up_y/staff_space) * staff_space;
 
   Real q = up_y - iv[SMALLER] <= iv[BIGGER] - up_y 
     ? iv[SMALLER] : iv[BIGGER];
   if (quant_dir)
-    q = iv[(Direction)quant_dir];
+    q = iv[ (Direction)quant_dir];
 
-  return q * directional_element (this).get ();
+  return q * Directional_element_interface::get (me);
 }
 
 void
-Beam::set_beaming (Beaming_info_list *beaming)
+Beam::set_beaming (Grob*me,Beaming_info_list *beaming)
 {
+  Link_array<Grob> stems=
+    Pointer_group_interface__extract_elements (me, (Grob*)0, "stems");
+  
   Direction d = LEFT;
-  for (int i=0; i  < stem_count (); i++)
+  for (int i=0; i  < stems.size (); i++)
     {
       do
        {
-         if (stem (i)->beam_count (d) == 0)
-           stem (i)->set_beaming ( beaming->infos_.elem (i).beams_i_drul_[d],d);
+         /* Don't overwrite user override (?) */
+         if (Stem::beam_count (stems[i], d) == 0
+             /* Don't set beaming for outside of outer stems */
+             && ! (d == LEFT && i == 0)
+             && ! (d == RIGHT && i == stems.size () -1))
+           {
+             int b = beaming->infos_.elem (i).beams_i_drul_[d];
+             Stem::set_beaming (stems[i], b, d);
+           }
        }
       while (flip (&d) != LEFT);
     }
@@ -584,64 +757,79 @@ Beam::set_beaming (Beaming_info_list *beaming)
 /*
   beams to go with one stem.
 
-  BURP
-  clean  me up.
+  FIXME: clean me up.
   */
 Molecule
-Beam::stem_beams (Stem *here, Stem *next, Stem *prev) const
+Beam::stem_beams (Grob*me,Item *here, Item *next, Item *prev,
+                 Real dy, Real dydx
+                 ) 
 {
-  if ((next && !(next->relative_coordinate (0, X_AXIS) > here->relative_coordinate (0, X_AXIS))) ||
-      (prev && !(prev->relative_coordinate (0, X_AXIS) < here->relative_coordinate (0, X_AXIS))))
+  // ugh -> use commonx
+  if ((next && ! (next->relative_coordinate (0, X_AXIS) > here->relative_coordinate (0, X_AXIS))) ||
+ (prev && ! (prev->relative_coordinate (0, X_AXIS) < here->relative_coordinate (0, X_AXIS))))
       programming_error ("Beams are not left-to-right");
 
-  Real staffline_f = paper_l ()->get_var ("stafflinethickness");
-  int   multiplicity = get_multiplicity ();
-
+  int multiplicity = get_multiplicity (me);
 
-  Real interbeam_f = paper_l ()->interbeam_f (multiplicity);
-  Real thick = gh_scm2double (get_elt_property ("beam-thickness"));
+  SCM space_proc = me->get_grob_property ("space-function");
+  SCM space = gh_call1 (space_proc, gh_int2scm (multiplicity));
 
+  Real thick = gh_scm2double (me->get_grob_property ("thickness")) ;
+  Real interbeam_f = gh_scm2double (space) ;
+    
   Real bdy = interbeam_f;
-  Real stemdx = staffline_f;
-
-  Real dx = visible_stem_count () ?
-    last_visible_stem ()->relative_coordinate (0, X_AXIS) - first_visible_stem ()->relative_coordinate (0, X_AXIS)
+  
+#if 0
+    // ugh -> use commonx
+  Real dx = visible_stem_count (me) ?
+    last_visible_stem (me)->relative_coordinate (0, X_AXIS) - first_visible_stem (me)->relative_coordinate (0, X_AXIS)
     : 0.0;
-  Real dy = gh_scm2double (get_elt_property ("height"));
-  Real dydx = dy && dx ? dy/dx : 0;
-
+#endif
+  
   Molecule leftbeams;
   Molecule rightbeams;
 
-  // UGH
   Real nw_f;
-  if (!here->first_head ())
+  if (!Stem::first_head (here))
     nw_f = 0;
-  else if (here->type_i ()== 1)
-    nw_f = paper_l ()->get_var ("wholewidth");
-  else if (here->type_i () == 2)
-    nw_f = paper_l ()->get_var ("notewidth") * 0.8;
-  else
-    nw_f = paper_l ()->get_var ("quartwidth");
+  else {
+    int t = Stem::type_i (here); 
+
+    SCM proc = me->get_grob_property ("flag-width-function");
+    SCM result = gh_call1 (proc, gh_int2scm (t));
+    nw_f = gh_scm2double (result);
+  }
 
 
-  Direction dir = directional_element (this).get ();
+  Direction dir = Directional_element_interface::get (me);
+  
+  /* [Tremolo] beams on whole notes may not have direction set? */
+ if (dir == CENTER)
+    dir = Directional_element_interface::get (here);
   
   /* half beams extending to the left. */
   if (prev)
     {
-      int lhalfs= lhalfs = here->beam_count (LEFT) - prev->beam_count (RIGHT);
-      int lwholebeams= here->beam_count (LEFT) <? prev->beam_count (RIGHT) ;
+      int lhalfs= lhalfs = Stem::beam_count (here,LEFT) - Stem::beam_count (prev,RIGHT);
+      int lwholebeams= Stem::beam_count (here,LEFT) <? Stem::beam_count (prev,RIGHT) ;
       /*
        Half beam should be one note-width, 
        but let's make sure two half-beams never touch
        */
-      Real w = here->relative_coordinate (0, X_AXIS) - prev->relative_coordinate (0, X_AXIS);
+
+      // FIXME: TODO (check) stem width / sloped beams
+      Real w = here->relative_coordinate (0, X_AXIS)
+       - prev->relative_coordinate (0, X_AXIS);
+      Real stem_w = gh_scm2double (prev->get_grob_property ("thickness"))
+       // URG
+       * me->paper_l ()->get_var ("stafflinethickness");
+
       w = w/2 <? nw_f;
       Molecule a;
       if (lhalfs)              // generates warnings if not
-       a =  lookup_l ()->beam (dydx, w, thick);
+       a =  Lookup::beam (dydx, w + stem_w, thick);
       a.translate (Offset (-w, -w * dydx));
+      a.translate_axis (-stem_w/2, X_AXIS);
       for (int j = 0; j  < lhalfs; j++)
        {
          Molecule b (a);
@@ -652,19 +840,27 @@ Beam::stem_beams (Stem *here, Stem *next, Stem *prev) const
 
   if (next)
     {
-      int rhalfs  = here->beam_count (RIGHT) - next->beam_count (LEFT);
-      int rwholebeams= here->beam_count (RIGHT) <? next->beam_count (LEFT) ;
+      int rhalfs  = Stem::beam_count (here,RIGHT)
+       - Stem::beam_count (next,LEFT);
+      int rwholebeams= Stem::beam_count (here,RIGHT)
+       <? Stem::beam_count (next,LEFT) ;
+
+      Real w = next->relative_coordinate (0, X_AXIS)
+       - here->relative_coordinate (0, X_AXIS);
+
+      Real stem_w = gh_scm2double (next->get_grob_property ("thickness"))
+       // URG
+       * me->paper_l ()->get_var ("stafflinethickness");
 
-      Real w = next->relative_coordinate (0, X_AXIS) - here->relative_coordinate (0, X_AXIS);
-      Molecule a = lookup_l ()->beam (dydx, w + stemdx, thick);
-      a.translate_axis( - stemdx/2, X_AXIS);
+      Molecule a = Lookup::beam (dydx, w + stem_w, thick);
+      a.translate_axis (- stem_w/2, X_AXIS);
       int j = 0;
       Real gap_f = 0;
-
-      SCM gap = get_elt_property ("beam-gap");
+      
+      SCM gap = me->get_grob_property ("gap");
       if (gh_number_p (gap))
        {
-         int gap_i = gh_scm2int ( (gap));
+         int gap_i = gh_scm2int ((gap));
          int nogap = rwholebeams - gap_i;
          
          for (; j  < nogap; j++)
@@ -673,22 +869,30 @@ Beam::stem_beams (Stem *here, Stem *next, Stem *prev) const
              b.translate_axis (-dir  * bdy * j, Y_AXIS);
              rightbeams.add_molecule (b);
            }
-         // TODO: notehead widths differ for different types
-         gap_f = nw_f / 2;
+         if (Stem::invisible_b (here))
+           gap_f = nw_f;
+         else
+           gap_f = nw_f / 2;
          w -= 2 * gap_f;
-         a = lookup_l ()->beam (dydx, w + stemdx, thick);
+         a = Lookup::beam (dydx, w + stem_w, thick);
        }
 
       for (; j  < rwholebeams; j++)
        {
          Molecule b (a);
-         b.translate (Offset (here->invisible_b () ? 0 : gap_f, -dir * bdy * j));
+         Real tx = 0;
+         if (Stem::invisible_b (here))
+           // ugh, see chord-tremolo.ly
+           tx = (-dir + 1) / 2 * nw_f * 1.5 + gap_f/4;
+         else
+           tx = gap_f;
+         b.translate (Offset (tx, -dir * bdy * j));
          rightbeams.add_molecule (b);
        }
 
       w = w/2 <? nw_f;
       if (rhalfs)
-       a = lookup_l ()->beam (dydx, w, thick);
+       a = Lookup::beam (dydx, w, thick);
 
       for (; j  < rwholebeams + rhalfs; j++)
        {
@@ -707,59 +911,76 @@ Beam::stem_beams (Stem *here, Stem *next, Stem *prev) const
   return leftbeams;
 }
 
-
-Molecule 
-Beam::do_brew_molecule () const
+MAKE_SCHEME_CALLBACK (Beam,brew_molecule,1);
+SCM
+Beam::brew_molecule (SCM smob)
 {
+  Grob * me =unsmob_grob (smob);
+
   Molecule mol;
-  if (!stem_count ())
-    return mol;
+  if (!gh_pair_p (me->get_grob_property ("stems")))
+    return SCM_EOL;
   Real x0,dx;
-  if (visible_stem_count ())
+  Link_array<Item>stems = 
+    Pointer_group_interface__extract_elements (me, (Item*) 0, "stems");  
+  if (visible_stem_count (me))
     {
-      x0 = first_visible_stem ()->relative_coordinate (0, X_AXIS);
-      dx = last_visible_stem ()->relative_coordinate (0, X_AXIS) - x0;
+  // ugh -> use commonx
+      x0 = first_visible_stem (me)->relative_coordinate (0, X_AXIS);
+      dx = last_visible_stem (me)->relative_coordinate (0, X_AXIS) - x0;
     }
   else
     {
-      x0 = stem (0)->relative_coordinate (0, X_AXIS);
-      dx = stem_top ()->relative_coordinate (0, X_AXIS) - x0;
+      x0 = stems[0]->relative_coordinate (0, X_AXIS);
+      dx = stems.top ()->relative_coordinate (0, X_AXIS) - x0;
     }
   
+
+
+  /*
+    TODO: the naming of the grob properties sucks.
+   */
+  SCM dy_s = me->get_grob_property ("dy");
+  SCM y_s = me->get_grob_property ("y");
+
   
-  Real dy = gh_scm2double (get_elt_property ("height"));
+  Real dy = gh_number_p (dy_s) ? gh_scm2double (dy_s) : 0.0;
   Real dydx = dy && dx ? dy/dx : 0;
-  Real y = gh_scm2double (get_elt_property ("y-position"));
-  for (int j=0; j <stem_count (); j++)
+  Real y = gh_number_p (y_s) ? gh_scm2double (y_s) : 0.0;
+
+
+  for (int j=0; j <stems.size (); j++)
     {
-      Stem *i = stem (j);
-      Stem * prev = (j > 0)? stem (j-1) : 0;
-      Stem * next = (j < stem_count ()-1) ? stem (j+1) :0;
+      Item *i = stems[j];
+      Item * prev = (j > 0)? stems[j-1] : 0;
+      Item * next = (j < stems.size ()-1) ? stems[j+1] :0;
 
-      Molecule sb = stem_beams (i, next, prev);
+      Molecule sb = stem_beams (me, i, next, prev, dy, dydx);
       Real x = i->relative_coordinate (0, X_AXIS)-x0;
       sb.translate (Offset (x, x * dydx + y));
       mol.add_molecule (sb);
     }
   mol.translate_axis (x0 
-    - get_bound (LEFT)->relative_coordinate (0, X_AXIS), X_AXIS);
+    - dynamic_cast<Spanner*> (me)->get_bound (LEFT)->relative_coordinate (0, X_AXIS), X_AXIS);
 
-  return mol;
+  return mol.smobbed_copy ();
 }
 
 int
-Beam::forced_stem_count () const
+Beam::forced_stem_count (Grob*me) 
 {
+  Link_array<Item>stems = 
+    Pointer_group_interface__extract_elements (me, (Item*) 0, "stems");
   int f = 0;
-  for (int i=0; i < stem_count (); i++)
+  for (int i=0; i < stems.size (); i++)
     {
-      Stem *s = stem (i);
+      Item *s = stems[i];
 
-      if (s->invisible_b ())
+      if (Stem::invisible_b (s))
        continue;
 
-      if (((int)s->chord_start_f ()) 
-        && (s->get_direction () != s->get_default_dir ()))
+      if (( (int)Stem::chord_start_f (s)) 
+        && (Stem::get_direction (s) != Stem::get_default_dir (s)))
         f++;
     }
   return f;
@@ -767,66 +988,126 @@ Beam::forced_stem_count () const
 
 
 
-/*
-  TODO: Fix this class. This is wildly inefficient.
-  And it sux.  Yet another array/list 'interface'.
- */
-Stem *
-Beam::stem (int i) const
-{
-  return Group_interface__extract_elements ((Beam*) this, (Stem*) 0, "stems")[i];
-}
-
-int
-Beam::stem_count () const
-{
-  Group_interface gi (this, "stems");
-  return gi.count ();
-}
-
-Stem*
-Beam::stem_top () const
-{
-  SCM s = get_elt_property ("stems");
-  
-  return gh_pair_p (s) ? dynamic_cast<Stem*> (unsmob_element (gh_car (s))) : 0;
-    
-  //Group_interface__extract_elements ((Beam*) this, (Stem*) 0, "stems")[stem_count () - 1];
-}
 
-/* burp */
+/* TODO:
+   use filter and standard list functions.
+ */
 int
-Beam::visible_stem_count () const
+Beam::visible_stem_count (Grob*me) 
 {
+  Link_array<Item>stems = 
+    Pointer_group_interface__extract_elements (me, (Item*) 0, "stems");
   int c = 0;
-  for (int i = 0; i < stem_count (); i++)
+  for (int i = stems.size (); i--;)
     {
-      if (!stem (i)->invisible_b ())
+      if (!Stem::invisible_b (stems[i]))
         c++;
     }
   return c;
 }
 
-Stem*
-Beam::first_visible_stem () const
+Item*
+Beam::first_visible_stem (Grob*me) 
 {
-  for (int i = 0; i < stem_count (); i++)
+  Link_array<Item>stems = 
+    Pointer_group_interface__extract_elements (me, (Item*) 0, "stems");
+  
+  for (int i = 0; i < stems.size (); i++)
     {
-      Stem* s = stem (i);
-      if (!s->invisible_b ())
-        return s;
+      if (!Stem::invisible_b (stems[i]))
+        return stems[i];
     }
   return 0;
 }
 
-Stem*
-Beam::last_visible_stem () const
+Item*
+Beam::last_visible_stem (Grob*me) 
 {
-  for (int i = stem_count (); i > 0; i--)
+  Link_array<Item>stems = 
+    Pointer_group_interface__extract_elements (me, (Item*) 0, "stems");
+  for (int i = stems.size (); i--;)
     {
-      Stem* s = stem (i - 1);
-      if (!s->invisible_b ())
-        return s;
+      if (!Stem::invisible_b (stems[i]))
+        return stems[i];
     }
   return 0;
 }
+
+
+/*
+  [TODO]
+  handle rest under beam (do_post: beams are calculated now)
+  what about combination of collisions and rest under beam.
+
+  Should lookup
+    
+    rest -> stem -> beam -> interpolate_y_position ()
+*/
+MAKE_SCHEME_CALLBACK (Beam,rest_collision_callback,2);
+SCM
+Beam::rest_collision_callback (SCM element_smob, SCM axis)
+{
+  Grob *rest = unsmob_grob (element_smob);
+  Axis a = (Axis) gh_scm2int (axis);
+  
+  assert (a == Y_AXIS);
+
+  Grob * st = unsmob_grob (rest->get_grob_property ("stem"));
+  Grob * stem = st;
+  if (!stem)
+    return gh_double2scm (0.0);
+  Grob * beam = unsmob_grob (stem->get_grob_property ("beam"));
+  if (!beam || !Beam::has_interface (beam) || !Beam::visible_stem_count (beam))
+    return gh_double2scm (0.0);
+
+  // make callback for rest from this.
+  Real beam_dy = 0;
+  Real beam_y = 0;
+
+
+  // todo: make sure this calced already.
+  SCM s = beam->get_grob_property ("dy");
+  if (gh_number_p (s))
+    beam_dy = gh_scm2double (s);
+  
+  s = beam->get_grob_property ("y");
+  if (gh_number_p (s))
+    beam_y = gh_scm2double (s);
+  
+  // ugh -> use commonx
+  Real x0 = first_visible_stem (beam)->relative_coordinate (0, X_AXIS);
+  Real dx = last_visible_stem (beam)->relative_coordinate (0, X_AXIS) - x0;
+  Real dydx = beam_dy && dx ? beam_dy/dx : 0;
+
+  Direction d = Stem::get_direction (stem);
+  Real beamy = (stem->relative_coordinate (0, X_AXIS) - x0) * dydx + beam_y;
+
+  Real staff_space =   Staff_symbol_referencer::staff_space (rest);
+
+  
+  Real rest_dim = rest->extent (rest, Y_AXIS)[d]*2.0 / staff_space ; // refp??
+
+  Real minimum_dist
+    = gh_scm2double (rest->get_grob_property ("minimum-beam-collision-distance"));
+  Real dist =
+    minimum_dist +  -d  * (beamy - rest_dim) >? 0;
+
+  int stafflines = Staff_symbol_referencer::line_count (rest);
+
+  // move discretely by half spaces.
+  int discrete_dist = int (ceil (dist));
+
+  // move by whole spaces inside the staff.
+  if (discrete_dist < stafflines+1)
+    discrete_dist = int (ceil (discrete_dist / 2.0)* 2.0);
+
+  return gh_double2scm (-d *  discrete_dist);
+}
+
+
+bool
+Beam::has_interface (Grob*me)
+{
+  return me->has_interface (ly_symbol2scm ("beam-interface"));
+}
+