X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fstaff-spacing.cc;h=b175987674282d68ac0b9ffb7826978c2de3a39f;hb=5eedf005834d8d3fdc64aee2554a9936c3ef1f57;hp=866c33dbb17eb662d1c0becf476240c8aab58492;hpb=163225c0cbb1055dfd3614615350ab9f3aaba74c;p=lilypond.git diff --git a/lily/staff-spacing.cc b/lily/staff-spacing.cc index 866c33dbb1..b175987674 100644 --- a/lily/staff-spacing.cc +++ b/lily/staff-spacing.cc @@ -1,17 +1,16 @@ -/* - staff-spacing.cc -- implement Staff_spacing - - source file of the GNU LilyPond music typesetter +/* + staff-spacing.cc -- implement Staff_spacing - (c) 2001--2004 Han-Wen Nienhuys + source file of the GNU LilyPond music typesetter + (c) 2001--2005 Han-Wen Nienhuys */ #include "staff-spacing.hh" #include -#include "paper-column.hh" +#include "paper-column.hh" #include "separation-item.hh" #include "warn.hh" #include "bar-line.hh" @@ -23,23 +22,22 @@ /* Insert some more space for the next note, in case it has a stem in the wrong direction - - */ +*/ Real -Staff_spacing::next_note_correction (Grob * me, - Grob * g, +Staff_spacing::next_note_correction (Grob *me, + Grob *g, Interval bar_size) { if (!g || !Note_column::has_interface (g)) return 0.0; - Item *col =dynamic_cast (g)->get_column (); + Item *col = dynamic_cast (g)->get_column (); Real max_corr = 0. >? (- g->extent (col, X_AXIS)[LEFT]); /* Duh. If this gets out of hand, we should invent something more generic. - */ - if (Grob * a = Note_column::accidentals (g)) + */ + if (Grob *a = Note_column::accidentals (g)) { Interval v; if (Accidental_placement::has_interface (a)) @@ -48,21 +46,21 @@ Staff_spacing::next_note_correction (Grob * me, } else v = a->extent (col, X_AXIS); - + max_corr = max_corr >? (- v[LEFT]); } - if (Grob* a = unsmob_grob (g->get_property ("arpeggio"))) + if (Grob *a = unsmob_grob (g->get_property ("arpeggio"))) { max_corr = max_corr >? (- a->extent (col, X_AXIS)[LEFT]); } - + /* Let's decrease the space a little if the problem is not located after a barline. */ if (bar_size.is_empty ()) max_corr *= 0.75; - + if (!bar_size.is_empty ()) if (Grob *stem = Note_column::get_stem (g)) { @@ -70,15 +68,15 @@ Staff_spacing::next_note_correction (Grob * me, if (d == DOWN) { Real stem_start = Stem::head_positions (stem) [DOWN]; - Real stem_end = Stem::stem_end_position (stem); + Real stem_end = Stem::stem_end_position (stem); Interval stem_posns (stem_start ? stem_start); stem_posns.intersect (bar_size); - Real corr = abs (stem_posns.length ()/7.) get_property ("stem-spacing-correction"), 1); + Real corr = abs (stem_posns.length () / 7.) get_property ("stem-spacing-correction"), 1); if (d != DOWN) corr = 0.0; @@ -88,7 +86,6 @@ Staff_spacing::next_note_correction (Grob * me, return max_corr; } - /* Y-positions that are covered by BAR_GROB, in the case that it is a barline. */ @@ -100,12 +97,12 @@ Staff_spacing::bar_y_positions (Grob *bar_grob) if (Bar_line::has_interface (bar_grob)) { SCM glyph = bar_grob->get_property ("glyph"); - + String glyph_string = scm_is_string (glyph) ? ly_scm2string (glyph) : ""; if (glyph_string.left_string (1) == "|" || glyph_string.left_string (1) == ".") { SCM sz = Bar_line::get_staff_bar_size (bar_grob->self_scm ()); - bar_size = Interval (-1,1); + bar_size = Interval (-1, 1); bar_size *= robust_scm2double (sz, 1) / Staff_symbol_referencer::staff_space (bar_grob); } @@ -119,44 +116,42 @@ Staff_spacing::bar_y_positions (Grob *bar_grob) This is slightly convoluted, since the staffspacing grob gets pointers to the separation-items, not the note-columns or note-spacings. - - */ +*/ Real -Staff_spacing::next_notes_correction (Grob *me, Grob * last_grob) +Staff_spacing::next_notes_correction (Grob *me, Grob *last_grob) { Interval bar_size = bar_y_positions (last_grob); Real max_corr = 0.0; for (SCM s = me->get_property ("right-items"); - scm_is_pair (s); s = scm_cdr (s)) + scm_is_pair (s); s = scm_cdr (s)) { - Grob * g = unsmob_grob (scm_car (s)); + Grob *g = unsmob_grob (scm_car (s)); - max_corr = max_corr >? next_note_correction (me, g, bar_size); + max_corr = max_corr >? next_note_correction (me, g, bar_size); for (SCM t = g->get_property ("elements"); - scm_is_pair (t); t = scm_cdr (t)) + scm_is_pair (t); t = scm_cdr (t)) max_corr = max_corr >? next_note_correction (me, unsmob_grob (scm_car (t)), bar_size); - } - + return max_corr; } void -Staff_spacing::get_spacing_params (Grob *me, Real * space, Real * fixed) +Staff_spacing::get_spacing_params (Grob *me, Real *space, Real *fixed) { *space = 1.0; *fixed = 1.0; - Grob * separation_item= 0; - Item * me_item = dynamic_cast (me); - + Grob *separation_item = 0; + Item *me_item = dynamic_cast (me); + for (SCM s = me->get_property ("left-items"); scm_is_pair (s); s = scm_cdr (s)) { - Grob * cand = unsmob_grob (scm_car (s)); + Grob *cand = unsmob_grob (scm_car (s)); if (cand && Separation_item::has_interface (cand)) - separation_item = cand ; + separation_item = cand; } // printf ("doing col %d\n" , Paper_column::get_rank (left_col)); @@ -169,30 +164,30 @@ Staff_spacing::get_spacing_params (Grob *me, Real * space, Real * fixed) Interval last_ext; Grob *last_grob = Separation_item::extremal_break_aligned_grob (separation_item, RIGHT, - &last_ext); + &last_ext); if (!last_grob) { /* TODO: - + Should insert a adjustable space here? For excercises, you might want to - use a staff without a clef in the beginning. - */ - + use a staff without a clef in the beginning. + */ + /* - we used to have a warning here, but itgenerates a lot of + we used to have a warning here, but it generates a lot of spurious error messages. */ - return ; + return; } *fixed = last_ext[RIGHT]; *space = *fixed + 1.0; - + SCM alist = last_grob->get_property ("space-alist"); if (!scm_list_p (alist)) - return ; - + return; + SCM space_def = scm_sloppy_assq (ly_symbol2scm ("first-note"), alist); if (me_item->break_status_dir () == CENTER) { @@ -200,32 +195,31 @@ Staff_spacing::get_spacing_params (Grob *me, Real * space, Real * fixed) if (scm_is_pair (nndef)) space_def = nndef; } - - + if (!scm_is_pair (space_def)) { - programming_error ("Unknown prefatory spacing. "); - return; + programming_error ("unknown prefatory spacing"); + return; } space_def = scm_cdr (space_def); Real distance = scm_to_double (scm_cdr (space_def)); - SCM type = scm_car (space_def) ; + SCM type = scm_car (space_def); *fixed = last_ext[RIGHT]; if (type == ly_symbol2scm ("fixed-space")) { - *fixed += distance; - *space = *fixed; + *fixed += distance; + *space = *fixed; } - else if (type == ly_symbol2scm ("extra-space")) + else if (type == ly_symbol2scm ("extra-space")) { *space = *fixed + distance; } else if (type == ly_symbol2scm ("semi-fixed-space")) { - *fixed += distance / 2; - *space = *fixed + distance/2; + *fixed += distance / 2; + *space = *fixed + distance / 2; } else if (type == ly_symbol2scm ("minimum-space")) { @@ -236,13 +230,12 @@ Staff_spacing::get_spacing_params (Grob *me, Real * space, Real * fixed) *space = last_ext[LEFT] + (last_ext.length () >? distance); *fixed = *space; } - + *space += next_notes_correction (me, last_grob); } - -ADD_INTERFACE (Staff_spacing,"staff-spacing-interface", +ADD_INTERFACE (Staff_spacing, "staff-spacing-interface", "This object calculates spacing details from a " " breakable symbol (left) to another object. For example, it takes care " " of optical spacing from a bar lines to a note.", - "stem-spacing-correction left-items right-items"); + "stem-spacing-correction left-items right-items");