X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fspaceable-grob.cc;h=725c4759c65c10464b42d36754aa4eef0a6d5163;hb=c48b83513ad53097baaf937ad278db32b3d362d8;hp=77793e5492fddb322b6e17c21c04c58ff4aedaae;hpb=2050a4a2cbf83bd9f75c91f17b03d1693e52e097;p=lilypond.git diff --git a/lily/spaceable-grob.cc b/lily/spaceable-grob.cc index 77793e5492..725c4759c6 100644 --- a/lily/spaceable-grob.cc +++ b/lily/spaceable-grob.cc @@ -56,51 +56,16 @@ Spaceable_grob::add_rod (Grob *me, Grob *p, Real d) } void -Spaceable_grob::add_spring (Grob *me, Grob *other, - Real distance, Real inverse_strength) +Spaceable_grob::add_spring (Grob *me, Grob *other, Spring sp) { - if (distance < 0.0 || inverse_strength < 0.0) - { - programming_error ("adding reverse spring, setting to unit"); - distance = 1.0; - inverse_strength = 1.0; - } - - if (isinf (distance) || isnan (distance) - || isnan (inverse_strength)) - { - /* strength == INF is possible. It means fixed distance. */ - programming_error ("insane distance found"); - distance = 1.0; - inverse_strength = 1.0; - } - -#ifndef NDEBUG - SCM mins = me->get_object ("ideal-distances"); - for (SCM s = mins; scm_is_pair (s); s = scm_cdr (s)) - { - Spring *sp = unsmob_spring (scm_car (s)); - if (sp->other_ == other) - { - programming_error ("already have that spring"); - return; - } - } -#endif - - Spring spring; - spring.inverse_stretch_strength_ = inverse_strength; - spring.inverse_compress_strength_ = inverse_strength; - spring.distance_ = distance; - spring.other_ = other; - SCM ideal = me->get_object ("ideal-distances"); - ideal = scm_cons (spring.smobbed_copy (), ideal); + + ideal = scm_cons (scm_cons (sp.smobbed_copy (), other->self_scm ()), ideal); me->set_object ("ideal-distances", ideal); } -void -Spaceable_grob::get_spring (Grob *this_col, Grob *next_col, Real *dist, Real *inv_strength) +Spring +Spaceable_grob::get_spring (Grob *this_col, Grob *next_col) { Spring *spring = 0; @@ -108,25 +73,23 @@ Spaceable_grob::get_spring (Grob *this_col, Grob *next_col, Real *dist, Real *in !spring && scm_is_pair (s); s = scm_cdr (s)) { - Spring *sp = unsmob_spring (scm_car (s)); - - if (sp && sp->other_ == next_col) - spring = sp; + if (scm_is_pair (scm_car (s)) + && unsmob_grob (scm_cdar (s)) == next_col + && unsmob_spring (scm_caar (s))) + spring = unsmob_spring (scm_caar (s)); } if (!spring) programming_error (_f ("No spring between column %d and next one", Paper_column::get_rank (this_col))); - *dist = (spring) ? spring->distance_ : 5.0; - *inv_strength = (spring) ? spring->inverse_stretch_strength_ : 1.0; + return spring ? *spring : Spring (); } ADD_INTERFACE (Spaceable_grob, - "A layout object that takes part in the spacing problem. ", - + "A layout object that takes part in the spacing problem.", /* properties */ "allow-loose-spacing " @@ -137,6 +100,5 @@ ADD_INTERFACE (Spaceable_grob, "minimum-distances " "right-neighbors " "spacing-wishes " - );