]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/dot-column.cc
(precision_string): >? fix.
[lilypond.git] / lily / dot-column.cc
index 46448cd95ff0fb20b959a8d8212d9b5b11274568..2b0e59494c1802f0566c74cb8e12babe31e195fd 100644 (file)
@@ -3,16 +3,17 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 1997--2004 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1997--2005 Han-Wen Nienhuys <hanwen@cs.uu.nl>
 */
 
-#include <stdio.h>
+#include "dot-column.hh"
+
+#include <cstdio>
+#include <math.h>
 #include <map>
 
 #include "dots.hh"
-#include "dot-column.hh"
 #include "rhythmic-head.hh"
-#include "group-interface.hh"
 #include "staff-symbol-referencer.hh"
 #include "directional-element-interface.hh"
 #include "side-position-interface.hh"
 
 /*
   TODO: let Dot_column communicate with stem via Note_column.
- */
+*/
 
-MAKE_SCHEME_CALLBACK (Dot_column,force_shift_callback,2);
+MAKE_SCHEME_CALLBACK (Dot_column, force_shift_callback, 2);
 SCM
 Dot_column::force_shift_callback (SCM element_smob, SCM axis)
 {
   Grob *me = unsmob_grob (element_smob);
-  Axis a = (Axis) gh_scm2int (axis);
-  assert (a == Y_AXIS);
+  (void) axis;
+  assert (scm_to_int (axis) == Y_AXIS);
   me = me->get_parent (X_AXIS);
 
   if (!to_boolean (me->get_property ("positioning-done")))
     {
       me->set_property ("positioning-done", SCM_BOOL_T);
-  
+
       do_shifts (me);
     }
-  return gh_double2scm (0.0);
+  return scm_make_real (0.0);
 }
 
-MAKE_SCHEME_CALLBACK (Dot_column,side_position, 2);
+MAKE_SCHEME_CALLBACK (Dot_column, side_position, 2);
 SCM
 Dot_column::side_position (SCM element_smob, SCM axis)
 {
   Grob *me = unsmob_grob (element_smob);
-  Axis a = (Axis) gh_scm2int (axis);
-  assert (a == X_AXIS);
+  (void) axis;
+  assert (scm_to_int (axis) == X_AXIS);
 
-  Grob * stem = unsmob_grob (me->get_property ("stem"));
+  Grob *stem = unsmob_grob (me->get_property ("stem"));
   if (stem
       && !Stem::get_beam (stem)
       && Stem::duration_log (stem) > 2
-      && !Stem::is_invisible (stem)
-      )
+      && !Stem::is_invisible (stem))
     {
       /*
        trigger stem end & direction calculation.
 
        This will add the stem to the support if a flag collision happens.
-       */
-      Stem::stem_end_position (stem); 
+      */
+      Stem::stem_end_position (stem);
     }
   return Side_position_interface::aligned_side (element_smob, axis);
 }
 
-
-
 struct Dot_position
 {
   int pos_;
@@ -75,24 +73,21 @@ struct Dot_position
   Grob *dot_;
   bool extremal_head_;
 
-  
   Dot_position ()
   {
     dot_ = 0;
-    pos_ =0;
+    pos_ = 0;
     dir_ = CENTER;
   }
 };
 
-
 typedef std::map<int, Dot_position> Dot_configuration;
 
 /*
   Value CFG according.
-
 */
 int
-dot_config_badness (Dot_configuration const  &cfg)
+dot_config_badness (Dot_configuration const &cfg)
 {
   int t = 0;
   for (Dot_configuration::const_iterator i (cfg.begin ());
@@ -105,18 +100,18 @@ dot_config_badness (Dot_configuration const  &cfg)
       if (i->second.extremal_head_)
        {
          if (i->second.dir_
-             &&  dot_move_dir != i->second.dir_)
+             && dot_move_dir != i->second.dir_)
            demerit += 3;
          else if (dot_move_dir != UP)
            demerit += 2;
        }
       else if (dot_move_dir != UP)
        demerit += 1;
-      
+
       t += demerit;
     }
 
-  return t;  
+  return t;
 }
 
 void
@@ -125,7 +120,7 @@ print_dot_configuration (Dot_configuration const &cfg)
   printf ("dotconf { ");
   for (Dot_configuration::const_iterator i (cfg.begin ());
        i != cfg.end (); i++)
-    printf ("%d, " , i->first);
+    printf ("%d, ", i->first);
   printf ("} \n");
 }
 
@@ -133,7 +128,7 @@ print_dot_configuration (Dot_configuration const &cfg)
   Shift K and following (preceding) entries up (down) as necessary to
   prevent staffline collisions if D is up (down).
 
-  If K is in CFG, then do nothing.  
+  If K is in CFG, then do nothing.
 */
 
 Dot_configuration
@@ -142,7 +137,7 @@ shift_one (Dot_configuration const &cfg,
 {
   Dot_configuration new_cfg;
   int offset = 0;
-  
+
   if (d > 0)
     {
       for (Dot_configuration::const_iterator i (cfg.begin ());
@@ -152,19 +147,19 @@ shift_one (Dot_configuration const &cfg,
          if (p == k)
            {
              if (Staff_symbol_referencer::on_staffline (i->second.dot_, p))
-               p += d ;
+               p += d;
              else
                p += 2* d;
 
              offset = 2*d;
 
-             new_cfg[p] = i->second; 
+             new_cfg[p] = i->second;
            }
          else
            {
              if (new_cfg.find (p) == new_cfg.end ())
                {
-                 offset =0;
+                 offset = 0;
                }
              new_cfg[p + offset] = i->second;
            }
@@ -175,25 +170,25 @@ shift_one (Dot_configuration const &cfg,
       Dot_configuration::const_iterator i (cfg.end ());
       do
        {
-         i --;
+         i--;
 
          int p = i->first;
          if (p == k)
            {
              if (Staff_symbol_referencer::on_staffline (i->second.dot_, p))
-               p += d ;
+               p += d;
              else
                p += 2* d;
 
              offset = 2*d;
 
-             new_cfg[p] = i->second; 
+             new_cfg[p] = i->second;
            }
          else
            {
              if (new_cfg.find (p) == new_cfg.end ())
                {
-                 offset =0;
+                 offset = 0;
                }
 
              new_cfg[p + offset] = i->second;
@@ -201,14 +196,14 @@ shift_one (Dot_configuration const &cfg,
        }
       while (i != cfg.begin ());
     }
-  
+
   return new_cfg;
 }
 
 /*
   Remove the collision in CFG either by shifting up or down, whichever
   is best.
- */
+*/
 void
 remove_collision (Dot_configuration &cfg, int p)
 {
@@ -218,64 +213,64 @@ remove_collision (Dot_configuration &cfg, int p)
     {
       Dot_configuration cfg_up = shift_one (cfg, p, UP);
       Dot_configuration cfg_down = shift_one (cfg, p, DOWN);
-      
-      int b_up =  dot_config_badness (cfg_up);
+
+      int b_up = dot_config_badness (cfg_up);
       int b_down = dot_config_badness (cfg_down);
 
-      cfg =  (b_up < b_down) ? cfg_up : cfg_down;
+      cfg = (b_up < b_down) ? cfg_up : cfg_down;
     }
 }
 
 SCM
-Dot_column::do_shifts (Grob*me)
+Dot_column::do_shifts (Grob *me)
 {
-  Link_array<Grob> dots =
-    Pointer_group_interface__extract_grobs (me, (Grob*)0, "dots");
+  Link_array<Grob> dots
+    = extract_grob_array (me, ly_symbol2scm ("dots"));
 
   { /*
       Trigger note collision resolution first, since that may kill off
       dots when merging.
-     */
-    Grob * c = 0;
-    for (int i = dots.size (); i-- ; )
+    */
+    Grob *c = 0;
+    for (int i = dots.size (); i--;)
       {
-       Grob * n = dots[i]->get_parent (Y_AXIS);
+       Grob *n = dots[i]->get_parent (Y_AXIS);
        if (c)
          c = n->common_refpoint (c, X_AXIS);
        else
          c = n;
       }
-    for (int i = dots.size (); i-- ; )
+    for (int i = dots.size (); i--;)
       {
-       Grob * n = dots[i]->get_parent (Y_AXIS);
-       n->relative_coordinate (c ,  X_AXIS);
+       Grob *n = dots[i]->get_parent (Y_AXIS);
+       n->relative_coordinate (c, X_AXIS);
       }
   }
-  
+
   dots.sort (compare_position);
   for (int i = dots.size (); i--;)
-    if (!dots[i]->live ())
+    if (!dots[i]->is_live ())
       dots.del (i);
-  
+
   Dot_configuration cfg;
-  for (int i =0;i < dots.size (); i++)
+  for (int i = 0;i < dots.size (); i++)
     {
       Dot_position dp;
       dp.dot_ = dots[i];
 
-      Grob * note = dots[i]->get_parent (Y_AXIS);
+      Grob *note = dots[i]->get_parent (Y_AXIS);
       if (note)
        {
          Grob *stem = unsmob_grob (note->get_property ("stem"));
          if (stem)
            dp.extremal_head_ = Stem::first_head (stem) == note;
        }
-      
-      int p = int (Staff_symbol_referencer::get_position (dp.dot_));
-      dp.pos_= p;
+
+      int p = Staff_symbol_referencer::get_rounded_position (dp.dot_);
+      dp.pos_ = p;
 
       if (dp.extremal_head_)
-       dp.dir_ = to_dir (dp.dot_->get_property  ("direction"));
+       dp.dir_ = to_dir (dp.dot_->get_property ("direction"));
 
       remove_collision (cfg, p);
       cfg[p] = dp;
@@ -286,32 +281,28 @@ Dot_column::do_shifts (Grob*me)
   for (Dot_configuration::const_iterator i (cfg.begin ());
        i != cfg.end (); i++)
     {
-      Staff_symbol_referencer::set_position (i->second.dot_, i->first);  
+      Staff_symbol_referencer::set_position (i->second.dot_, i->first);
     }
 
   return SCM_UNSPECIFIED;
 }
 
 void
-Dot_column::add_head (Grob * me, Grob *rh)
+Dot_column::add_head (Grob *me, Grob *rh)
 {
-  Grob * d = unsmob_grob (rh->get_property ("dot"));
+  Grob *d = unsmob_grob (rh->get_property ("dot"));
   if (d)
     {
-      Side_position_interface::add_support (me,rh);
+      Side_position_interface::add_support (me, rh);
 
       Pointer_group_interface::add_grob (me, ly_symbol2scm ("dots"), d);
-      d->add_offset_callback (Dot_column::force_shift_callback_proc , Y_AXIS);
+      d->add_offset_callback (Dot_column::force_shift_callback_proc, Y_AXIS);
       Axis_group_interface::add_element (me, d);
     }
 }
 
-
-
-
 ADD_INTERFACE (Dot_column, "dot-column-interface",
               "Groups dot objects so they form a column, and position dots so they do not "
-              "clash with staff lines "
-              ,
+              "clash with staff lines ",
               "positioning-done direction stem");