X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fsimultaneous-music-iterator.cc;h=e892776b61a39e6f4814eb350500ab706a45c4ee;hb=97a0169312a260933246ab224e4f8b0969871dd5;hp=8e9eea1e94339877c238763b57595c33ae744c01;hpb=ff498ed01dc80778ac61254566dc811301741003;p=lilypond.git diff --git a/lily/simultaneous-music-iterator.cc b/lily/simultaneous-music-iterator.cc index 8e9eea1e94..e892776b61 100644 --- a/lily/simultaneous-music-iterator.cc +++ b/lily/simultaneous-music-iterator.cc @@ -1,61 +1,45 @@ /* - Simultaneous_music-iterator.cc -- implement Simultaneous_music_iterator + This file is part of LilyPond, the GNU music typesetter. - source file of the GNU LilyPond music typesetter + Copyright (C) 1997--2015 Han-Wen Nienhuys - (c) 1997--2003 Han-Wen Nienhuys + LilyPond is free software: you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation, either version 3 of the License, or + (at your option) any later version. + + LilyPond is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with LilyPond. If not, see . */ -#include "translator-group.hh" -#include "warn.hh" #include "simultaneous-music-iterator.hh" -#include "music-list.hh" -#include "translator-def.hh" - +#include "music.hh" +#include "context.hh" +#include "warn.hh" +#include "context-def.hh" Simultaneous_music_iterator::Simultaneous_music_iterator () { - separate_contexts_b_ = false; + create_separate_contexts_ = false; children_list_ = SCM_EOL; } -Simultaneous_music_iterator::Simultaneous_music_iterator (Simultaneous_music_iterator const& src) - : Music_iterator (src) -{ - separate_contexts_b_ - = src.separate_contexts_b_; - children_list_ = SCM_EOL; - - SCM children_list = SCM_EOL; - SCM *tail = &children_list; - for (SCM s = src.children_list_; gh_pair_p (s); s = gh_cdr(s)) - { - Music_iterator *i = unsmob_iterator (gh_car (s)); - SCM cl = i->clone ()->self_scm(); - *tail = scm_cons (cl, *tail); - tail = SCM_CDRLOC (*tail); - scm_gc_unprotect_object (cl); - } - - children_list_ = children_list; - scm_remember_upto_here_1 (children_list); -} - void -Simultaneous_music_iterator::derived_mark()const +Simultaneous_music_iterator::derived_mark () const { scm_gc_mark (children_list_); } -SCM -Simultaneous_music_iterator::get_pending_events (Moment m)const +void +Simultaneous_music_iterator::derived_substitute (Context *f, Context *t) { - SCM l = SCM_EOL; - for (SCM s = children_list_; gh_pair_p (s); s = gh_cdr(s)) - { - l = gh_append2 (unsmob_iterator (gh_car (s))->get_pending_events (m), l); - } - return l; + for (SCM s = children_list_; scm_is_pair (s); s = scm_cdr (s)) + unsmob (scm_car (s))->substitute_outlet (f, t); } void @@ -63,85 +47,82 @@ Simultaneous_music_iterator::construct_children () { int j = 0; - SCM i = get_music ()->get_mus_property ("elements"); + SCM i = get_music ()->get_property ("elements"); children_list_ = SCM_EOL; - SCM * tail = &children_list_; - for (; gh_pair_p (i); i = ly_cdr (i), j++) + SCM *tail = &children_list_; + for (; scm_is_pair (i); i = scm_cdr (i), j++) { - Music *mus = unsmob_music (ly_car (i)); + Music *mus = unsmob (scm_car (i)); SCM scm_iter = get_static_get_iterator (mus); - Music_iterator * mi = unsmob_iterator (scm_iter); + Music_iterator *mi = unsmob (scm_iter); - /* if separate_contexts_b_ is set, create a new context with the - number number as name */ + /* if create_separate_contexts_ is set, create a new context with the + number number as name */ - SCM name = unsmob_context_def (report_to ()->definition_)->get_context_name (); - Translator_group * t = (j && separate_contexts_b_) - ? report_to ()->find_create_translator (name, to_string (j), SCM_EOL) - : report_to (); + SCM name = ly_symbol2scm (get_outlet ()->context_name ().c_str ()); + Context *c = (j && create_separate_contexts_) + ? get_outlet ()->find_create_context (name, ::to_string (j), SCM_EOL) + : get_outlet (); - if (!t) - t = report_to (); + if (!c) + c = get_outlet (); - mi->init_translator (mus, t); + mi->init_context (mus, c); mi->construct_children (); - if (mi->ok ()) - { - *tail = scm_cons (scm_iter, *tail); - tail = SCM_CDRLOC (*tail); - } + if (mi->ok ()) + { + *tail = scm_cons (scm_iter, *tail); + tail = SCM_CDRLOC (*tail); + } else - mi->set_translator (0); + mi->quit (); } } +// If we have some iterators with definite next moment and no of them +// remain after processing, we take the iterators with indefinite next +// moment along. That makes sure that no Lyric_combine_music_iterator +// will outstay its welcome (issue 2010). + void Simultaneous_music_iterator::process (Moment until) { - SCM *proc = &children_list_; - while(gh_pair_p (*proc)) + SCM *proc = &children_list_; + bool finite = !pending_moment ().main_part_.is_infinity (); + while (scm_is_pair (*proc)) { - Music_iterator * i = unsmob_iterator (gh_car (*proc)); - if (i->run_always () - || i->pending_moment () == until) - { - i->process (until); - } + Music_iterator *i = unsmob (scm_car (*proc)); + if (i->run_always () || i->pending_moment () == until) + i->process (until); if (!i->ok ()) - { - i->quit (); - *proc = gh_cdr (*proc); - } + { + i->quit (); + *proc = scm_cdr (*proc); + } else - { - proc = SCM_CDRLOC(*proc); - } + { + proc = SCM_CDRLOC (*proc); + } } -} - -void -Simultaneous_music_iterator::skip (Moment until) -{ - SCM *proc = &children_list_; - while(gh_pair_p (*proc)) + // If there were definite-ended iterators and all of them died, take + // the rest of the iterators along with them. They have + // likely lost their reference iterators. Basing this on the actual + // music contexts is not reliable since something like + // \new Voice = blah { + // << \context Voice = blah { c4 d } + // \addlyrics { oh no } + // >> e f + // } + // cannot wait for the death of context blah before ending the + // simultaneous iterator. + if (finite && pending_moment ().main_part_.is_infinity ()) { - Music_iterator * i = unsmob_iterator (gh_car (*proc)); - if (i->pending_moment () <= until) - { - i->skip (until); - } - if (!i->ok ()) - { - i->quit (); - *proc = gh_cdr (*proc); - } - else - { - proc = SCM_CDRLOC(*proc); - } + for (SCM p = children_list_; scm_is_pair (p); p = scm_cdr (p)) + unsmob (scm_car (p))->quit (); + children_list_ = SCM_EOL; } } @@ -150,59 +131,45 @@ Simultaneous_music_iterator::pending_moment () const { Moment next; next.set_infinite (1); - - for (SCM s = children_list_; gh_pair_p (s); s = gh_cdr(s)) + + for (SCM s = children_list_; scm_is_pair (s); s = scm_cdr (s)) { - Music_iterator * it = unsmob_iterator (gh_car (s)); - if (!it-> run_always ()) - next = next pending_moment (); + Music_iterator *it = unsmob (scm_car (s)); + next = min (next, it->pending_moment ()); } - + return next; } bool Simultaneous_music_iterator::ok () const { - bool run_always_ok = false; - for (SCM s = children_list_; gh_pair_p (s); s = gh_cdr(s)) + for (SCM s = children_list_; scm_is_pair (s); s = scm_cdr (s)) { - Music_iterator * it = unsmob_iterator (gh_car (s)); - if (!it->run_always ()) - return true; - else - run_always_ok = run_always_ok || it->ok (); + Music_iterator *it = unsmob (scm_car (s)); + if (it->ok ()) + return true; } - return run_always_ok; + return false; } bool Simultaneous_music_iterator::run_always () const { - for (SCM s = children_list_; gh_pair_p (s); s = gh_cdr(s)) + for (SCM s = children_list_; scm_is_pair (s); s = scm_cdr (s)) { - Music_iterator * it = unsmob_iterator (gh_car (s)); + Music_iterator *it = unsmob (scm_car (s)); if (it->run_always ()) - return true; + return true; } return false; } -Music_iterator* -Simultaneous_music_iterator::try_music_in_children (Music *m) const -{ - Music_iterator * b=0; - for (SCM s = children_list_; !b && gh_pair_p (s); s = gh_cdr(s)) - b =unsmob_iterator (gh_car (s))->try_music (m); - return b; -} - void Simultaneous_music_iterator::do_quit () { - for (SCM s = children_list_; gh_pair_p (s); s = gh_cdr(s)) - unsmob_iterator (gh_car (s))->quit(); + for (SCM s = children_list_; scm_is_pair (s); s = scm_cdr (s)) + unsmob (scm_car (s))->quit (); } - IMPLEMENT_CTOR_CALLBACK (Simultaneous_music_iterator);