X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fstem-engraver.cc;h=f8d19adcbe7a17e00de6223c90dbef15cb94c383;hb=7e72a1e50e94a7f9738d62599de79fe7745f600c;hp=aba73c027fe77d0a6ef2717649863427b646362a;hpb=8dbb0048c1ac59125562d5adc1c68512eab8541b;p=lilypond.git diff --git a/lily/stem-engraver.cc b/lily/stem-engraver.cc index aba73c027f..f8d19adcbe 100644 --- a/lily/stem-engraver.cc +++ b/lily/stem-engraver.cc @@ -3,13 +3,13 @@ source file of the GNU LilyPond music typesetter - (c) 1997--2001 Han-Wen Nienhuys + (c) 1997--2004 Han-Wen Nienhuys */ #include "staff-symbol-referencer.hh" #include "rhythmic-head.hh" #include "stem.hh" -#include "musical-request.hh" +#include "event.hh" #include "misc.hh" #include "stem-tremolo.hh" #include "item.hh" @@ -30,157 +30,156 @@ protected: virtual bool try_music (Music*); private: - Grob *stem_p_; - Grob *tremolo_p_; - Rhythmic_req *rhythmic_req_l_; - Tremolo_req* tremolo_req_l_; + Grob *stem_; + Grob *tremolo_; + Music *rhythmic_ev_; + Music* tremolo_ev_; }; Stem_engraver::Stem_engraver () { - tremolo_req_l_ = 0; - stem_p_ = 0; - tremolo_p_ = 0; - rhythmic_req_l_ =0; + tremolo_ev_ = 0; + stem_ = 0; + tremolo_ = 0; + rhythmic_ev_ =0; } void Stem_engraver::acknowledge_grob (Grob_info i) { - Grob* h = i.grob_l_; + Grob* h = i.grob_; if (Rhythmic_head::has_interface (h)) { - if (Rhythmic_head::stem_l (h)) + if (Rhythmic_head::get_stem (h)) return; - /* - We take the duration-log of the head; this is because - auto-tieing does strange things with the rhythmics. - */ - int duration_log =gh_scm2int (h->get_grob_property ("duration-log")); - if (!stem_p_) + /* Reverted to the old method so chord tremolos work again. /MB + */ + int duration_log = 0; + + Music * m = i.music_cause (); + if (m->is_mus_type ("rhythmic-event")) + duration_log = unsmob_duration (m->get_mus_property ("duration"))-> duration_log (); + + if (!stem_) { - stem_p_ = new Item (get_property ("Stem")); - Stem::set_interface (stem_p_); - Staff_symbol_referencer::set_interface (stem_p_); - - stem_p_->set_grob_property ("duration-log", gh_int2scm (duration_log)); + stem_ = make_item ("Stem"); + + stem_->set_grob_property ("duration-log", gh_int2scm (duration_log)); - if (tremolo_req_l_) + if (tremolo_ev_) { /* Stem tremolo is never applied to a note by default, - is must me requested. But there is a default for the + is must me evuested. But there is a default for the tremolo value: c4:8 c c: the first and last (quarter) note bothe get one tremolo flag. */ - int requested_type = gh_scm2int (tremolo_req_l_->get_mus_property ("tremolo-type")); - + int requested_type = gh_scm2int (tremolo_ev_->get_mus_property ("tremolo-type")); SCM f = get_property ("tremoloFlags"); - if (!requested_type && gh_number_p (f)) - requested_type = gh_scm2int (f); + if (!requested_type) + if (gh_number_p (f)) + requested_type = gh_scm2int (f); + else + requested_type = 8; else - daddy_trans_l_->set_property ("tremoloFlags", gh_int2scm (requested_type)); + daddy_trans_->set_property ("tremoloFlags", gh_int2scm (requested_type)); + + int tremolo_flags = intlog2 (requested_type) - 2 + - (duration_log > 2 ? duration_log - 2 : 0); + if (tremolo_flags <= 0) + { + tremolo_ev_->origin()->warning (_("tremolo duration is too long")); + tremolo_flags = 0; + } - if (requested_type) + if (tremolo_flags) { - tremolo_p_ = new Item (get_property ("StemTremolo")); - Stem_tremolo::set_interface (tremolo_p_); + tremolo_ = make_item ("StemTremolo"); + announce_grob(tremolo_, tremolo_ev_->self_scm()); - announce_grob (tremolo_p_, tremolo_req_l_); /* The number of tremolo flags is the number of flags of the tremolo-type minus the number of flags of the note itself. */ - int tremolo_flags = intlog2 (requested_type) - 2 - - (duration_log > 2 ? duration_log - 2 : 0); - if (tremolo_flags < 0) - tremolo_flags = 0; - tremolo_p_->set_grob_property ("tremolo-flags", - gh_int2scm (tremolo_flags)); + tremolo_->set_grob_property ("flag-count", + gh_int2scm (tremolo_flags)); + tremolo_->set_parent (stem_, X_AXIS); + stem_->set_grob_property ("tremolo-flag", tremolo_->self_scm ()); } } /* We announce the cause of the head as cause of the stem. The stem needs a rhythmic structure to fit it into a beam. */ - announce_grob (stem_p_, i.music_cause ()); + announce_grob(stem_, i.music_cause ()->self_scm()); } - if (Stem::flag_i (stem_p_) != duration_log) + if (Stem::duration_log (stem_) != duration_log) { - i.music_cause ()->origin ()->warning (_f ("Adding note head to incompatible stem (type = %d)", 1 << Stem::flag_i (stem_p_))); + i.music_cause ()->origin ()->warning (_f ("Adding note head to incompatible stem (type = %d)", 1 << Stem::duration_log (stem_)) + + _f ("Don't you want polyphonic voices instead?") + ); } - Stem::add_head (stem_p_,h); + Stem::add_head (stem_,h); } } void Stem_engraver::stop_translation_timestep () { - if (tremolo_p_) + if (tremolo_) { - Stem_tremolo::set_stem (tremolo_p_, stem_p_); - typeset_grob (tremolo_p_); - tremolo_p_ = 0; + Stem_tremolo::set_stem (tremolo_, stem_); + typeset_grob (tremolo_); + tremolo_ = 0; } - if (stem_p_) + if (stem_) { SCM prop = get_property ("stemLeftBeamCount"); if (gh_number_p (prop)) { - Stem::set_beaming (stem_p_,gh_scm2int (prop),LEFT); - daddy_trans_l_->set_property ("stemLeftBeamCount", SCM_UNDEFINED); + Stem::set_beaming (stem_,gh_scm2int (prop),LEFT); + daddy_trans_->unset_property (ly_symbol2scm ("stemLeftBeamCount")); } prop = get_property ("stemRightBeamCount"); if (gh_number_p (prop)) { - Stem::set_beaming (stem_p_,gh_scm2int (prop), RIGHT); - daddy_trans_l_->set_property ("stemRightBeamCount", SCM_UNDEFINED); - } - - - // UGH. Should mark non-forced instead. - /* - aargh: I don't get it. direction is being set (and then set - to forced), if we have a Chord_tremolo. - */ - SCM dir = stem_p_->get_grob_property ("direction"); - if (gh_number_p (dir) && to_dir (dir)) - { - stem_p_->set_grob_property ("dir-forced", SCM_BOOL_T); + Stem::set_beaming (stem_,gh_scm2int (prop), RIGHT); + daddy_trans_->unset_property (ly_symbol2scm ("stemRightBeamCount")); } - typeset_grob (stem_p_); - stem_p_ = 0; + typeset_grob (stem_); + stem_ = 0; } - tremolo_req_l_ = 0; + tremolo_ev_ = 0; } bool Stem_engraver::try_music (Music* r) { - if (Tremolo_req* a = dynamic_cast (r)) + if (r->is_mus_type ("tremolo-event")) { - tremolo_req_l_ = a; + tremolo_ev_ = r; return true; } return false; } ENTER_DESCRIPTION(Stem_engraver, -/* descr */ "Create stems and single-stem tremolos. It also works together with -the beam engraver for overriding beaming.", +/* descr */ "Create stems and single-stem tremolos. It also works together with " +"the beam engraver for overriding beaming.", /* creats*/ "Stem StemTremolo", -/* acks */ "rhythmic-head-interface", +/* accepts */ "tremolo-event", +/* acks */ "rhythmic-head-interface", /* reads */ "tremoloFlags stemLeftBeamCount stemRightBeamCount", /* write */ "");