X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fa2-engraver.cc;h=e50fffae20eb9808aa89a03b51c0d5807e9ded33;hb=33f2e59ea0b2a1ba41e943b5afa29e5fee97ba94;hp=d155be0d151ab95a0740695d72fa66847800d0fb;hpb=6f96331d77b66f223cd9f4f68b857b53a2dd6c03;p=lilypond.git diff --git a/lily/a2-engraver.cc b/lily/a2-engraver.cc index d155be0d15..e50fffae20 100644 --- a/lily/a2-engraver.cc +++ b/lily/a2-engraver.cc @@ -3,7 +3,7 @@ source file of the GNU LilyPond music typesetter - (c) 2000 Jan Nieuwenhuizen + (c) 2000--2003 Jan Nieuwenhuizen */ #include "engraver.hh" @@ -14,36 +14,36 @@ #include "translator-group.hh" #include "side-position-interface.hh" #include "directional-element-interface.hh" - +#include "multi-measure-rest.hh" +#include "tie.hh" class A2_engraver : public Engraver { -public: - A2_engraver (); - VIRTUAL_COPY_CONS (Translator); + TRANSLATOR_DECLARATIONS(A2_engraver); + protected: virtual void acknowledge_grob (Grob_info); - virtual void create_grobs (); + virtual void process_acknowledged_grobs (); virtual void stop_translation_timestep (); private: - Item* text_p_; + Item* text_; enum State { SOLO, SPLIT_INTERVAL, UNIRHYTHM, UNISILENCE, UNISON } state_; }; -ADD_THIS_TRANSLATOR (A2_engraver); + A2_engraver::A2_engraver () { - text_p_ = 0; + text_ = 0; state_ = UNISILENCE; } void -A2_engraver::create_grobs () +A2_engraver::process_acknowledged_grobs () { if (!to_boolean (get_property ("combineParts"))) return ; - if (!text_p_) + if (!text_) { SCM unison = get_property ("unison"); SCM solo = get_property ("solo"); @@ -52,18 +52,18 @@ A2_engraver::create_grobs () if (solo_adue == SCM_BOOL_T && ((solo == SCM_BOOL_T && state_ != SOLO) || (unison == SCM_BOOL_T && state_ != UNISON - && daddy_trans_l_->id_str_.left_str (3) == "one"))) + && daddy_trans_->id_string_.left_string (3) == "one"))) { - text_p_ = new Item (get_property ("TextScript")); - Side_position::set_axis (text_p_, Y_AXIS); - announce_grob (text_p_, 0); + text_ = make_item ("TextScript"); + Side_position_interface::set_axis (text_, Y_AXIS); + announce_grob(text_, SCM_EOL); Direction dir = UP; - SCM text; + SCM text = SCM_EOL; if (solo == SCM_BOOL_T) { state_ = SOLO; - if (daddy_trans_l_->id_str_.left_str (3) == "one") + if (daddy_trans_->id_string_.left_string (3) == "one") { text = get_property ("soloText"); } @@ -76,12 +76,12 @@ A2_engraver::create_grobs () else if (unison == SCM_BOOL_T) { state_ = UNISON; - if (daddy_trans_l_->id_str_.left_str (3) == "one") + if (daddy_trans_->id_string_.left_string (3) == "one") text = get_property ("aDueText"); } - Side_position::set_direction (text_p_, dir); - text_p_->set_grob_property ("text", text); + set_grob_direction (text_, dir); + text_->set_grob_property ("text", text); } } } @@ -92,19 +92,19 @@ A2_engraver::acknowledge_grob (Grob_info i) if (!to_boolean (get_property ("combineParts"))) return ; - if (text_p_) + if (text_) { - if (Note_head::has_interface (i.elem_l_)) + if (Note_head::has_interface (i.grob_)) { - Grob*t = text_p_; - Side_position::add_support (t, i.elem_l_); - if (Side_position::get_axis (t) == X_AXIS - && !t->parent_l (Y_AXIS)) - t->set_parent (i.elem_l_, Y_AXIS); + Grob*t = text_; + Side_position_interface::add_support (t, i.grob_); + if (Side_position_interface::get_axis (t) == X_AXIS + && !t->get_parent (Y_AXIS)) + t->set_parent (i.grob_, Y_AXIS); } - if (Stem::has_interface (i.elem_l_)) + if (Stem::has_interface (i.grob_)) { - Side_position::add_support (text_p_, i.elem_l_); + Side_position_interface::add_support (text_, i.grob_); } } @@ -129,51 +129,87 @@ A2_engraver::acknowledge_grob (Grob_info i) state_ = SPLIT_INTERVAL; else if (unirhythm) state_ = UNIRHYTHM; - - if (Stem::has_interface (i.elem_l_) - || Slur::has_interface (i.elem_l_) - // || Tie::has_interface (i.elem_l_) - || i.elem_l_->has_interface (ly_symbol2scm ("tie-interface")) - /* - Usually, dynamics are removed by *_devnull_engravers for the - second voice. On the one hand, we don't want all dynamics for - the first voice to be placed above the staff. On the other - hand, colliding of scripts may be worse */ -#if 0 - || i.elem_l_->has_interface (ly_symbol2scm ("dynamic-interface")) - || i.elem_l_->has_interface (ly_symbol2scm ("text-interface")) -#endif - ) + + Direction d = CENTER; + if (daddy_trans_->id_string_.left_string (3) == "one") + d = UP; + else if (daddy_trans_->id_string_.left_string (3) == "two") + d = DOWN; + + /* Must only set direction for VoiceCombines, not for StaffCombines: + we can't detect that here, so we use yet another property */ + if (!to_boolean (get_property ("noDirection")) + && (Stem::has_interface (i.grob_) + || Slur::has_interface (i.grob_) + || Tie::has_interface (i.grob_) + /* Usually, dynamics are removed by *_devnull_engravers for + the second voice. On the one hand, we don't want all + dynamics for the first voice to be placed above the + staff. On the other hand, colliding of scripts may be + worse. So, we don't set directions for these when we're + playing solo. */ + || (i.grob_->internal_has_interface (ly_symbol2scm + ("dynamic-interface")) + && state_ != SOLO) + || (i.grob_->internal_has_interface (ly_symbol2scm + ("text-interface")) + && state_ != SOLO) + )) { - /* - Hmm. We must set dir when solo, in order to get - the rests collided to the right position - */ - if ((unirhythm != SCM_BOOL_T) || (solo == SCM_BOOL_T) - || ((unisilence == SCM_BOOL_T && previous_state != UNISON)) - || (unirhythm == SCM_BOOL_T && split_interval == SCM_BOOL_T - && (unison != SCM_BOOL_T || solo_adue != SCM_BOOL_T))) + /* When in solo a due mode, and we have solo, every grob in + other thread gets annihilated, so we don't set dir. + + Maybe that should be optional? */ + if ((solo != SCM_BOOL_T && solo_adue == SCM_BOOL_T) + + /* When not same rhythm, we set dir */ + && (unirhythm != SCM_BOOL_T + /* When both have rests, but previously played something + different, we set dir */ + || ((unisilence == SCM_BOOL_T && previous_state != UNISON)) + /* When same rhythm, and split stems, but not same pitch + or not solo a du mode, we set dir */ + || (unirhythm == SCM_BOOL_T && split_interval == SCM_BOOL_T + && (unison != SCM_BOOL_T || solo_adue != SCM_BOOL_T)))) { - if (daddy_trans_l_->id_str_.left_str (3) == "one") - { - i.elem_l_->set_grob_property ("direction", gh_int2scm (1)); - } - else if (daddy_trans_l_->id_str_.left_str (3) == "two") - { - i.elem_l_->set_grob_property ("direction", gh_int2scm (-1)); - } + + /* Blunt axe method: every grob gets a propertysetting. */ + i.grob_->set_grob_property ("direction", scm_int2num (d)); } } + + /* Should we have separate state variable for being "rest + while other has solo?" */ + if (Multi_measure_rest::has_interface (i.grob_) && d) + if (state_ == UNIRHYTHM + && unisilence != SCM_BOOL_T) + { + i.grob_->set_grob_property ("staff-position", scm_int2num (d * 6)); + } } void A2_engraver::stop_translation_timestep () { - if (text_p_) + if (text_) { - Side_position::add_staff_support (text_p_); - typeset_grob (text_p_); - text_p_ = 0; + typeset_grob (text_); + text_ = 0; } } +ENTER_DESCRIPTION(A2_engraver, +/* descr */ "Part combine engraver for orchestral scores. " +"The markings @emph{a2}, @emph{Solo} and @emph{Solo II}, are " +"created by this engraver. It also acts upon instructions of the part " +"combiner. Another thing that the this engraver, is forcing of stem, " +"slur and tie directions, always when both threads are not identical; " +"up for the musicexpr called @code{one}, down for the musicexpr called " +"@code{two}. " +, +/* creats*/ "TextScript", +/* accepts */ "", +/* acks */ "multi-measure-rest-interface " +"slur-interface stem-interface tie-interface note-head-interface dynamic-interface text-interface" +,/* reads */ "combineParts noDirection soloADue soloText soloIIText aDueText split-interval unison solo unisilence unirhythm", +/* write */ "");