]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/stem.cc
* Documentation/user/changing-defaults.itely (Text encoding): node
[lilypond.git] / lily / stem.cc
index 4b9462376357e50bc4777559994f0f93c74f2a0a..5aa88448492b8452b70544c76ba732ed0bc14708 100644 (file)
@@ -20,7 +20,7 @@
 #include "note-head.hh"
 #include "stem.hh"
 #include "warn.hh"
-#include "paper-def.hh"
+#include "output-def.hh"
 #include "rhythmic-head.hh"
 #include "font-interface.hh"
 #include "stencil.hh"
 #include "stem-tremolo.hh"
 
 void
-Stem::set_beaming (Grob*me, int beam_count,  Direction d)
+Stem::set_beaming (Grob *me, int beam_count, Direction d)
 {
   SCM pair = me->get_property ("beaming");
-  
-  if (!gh_pair_p (pair))
+
+  if (!ly_c_pair_p (pair))
     {
-      pair = gh_cons (SCM_EOL, SCM_EOL);
+      pair = scm_cons (SCM_EOL, SCM_EOL);
       me->set_property ("beaming", pair);
     }
 
-  SCM l = index_get_cell (pair, d);
-  for( int i = 0; i<  beam_count; i++)
-    {
-      l = gh_cons (gh_int2scm (i), l);
-    }
-  index_set_cell (pair, d, l);         
+  SCM lst = index_get_cell (pair, d);
+  for (int i = 0; i < beam_count; i++)
+    lst = scm_cons (scm_int2num (i), lst);
+  index_set_cell (pair, d, lst);
+}
+
+int
+Stem::get_beaming (Grob *me, Direction d)
+{
+  SCM pair = me->get_property ("beaming");
+  if (!ly_c_pair_p (pair))
+    return 0;
+
+  SCM lst = index_get_cell (pair, d);
+  return scm_ilength (lst);
 }
 
 
 Interval
-Stem::head_positions (Grob*me) 
+Stem::head_positions (Grob *me)
 {
-  if (!head_count (me))
+  if (head_count (me))
     {
-      Interval iv;
-      return iv;
+      Drul_array<Grob*> e (extremal_heads (me));
+      return Interval (Staff_symbol_referencer::get_position (e[DOWN]),
+                      Staff_symbol_referencer::get_position (e[UP]));
     }
-
-  Drul_array<Grob*> e (extremal_heads (me));
-
-  return Interval (Staff_symbol_referencer::get_position (e[DOWN]),
-                  Staff_symbol_referencer::get_position (e[UP]));
+  return Interval ();
 }
 
-
 Real
-Stem::chord_start_y (Grob*me) 
+Stem::chord_start_y (Grob *me)
 {
-  return head_positions (me)[get_direction (me)]
-    * Staff_symbol_referencer::staff_space (me)/2.0;
+  Interval hp = head_positions (me);
+  if (!hp.is_empty ())
+    return hp[get_direction (me)] * Staff_symbol_referencer::staff_space (me)
+      * 0.5;
+  return 0;
 }
 
 Real
-Stem::stem_end_position (Grob*me) 
+Stem::stem_end_position (Grob *me)
 {
-  SCM p =me->get_property ("stem-end-position");
+  SCM p = me->get_property ("stem-end-position");
   Real pos;
-  if (!gh_number_p (p))
+  if (!ly_c_number_p (p))
     {
       pos = get_default_stem_end_position (me);
-      me->set_property ("stem-end-position", gh_double2scm (pos));
+      me->set_property ("stem-end-position", scm_make_real (pos));
     }
   else
-    pos = gh_scm2double (p);
+    pos = ly_scm2double (p);
 
   return pos;
 }
 
 Direction
-Stem::get_direction (Grob*me)
+Stem::get_direction (Grob *me)
 {
   Direction d = get_grob_direction (me);
 
   if (!d)
     {
-       d = get_default_dir (me);
-       // urg, AAARGH!
-       set_grob_direction (me, d);
+      d = get_default_dir (me);
+      // urg, AAARGH!
+      set_grob_direction (me, d);
     }
-  return d ;
+  return d;
 }
 
-
 void
-Stem::set_stemend (Grob*me, Real se)
+Stem::set_stemend (Grob *me, Real se)
 {
   // todo: margins
-  Direction d= get_direction (me);
-  
+  Direction d = get_direction (me);
+
   if (d && d * head_positions (me)[get_direction (me)] >= se*d)
     me->warning (_ ("Weird stem size; check for narrow beams"));
 
-  me->set_property ("stem-end-position", gh_double2scm (se));
+  me->set_property ("stem-end-position", scm_make_real (se));
 }
 
-
-/*
-  Note head that determines hshift for upstems
-
-  WARNING: triggers direction
-*/ 
-Grob*
-Stem::support_head (Grob*me)
+/* Note head that determines hshift for upstems
+   WARNING: triggers direction  */
+Grob *
+Stem::support_head (Grob *me)
 {
   if (head_count (me) == 1)
-    {
-      /*
-       UGH.
-       */
-      
-      return unsmob_grob (ly_car (me->get_property ("note-heads")));
-    }
-  else
-    return first_head (me);
+    /* UGH. */
+    return unsmob_grob (ly_car (me->get_property ("note-heads")));
+  return first_head (me);
 }
 
-
 int
-Stem::head_count (Grob*me)
+Stem::head_count (Grob *me)
 {
-  return  Pointer_group_interface::count (me, "note-heads");
+  return Pointer_group_interface::count (me, "note-heads");
 }
 
-/*
-  The note head which forms one end of the stem.  
-
-  WARNING: triggers direction
-*/
-Grob*
-Stem::first_head (Grob*me)
+/* The note head which forms one end of the stem.
+   WARNING: triggers direction  */
+Grob *
+Stem::first_head (Grob *me)
 {
   Direction d = get_direction (me);
-  if (!d)
-    return 0;
-  return extremal_heads (me)[-d];
+  if (d)
+    return extremal_heads (me)[-d];
+  return 0;
 }
 
-/*
-  The note head opposite to the first head.
- */
-Grob*
-Stem::last_head (Grob*me)
+/* The note head opposite to the first head.  */
+Grob *
+Stem::last_head (Grob *me)
 {
   Direction d = get_direction (me);
-  if (!d)
-    return 0;
-  return extremal_heads (me)[d];
+  if (d)
+    return extremal_heads (me)[d];
+  return 0;  
 }
 
-/*
-  START is part where stem reaches `last' head. 
- */
+/* START is part where stem reaches `last' head.
+
+This function returns a drul with (bottom-head, top-head). 
+
+*/
 Drul_array<Grob*>
-Stem::extremal_heads (Grob*me) 
+Stem::extremal_heads (Grob *me)
 {
   const int inf = 1000000;
   Drul_array<int> extpos;
   extpos[DOWN] = inf;
-  extpos[UP] = -inf;  
-  
+  extpos[UP] = -inf;
+
   Drul_array<Grob *> exthead;
   exthead[LEFT] = exthead[RIGHT] =0;
-  
-  for (SCM s = me->get_property ("note-heads"); gh_pair_p (s); s = ly_cdr (s))
-    {
-      Grob * n = unsmob_grob (ly_car (s));
 
-      
-      int p = int (Staff_symbol_referencer::get_position (n));
+  for (SCM s = me->get_property ("note-heads"); ly_c_pair_p (s);
+       s = ly_cdr (s))
+    {
+      Grob *n = unsmob_grob (ly_car (s));
+      int p = Staff_symbol_referencer::get_rounded_position (n);
 
       Direction d = LEFT;
-      do {
-      if (d* p > d* extpos[d])
+      do
        {
-         exthead[d] = n;
-         extpos[d] = p;
-       }
-      } while (flip (&d) != DOWN);
+         if (d * p > d * extpos[d])
+           {
+             exthead[d] = n;
+             extpos[d] = p;
+           }
+       } while (flip (&d) != DOWN);
     }
-
   return exthead;
 }
 
 static int
 icmp (int const &a, int const &b)
 {
-  return a-b;
+  return a - b;
 }
 
-/*
-  The positions, in ascending order.
- */
+/* The positions, in ascending order.  */
 Array<int>
 Stem::note_head_positions (Grob *me)
 {
   Array<int> ps ;
-  for (SCM s = me->get_property ("note-heads"); gh_pair_p (s); s = ly_cdr (s))
+  for (SCM s = me->get_property ("note-heads"); ly_c_pair_p (s);
+       s = ly_cdr (s))
     {
-      Grob * n = unsmob_grob (ly_car (s));
-      int p = int (Staff_symbol_referencer::get_position (n));
+      Grob *n = unsmob_grob (ly_car (s));
+      int p = Staff_symbol_referencer::get_rounded_position (n);
 
       ps.push (p);
     }
-  
+
   ps.sort (icmp);
-  return ps; 
+  return ps;
 }
 
-
 void
-Stem::add_head (Grob*me, Grob *n)
+Stem::add_head (Grob *me, Grob *n)
 {
   n->set_property ("stem", me->self_scm ());
   n->add_dependency (me);
 
-  /*
-    TODO: why not store Rest pointers? 
-  */
+  /* TODO: why not store Rest pointers? */
   if (Note_head::has_interface (n))
-    {
-      Pointer_group_interface::add_grob (me, ly_symbol2scm ("note-heads"), n);
-    }
+    Pointer_group_interface::add_grob (me, ly_symbol2scm ("note-heads"), n);
 }
 
 bool
-Stem::is_invisible (Grob*me)
+Stem::is_invisible (Grob *me)
 {
-  return ! (head_count (me)
-           && gh_scm2int (me->get_property ("duration-log")) >= 1);
+  return !(head_count (me)
+          && ly_scm2int (me->get_property ("duration-log")) >= 1);
 }
 
 Direction
-Stem::get_default_dir (Grob*me) 
+Stem::get_default_dir (Grob *me)
 {
   int staff_center = 0;
   Interval hp = head_positions (me);
   if (hp.is_empty ())
-    {
-      return CENTER;
-    }
-  
+    return CENTER;
+
   int udistance = (int) (UP * hp[UP] - staff_center);
-  int ddistance = (int) (DOWN* hp[DOWN] - staff_center);  
-  
+  int ddistance = (int) (DOWN * hp[DOWN] - staff_center);
+
   if (sign (ddistance - udistance))
-    return Direction (sign (ddistance -udistance));
+    return Direction (sign (ddistance - udistance));
 
   return to_dir (me->get_property ("neutral-direction"));
 }
 
 Real
-Stem::get_default_stem_end_position (Grob*me) 
+Stem::get_default_stem_end_position (Grob *me)
 {
-  Real ss = Staff_symbol_referencer::staff_space (me); 
-
+  Real ss = Staff_symbol_referencer::staff_space (me);
   int durlog = duration_log (me);
-    
   SCM s;
   Array<Real> a;
 
-  
-  Real length = 7;             // WARNING: IN HALF SPACES
+  /* WARNING: IN HALF SPACES */
+  Real length = 7;
   SCM scm_len = me->get_property ("length");
-  if (gh_number_p (scm_len))
-    {
-      length = gh_scm2double (scm_len);
-    }
+  if (ly_c_number_p (scm_len))
+    length = ly_scm2double (scm_len);
   else
     {
       s = me->get_property ("lengths");
-      if (gh_pair_p (s))
-       {
-         length = 2* gh_scm2double (robust_list_ref (durlog -2, s));
-       }
+      if (ly_c_pair_p (s))
+       length = 2 * ly_scm2double (robust_list_ref (durlog - 2, s));
     }
 
-
-
   /* URGURGURG
-     'set-default-stemlen' sets direction too
-   */
+     'set-default-stemlen' sets direction too.   */
   Direction dir = get_direction (me);
   if (!dir)
     {
@@ -314,78 +289,60 @@ Stem::get_default_stem_end_position (Grob*me)
       set_grob_direction (me, dir);
     }
 
-
-  /* stems in unnatural (forced) direction should be shortened, 
-    according to [Roush & Gourlay] */
-  if (!chord_start_y (me)
-      || (get_direction (me) != get_default_dir (me)))
+  /* Stems in unnatural (forced) direction should be shortened,
+     according to [Roush & Gourlay] */
+  Interval hp = head_positions (me);
+  if (dir && dir * hp[dir] >= 0)
     {
-      
-  
       SCM sshorten = me->get_property ("stem-shorten");
-      SCM scm_shorten = gh_pair_p (sshorten) ?
+      SCM scm_shorten = ly_c_pair_p (sshorten) ?
        robust_list_ref ((duration_log (me) - 2) >? 0, sshorten): SCM_EOL;
       Real shorten = 2* robust_scm2double (scm_shorten,0);
-      
-  
+
       /* On boundary: shorten only half */
-      if (abs (head_positions (me)[get_direction (me)]) <= 1)
+      if (abs (head_positions (me)[dir]) <= 1)
        shorten *= 0.5;
-  
+
       length -= shorten;
     }
 
-  /*
-    Tremolo stuff: 
-  */
-  Grob * trem = unsmob_grob (me->get_property ("tremolo-flag"));
-  if (trem &&  !unsmob_grob (me->get_property ("beam")))
+  /* Tremolo stuff.  */
+  Grob *t_flag = unsmob_grob (me->get_property ("tremolo-flag"));
+  if (t_flag && !unsmob_grob (me->get_property ("beam")))
     {
-      /*
-       Crude hack: add extra space if tremolo flag is there.
+      /* Crude hack: add extra space if tremolo flag is there.
 
        We can't do this for the beam, since we get into a loop
-       (Stem_tremolo::raw_stencil() looks at the beam.)
-       
-        --hwn 
-      */
-      
-      Real minlen =
-       1.0 + 2 * Stem_tremolo::raw_stencil (trem).extent (Y_AXIS).length  () / ss;
-      
+       (Stem_tremolo::raw_stencil () looks at the beam.) --hwn  */
+
+      Real minlen = 1.0
+       + 2 * Stem_tremolo::raw_stencil (t_flag).extent (Y_AXIS).length  ()
+       / ss;
+
       if (durlog >= 3)
        {
-         Interval flag_ext = flag (me).extent (Y_AXIS) ;
+         Interval flag_ext = flag (me).extent (Y_AXIS);
          if (!flag_ext.is_empty ())
-           minlen += 2 * flag_ext.length () / ss ;
+           minlen += 2 * flag_ext.length () / ss;
 
-         /*
-           The clash is smaller for down stems (since the tremolo is
-           angled up.)
-          */
+         /* The clash is smaller for down stems (since the tremolo is
+            angled up.) */
          if (dir == DOWN)
            minlen -= 1.0;
        }
-      
       length = length >? (minlen + 1.0);
     }
-   
-  Interval hp = head_positions (me);  
-  Real st = hp[dir] + dir * length;
 
-  /*
-    TODO: change name  to extend-stems to staff/center/'()
-  */
+  Real st = dir ? hp[dir] + dir * length : 0;
+
+  /* TODO: change name  to extend-stems to staff/center/'()  */
   bool no_extend_b = to_boolean (me->get_property ("no-stem-extend"));
-  if (!no_extend_b && dir * st < 0) // junkme?
+  if (!no_extend_b && dir * st < 0)
     st = 0.0;
 
-  /*
-    Make a little room if we have a upflag and there is a dot.
-    previous approach was to lengthen the stem. This is not
-    good typesetting practice. 
-    
-  */
+  /* Make a little room if we have a upflag and there is a dot.
+     previous approach was to lengthen the stem. This is not
+     good typesetting practice.  */
   if (!get_beam (me) && dir == UP
       && durlog > 2)
     {
@@ -396,13 +353,10 @@ Stem::get_default_stem_end_position (Grob*me)
       if (dots)
        {
          Real dp = Staff_symbol_referencer::get_position (dots);
-         Real flagy =  flag (me).extent (Y_AXIS)[-dir] * 2
-           / ss;
+         Real flagy = flag (me).extent (Y_AXIS)[-dir] * 2 / ss;
 
-         /*
-           Very gory: add myself to the X-support of the parent,
-           which should be a dot-column.
-          */
+         /* Very gory: add myself to the X-support of the parent,
+            which should be a dot-column. */
          if (dir * (st + flagy -  dp) < 0.5)
            {
              Grob *par = dots->get_parent (X_AXIS);
@@ -411,62 +365,51 @@ Stem::get_default_stem_end_position (Grob*me)
                {
                  Side_position_interface::add_support (par, me);
 
-                 /*
-                   TODO: apply some better logic here. The flag is
-                   curved inwards, so this will typically be too
-                   much.
-                 */
+                 /* TODO: apply some better logic here. The flag is
+                    curved inwards, so this will typically be too
+                    much. */
                }
            }
        }
     }
-
-
   return st;
 }
 
-
-
-/*
-  
-  the log of the duration (Number of hooks on the flag minus two)
- */
+/* The log of the duration (Number of hooks on the flag minus two)  */
 int
-Stem::duration_log (Grob*me) 
+Stem::duration_log (Grob *me)
 {
   SCM s = me->get_property ("duration-log");
-  return (gh_number_p (s)) ? gh_scm2int (s) : 2;
+  return (ly_c_number_p (s)) ? ly_scm2int (s) : 2;
 }
 
 void
-Stem::position_noteheads (Grob*me)
+Stem::position_noteheads (Grob *me)
 {
   if (!head_count (me))
     return;
-  
+
   Link_array<Grob> heads =
-    Pointer_group_interface__extract_grobs (me, (Grob*)0, "note-heads");
+    Pointer_group_interface__extract_grobs (me, (Grob*) 0, "note-heads");
 
   heads.sort (compare_position);
-  Direction dir =get_direction (me);
-  
+  Direction dir = get_direction (me);
+
   if (dir < 0)
     heads.reverse ();
 
-
   Real thick = thickness (me);
-      
+
   Grob *hed = support_head (me);
-  Real w = Note_head::head_extent (hed,X_AXIS)[dir];
-  for (int i=0; i < heads.size (); i++)
-    {
-      heads[i]->translate_axis (w - Note_head::head_extent (heads[i],X_AXIS)[dir],
-                               X_AXIS);
-    }
-  
-  bool parity= true;
+  Real w = Note_head::head_extent (hed, X_AXIS)[dir];
+  for (int i = 0; i < heads.size (); i++)
+    heads[i]->translate_axis (w - Note_head::head_extent (heads[i],
+                                                         X_AXIS)[dir],
+                             X_AXIS);
+
+  bool parity = true;
   Real lastpos = Real (Staff_symbol_referencer::get_position (heads[0]));
-  for (int i=1; i < heads.size (); i ++)
+  for (int i = 1; i < heads.size (); i ++)
     {
       Real p = Staff_symbol_referencer::get_position (heads[i]);
       Real dy =fabs (lastpos- p);
@@ -479,45 +422,45 @@ Stem::position_noteheads (Grob*me)
        {
          if (parity)
            {
-             Real l = Note_head::head_extent (heads[i], X_AXIS).length ();
+             Real ell = Note_head::head_extent (heads[i], X_AXIS).length ();
 
              Direction d = get_direction (me);
              /*
-               Reversed head should be shifted l-thickness, but this
-               looks too crowded, so we only shift l-0.5*thickness.
+               Reversed head should be shifted ell-thickness, but this
+               looks too crowded, so we only shift ell-0.5*thickness.
 
                This leads to assymetry: Normal heads overlap the
                stem 100% whereas reversed heads only overlaps the
                stem 50%
-
              */
 
-             Real reverse_overlap =0.5;
-             heads[i]->translate_axis ((l-thick*reverse_overlap) * d, X_AXIS);
+             Real reverse_overlap = 0.5;
+             heads[i]->translate_axis ((ell - thick * reverse_overlap) * d,
+                                       X_AXIS);
 
-             if (is_invisible(me))
-               heads[i]->translate_axis (-thick*(2 - reverse_overlap) * d , X_AXIS);
+             if (is_invisible (me))
+               heads[i]->translate_axis (-thick * (2 - reverse_overlap) * d,
+                                         X_AXIS);
 
-             
             /* TODO:
-                
+               
              For some cases we should kern some more: when the
-             distance between the next or prev note is too large, we'd 
+             distance between the next or prev note is too large, we'd
              get large white gaps, eg.
-             
+       
                |
               X|
               |X  <- kern this.
               |
              X
-             
+       
              */
            }
          parity = !parity;
        }
       else
        parity = true;
-      
+
       lastpos = int (p);
     }
 }
@@ -526,8 +469,7 @@ MAKE_SCHEME_CALLBACK (Stem,before_line_breaking,1);
 SCM
 Stem::before_line_breaking (SCM smob)
 {
-  Grob*me = unsmob_grob (smob);
-
+  Grob *me = unsmob_grob (smob);
 
   /*
     Do the calculations for visible stems, but also for invisible stems
@@ -539,9 +481,7 @@ Stem::before_line_breaking (SCM smob)
       position_noteheads (me);
     }
   else
-    {
-      me->set_property ("print-function", SCM_EOL);
-    }
+    me->set_property ("print-function", SCM_EOL);
   
   return SCM_UNSPECIFIED;
 }
@@ -555,10 +495,20 @@ MAKE_SCHEME_CALLBACK (Stem, height, 2);
 SCM
 Stem::height (SCM smob, SCM ax)
 {
-  Axis a = (Axis)gh_scm2int (ax);
-  Grob * me = unsmob_grob (smob);
+  Axis a = (Axis)ly_scm2int (ax);
+  Grob *me = unsmob_grob (smob);
   assert (a == Y_AXIS);
 
+  /*
+    ugh. - this dependency should be automatic.
+  */
+  Grob *beam= get_beam (me);
+  if (beam)
+    {
+      Beam::after_line_breaking (beam->self_scm ());
+    }
+  
+
   SCM mol = me->get_uncached_stencil ();
   Interval iv;
   if (mol != SCM_EOL)
@@ -566,7 +516,7 @@ Stem::height (SCM smob, SCM ax)
   if (Grob *b =get_beam (me))
     {
       Direction d = get_direction (me);
-      iv[d] += d * Beam::get_thickness (b) /2.0 ;
+      iv[d] += d * Beam::get_thickness (b) * 0.5 ;
     }
 
   return ly_interval2scm (iv);
@@ -574,25 +524,21 @@ Stem::height (SCM smob, SCM ax)
 
 
 Stencil
-Stem::flag (Grob*me)
+Stem::flag (Grob *me)
 {
   /* TODO: maybe property stroke-style should take different values,
      e.g. "" (i.e. no stroke), "single" and "double" (currently, it's
      '() or "grace").  */
   String flag_style;
-  
-  SCM flag_style_scm = me->get_property ("flag-style");
-  if (gh_symbol_p (flag_style_scm))
-    {
-      flag_style = ly_symbol2string (flag_style_scm);
-    }
 
+  SCM flag_style_scm = me->get_property ("flag-style");
+  if (ly_c_symbol_p (flag_style_scm))
+    flag_style = ly_symbol2string (flag_style_scm);
+  
   if (flag_style == "no-flag")
-    {
-      return Stencil ();
-    }
+    return Stencil ();
 
-  bool adjust = to_boolean (me->get_property ("adjust-if-on-staffline"));
+  bool adjust = true;
 
   String staffline_offs;
   if (String::compare (flag_style, "mensural") == 0)
@@ -608,7 +554,7 @@ Stem::flag (Grob*me)
         {
          /* Urrgh!  We have to detect wether this stem ends on a staff
             line or between two staff lines.  But we can not call
-            stem_end_position(me) or get_default_stem_end_position(me),
+            stem_end_position (me) or get_default_stem_end_position (me),
             since this encounters the flag and hence results in an
             infinite recursion.  However, in pure mensural notation,
             there are no multiple note heads attached to a single stem,
@@ -627,12 +573,12 @@ Stem::flag (Grob*me)
          /*
            perhaps the detection whether this correction is needed should
            happen in a different place  to avoid the recursion.
-           
+       
            --hwn.
          */
-         int p = (int)rint (Staff_symbol_referencer::get_position (first_head (me)));
-         staffline_offs = Staff_symbol_referencer::on_staffline (me, p) ?
-           "1" : "0";
+         int p = Staff_symbol_referencer::get_rounded_position (me);
+         staffline_offs = Staff_symbol_referencer::on_staffline (me, p)
+           "1" : "0";
        }
       else
         {
@@ -645,17 +591,15 @@ Stem::flag (Grob*me)
     }
 
   char dir = (get_direction (me) == UP) ? 'u' : 'd';
-  String font_char =
-    flag_style + to_string (dir) + staffline_offs + to_string (duration_log (me));
+  String font_char = flag_style
+    + to_string (dir) + staffline_offs + to_string (duration_log (me));
   Font_metric *fm = Font_interface::get_default_font (me);
   Stencil flag = fm->find_by_name ("flags-" + font_char);
   if (flag.is_empty ())
-    {
-      me->warning (_f ("flag `%s' not found", font_char));
-    }
+    me->warning (_f ("flag `%s' not found", font_char));
 
   SCM stroke_style_scm = me->get_property ("stroke-style");
-  if (gh_string_p (stroke_style_scm))
+  if (ly_c_string_p (stroke_style_scm))
     {
       String stroke_style = ly_scm2string (stroke_style_scm);
       if (!stroke_style.is_empty ())
@@ -663,13 +607,9 @@ Stem::flag (Grob*me)
          String font_char = to_string (dir) + stroke_style;
          Stencil stroke = fm->find_by_name ("flags-" + font_char);
          if (stroke.is_empty ())
-           {
-             me->warning (_f ("flag stroke `%s' not found", font_char));
-           }
+           me->warning (_f ("flag stroke `%s' not found", font_char));
          else
-           {
-             flag.add_stencil (stroke);
-           }
+           flag.add_stencil (stroke);
        }
     }
 
@@ -680,42 +620,36 @@ MAKE_SCHEME_CALLBACK (Stem,dim_callback,2);
 SCM
 Stem::dim_callback (SCM e, SCM ax)
 {
-  Axis a = (Axis) gh_scm2int (ax);
+  Axis a = (Axis) ly_scm2int (ax);
   assert (a == X_AXIS);
   Grob *me = unsmob_grob (e);
   Interval r (0, 0);
   if (unsmob_grob (me->get_property ("beam")) || abs (duration_log (me)) <= 2)
     ;  // TODO!
   else
-    {
-      r = flag (me).extent (X_AXIS)
-       + thickness (me)/2;
-    }
+    r = flag (me).extent (X_AXIS)
+      + thickness (me)/2;
   return ly_interval2scm (r);
 }
+
 Real
-Stem::thickness (Grobme)
+Stem::thickness (Grob *me)
 {
-  return gh_scm2double (me->get_property ("thickness"))
+  return ly_scm2double (me->get_property ("thickness"))
     * Staff_symbol_referencer::line_thickness (me);
 }
 
-MAKE_SCHEME_CALLBACK (Stem,print,1);
-
+MAKE_SCHEME_CALLBACK (Stem, print, 1);
 SCM
-Stem::print (SCM smob) 
+Stem::print (SCM smob)
 {
-  Grob*me = unsmob_grob (smob);
+  Grob *me = unsmob_grob (smob);
   Stencil mol;
   Direction d = get_direction (me);
-     
-  /*
-    TODO: make the stem start a direction ?
 
-    This is required to avoid stems passing in tablature chords...
-  */
-  Grob *lh = to_boolean (me->get_property ("avoid-note-head")) 
+  /* TODO: make the stem start a direction ?
+     This is required to avoid stems passing in tablature chords.  */
+  Grob *lh = to_boolean (me->get_property ("avoid-note-head"))
     ? last_head (me) :  lh = first_head (me);
 
   if (!lh)
@@ -723,12 +657,11 @@ Stem::print (SCM smob)
 
   if (is_invisible (me))
     return SCM_EOL;
-  
+
   Real y1 = Staff_symbol_referencer::get_position (lh);
   Real y2 = stem_end_position (me);
-  
+
   Interval stem_y (y1 <? y2,y2 >? y1);
 
   // dy?
   Real dy = Staff_symbol_referencer::staff_space (me) * 0.5;
@@ -745,12 +678,12 @@ Stem::print (SCM smob)
       stem_y[Direction (-d)] += d * y_attach/dy;
     }
 
-  
+
   // URG
   Real stem_width = thickness (me);
-  Real blot = 
-       me->get_paper ()->get_realvar (ly_symbol2scm ("blotdiameter"));
-  
+  Real blot =
+       me->get_paper ()->get_dimension (ly_symbol2scm ("blotdiameter"));
+
   Box b = Box (Interval (-stem_width/2, stem_width/2),
               Interval (stem_y[DOWN]*dy, stem_y[UP]*dy));
 
@@ -771,58 +704,43 @@ Stem::print (SCM smob)
 /*
   move the stem to right of the notehead if it is up.
  */
-MAKE_SCHEME_CALLBACK (Stem,off_callback,2);
+MAKE_SCHEME_CALLBACK (Stem, off_callback, 2);
 SCM
 Stem::off_callback (SCM element_smob, SCM)
 {
   Grob *me = unsmob_grob (element_smob);
+  Real r = 0.0;
   
-  Real r=0;
-
-  if (head_count (me) == 0)
-    {
-      return gh_double2scm (0.0);
-    }
-  
-  if (Grob * f = first_head (me))
-    {
-      Interval head_wid = Note_head::head_extent(f, X_AXIS);
-      
-      Real attach =0.0;
-
-      if (is_invisible (me))
-       {
+  if (head_count (me))
+    if (Grob *f = first_head (me))
+      {
+       Interval head_wid = Note_head::head_extent (f, X_AXIS);
+       Real attach = 0.0;
+       
+       if (is_invisible (me))
          attach = 0.0;
-       }
-      else
-       attach = Note_head::stem_attachment_coordinate(f, X_AXIS);
-
-      Direction d = get_direction (me);
-
-      Real real_attach = head_wid.linear_combination (d * attach);
-
-      r = real_attach;
-
-      /*
-       If not centered: correct for stem thickness.
-       */
-      if (attach)
-       {
-         Real rule_thick
-           = thickness (me);
-         
-         r += - d * rule_thick * 0.5;
-       }
-    }
-  return gh_double2scm (r);
+       else
+       attach = Note_head::stem_attachment_coordinate (f, X_AXIS);
+       
+       Direction d = get_direction (me);
+       Real real_attach = head_wid.linear_combination (d * attach);
+       r = real_attach;
+       
+       /* If not centered: correct for stem thickness.  */
+       if (attach)
+         {
+           Real rule_thick = thickness (me);
+           r += - d * rule_thick * 0.5;
+         }
+      }
+  return scm_make_real (r);
 }
 
-
-Grob*
-Stem::get_beam (Grob*me)
+Spanner *
+Stem::get_beam (Grob *me)
 {
-  SCM b me->get_property ("beam");
-  return unsmob_grob (b);
+  SCM b = me->get_property ("beam");
+  return dynamic_cast<Spanner*> (unsmob_grob (b));
 }
 
 Stem_info
@@ -830,23 +748,21 @@ Stem::get_stem_info (Grob *me)
 {
   /* Return cached info if available */
   SCM scm_info = me->get_property ("stem-info");
-  if (!gh_pair_p (scm_info))
+  if (!ly_c_pair_p (scm_info))
     {
       calc_stem_info (me);
       scm_info = me->get_property ("stem-info");
     }
-  
+
   Stem_info si;
-  si.dir_ = get_grob_direction (me); 
-  si.ideal_y_ = gh_scm2double (gh_car (scm_info)); 
-  si.shortest_y_ = gh_scm2double (gh_cadr (scm_info));
+  si.dir_ = get_grob_direction (me);
+  si.ideal_y_ = ly_scm2double (ly_car (scm_info));
+  si.shortest_y_ = ly_scm2double (ly_cadr (scm_info));
   return si;
 }
 
 
-/*
-  TODO: add extra space for tremolos!
- */
+/* TODO: add extra space for tremolos!  */
 void
 Stem::calc_stem_info (Grob *me)
 {
@@ -857,7 +773,7 @@ Stem::calc_stem_info (Grob *me)
       programming_error ("No stem dir set?");
       my_dir  = UP;
     }
-  
+
   Real staff_space = Staff_symbol_referencer::staff_space (me);
   Grob *beam = get_beam (me);
   Real beam_translation = Beam::get_beam_translation (beam);
@@ -868,25 +784,27 @@ Stem::calc_stem_info (Grob *me)
   /* Simple standard stem length */
   SCM lengths = me->get_property ("beamed-lengths");
   Real ideal_length =
-    gh_scm2double (robust_list_ref (beam_count - 1,lengths))
+    ly_scm2double (robust_list_ref (beam_count - 1,lengths))
                
     * staff_space
-    /* stem only extends to center of beam */
-    - 0.5 * beam_thickness;
-  
+    /* stem only extends to center of beam
+    */
+    - 0.5 * beam_thickness
+    ;
+
   /* Condition: sane minimum free stem length (chord to beams) */
   lengths = me->get_property ("beamed-minimum-free-lengths");
   Real ideal_minimum_free =
-    gh_scm2double (robust_list_ref (beam_count - 1, lengths))
+    ly_scm2double (robust_list_ref (beam_count - 1, lengths))
     * staff_space;
-  
+
 
   /* UGH
      It seems that also for ideal minimum length, we must use
      the maximum beam count (for this direction):
-     
+
      \score{ \notes\relative c''{ [a8 a32] }}
-     
+
      must be horizontal. */
   Real height_of_my_beams = beam_thickness
     + (beam_count - 1) * beam_translation;
@@ -898,7 +816,6 @@ Stem::calc_stem_info (Grob *me)
 
   ideal_length = ideal_length >? ideal_minimum_length;
 
-  
   /* Convert to Y position, calculate for dir == UP */
   Real note_start =
     /* staff positions */
@@ -910,9 +827,9 @@ Stem::calc_stem_info (Grob *me)
   /* Conditions for Y position */
 
   /* Lowest beam of (UP) beam must never be lower than second staffline
+
      Reference?
+
      Although this (additional) rule is probably correct,
      I expect that highest beam (UP) should also never be lower
      than middle staffline, just as normal stems.
@@ -920,7 +837,7 @@ Stem::calc_stem_info (Grob *me)
      Reference?
 
      Obviously not for grace beams.
-     
+
      Also, not for knees.  Seems to be a good thing. */
   bool no_extend_b = to_boolean (me->get_property ("no-stem-extend"));
   bool is_knee = to_boolean (beam->get_property ("knee"));
@@ -938,7 +855,7 @@ Stem::calc_stem_info (Grob *me)
   ideal_y -= robust_scm2double (beam->get_property ("shorten"), 0);
 
   Real minimum_free =
-    gh_scm2double (robust_list_ref
+    ly_scm2double (robust_list_ref
                   (beam_count - 1,
                    me->get_property
                    ("beamed-extreme-minimum-free-lengths")))
@@ -949,55 +866,45 @@ Stem::calc_stem_info (Grob *me)
     /* stem only extends to center of beam */
     - 0.5 * beam_thickness;
 
-  Real minimum_y = note_start + minimum_length;
-  
-  
   ideal_y *= my_dir;
-  Real shortest_y = minimum_y * my_dir; 
-  
+  Real minimum_y = note_start + minimum_length;
+  Real shortest_y = minimum_y * my_dir;
+
   me->set_property ("stem-info",
-                        scm_list_n (gh_double2scm (ideal_y),
-                                    gh_double2scm (shortest_y),
-                                    SCM_UNDEFINED));
+                   scm_list_2 (scm_make_real (ideal_y),
+                               scm_make_real (shortest_y)));
 }
 
 Slice
 Stem::beam_multiplicity (Grob *stem)
 {
   SCM beaming= stem->get_property ("beaming");
-  Slice l = int_list_to_slice (gh_car (beaming));
-  Slice r = int_list_to_slice (gh_cdr (beaming));
-  l.unite (r);
-
-  return l;
+  Slice le = int_list_to_slice (ly_car (beaming));
+  Slice ri = int_list_to_slice (ly_cdr (beaming));
+  le.unite (ri);
+  return le;
 }
 
 
-/*
-  these are too many props.
- */
-ADD_INTERFACE (Stem,"stem-interface",
-              "The stem represent the graphical  stem. "
-              "  In addition, it internally connects note heads, beams, tremolos. Rests "
-              " and whole notes have invisible stems."
-
-,
-              
+/* FIXME:  Too many properties  */
+ADD_INTERFACE (Stem, "stem-interface",
+              "The stem represent the graphical stem.  "
+              "In addition, it internally connects note heads, beams and"
+              "tremolos. "
+              "Rests and whole notes have invisible stems.",
               "tremolo-flag french-beaming "
-              "avoid-note-head adjust-if-on-staffline thickness "
+              "avoid-note-head thickness "
               "stem-info beamed-lengths beamed-minimum-free-lengths "
               "beamed-extreme-minimum-free-lengths lengths beam stem-shorten "
               "duration-log beaming neutral-direction stem-end-position "
               "note-heads direction length flag-style "
               "no-stem-extend stroke-style");
 
-
-
 /****************************************************************/
 
-Stem_info::Stem_info()
+Stem_info::Stem_info ()
 {
-  ideal_y_ = shortest_y_ =0;
+  ideal_y_ = shortest_y_ = 0;
   dir_ = CENTER;
 }