X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fdot-column.cc;h=2b0e59494c1802f0566c74cb8e12babe31e195fd;hb=da18ce55e6c6755ba018ec35a8ce8d59c9dde1e0;hp=46448cd95ff0fb20b959a8d8212d9b5b11274568;hpb=947454c5cc430a9627262de4f25274a64789d3c2;p=lilypond.git diff --git a/lily/dot-column.cc b/lily/dot-column.cc index 46448cd95f..2b0e59494c 100644 --- a/lily/dot-column.cc +++ b/lily/dot-column.cc @@ -3,16 +3,17 @@ source file of the GNU LilyPond music typesetter - (c) 1997--2004 Han-Wen Nienhuys + (c) 1997--2005 Han-Wen Nienhuys */ -#include +#include "dot-column.hh" + +#include +#include #include #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" @@ -21,53 +22,50 @@ /* 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 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 dots = - Pointer_group_interface__extract_grobs (me, (Grob*)0, "dots"); + Link_array 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");