*** empty log message ***

[lilypond.git] / lily / note-column.cc

diff --git a/lily/note-column.cc b/lily/note-column.cc
index 5f523ea26b13fdcebfc6f7cf4cccb2d3ae671a7d..a3afffe4264bcf4909d4ab17287d170436bfacb4 100644 (file)
--- a/lily/note-column.cc
+++ b/lily/note-column.cc
@@ -3,7 +3,7 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997--2003 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1997--2004 Han-Wen Nienhuys <hanwen@cs.uu.nl>
 */
 #include <math.h>              // ceil
 
@@ -11,34 +11,39 @@
 #include "note-column.hh"
 #include "stem.hh"
 #include "warn.hh"
-#include "paper-def.hh"
+#include "output-def.hh"
 #include "group-interface.hh"
 #include "staff-symbol-referencer.hh"
 #include "rest.hh"
 #include "note-head.hh"
 #include "accidental-placement.hh"
 
+/*
+  TODO: figure out if we can prune this class. This is just an
+  annoying layer between (rest)collision & (note-head + stem)
+ */
+
 bool
-Note_column::rest_b (Grob*me) 
+Note_column::has_rests (Grob*me) 
 {
-  return unsmob_grob (me->get_grob_property ("rest"));
+  return unsmob_grob (me->get_property ("rest"));
 }
 
 int
 Note_column::shift_compare (Grob *const &p1, Grob *const&p2)
 {
-  SCM s1 = p1->get_grob_property ("horizontal-shift");
-  SCM s2 = p2->get_grob_property ("horizontal-shift");
+  SCM s1 = p1->get_property ("horizontal-shift");
+  SCM s2 = p2->get_property ("horizontal-shift");
 
-  int h1 = (gh_number_p (s1))?  gh_scm2int (s1) :0;
-  int h2 = (gh_number_p (s2)) ? gh_scm2int (s2):0;
+  int h1 = (scm_is_number (s1))?  scm_to_int (s1) :0;
+  int h2 = (scm_is_number (s2)) ? scm_to_int (s2):0;
   return h1 - h2;
 }
 
 Item *
 Note_column::get_stem (Grob*me) 
 {
-  SCM s = me->get_grob_property ("stem");
+  SCM s = me->get_property ("stem");
   return  unsmob_item (s);
 }
   
@@ -49,12 +54,12 @@ Note_column::head_positions_interval (Grob *me)
 
   iv.set_empty ();
 
-  SCM h = me->get_grob_property ("note-heads");
-  for (; gh_pair_p (h); h = ly_cdr (h))
+  SCM h = me->get_property ("note-heads");
+  for (; scm_is_pair (h); h = scm_cdr (h))
     {
-      Grob *se = unsmob_grob (ly_car (h));
+      Grob *se = unsmob_grob (scm_car (h));
       
-      int j = int (Staff_symbol_referencer::get_position (se));
+      int j = Staff_symbol_referencer::get_rounded_position (se);
       iv.unite (Slice (j,j));
     }
   return iv;
@@ -63,10 +68,10 @@ Note_column::head_positions_interval (Grob *me)
 Direction
 Note_column::dir (Grob*  me)
 {
-  Grob *stem = unsmob_grob (me->get_grob_property ("stem"));
+  Grob *stem = unsmob_grob (me->get_property ("stem"));
   if (stem && Stem::has_interface (stem))
     return Stem::get_direction (stem);
-  else if (gh_pair_p (me->get_grob_property ("note-heads")))
+  else if (scm_is_pair (me->get_property ("note-heads")))
     return (Direction)sign (head_positions_interval (me).center ());
 
   programming_error ("Note column without heads and stem!");
@@ -77,36 +82,53 @@ Note_column::dir (Grob*  me)
 void
 Note_column::set_stem (Grob*me,Grob * stem)
 {
-  me->set_grob_property ("stem", stem->self_scm ());
+  me->set_property ("stem", stem->self_scm ());
   me->add_dependency (stem);
   Axis_group_interface::add_element (me, stem);
 }
 
+
+Grob*
+Note_column::get_rest (Grob*me)
+{
+  return unsmob_grob (me->get_property ("rest"));
+}
+  
 void
 Note_column::add_head (Grob*me,Grob *h)
 {
+  bool both = false;
   if (Rest::has_interface (h))
     {
-      me->set_grob_property ("rest", h->self_scm ());
+      if (scm_is_pair (me->get_property ("note-heads")))
+       both = true;
+      else
+       me->set_property ("rest", h->self_scm ());
     }
   else if (Note_head::has_interface (h))
     {
+      if (unsmob_grob (me->get_property ("rest")))
+       both = true;
       Pointer_group_interface::add_grob (me, ly_symbol2scm ("note-heads"),h);
     }
-  Axis_group_interface::add_element (me, h);
+
+  if (both)
+    me->warning ("Can't have rests and note heads together on a stem.");
+  else
+    Axis_group_interface::add_element (me, h);
 }
 
 /**
-  translate the rest symbols vertically by amount DY_I, but only if
+  translate the rest symbols vertically by amount DY, but only if
   they have no staff-position set.
 */
 void
-Note_column::translate_rests (Grob*me,int dy_i)
+Note_column::translate_rests (Grob*me, int dy)
 {
-  Grob * r = unsmob_grob (me->get_grob_property ("rest"));
-  if (r && !gh_number_p (r->get_grob_property ("staff-position")))
+  Grob * r = unsmob_grob (me->get_property ("rest"));
+  if (r && !scm_is_number (r->get_property ("staff-position")))
     {
-      r->translate_axis (dy_i * Staff_symbol_referencer::staff_space (r)/2.0, Y_AXIS);
+      r->translate_axis (dy * Staff_symbol_referencer::staff_space (r)/2.0, Y_AXIS);
     }
 }
 
@@ -134,12 +156,12 @@ Note_column::first_head (Grob*me)
 Grob* 
 Note_column::accidentals (Grob *me)
 {
-  SCM heads = me->get_grob_property ("note-heads");
+  SCM heads = me->get_property ("note-heads");
   Grob * acc = 0;
-  for (;gh_pair_p (heads); heads =gh_cdr (heads))
+  for (;scm_is_pair (heads); heads =scm_cdr (heads))
     {
-      Grob * h = unsmob_grob (gh_car (heads));
-      acc = h ? unsmob_grob (h->get_grob_property ("accidental-grob")) : 0;
+      Grob * h = unsmob_grob (scm_car (heads));
+      acc = h ? unsmob_grob (h->get_property ("accidental-grob")) : 0;
       if (acc)
        break;
     }