X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fmusic-iterator.cc;h=009583d8e3dbb9a45c8b0750b6f364c354528ac8;hb=5bbfc22fce036b9b69df5e420de93e11da23c05e;hp=83838427df32aac99a381dd0b2f157bff6b6faaa;hpb=d7180ea0d67465b319d3e27ae15ac9de379ac779;p=lilypond.git diff --git a/lily/music-iterator.cc b/lily/music-iterator.cc index 83838427df..009583d8e3 100644 --- a/lily/music-iterator.cc +++ b/lily/music-iterator.cc @@ -1,18 +1,27 @@ /* - music-iterator.cc -- implement Music_iterator + This file is part of LilyPond, the GNU music typesetter. - source file of the GNU LilyPond music typesetter + Copyright (C) 1997--2014 Han-Wen Nienhuys - (c) 1997--2006 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. -#include "music-iterator.hh" + You should have received a copy of the GNU General Public License + along with LilyPond. If not, see . +*/ #include using namespace std; #include "warn.hh" +#include "music.hh" #include "context.hh" #include "event-iterator.hh" #include "input.hh" @@ -41,7 +50,7 @@ Music_iterator::~Music_iterator () Context * Music_iterator::get_outlet () const { - return handle_.get_outlet (); + return handle_.get_context (); } void @@ -82,16 +91,16 @@ Music_iterator::get_static_get_iterator (Music *m) if (ly_is_procedure (ctor)) { iter = scm_call_0 (ctor); - p = unsmob_iterator (iter); + p = Music_iterator::unsmob (iter); } else { if (dynamic_cast (m)) - p = new Music_wrapper_iterator; + p = new Music_wrapper_iterator; else if (m->is_mus_type ("event")) p = new Event_iterator; else - p = new Simple_music_iterator; + p = new Simple_music_iterator; iter = p->self_scm (); p->unprotect (); @@ -118,7 +127,7 @@ Music_iterator::music_start_mom ()const } void -Music_iterator::init_translator (Music *m, Context *report) +Music_iterator::init_context (Music *m, Context *report) { music_ = m; assert (m); @@ -143,9 +152,9 @@ SCM Music_iterator::get_iterator (Music *m) const { SCM ip = get_static_get_iterator (m); - Music_iterator *p = unsmob_iterator (ip); + Music_iterator *p = Music_iterator::unsmob (ip); - p->init_translator (m, get_outlet ()); + p->init_context (m, get_outlet ()); p->construct_children (); return ip; @@ -160,16 +169,16 @@ Music_iterator::descend_to_bottom_context () set_context (get_outlet ()->get_default_interpreter ()); } -void +void Music_iterator::report_event (Music *m) { descend_to_bottom_context (); /* - FIXME: then don't do it. + FIXME: then don't do it. */ if (!m->is_mus_type ("event")) - m->origin ()->programming_error (_f ("Sending non-event to context")); + m->origin ()->programming_error ("Sending non-event to context"); m->send_to_context (get_outlet ()); } @@ -198,8 +207,11 @@ Music_iterator::mark_smob (SCM smob) Careful with GC, although we intend the following as pointers only, we _must_ mark them. */ - if (mus->get_outlet ()) - scm_gc_mark (mus->get_outlet ()->self_scm ()); + /* Use handle_ directly as get_outlet is a virtual function and we + need to protect the context until Music_iterator::quit is being + run. */ + if (mus->handle_.get_context ()) + scm_gc_mark (mus->handle_.get_context ()->self_scm ()); if (mus->music_) scm_gc_mark (mus->music_->self_scm ()); @@ -207,11 +219,11 @@ Music_iterator::mark_smob (SCM smob) } int -Music_iterator::print_smob (SCM sm, SCM port, scm_print_state*) +Music_iterator::print_smob (SCM sm, SCM port, scm_print_state *) { char s[1000]; - Music_iterator *iter = unsmob_iterator (sm); + Music_iterator *iter = Music_iterator::unsmob (sm); sprintf (s, "#<%s>", iter->class_name ()); scm_puts (s, port); return 1;