X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fstem.cc;h=4dc00e7bf31e9fcfa682c13fc4ab24ef678e7fa3;hb=a7f82e4dc22fc6219a9fe0f6874f8c91e3f31f24;hp=ad049f1935f1122afd7be9b815694f41947bd926;hpb=38f0c5f988029aa7cf26ff84dc68645d7dbd7bf5;p=lilypond.git diff --git a/lily/stem.cc b/lily/stem.cc index ad049f1935..4dc00e7bf3 100644 --- a/lily/stem.cc +++ b/lily/stem.cc @@ -3,8 +3,8 @@ source file of the GNU LilyPond music typesetter - (c) 1996--2004 Han-Wen Nienhuys - Jan Nieuwenhuizen + (c) 1996--2005 Han-Wen Nienhuys + Jan Nieuwenhuizen TODO: This is way too hairy @@ -13,24 +13,23 @@ Stem-end, chord-start, etc. is all confusing naming. */ +#include "stem.hh" + #include // rint #include "lookup.hh" #include "directional-element-interface.hh" #include "note-head.hh" -#include "stem.hh" #include "warn.hh" #include "output-def.hh" #include "rhythmic-head.hh" #include "font-interface.hh" -#include "stencil.hh" #include "paper-column.hh" #include "misc.hh" #include "beam.hh" #include "rest.hh" -#include "group-interface.hh" +#include "pointer-group-interface.hh" #include "staff-symbol-referencer.hh" -#include "spanner.hh" #include "side-position-interface.hh" #include "dot-column.hh" #include "stem-tremolo.hh" @@ -63,13 +62,12 @@ Stem::get_beaming (Grob *me, Direction d) return scm_ilength (lst); } - Interval Stem::head_positions (Grob *me) { if (head_count (me)) { - Drul_array e (extremal_heads (me)); + Drul_array e (extremal_heads (me)); return Interval (Staff_symbol_referencer::get_position (e[DOWN]), Staff_symbol_referencer::get_position (e[UP])); } @@ -122,8 +120,8 @@ Stem::set_stemend (Grob *me, Real se) // todo: margins 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")); + 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", scm_make_real (se)); } @@ -133,16 +131,17 @@ Stem::set_stemend (Grob *me, Real se) Grob * Stem::support_head (Grob *me) { - if (head_count (me) == 1) - /* UGH. */ - return unsmob_grob (scm_car (me->get_property ("note-heads"))); + extract_grob_set (me, "note-heads", heads); + if (heads.size () == 1) + return heads[0]; + return first_head (me); } int Stem::head_count (Grob *me) { - return Pointer_group_interface::count (me, "note-heads"); + return Pointer_group_interface::count (me, ly_symbol2scm ("note-heads")); } /* The note head which forms one end of the stem. @@ -163,16 +162,15 @@ Stem::last_head (Grob *me) Direction d = get_direction (me); if (d) return extremal_heads (me)[d]; - return 0; + return 0; } /* START is part where stem reaches `last' head. - This function returns a drul with (bottom-head, top-head). - + This function returns a drul with (bottom-head, top-head). */ -Drul_array +Drul_array Stem::extremal_heads (Grob *me) { const int inf = 1000000; @@ -181,10 +179,11 @@ Stem::extremal_heads (Grob *me) extpos[UP] = -inf; Drul_array exthead (0, 0); - for (SCM s = me->get_property ("note-heads"); scm_is_pair (s); - s = scm_cdr (s)) + extract_grob_set (me, "note-heads", heads); + + for (int i = heads.size (); i--;) { - Grob *n = unsmob_grob (scm_car (s)); + Grob *n = heads[i]; int p = Staff_symbol_referencer::get_rounded_position (n); Direction d = LEFT; @@ -195,7 +194,8 @@ Stem::extremal_heads (Grob *me) exthead[d] = n; extpos[d] = p; } - } while (flip (&d) != DOWN); + } + while (flip (&d) != DOWN); } return exthead; } @@ -210,11 +210,12 @@ integer_compare (int const &a, int const &b) Array Stem::note_head_positions (Grob *me) { - Array ps ; - for (SCM s = me->get_property ("note-heads"); scm_is_pair (s); - s = scm_cdr (s)) + Array ps; + extract_grob_set (me, "note-heads", heads); + + for (int i = heads.size (); i--;) { - Grob *n = unsmob_grob (scm_car (s)); + Grob *n = heads[i]; int p = Staff_symbol_referencer::get_rounded_position (n); ps.push (p); @@ -227,7 +228,7 @@ Stem::note_head_positions (Grob *me) void Stem::add_head (Grob *me, Grob *n) { - n->set_property ("stem", me->self_scm ()); + n->set_object ("stem", me->self_scm ()); n->add_dependency (me); if (Note_head::has_interface (n)) @@ -241,7 +242,7 @@ Stem::is_invisible (Grob *me) { Real stemlet_length = robust_scm2double (me->get_property ("stemlet-length"), 0.0); - + return !((head_count (me) || stemlet_length > 0.0) && scm_to_int (me->get_property ("duration-log")) >= 1); @@ -255,8 +256,8 @@ Stem::get_default_dir (Grob *me) if (hp.is_empty ()) return CENTER; - int udistance = (int) (UP * hp[UP] - staff_center); - int ddistance = (int) (DOWN * hp[DOWN] - staff_center); + int udistance = (int) (UP *hp[UP] - staff_center); + int ddistance = (int) (DOWN *hp[DOWN] - staff_center); if (sign (ddistance - udistance)) return Direction (sign (ddistance - udistance)); @@ -299,9 +300,9 @@ Stem::get_default_stem_end_position (Grob *me) if (dir && dir * hp[dir] >= 0) { SCM sshorten = me->get_property ("stem-shorten"); - SCM scm_shorten = scm_is_pair (sshorten) ? - robust_list_ref ((duration_log (me) - 2) >? 0, sshorten): SCM_EOL; - Real shorten = 2* robust_scm2double (scm_shorten,0); + SCM scm_shorten = scm_is_pair (sshorten) + ? robust_list_ref (max (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)[dir]) <= 1) @@ -311,16 +312,16 @@ Stem::get_default_stem_end_position (Grob *me) } /* Tremolo stuff. */ - Grob *t_flag = unsmob_grob (me->get_property ("tremolo-flag")); - if (t_flag && !unsmob_grob (me->get_property ("beam"))) + Grob *t_flag = unsmob_grob (me->get_object ("tremolo-flag")); + if (t_flag && !unsmob_grob (me->get_object ("beam"))) { /* 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 */ + 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 (t_flag).extent (Y_AXIS).length () + + 2 * Stem_tremolo::raw_stencil (t_flag).extent (Y_AXIS).length () / ss; if (durlog >= 3) @@ -334,7 +335,7 @@ Stem::get_default_stem_end_position (Grob *me) if (dir == DOWN) minlen -= 1.0; } - length = length >? (minlen + 1.0); + length = max (length, minlen + 1.0); } Real st = dir ? hp[dir] + dir * length : 0; @@ -350,9 +351,9 @@ Stem::get_default_stem_end_position (Grob *me) if (!get_beam (me) && dir == UP && durlog > 2) { - Grob * closest_to_flag = extremal_heads (me)[dir]; - Grob * dots = closest_to_flag - ? Rhythmic_head::get_dots (closest_to_flag ) : 0; + Grob *closest_to_flag = extremal_heads (me)[dir]; + Grob *dots = closest_to_flag + ? Rhythmic_head::get_dots (closest_to_flag) : 0; if (dots) { @@ -393,9 +394,8 @@ Stem::position_noteheads (Grob *me) if (!head_count (me)) return; - Link_array heads = - Pointer_group_interface__extract_grobs (me, (Grob*) 0, "note-heads"); - + extract_grob_set (me, "note-heads", ro_heads); + Link_array heads (ro_heads); heads.sort (compare_position); Direction dir = get_direction (me); @@ -412,16 +412,16 @@ Stem::position_noteheads (Grob *me) 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); + Real dy = fabs (lastpos- p); /* - dy should always be 0.5, 0.0, 1.0, but provide safety margin - for rounding errors. + dy should always be 0.5, 0.0, 1.0, but provide safety margin + for rounding errors. */ - if (dy < 1.1) + if (dy < 1.1) { if (parity) { @@ -445,18 +445,18 @@ Stem::position_noteheads (Grob *me) heads[i]->translate_axis (-thick * (2 - reverse_overlap) * d, X_AXIS); - /* TODO: - + /* TODO: + For some cases we should kern some more: when the distance between the next or prev note is too large, we'd get large white gaps, eg. - - | + + | X| - |X <- kern this. - | + |X <- kern this. + | X - + */ } parity = !parity; @@ -468,7 +468,7 @@ Stem::position_noteheads (Grob *me) } } -MAKE_SCHEME_CALLBACK (Stem,before_line_breaking,1); +MAKE_SCHEME_CALLBACK (Stem, before_line_breaking, 1); SCM Stem::before_line_breaking (SCM smob) { @@ -477,13 +477,13 @@ Stem::before_line_breaking (SCM smob) /* Do the calculations for visible stems, but also for invisible stems with note heads (i.e. half notes.) - */ + */ if (head_count (me)) { stem_end_position (me); // ugh. Trigger direction calc. position_noteheads (me); } - + return SCM_UNSPECIFIED; } @@ -491,7 +491,7 @@ Stem::before_line_breaking (SCM smob) ugh. When in a beam with tuplet brackets, brew_mol is called early, caching a wrong value. - */ +*/ MAKE_SCHEME_CALLBACK (Stem, height, 2); SCM Stem::height (SCM smob, SCM ax) @@ -503,27 +503,30 @@ Stem::height (SCM smob, SCM ax) /* ugh. - this dependency should be automatic. */ - Grob *beam= get_beam (me); + 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) iv = unsmob_stencil (mol)->extent (a); - if (Grob *b =get_beam (me)) + if (Grob *b = get_beam (me)) { Direction d = get_direction (me); - iv[d] += d * Beam::get_thickness (b) * 0.5 ; + if (d == CENTER) + { + programming_error ("no stem direction"); + d = UP; + } + iv[d] += d * Beam::get_thickness (b) * 0.5; } return ly_interval2scm (iv); } - Stencil Stem::flag (Grob *me) { @@ -535,7 +538,7 @@ Stem::flag (Grob *me) SCM flag_style_scm = me->get_property ("flag-style"); if (scm_is_symbol (flag_style_scm)) flag_style = ly_symbol2string (flag_style_scm); - + if (flag_style == "no-flag") return Stencil (); @@ -553,9 +556,9 @@ Stem::flag (Grob *me) { if (adjust) { - int p = (int)(rint (stem_end_position (me))); - staffline_offs = - Staff_symbol_referencer::on_staffline (me, p) ? "0" : "1"; + int p = (int) (rint (stem_end_position (me))); + staffline_offs + = Staff_symbol_referencer::on_staffline (me, p) ? "0" : "1"; } else { @@ -571,7 +574,7 @@ Stem::flag (Grob *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); + Stencil flag = fm->find_by_name ("flags." + font_char); if (flag.is_empty ()) me->warning (_f ("flag `%s' not found", font_char)); @@ -582,7 +585,7 @@ Stem::flag (Grob *me) if (!stroke_style.is_empty ()) { String font_char = to_string (dir) + stroke_style; - Stencil stroke = fm->find_by_name ("flags-" + font_char); + Stencil stroke = fm->find_by_name ("flags." + font_char); if (stroke.is_empty ()) me->warning (_f ("flag stroke `%s' not found", font_char)); else @@ -593,30 +596,30 @@ Stem::flag (Grob *me) return flag; } -MAKE_SCHEME_CALLBACK (Stem,dim_callback,2); +MAKE_SCHEME_CALLBACK (Stem, width_callback, 2); SCM -Stem::dim_callback (SCM e, SCM ax) +Stem::width_callback (SCM e, SCM ax) { - Axis a = (Axis) scm_to_int (ax); - assert (a == X_AXIS); + (void) ax; + assert (scm_to_int (ax) == X_AXIS); Grob *me = unsmob_grob (e); Interval r; - + if (is_invisible (me)) { r.set_empty (); - } - else if (unsmob_grob (me->get_property ("beam")) || abs (duration_log (me)) <= 2) + } + else if (unsmob_grob (me->get_object ("beam")) || abs (duration_log (me)) <= 2) { - r = Interval (-1,1); - r *= thickness (me)/2; + r = Interval (-1, 1); + r *= thickness (me) / 2; } else { - r = flag (me).extent (X_AXIS) - + thickness (me)/2; - } + r = Interval (-1,1) * thickness (me) * 0.5; + r.unite (flag (me).extent (X_AXIS)); + } return ly_interval2scm (r); } @@ -638,21 +641,21 @@ Stem::print (SCM smob) Real stemlet_length = robust_scm2double (me->get_property ("stemlet-length"), 0.0); bool stemlet = stemlet_length > 0.0; - + /* 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")) + Grob *lh + = to_boolean (me->get_property ("avoid-note-head")) ? last_head (me) - : lh = first_head (me); + : first_head (me); Grob *beam = get_beam (me); if (!lh && !stemlet) return SCM_EOL; - if (stemlet && !beam) + if (!lh && stemlet && !beam) return SCM_EOL; - + if (is_invisible (me)) return SCM_EOL; @@ -660,7 +663,6 @@ Stem::print (SCM smob) Real y1 = y2; Real half_space = Staff_symbol_referencer::staff_space (me) * 0.5; - if (lh) y2 = Staff_symbol_referencer::get_position (lh); else if (stemlet) @@ -671,32 +673,31 @@ Stem::print (SCM smob) y2 -= d * (0.5 * beam_thickness - + beam_translation * (0 >? (beam_count - 1)) + + beam_translation * max (0, (beam_count - 1)) + stemlet_length) / half_space; } - Interval stem_y (y1 ? y1); + Interval stem_y (min (y1, y2), max (y2, y1)); if (Grob *hed = support_head (me)) { /* must not take ledgers into account. - */ - Interval head_height = hed->extent (hed,Y_AXIS); + */ + Interval head_height = hed->extent (hed, Y_AXIS); Real y_attach = Note_head::stem_attachment_coordinate (hed, Y_AXIS); y_attach = head_height.linear_combination (y_attach); - stem_y[Direction (-d)] += d * y_attach/half_space; + stem_y[Direction (-d)] += d * y_attach / half_space; } - // URG Real stem_width = thickness (me); - Real blot = - me->get_layout ()->get_dimension (ly_symbol2scm ("blotdiameter")); + Real blot + = me->get_layout ()->get_dimension (ly_symbol2scm ("blotdiameter")); - Box b = Box (Interval (-stem_width/2, stem_width/2), - Interval (stem_y[DOWN]*half_space, stem_y[UP]*half_space)); + Box b = Box (Interval (-stem_width / 2, stem_width / 2), + Interval (stem_y[DOWN] * half_space, stem_y[UP] * half_space)); Stencil ss = Lookup::round_filled_box (b, blot); mol.add_stencil (ss); @@ -704,8 +705,8 @@ Stem::print (SCM smob) if (!get_beam (me) && abs (duration_log (me)) > 2) { Stencil fl = flag (me); - fl.translate_axis (stem_y[d]*half_space - d * blot/2, Y_AXIS); - fl.translate_axis (stem_width/2, X_AXIS); + fl.translate_axis (stem_y[d] * half_space - d * blot / 2, Y_AXIS); + fl.translate_axis (stem_width / 2, X_AXIS); mol.add_stencil (fl); } @@ -714,41 +715,41 @@ 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, offset_callback, 2); SCM -Stem::off_callback (SCM element_smob, SCM) +Stem::offset_callback (SCM element_smob, SCM) { Grob *me = unsmob_grob (element_smob); Real r = 0.0; - + if (Grob *f = first_head (me)) { Interval head_wid = f->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; + r += -d * rule_thick * 0.5; } } else { - SCM rests = me->get_property ("rests"); - if (scm_is_pair (rests)) + extract_grob_set (me, "rests", rests); + if (rests.size ()) { - Grob * rest = unsmob_grob (scm_car (rests)); + Grob *rest = rests.top (); r = rest->extent (rest, X_AXIS).center (); } } @@ -758,8 +759,8 @@ Stem::off_callback (SCM element_smob, SCM) Spanner * Stem::get_beam (Grob *me) { - SCM b = me->get_property ("beam"); - return dynamic_cast (unsmob_grob (b)); + SCM b = me->get_object ("beam"); + return dynamic_cast (unsmob_grob (b)); } Stem_info @@ -780,7 +781,6 @@ Stem::get_stem_info (Grob *me) return si; } - /* TODO: add extra space for tremolos! */ void Stem::calc_stem_info (Grob *me) @@ -789,8 +789,8 @@ Stem::calc_stem_info (Grob *me) if (!my_dir) { - programming_error ("No stem dir set?"); - my_dir = UP; + programming_error ("no stem dir set"); + my_dir = UP; } Real staff_space = Staff_symbol_referencer::staff_space (me); @@ -799,25 +799,22 @@ Stem::calc_stem_info (Grob *me) Real beam_thickness = Beam::get_thickness (beam); int beam_count = Beam::get_direction_beam_count (beam, my_dir); - /* Simple standard stem length */ SCM lengths = me->get_property ("beamed-lengths"); - Real ideal_length = - scm_to_double (robust_list_ref (beam_count - 1,lengths)) - + Real ideal_length + = scm_to_double (robust_list_ref (beam_count - 1, lengths)) + * staff_space /* stem only extends to center of beam - */ - - 0.5 * beam_thickness - ; + */ + - 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 = - scm_to_double (robust_list_ref (beam_count - 1, lengths)) + Real ideal_minimum_free + = scm_to_double (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): @@ -833,51 +830,49 @@ Stem::calc_stem_info (Grob *me) /* stem only extends to center of beam */ - 0.5 * beam_thickness; - ideal_length = ideal_length >? ideal_minimum_length; + ideal_length = max (ideal_length, ideal_minimum_length); /* Convert to Y position, calculate for dir == UP */ - Real note_start = - /* staff positions */ + Real note_start + = /* staff positions */ head_positions (me)[my_dir] * 0.5 * my_dir * staff_space; Real ideal_y = note_start + ideal_length; - /* Conditions for Y position */ /* Lowest beam of (UP) beam must never be lower than second staffline - Reference? + 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. + 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. - Reference? + Reference? - Obviously not for grace beams. + Obviously not for grace beams. - Also, not for knees. Seems to be a good thing. */ + 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")); if (!no_extend_b && !is_knee) { /* Highest beam of (UP) beam must never be lower than middle staffline */ - ideal_y = ideal_y >? 0; + ideal_y = max (ideal_y, 0.0); /* Lowest beam of (UP) beam must never be lower than second staffline */ - ideal_y = ideal_y >? (-staff_space - - beam_thickness + height_of_my_beams); + ideal_y = max (ideal_y, (-staff_space + - beam_thickness + height_of_my_beams)); } - ideal_y -= robust_scm2double (beam->get_property ("shorten"), 0); - Real minimum_free = - scm_to_double (robust_list_ref - (beam_count - 1, - me->get_property - ("beamed-extreme-minimum-free-lengths"))) + Real minimum_free + = scm_to_double (robust_list_ref + (beam_count - 1, + me->get_property + ("beamed-extreme-minimum-free-lengths"))) * staff_space; Real minimum_length = minimum_free @@ -885,6 +880,13 @@ Stem::calc_stem_info (Grob *me) /* stem only extends to center of beam */ - 0.5 * beam_thickness; + if (Grob *tremolo = unsmob_grob (me->get_object ("tremolo-flag"))) + { + Interval y_ext = tremolo->extent (tremolo, Y_AXIS); + y_ext.widen (0.5); // FIXME. Should be tunable? + minimum_length = max (minimum_length, y_ext.length ()); + } + ideal_y *= my_dir; Real minimum_y = note_start + minimum_length; Real shortest_y = minimum_y * my_dir; @@ -897,14 +899,13 @@ Stem::calc_stem_info (Grob *me) Slice Stem::beam_multiplicity (Grob *stem) { - SCM beaming= stem->get_property ("beaming"); + SCM beaming = stem->get_property ("beaming"); Slice le = int_list_to_slice (scm_car (beaming)); Slice ri = int_list_to_slice (scm_cdr (beaming)); le.unite (ri); return le; } - /* FIXME: Too many properties */ ADD_INTERFACE (Stem, "stem-interface", "The stem represent the graphical stem. "