X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fspan-bar.cc;h=2d76103c381829c1e82c9fc2b367bb9c223dd80d;hb=32b02b2642bd132a548afc163c28667aff074829;hp=c90a2837263658ef493f8ed00cc2ad7029a57b6a;hpb=b1323f33e9aa4b9eea05eefb8755c907d4d762d4;p=lilypond.git diff --git a/lily/span-bar.cc b/lily/span-bar.cc index c90a283726..2d76103c38 100644 --- a/lily/span-bar.cc +++ b/lily/span-bar.cc @@ -3,59 +3,58 @@ source file of the GNU LilyPond music typesetter - (c) 1997--2004 Han-Wen Nienhuys + (c) 1997--2005 Han-Wen Nienhuys */ #include "span-bar.hh" + #include "font-interface.hh" #include "dimensions.hh" -#include "paper-def.hh" +#include "output-def.hh" #include "stencil.hh" #include "warn.hh" #include "axis-group-interface.hh" -#include "group-interface.hh" -#include "grob.hh" #include "bar-line.hh" void -Span_bar::add_bar (Grob*me, Grob*b) +Span_bar::add_bar (Grob *me, Grob *b) { - Pointer_group_interface::add_grob (me, ly_symbol2scm ("elements"), b); + Pointer_group_interface::add_grob (me, ly_symbol2scm ("elements"), b); me->add_dependency (b); } -MAKE_SCHEME_CALLBACK (Span_bar,print,1); +MAKE_SCHEME_CALLBACK (Span_bar, print, 1); /* Limitations/Bugs: - (1) Elements from 'me->get_property ("elements")' must be - ordered according to their y coordinates relative to their common - axis group parent. Otherwise, the computation goes mad. +(1) Elements from 'me->get_property ("elements")' must be +ordered according to their y coordinates relative to their common +axis group parent. Otherwise, the computation goes mad. - (TODO: - apply a sort algorithm that ensures this precondition.) However, - until now, I have seen no case where lily has not fulfilled this - precondition. +(TODO: +apply a sort algorithm that ensures this precondition.) However, +until now, I have seen no case where lily has not fulfilled this +precondition. - (2) This method depends on bar_engraver not being removed from - staff context. If bar_engraver is removed, the size of the staff - lines is evaluated as 0, which results in a solid span bar line - with faulty y coordinate. */ +(2) This method depends on bar_engraver not being removed from +staff context. If bar_engraver is removed, the size of the staff +lines is evaluated as 0, which results in a solid span bar line +with faulty y coordinate. */ /* This routine was originally by Juergen Reuter, but it was a on the bulky side. Rewritten by Han-Wen. */ SCM -Span_bar::print (SCM smobbed_me) +Span_bar::print (SCM smobbed_me) { Grob *me = unsmob_grob (smobbed_me); SCM first_elt = me->get_property ("elements"); /* compute common refpoint of elements */ Grob *refp = me; - for (SCM elts = first_elt; gh_pair_p (elts); elts = ly_cdr (elts)) + for (SCM elts = first_elt; scm_is_pair (elts); elts = scm_cdr (elts)) { - SCM smobbed_staff_bar = ly_car (elts); + SCM smobbed_staff_bar = scm_car (elts); Grob *staff_bar = unsmob_grob (smobbed_staff_bar); refp = staff_bar->common_refpoint (refp, Y_AXIS); } @@ -64,28 +63,28 @@ Span_bar::print (SCM smobbed_me) SCM glyph = me->get_property ("glyph"); /* glyph may not be a string, when ME is killed by Hara Kiri in - between. */ - if (!gh_string_p (glyph)) + between. */ + if (!scm_is_string (glyph)) return SCM_EOL; - + String glyph_string = ly_scm2string (glyph); /* compose span_bar_mol */ Stencil span_bar_mol; Interval prev_extent; - for (SCM elts = first_elt; gh_pair_p (elts); elts = ly_cdr (elts)) + for (SCM elts = first_elt; scm_is_pair (elts); elts = scm_cdr (elts)) { - SCM smobbed_staff_bar = ly_car (elts); + SCM smobbed_staff_bar = scm_car (elts); Grob *staff_bar = unsmob_grob (smobbed_staff_bar); Interval ext = staff_bar->extent (refp, Y_AXIS); if (ext.is_empty ()) - continue; - + continue; + if (!prev_extent.is_empty ()) { Interval l (prev_extent [UP], - ext[DOWN]); + ext[DOWN]); if (l.is_empty ()) { @@ -94,8 +93,8 @@ Span_bar::print (SCM smobbed_me) else { Stencil interbar = Bar_line::compound_barline (staff_bar, - glyph_string, - l.length ()); + glyph_string, + l.length ()); interbar.translate_axis (l.center (), Y_AXIS); span_bar_mol.add_stencil (interbar); } @@ -105,16 +104,16 @@ Span_bar::print (SCM smobbed_me) span_bar_mol.translate_axis (- me->relative_coordinate (refp, Y_AXIS), Y_AXIS); - + return span_bar_mol.smobbed_copy (); } -MAKE_SCHEME_CALLBACK (Span_bar,width_callback,2); +MAKE_SCHEME_CALLBACK (Span_bar, width_callback, 2); SCM Span_bar::width_callback (SCM element_smob, SCM scm_axis) { Grob *se = unsmob_grob (element_smob); - Axis a = (Axis) gh_scm2int (scm_axis); + Axis a = (Axis) scm_to_int (scm_axis); assert (a == X_AXIS); String gl = ly_scm2string (se->get_property ("glyph")); @@ -122,16 +121,17 @@ Span_bar::width_callback (SCM element_smob, SCM scm_axis) urg. */ Stencil m = Bar_line::compound_barline (se, gl, 40 PT); - + return ly_interval2scm (m.extent (X_AXIS)); } -MAKE_SCHEME_CALLBACK (Span_bar,before_line_breaking,1); +MAKE_SCHEME_CALLBACK (Span_bar, before_line_breaking, 1); SCM Span_bar::before_line_breaking (SCM smob) { - evaluate_empty (unsmob_grob (smob)); - evaluate_glyph (unsmob_grob (smob)); + Grob *g = unsmob_grob (smob); + evaluate_empty (g); + evaluate_glyph (g); /* No need to call Bar_line::before_line_breaking (), because the info in ELEMENTS already has been procced by @@ -139,71 +139,71 @@ Span_bar::before_line_breaking (SCM smob) return SCM_UNSPECIFIED; } -MAKE_SCHEME_CALLBACK (Span_bar,center_on_spanned_callback,2); +MAKE_SCHEME_CALLBACK (Span_bar, center_on_spanned_callback, 2); SCM Span_bar::center_on_spanned_callback (SCM element_smob, SCM axis) { Grob *me = unsmob_grob (element_smob); - Axis a = (Axis) gh_scm2int (axis); + Axis a = (Axis) scm_to_int (axis); assert (a == Y_AXIS); Interval i (get_spanned_interval (me)); - /* Bar_line::print delivers a barline of y-extent (-h/2,h/2), so - we have to translate ourselves to be in the center of the + /* Bar_line::print delivers a barline of y-extent (-h/2, h/2), so + we have to translate ourselves to be in the center of the interval that we span. */ if (i.is_empty ()) { me->suicide (); - return gh_double2scm (0.0); + return scm_make_real (0.0); } - - return gh_double2scm (i.center ()); + + return scm_make_real (i.center ()); } void -Span_bar::evaluate_empty (Grob*me) +Span_bar::evaluate_empty (Grob *me) { /* TODO: filter all hara-kiried out of ELEMENS list, and then optionally do suicide. Call this cleanage function from center_on_spanned_callback () as well. */ - if (!gh_pair_p (me->get_property ("elements"))) + if (!scm_is_pair (me->get_property ("elements"))) { me->suicide (); } } void -Span_bar::evaluate_glyph (Grob*me) +Span_bar::evaluate_glyph (Grob *me) { SCM gl = me->get_property ("glyph"); - if (gh_string_p (gl)) - return ; - + if (scm_is_string (gl)) + return; + for (SCM s = me->get_property ("elements"); - !gh_string_p (gl) && gh_pair_p (s); s = gh_cdr (s)) + !scm_is_string (gl) && scm_is_pair (s); s = scm_cdr (s)) { - gl = unsmob_grob (gh_car (s)) + gl = unsmob_grob (scm_car (s)) ->get_property ("glyph"); } - if (!gh_string_p (gl)) + if (!scm_is_string (gl)) { me->suicide (); return; } - + String type = ly_scm2string (gl); - if (type == "|:") + if (type == "|:") { type = ".|"; } - else if (type== ":|") + else if (type == ":|") { type = "|."; } - else if (type== ":|:") + else if (type == ":|:") { type = ".|."; } @@ -215,33 +215,29 @@ Span_bar::evaluate_glyph (Grob*me) } Interval -Span_bar::get_spanned_interval (Grob*me) +Span_bar::get_spanned_interval (Grob *me) { return ly_scm2interval (Axis_group_interface::group_extent_callback - (me->self_scm (), gh_int2scm (Y_AXIS))); + (me->self_scm (), scm_int2num (Y_AXIS))); } - -MAKE_SCHEME_CALLBACK (Span_bar,get_bar_size,1); +MAKE_SCHEME_CALLBACK (Span_bar, get_bar_size, 1); SCM Span_bar::get_bar_size (SCM smob) { - Grob* me = unsmob_grob (smob); + Grob *me = unsmob_grob (smob); Interval iv (get_spanned_interval (me)); if (iv.is_empty ()) { /* This happens if the bars are hara-kiried from under us. */ me->suicide (); - return gh_double2scm (-1); + return scm_make_real (-1); } - return gh_double2scm (iv.length ()); + return scm_make_real (iv.length ()); } - - -ADD_INTERFACE (Span_bar,"span-bar-interface", - "A bar line that spanned between other barlines. This interface is " +ADD_INTERFACE (Span_bar, "span-bar-interface", + "A bar line that spanned between other barlines. This interface is " " used for bar lines that connect different staves.", - "elements"); - + "elements");