]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/rest-collision.cc
2003 -> 2004
[lilypond.git] / lily / rest-collision.cc
index 6252f008666c8e413810fd396738f2a008eb60ac..ea072a3b1e05fd7cfc86bcda0512ba949450c6c7 100644 (file)
@@ -3,12 +3,12 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997--2000 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1997--2004 Han-Wen Nienhuys <hanwen@cs.uu.nl>
 */
 
 #include <math.h>              // ceil.
 
-#include "debug.hh"
+#include "warn.hh"
 #include "rest-collision.hh"
 #include "note-column.hh"
 #include "stem.hh"
 #include "staff-symbol-referencer.hh"
 #include "duration.hh"
 
-MAKE_SCHEME_CALLBACK(Rest_collision,force_shift_callback,2);
+MAKE_SCHEME_CALLBACK (Rest_collision,force_shift_callback,2);
 SCM
 Rest_collision::force_shift_callback (SCM element_smob, SCM axis)
 {
-  Grob *them = unsmob_element (element_smob);
+  Grob *them = unsmob_grob (element_smob);
   Axis a = (Axis) gh_scm2int (axis);
   assert (a == Y_AXIS);
 
-  Grob * rc = unsmob_element (them->get_grob_property ("rest-collision"));
+  Grob * rc = unsmob_grob (them->get_grob_property ("rest-collision"));
 
-  if (rc)
+  if (rc && !to_boolean (rc->get_grob_property ("positioning-done")))
     {
-      /*
-       Done: destruct pointers, so we do the shift only once.
-      */
-      SCM elts = rc->get_grob_property ("elements");
-      rc->set_grob_property ("elements", SCM_EOL);
+      rc->set_grob_property ("positioning-done", SCM_BOOL_T);
 
-      do_shift (rc, elts);
+      do_shift (rc);
     }
   
   return gh_double2scm (0.0);
@@ -47,15 +43,16 @@ void
 Rest_collision::add_column (Grob*me,Grob *p)
 {
   me->add_dependency (p);
-  Pointer_group_interface::add_element (me, "elements", p);
+  Pointer_group_interface::add_grob (me, ly_symbol2scm ("elements"), p);
 
   /*
-    only add callback for the rests, since we don't move anything else.
+    only add callback for the rests, since we don't move anything
+    else.
 
   (not?)
+ (not?)
   */
-      p->add_offset_callback (Rest_collision::force_shift_callback_proc, Y_AXIS);
-      p->set_grob_property ("rest-collision", me->self_scm ());
+  p->add_offset_callback (Rest_collision::force_shift_callback_proc, Y_AXIS);
+  p->set_grob_property ("rest-collision", me->self_scm ());
 }
 
 
@@ -65,7 +62,7 @@ Rest_collision::add_column (Grob*me,Grob *p)
 static SCM
 head_characteristic (Grob * col)
 {
-  Grob * s = unsmob_element (col->get_grob_property ("rest"));
+  Grob * s = unsmob_grob (col->get_grob_property ("rest"));
 
   if (!s)
     return SCM_BOOL_F;
@@ -76,29 +73,25 @@ head_characteristic (Grob * col)
 
 /*
   TODO: fixme, fucks up if called twice on the same set of rests.
+
+  TODO: look at horizontal-shift to determine ordering between rests
+  for more than two voices.
+
+  TODO: look at previous note to determine vertical position?
+  
  */
 SCM
-Rest_collision::do_shift (Grob *me, SCM elts)
+Rest_collision::do_shift (Grob *me)
 {
-  /*
-    ugh. -> score  elt type
-   */
+  SCM elts = me->get_grob_property ("elements");
+
   Link_array<Grob> rests;
   Link_array<Grob> notes;
-  Grob * commony = 0;
-  for (SCM s = elts; gh_pair_p (s); s = gh_cdr (s))
+
+  for (SCM s = elts; gh_pair_p (s); s = ly_cdr (s))
     {
-      
-      Grob * e = unsmob_element (gh_car (s));
-      if (!e)
-       continue;
-      
-      if (!commony)
-       commony = e;
-      else
-       commony= commony->common_refpoint  (e, Y_AXIS);
-      
-      if (unsmob_element (e->get_grob_property ("rest")))
+      Grob * e = unsmob_grob (ly_car (s));
+      if (unsmob_grob (e->get_grob_property ("rest")))
        rests.push (e);
       else
        notes.push (e);
@@ -115,21 +108,21 @@ Rest_collision::do_shift (Grob *me, SCM elts)
    */
 
   // no rests to collide
-  if (!rests.size())
+  if (!rests.size ())
     return SCM_UNSPECIFIED;
 
   // no partners to collide with
-  if (rests.size() + notes.size () < 2)
+  if (rests.size () + notes.size () < 2)
     return SCM_UNSPECIFIED;
 
   // meisjes met meisjes
-  if (!notes.size()) 
+  if (!notes.size ()) 
     {
-      SCM characteristic = head_characteristic  (rests[0]);
+      SCM characteristic = head_characteristic (rests[0]);
       int i = 1;
       for (; i < rests.size (); i++)
        {
-         if (!gh_equal_p (head_characteristic  (rests[i]), characteristic))
+         if (!gh_equal_p (head_characteristic (rests[i]), characteristic))
            break;
        }
 
@@ -137,7 +130,7 @@ Rest_collision::do_shift (Grob *me, SCM elts)
        If all durations are the same, we'll check if there are more
        rests than maximum-rest-count.
        Otherwise (different durations), we'll try to display them all
      (urg: all 3 of them, currently).
+ (urg: all 3 of them, currently).
        */
       int display_count;
       SCM s = me->get_grob_property ("maximum-rest-count");
@@ -147,9 +140,14 @@ Rest_collision::do_shift (Grob *me, SCM elts)
          display_count = gh_scm2int (s);
          for (; i > display_count; i--)
            {
-             Grob* r = unsmob_element (rests[i-1]->get_grob_property ("rest"));
+             Grob* r = unsmob_grob (rests[i-1]->get_grob_property ("rest"));
              if (r)
-               r->suicide ();
+               {
+                 Grob * d = unsmob_grob (r->get_grob_property ("dot"));
+                 if (d)
+                   d->suicide();
+                 r->suicide ();
+               }
              rests[i-1]->suicide ();
            }
        }
@@ -187,37 +185,33 @@ Rest_collision::do_shift (Grob *me, SCM elts)
     {
       if (rests.size () > 1)
        {
-         warning (_("too many colliding rests"));
-       }
-      if (notes.size () > 1)
-       {
-         warning (_("too many notes for rest collision"));
+         warning (_ ("too many colliding rests"));
        }
       Grob * rcol = rests[0];
+      Direction dir = Note_column::dir (rests[0]);
 
-      // try to be opposite of noteheads. 
-      Direction dir = - Note_column::dir (notes[0]);
-
-      Grob * r = unsmob_element (rcol->get_grob_property ("rest"));
+      if (!dir)
+       {
+         dir = - Note_column::dir (notes[0]);
+       }
+      Grob * r = unsmob_grob (rcol->get_grob_property ("rest"));
       Interval restdim = r->extent (r, Y_AXIS);        // ??
 
-      if (restdim.empty_b ())
+      if (restdim.is_empty ())
        return SCM_UNSPECIFIED;
       
-      // FIXME: staff ref'd?
-      Real staff_space = 1.0;
 
-      Real minimum_dist = gh_scm2double (me->get_grob_property ("minimum-distance")) * staff_space;
-      
-      /*
-       assumption: ref points are the same. 
-       */
+      Real staff_space = Staff_symbol_referencer::staff_space (rcol);
+
+      Real minimum_dist = robust_scm2double (me->get_grob_property ("minimum-distance"), 1.0) * staff_space;
+
+
+      Grob *common = common_refpoint_of_array (notes, rcol, Y_AXIS);
+
       Interval notedim;
-      for (int i = 0; i < notes.size(); i++) 
+      for (int i = 0; i < notes.size (); i++) 
        {
-         Grob * stem = Note_column::stem_l (notes[i]);
-         Grob * head = Stem::first_head (stem);
-         notedim.unite (head->extent (commony, Y_AXIS));
+         notedim.unite (notes[i]->extent (common, Y_AXIS));
        }
 
       Interval inter (notedim);
@@ -226,12 +220,12 @@ Rest_collision::do_shift (Grob *me, SCM elts)
       Real dist =
        minimum_dist +  dir * (notedim[dir] - restdim[-dir]) >? 0;
 
-
-      // FIXME
-      //int stafflines = 5; // rcol->rests[0]->line_count;
       int stafflines = Staff_symbol_referencer::line_count (me);
-      // hurg?
-      stafflines = stafflines != 0 ? stafflines : 5;
+      if (!stafflines)
+       {
+         programming_error ("No staff line count ? ");
+         stafflines =5;
+       }
       
       // move discretely by half spaces.
       int discrete_dist = int (ceil (dist / (0.5 *staff_space)));
@@ -245,10 +239,9 @@ Rest_collision::do_shift (Grob *me, SCM elts)
   return SCM_UNSPECIFIED;
 }
 
-void
-Rest_collision::set_interface (Grob*me)
-{
-  me->set_extent_callback (SCM_EOL, X_AXIS);
-  me->set_extent_callback (SCM_EOL, Y_AXIS);
-}
+
+ADD_INTERFACE (Rest_collision,"rest-collision-interface",
+  "Move around ordinary rests (not multi-measure-rests) to avoid "
+"conflicts.",
+  "maximum-rest-count minimum-distance positioning-done elements");