]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/beam.cc
* ly/engraver-init.ly (AncientRemoveEmptyStaffContext): use length-fraction.
[lilypond.git] / lily / beam.cc
index 868470c3b52ae07a6339570c3b7ab32f4e49c870..c4fc1147c4a26f4c084d3a82c715a9f44ab5c171 100644 (file)
@@ -24,8 +24,6 @@
   - beam_translation is the offset between Y centers of the beam.
 */
 
-#include <math.h> // tanh.
-
 #include "beam.hh"
 #include "interval-set.hh"
 #include "directional-element-interface.hh"
@@ -35,7 +33,7 @@
 #include "stem.hh"
 #include "output-def.hh"
 #include "lookup.hh"
-#include "group-interface.hh"
+#include "pointer-group-interface.hh"
 #include "staff-symbol-referencer.hh"
 #include "item.hh"
 #include "spanner.hh"
 void
 Beam::add_stem (Grob *me, Grob *s)
 {
-  Pointer_group_interface::add_grob (me, ly_symbol2scm ("stems"), s);
-
-  s->add_dependency (me);
-
-  assert (!Stem::get_beam (s));
-  s->set_property ("beam", me->self_scm ());
+  if (Stem::get_beam (s))
+    {
+      programming_error ("Stem already has beam");
+      return ;
+    }
 
+  Pointer_group_interface::add_grob (me, ly_symbol2scm ("stems"), s);
+  s->set_object ("beam", me->self_scm ());
   add_bound_item (dynamic_cast<Spanner *> (me), dynamic_cast<Item *> (s));
 }
 
@@ -72,15 +71,13 @@ Beam::get_beam_translation (Grob *me)
 {
   SCM func = me->get_property ("space-function");
 
-  if (ly_c_procedure_p (func))
+  if (ly_is_procedure (func))
     {
-      SCM s = scm_call_2 (func, me->self_scm (), scm_int2num (get_beam_count (me)));
+      SCM s = scm_call_2 (func, me->self_scm (), scm_from_int (get_beam_count (me)));
       return scm_to_double (s);
     }
   else
-    {
-      return 0.81;
-    }
+    return 0.81;
 }
 
 /* Maximum beam_count. */
@@ -88,9 +85,11 @@ int
 Beam::get_beam_count (Grob *me)
 {
   int m = 0;
-  for (SCM s = me->get_property ("stems"); scm_is_pair (s); s = scm_cdr (s))
+
+  extract_grob_set (me, "stems", stems);
+  for (int i = 0; i < stems.size (); i++)
     {
-      Grob *stem = unsmob_grob (scm_car (s));
+      Grob *stem = stems[i];
       m = max (m, (Stem::beam_multiplicity (stem).length () + 1));
     }
   return m;
@@ -113,7 +112,7 @@ Beam::space_function (SCM smob, SCM beam_count)
     ? (2 * staff_space + line - thickness) / 2.0
     : (3 * staff_space + line - thickness) / 3.0;
 
-  return scm_make_real (beam_translation);
+  return scm_from_double (beam_translation);
 }
 
 /* After pre-processing all directions should be set.
@@ -122,9 +121,9 @@ Beam::space_function (SCM smob, SCM beam_count)
    Currenly, this means that beam has set all stem's directions.
    [Alternatively, stems could set its own directions, according to
    their beam, during 'final-pre-processing'.] */
-MAKE_SCHEME_CALLBACK (Beam, before_line_breaking, 1);
+MAKE_SCHEME_CALLBACK (Beam, calc_direction, 1);
 SCM
-Beam::before_line_breaking (SCM smob)
+Beam::calc_direction (SCM smob)
 {
   Grob *me = unsmob_grob (smob);
 
@@ -139,26 +138,28 @@ Beam::before_line_breaking (SCM smob)
   int count = visible_stem_count (me);
   if (count < 2)
     {
-      SCM stems = me->get_property ("stems");
-      if (scm_ilength (stems) == 1)
+      extract_grob_set (me, "stems", stems);
+      if (stems.size () == 1)
        {
          me->warning (_ ("removing beam with less than two stems"));
 
-         unsmob_grob (scm_car (stems))->set_property ("beam", SCM_EOL);
+         stems[0]->set_object ("beam", SCM_EOL);
          me->suicide ();
 
          return SCM_UNSPECIFIED;
        }
-      else if (scm_ilength (stems) == 0)
+      else if (stems.size () == 0)
        {
          me->suicide ();
          return SCM_UNSPECIFIED;
        }
     }
+
+  Direction d = CENTER;
+  
   if (count >= 1)
     {
-      Direction d = get_default_dir (me);
-
+      d = get_default_dir (me);
       consider_auto_knees (me);
       set_stem_directions (me, d);
 
@@ -167,7 +168,7 @@ Beam::before_line_breaking (SCM smob)
       set_stem_shorten (me);
     }
 
-  return SCM_EOL;
+  return scm_from_int (d);
 }
 
 /* We want a maximal number of shared beams, but if there is choice, we
@@ -198,7 +199,7 @@ position_with_maximal_common_beams (SCM left_beaming, SCM right_beaming,
       for (SCM s = scm_car (right_beaming); scm_is_pair (s); s = scm_cdr (s))
        {
          int k = -right_dir * scm_to_int (scm_car (s)) + i;
-         if (scm_c_memq (scm_int2num (k), left_beaming) != SCM_BOOL_F)
+         if (scm_c_memq (scm_from_int (k), left_beaming) != SCM_BOOL_F)
            count++;
        }
 
@@ -215,12 +216,13 @@ position_with_maximal_common_beams (SCM left_beaming, SCM right_beaming,
 void
 Beam::connect_beams (Grob *me)
 {
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
 
   Slice last_int;
   last_int.set_empty ();
-  SCM last_beaming = SCM_EOL;
+  
+  //  SCM last_beaming = SCM_EOL;
+  SCM last_beaming = scm_cons (SCM_EOL, scm_list_1 (scm_from_int (0)));
   Direction last_dir = CENTER;
   for (int i = 0; i < stems.size (); i++)
     {
@@ -232,15 +234,13 @@ Beam::connect_beams (Grob *me)
        {
          int start_point = position_with_maximal_common_beams
            (last_beaming, this_beaming,
-            last_dir, this_dir);
+            last_dir ? last_dir : this_dir,
+            this_dir);
 
          Direction d = LEFT;
          Slice new_slice;
          do
            {
-             if (d == RIGHT && i == stems.size () - 1)
-               continue;
-
              new_slice.set_empty ();
              SCM s = index_get_cell (this_beaming, d);
              for (; scm_is_pair (s); s = scm_cdr (s))
@@ -249,7 +249,7 @@ Beam::connect_beams (Grob *me)
                    = start_point - this_dir * scm_to_int (scm_car (s));
 
                  new_slice.add_point (new_beam_pos);
-                 scm_set_car_x (s, scm_int2num (new_beam_pos));
+                 scm_set_car_x (s, scm_from_int (new_beam_pos));
                }
            }
          while (flip (&d) != LEFT);
@@ -259,21 +259,15 @@ Beam::connect_beams (Grob *me)
        }
       else
        {
-         scm_set_car_x (this_beaming, SCM_EOL);
          SCM s = scm_cdr (this_beaming);
          for (; scm_is_pair (s); s = scm_cdr (s))
            {
              int np = -this_dir * scm_to_int (scm_car (s));
-             scm_set_car_x (s, scm_int2num (np));
+             scm_set_car_x (s, scm_from_int (np));
              last_int.add_point (np);
            }
        }
-
-      if (i == stems.size () -1)
-       {
-         scm_set_cdr_x (this_beaming, SCM_EOL);
-       }
-
+      
       if (scm_ilength (scm_cdr (this_beaming)) > 0)
        {
          last_beaming = this_beaming;
@@ -283,17 +277,18 @@ Beam::connect_beams (Grob *me)
 }
 
 /*
-  TODO: should not make beams per stem, but per Y-level.
+  I really enjoy spaghetti, but spaghetti should be kept on a plate
+  with a little garlic and olive oil. This is too much.
+
+  rewrite-me
 */
 MAKE_SCHEME_CALLBACK (Beam, print, 1);
 SCM
 Beam::print (SCM grob)
 {
   Spanner *me = unsmob_spanner (grob);
-  position_beam (me);
 
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
   Grob *xcommon = common_refpoint_of_array (stems, me, X_AXIS);
 
   xcommon = me->get_bound (LEFT)->common_refpoint (xcommon, X_AXIS);
@@ -341,19 +336,19 @@ Beam::print (SCM grob)
 
   for (int i = 0; i <= stems.size (); i++)
     {
-      Grob *st = (i < stems.size ()) ? stems[i] : 0;
+      Grob *stem = (i < stems.size ()) ? stems[i] : 0;
 
-      SCM this_beaming = st ? st->get_property ("beaming") : SCM_EOL;
-      Real xposn = st ? st->relative_coordinate (xcommon, X_AXIS) : 0.0;
-      Real stem_width = st ? robust_scm2double (st->get_property ("thickness"), 1.0) * lt : 0;
-      Direction stem_dir = st ? to_dir (st->get_property ("direction")) : CENTER;
+      SCM this_beaming = stem ? stem->get_property ("beaming") : SCM_EOL;
+      Real xposn = stem ? stem->relative_coordinate (xcommon, X_AXIS) : 0.0;
+      Real stem_width = stem ? robust_scm2double (stem->get_property ("thickness"), 1.0) * lt : 0;
+      Direction stem_dir = stem ? to_dir (stem->get_property ("direction")) : CENTER;
       /*
        We do the space left of ST, with lfliebertjes pointing to the
        right from the left stem, and rfliebertjes pointing left from
        right stem.
       */
       SCM left = (i > 0) ? scm_cdr (last_beaming) : SCM_EOL;
-      SCM right = st ? scm_car (this_beaming) : SCM_EOL;
+      SCM right = stem ? scm_car (this_beaming) : SCM_EOL;
 
       Array<int> full_beams;
       Array<int> lfliebertjes;
@@ -364,29 +359,25 @@ Beam::print (SCM grob)
        {
          int b = scm_to_int (scm_car (s));
          if (scm_c_memq (scm_car (s), right) != SCM_BOOL_F)
-           {
-             full_beams.push (b);
-           }
+           full_beams.push (b);
          else
-           {
-             lfliebertjes.push (b);
-           }
+           lfliebertjes.push (b);
        }
       for (SCM s = right;
           scm_is_pair (s); s = scm_cdr (s))
        {
          int b = scm_to_int (scm_car (s));
          if (scm_c_memq (scm_car (s), left) == SCM_BOOL_F)
-           {
-             rfliebertjes.push (b);
-           }
+           rfliebertjes.push (b);
        }
 
-      /*
-       how much to stick out for beams across linebreaks
-      */
-      Real break_overshoot = 3.0;
-      Real w = (i > 0 && st) ? (xposn - last_xposn) : break_overshoot;
+      Drul_array<Real> break_overshoot
+       = robust_scm2drul (me->get_property ("break-overshoot"),
+                          Drul_array<Real> (-0.5, 0.0));
+
+      Real w = (i > 0 && stem)
+       ? (xposn - last_xposn)
+       : break_overshoot[ (i == 0) ? LEFT : RIGHT];
 
       Real stem_offset = 0.0;
       if (i > 0)
@@ -395,7 +386,7 @@ Beam::print (SCM grob)
          stem_offset = -last_stem_width / 2;
        }
 
-      if (st)
+      if (stem)
        w += stem_width / 2;
 
       Real blot = me->get_layout ()->get_dimension (ly_symbol2scm ("blotdiameter"));
@@ -433,16 +424,16 @@ Beam::print (SCM grob)
        {
          Real nw_f;
 
-         if (st)
+         if (stem)
            {
-             int t = Stem::duration_log (st);
+             int t = Stem::duration_log (stem);
 
              SCM proc = me->get_property ("flag-width-function");
-             SCM result = scm_call_1 (proc, scm_int2num (t));
+             SCM result = scm_call_1 (proc, scm_from_int (t));
              nw_f = scm_to_double (result);
            }
          else
-           nw_f = break_overshoot / 2;
+           nw_f = break_overshoot[RIGHT] / 2;
 
          /* Half beam should be one note-width,
             but let's make sure two half-beams never touch */
@@ -451,32 +442,43 @@ Beam::print (SCM grob)
          if (i > 0)
            rw = min (nw_f, ((xposn - last_xposn) / 2));
          else
-           /*
-             TODO: 0.5 is a guess.
-           */
-           rw = xposn - me->get_bound (LEFT)->extent (xcommon, X_AXIS)[RIGHT]
-             - 0.5;
-
-         if (st)
+           {
+             if (me->get_bound (LEFT)->break_status_dir ())
+               rw = xposn - me->get_bound (LEFT)->extent (xcommon, X_AXIS)[RIGHT]
+                 + break_overshoot[LEFT];
+             else
+               rw = 1.0;       // ugh.
+           }
+         
+         if (stem)
            lw = min (nw_f, ((xposn - last_xposn) / 2));
          else
-           lw = me->get_bound (RIGHT)->relative_coordinate (xcommon, X_AXIS)
-             - last_xposn;
+           {
+             lw = me->get_bound (RIGHT)->relative_coordinate (xcommon, X_AXIS)
+               - last_xposn
+               + break_overshoot[RIGHT];
+           }
+         rw += stem_width / 2;
+         lw += last_stem_width / 2;
 
          Stencil rhalf = Lookup::beam (slope, rw, thick, blot);
          Stencil lhalf = Lookup::beam (slope, lw, thick, blot);
          for (int j = lfliebertjes.size (); j--;)
            {
              Stencil b (lhalf);
-             b.translate_axis (last_xposn - x0, X_AXIS);
-             b.translate_axis (slope * (last_xposn - x0) + bdy * lfliebertjes[j], Y_AXIS);
+             b.translate_axis (last_xposn - x0 - last_stem_width /2,
+                               X_AXIS);
+             b.translate_axis (slope * (last_xposn - x0)
+                               + bdy * lfliebertjes[j],
+                               Y_AXIS);
              the_beam.add_stencil (b);
            }
          for (int j = rfliebertjes.size (); j--;)
            {
              Stencil b (rhalf);
-             b.translate_axis (xposn - x0 - rw, X_AXIS);
-             b.translate_axis (slope * (xposn - x0 -rw) + bdy * rfliebertjes[j], Y_AXIS);
+             b.translate_axis (xposn - x0 - rw + stem_width / 2, X_AXIS);
+             b.translate_axis (slope * (xposn - x0 - rw)
+                               + bdy * rfliebertjes[j], Y_AXIS);
              the_beam.add_stencil (b);
            }
        }
@@ -486,13 +488,14 @@ Beam::print (SCM grob)
       last_beaming = this_beaming;
     }
 
-  the_beam.translate_axis (x0 - me->relative_coordinate (xcommon, X_AXIS), X_AXIS);
+  the_beam.translate_axis (x0 - me->relative_coordinate (xcommon, X_AXIS),
+                          X_AXIS);
   the_beam.translate_axis (pos[LEFT], Y_AXIS);
 
 #if (DEBUG_QUANTING)
   SCM quant_score = me->get_property ("quant-score");
-  if (to_boolean (me->get_layout ()->lookup_variable (ly_symbol2scm ("debug-beam-quanting")))
-      && scm_is_string (quant_score))
+  SCM debug = me->get_layout ()->lookup_variable (ly_symbol2scm ("debug-beam-quanting"));
+  if (to_boolean (debug) && scm_is_string (quant_score))
     {
 
       /*
@@ -521,40 +524,38 @@ Beam::get_default_dir (Grob *me)
   total[UP] = total[DOWN] = 0;
   Drul_array<int> count;
   count[UP] = count[DOWN] = 0;
-  Direction d = DOWN;
 
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
 
   for (int i = 0; i < stems.size (); i++)
-    do
-      {
-       Grob *s = stems[i];
-       Direction sd = get_grob_direction (s);
-
-       int center_distance = max (int (- d * Stem::head_positions (s) [-d]), 0);
-       int current = sd ? (1 + d * sd) / 2 : center_distance;
-
-       if (current)
-         {
-           total[d] += current;
-           count[d]++;
-         }
-      }
-    while (flip (&d) != DOWN);
-
-  SCM func = me->get_property ("dir-function");
-  SCM s = scm_call_2 (func,
-                     scm_cons (scm_int2num (count[UP]),
-                               scm_int2num (count[DOWN])),
-                     scm_cons (scm_int2num (total[UP]),
-                               scm_int2num (total[DOWN])));
-
-  if (scm_is_number (s) && scm_to_int (s))
-    return to_dir (s);
-
-  /* If dir is not determined: get default */
-  return to_dir (me->get_property ("neutral-direction"));
+    {
+      Grob *s = stems[i];
+      Direction stem_dir = CENTER;
+      SCM stem_dir_scm = s->get_property_data (ly_symbol2scm ("direction"));
+      if (is_direction (stem_dir_scm))
+       stem_dir = to_dir (stem_dir_scm);
+      else
+       stem_dir = Stem::get_default_dir (s);
+
+      if (stem_dir)
+       {
+         count[stem_dir] ++;
+         total[stem_dir] += max (int (- stem_dir * Stem::head_positions (s) [-stem_dir]), 0);
+       }
+    }
+
+  Direction dir = CENTER;
+  
+  if (Direction d =  (Direction) sign (count[UP] - count[DOWN]))
+    dir = d;
+  else if (Direction d = (Direction)  sign (total[UP] / count[UP] - total[DOWN]/count[DOWN]))
+    dir = d;
+  else if (Direction d = (Direction)  sign (total[UP] - total[DOWN]))
+    dir = d;
+  else
+    dir = to_dir (me->get_property ("neutral-direction"));
+  
+  return dir;
 }
 
 /* Set all stems with non-forced direction to beam direction.
@@ -563,14 +564,13 @@ Beam::get_default_dir (Grob *me)
 void
 Beam::set_stem_directions (Grob *me, Direction d)
 {
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
 
   for (int i = 0; i < stems.size (); i++)
     {
       Grob *s = stems[i];
 
-      SCM forcedir = s->get_property ("direction");
+      SCM forcedir = s->get_property_data (ly_symbol2scm ("direction"));
       if (!to_dir (forcedir))
        set_grob_direction (s, d);
     }
@@ -599,8 +599,7 @@ Beam::consider_auto_knees (Grob *me)
 
   gaps.set_full ();
 
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
 
   Grob *common = common_refpoint_of_array (stems, me, Y_AXIS);
   Real staff_space = Staff_symbol_referencer::staff_space (me);
@@ -626,7 +625,7 @@ Beam::consider_auto_knees (Grob *me)
          */
          head_extents += stem->relative_coordinate (common, Y_AXIS);
 
-         if (to_dir (stem->get_property ("direction")))
+         if (to_dir (stem->get_property_data (ly_symbol2scm ("direction"))))
            {
              Direction stemdir = to_dir (stem->get_property ("direction"));
              head_extents[-stemdir] = -stemdir * infinity_f;
@@ -678,7 +677,7 @@ Beam::consider_auto_knees (Grob *me)
          Direction d = (head_extents.center () < max_gap.center ())
            ? UP : DOWN;
 
-         stem->set_property ("direction", scm_int2num (d));
+         stem->set_property ("direction", scm_from_int (d));
 
          head_extents.intersect (max_gap);
          assert (head_extents.is_empty () || head_extents.length () < 1e-6);
@@ -717,53 +716,39 @@ Beam::set_stem_shorten (Grob *me)
 
   SCM shorten_elt
     = robust_list_ref (beam_count -1, shorten_list);
-  Real shorten_f = scm_to_double (shorten_elt) * staff_space;
+  Real shorten = scm_to_double (shorten_elt) * staff_space;
 
-  /* your similar cute comment here */
-  shorten_f *= forced_fraction;
+  shorten *= forced_fraction;
 
-  if (shorten_f)
-    me->set_property ("shorten", scm_make_real (shorten_f));
+  if (shorten)
+    me->set_property ("shorten", scm_from_double (shorten));
 }
 
-/*  Call list of y-dy-callbacks, that handle setting of
-    grob-properties
-*/
-MAKE_SCHEME_CALLBACK (Beam, after_line_breaking, 1);
+MAKE_SCHEME_CALLBACK (Beam, calc_positions, 1);
 SCM
-Beam::after_line_breaking (SCM smob)
+Beam::calc_positions (SCM smob)
 {
   Grob *me = unsmob_grob (smob);
-
-  position_beam (me);
-  return SCM_UNSPECIFIED;
-}
-
-void
-Beam::position_beam (Grob *me)
-{
   if (!me->is_live ())
-    return;
-  if (to_boolean (me->get_property ("positioning-done")))
-    return;
-
-  me->set_property ("positioning-done", SCM_BOOL_T);
+    return SCM_EOL;
 
-  /* Copy to mutable list. */
-  SCM s = ly_deep_copy (me->get_property ("positions"));
-  me->set_property ("positions", s);
+  (void) me->get_property ("direction");
+  
+  SCM posns = scm_cons (SCM_BOOL_F, SCM_BOOL_F);
+  me->set_property ("positions", posns);
 
-  if (scm_car (s) == SCM_BOOL_F)
-    {
-      // one wonders if such genericity is necessary  --hwn.
-      SCM callbacks = me->get_property ("position-callbacks");
-      for (SCM i = callbacks; scm_is_pair (i); i = scm_cdr (i))
-       scm_call_1 (scm_car (i), me->self_scm ());
-    }
+  SCM callbacks = me->get_property ("position-callbacks");
+  for (SCM i = callbacks; scm_is_pair (i); i = scm_cdr (i))
+    scm_call_1 (scm_car (i), me->self_scm ());
 
+  /*
+    TODO: move this in separate calc function.
+   */
   set_stem_lengths (me);
+  return SCM_UNSPECIFIED;
 }
 
+
 void
 set_minimum_dy (Grob *me, Real *dy)
 {
@@ -805,8 +790,7 @@ Beam::least_squares (SCM smob)
     }
 
   Array<Real> x_posns;
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
   Grob *commonx = common_refpoint_of_array (stems, me, X_AXIS);
   Grob *commony = common_refpoint_of_array (stems, me, Y_AXIS);
 
@@ -854,9 +838,7 @@ Beam::least_squares (SCM smob)
          pos[-d] = -pos[d];
        }
       else
-       {
-         pos = ideal;
-       }
+       pos = ideal;
 
       /*
        For broken beams this doesn't work well. In this case, the
@@ -864,7 +846,7 @@ Beam::least_squares (SCM smob)
        where the second part goes.
       */
       me->set_property ("least-squares-dy",
-                       scm_make_real (pos[RIGHT] - pos[LEFT]));
+                       scm_from_double (pos[RIGHT] - pos[LEFT]));
     }
   else
     {
@@ -885,7 +867,7 @@ Beam::least_squares (SCM smob)
       dy = slope * dx;
 
       set_minimum_dy (me, &dy);
-      me->set_property ("least-squares-dy", scm_make_real (dy));
+      me->set_property ("least-squares-dy", scm_from_double (dy));
       pos = Interval (y, (y + dy));
     }
 
@@ -915,8 +897,7 @@ Beam::shift_region_to_valid (SCM grob)
     Code dup.
   */
   Array<Real> x_posns;
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
   Grob *commonx = common_refpoint_of_array (stems, me, X_AXIS);
   Grob *commony = common_refpoint_of_array (stems, me, Y_AXIS);
 
@@ -946,7 +927,7 @@ Beam::shift_region_to_valid (SCM grob)
 
   Real dy = pos[RIGHT] - pos[LEFT];
   Real y = pos[LEFT];
-  Real slope = dy / dx;
+  Real slope = dx ? (dy / dx) : 0.0;
 
   /*
     Shift the positions so that we have a chance of finding good
@@ -960,7 +941,7 @@ Beam::shift_region_to_valid (SCM grob)
       if (Stem::is_invisible (s))
        continue;
 
-      Direction d = Stem::get_direction (s);
+      Direction d = get_grob_direction (s);
 
       Real left_y
        = Stem::get_stem_info (s).shortest_y_
@@ -985,6 +966,7 @@ Beam::shift_region_to_valid (SCM grob)
     warning (_ ("no viable initial configuration found: may not find good beam slope"));
   else if (!feasible_left_point.contains (y))
     {
+      const int REGION_SIZE = 2; // UGH UGH
       if (isinf (feasible_left_point[DOWN]))
        y = feasible_left_point[UP] - REGION_SIZE;
       else if (isinf (feasible_left_point[UP]))
@@ -1116,9 +1098,7 @@ Beam::calc_stem_y (Grob *me, Grob *s, Grob ** common,
 void
 Beam::set_stem_lengths (Grob *me)
 {
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
-
+  extract_grob_set (me, "stems", stems);
   if (!stems.size ())
     return;
 
@@ -1169,10 +1149,9 @@ Beam::set_stem_lengths (Grob *me)
 }
 
 void
-Beam::set_beaming (Grob *me, Beaming_info_list *beaming)
+Beam::set_beaming (Grob *me, Beaming_info_list const *beaming)
 {
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
 
   Direction d = LEFT;
   for (int i = 0; i < stems.size (); i++)
@@ -1183,23 +1162,18 @@ Beam::set_beaming (Grob *me, Beaming_info_list *beaming)
 
       do
        {
-         /* Don't set beaming for outside of outer stems */
-         if ((d == LEFT && i == 0)
-             || (d == RIGHT && i == stems.size () -1))
-           continue;
-
-         Grob *st = stems[i];
-         SCM beaming_prop = st->get_property ("beaming");
+         Grob *stem = stems[i];
+         SCM beaming_prop = stem->get_property ("beaming");
          if (beaming_prop == SCM_EOL
              || index_get_cell (beaming_prop, d) == SCM_EOL)
            {
              int b = beaming->infos_.elem (i).beams_i_drul_[d];
              if (i > 0
                  && i < stems.size () -1
-                 && Stem::is_invisible (st))
+                 && Stem::is_invisible (stem))
                b = min (b, beaming->infos_.elem (i).beams_i_drul_[-d]);
 
-             Stem::set_beaming (st, b, d);
+             Stem::set_beaming (stem, b, d);
            }
        }
       while (flip (&d) != LEFT);
@@ -1209,8 +1183,8 @@ Beam::set_beaming (Grob *me, Beaming_info_list *beaming)
 int
 Beam::forced_stem_count (Grob *me)
 {
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
+
   int f = 0;
   for (int i = 0; i < stems.size (); i++)
     {
@@ -1222,7 +1196,7 @@ Beam::forced_stem_count (Grob *me)
       /* I can imagine counting those boundaries as a half forced stem,
         but let's count them full for now. */
       if (abs (Stem::chord_start_y (s)) > 0.1
-         && (Stem::get_direction (s) != Stem::get_default_dir (s)))
+         && (get_grob_direction (s) != Stem::get_default_dir (s)))
        f++;
     }
   return f;
@@ -1231,8 +1205,7 @@ Beam::forced_stem_count (Grob *me)
 int
 Beam::visible_stem_count (Grob *me)
 {
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
   int c = 0;
   for (int i = stems.size (); i--;)
     {
@@ -1245,8 +1218,7 @@ Beam::visible_stem_count (Grob *me)
 Grob *
 Beam::first_visible_stem (Grob *me)
 {
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
 
   for (int i = 0; i < stems.size (); i++)
     {
@@ -1259,8 +1231,8 @@ Beam::first_visible_stem (Grob *me)
 Grob *
 Beam::last_visible_stem (Grob *me)
 {
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
+
   for (int i = stems.size (); i--;)
     {
       if (!Stem::is_invisible (stems[i]))
@@ -1287,24 +1259,27 @@ Beam::rest_collision_callback (SCM element_smob, SCM axis)
   (void) axis;
 
   if (scm_is_number (rest->get_property ("staff-position")))
-    return scm_int2num (0);
+    return scm_from_int (0);
 
   assert (scm_to_int (axis) == Y_AXIS);
 
-  Grob *st = unsmob_grob (rest->get_property ("stem"));
+  Grob *st = unsmob_grob (rest->get_object ("stem"));
   Grob *stem = st;
   if (!stem)
-    return scm_make_real (0.0);
-  Grob *beam = unsmob_grob (stem->get_property ("beam"));
+    return scm_from_double (0.0);
+  Grob *beam = unsmob_grob (stem->get_object ("beam"));
   if (!beam
       || !Beam::has_interface (beam)
       || !Beam::visible_stem_count (beam))
-    return scm_make_real (0.0);
+    return scm_from_double (0.0);
 
   Drul_array<Real> pos (0, 0);
   SCM s = beam->get_property ("positions");
   if (scm_is_pair (s) && scm_is_number (scm_car (s)))
     pos = ly_scm2interval (s);
+  else
+    programming_error ("positions property should always be pair of numbers.");
+
   Real staff_space = Staff_symbol_referencer::staff_space (rest);
 
   scale_drul (&pos, staff_space);
@@ -1316,7 +1291,7 @@ Beam::rest_collision_callback (SCM element_smob, SCM axis)
   Real dx = last_visible_stem (beam)->relative_coordinate (0, X_AXIS) - x0;
   Real slope = dy && dx ? dy / dx : 0;
 
-  Direction d = Stem::get_direction (stem);
+  Direction d = get_grob_direction (stem);
   Real stem_y = pos[LEFT] + (stem->relative_coordinate (0, X_AXIS) - x0) * slope;
 
   Real beam_translation = get_beam_translation (beam);
@@ -1354,7 +1329,7 @@ Beam::rest_collision_callback (SCM element_smob, SCM axis)
       < rad)
     shift = ceil (fabs (shift)) * sign (shift);
 
-  return scm_make_real (staff_space * shift);
+  return scm_from_double (staff_space * shift);
 }
 
 bool
@@ -1366,9 +1341,10 @@ Beam::is_knee (Grob *me)
 
   bool knee = false;
   int d = 0;
-  for (SCM s = me->get_property ("stems"); scm_is_pair (s); s = scm_cdr (s))
+  extract_grob_set (me, "stems", stems);
+  for (int i = stems.size (); i--;)
     {
-      Direction dir = get_grob_direction (unsmob_grob (scm_car (s)));
+      Direction dir = get_grob_direction (stems[i]);
       if (d && d != dir)
        {
          knee = true;
@@ -1385,8 +1361,7 @@ Beam::is_knee (Grob *me)
 int
 Beam::get_direction_beam_count (Grob *me, Direction d)
 {
-  Link_array<Grob> stems
-    = extract_grob_array (me, ly_symbol2scm ("stems"));
+  extract_grob_set (me, "stems", stems);
   int bc = 0;
 
   for (int i = stems.size (); i--;)
@@ -1394,20 +1369,43 @@ Beam::get_direction_beam_count (Grob *me, Direction d)
       /*
        Should we take invisible stems into account?
       */
-      if (Stem::get_direction (stems[i]) == d)
+      if (get_grob_direction (stems[i]) == d)
        bc = max (bc, (Stem::beam_multiplicity (stems[i]).length () + 1));
     }
 
   return bc;
 }
 
-ADD_INTERFACE (Beam, "beam-interface",
-              "A beam. \n\n"
-              "The @code{thickness} property is the weight of beams, and is measured "
-              "in  staffspace",
-              "knee positioning-done position-callbacks "
-              "concaveness dir-function quant-score auto-knee-gap gap "
-              "gap-count chord-tremolo beamed-stem-shorten shorten least-squares-dy "
-              "damping inspect-quants flag-width-function neutral-direction positions space-function "
-              "thickness");
+ADD_INTERFACE (Beam,
+              "beam-interface",
 
+              "A beam. \n\n"
+              "The @code{thickness} property is the weight of beams, "
+              "measured in staffspace.  The @code{direction} property is not user-serviceable. Use "
+              "the @code{direction} property of @code{Stem} instead. "
+
+              ,
+              
+              /* properties */
+              "auto-knee-gap "
+              "beamed-stem-shorten "
+              "break-overshoot "
+              "chord-tremolo "
+              "concaveness "
+              "damping "
+              "details "
+              "direction " 
+              "flag-width-function "
+              "gap "
+              "gap-count "
+              "inspect-quants "
+              "knee "
+              "least-squares-dy "
+              "neutral-direction "
+              "position-callbacks "
+              "positions "
+              "quant-score "
+              "shorten "
+              "space-function "
+              "thickness "
+              );