Fix some bugs in the dynamic engraver and PostScript backend

[lilypond.git] / lily / recording-group-engraver.cc

diff --git a/lily/recording-group-engraver.cc b/lily/recording-group-engraver.cc
index 70b7f79f27bbc21a49a11e813b2af2819011d5b7..c0091cc834571ee68abf6d835ac679a674ee5861 100644 (file)
--- a/lily/recording-group-engraver.cc
+++ b/lily/recording-group-engraver.cc
@@ -1,89 +1,81 @@
-/*   
+/*
   recording-group-engraver.cc -- implement Recording_group_engraver
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 2003--2004 Han-Wen Nienhuys <hanwen@xs4all.nl>
-
- */
+  (c) 2003--2006 Han-Wen Nienhuys <hanwen@xs4all.nl>
+*/
 
+#include "recording-group-engraver.hh"
 #include "context.hh"
-#include "engraver-group-engraver.hh"
-#include "protected-scm.hh"
-
-class Recording_group_engraver : public Engraver_group_engraver
-{
-public:
-  TRANSLATOR_DECLARATIONS(Recording_group_engraver);
-  virtual bool try_music (Music *m);
-  virtual void start_translation_timestep ();
-  virtual void stop_translation_timestep ();
-  virtual void finalize ();
-  virtual void initialize ();
-  Protected_scm accumulator_;
-};
 
 void
-Recording_group_engraver::initialize ()
+Recording_group_engraver::derived_mark () const
 {
-  Engraver_group_engraver::initialize ();
-  accumulator_ = gh_cons (SCM_EOL, SCM_EOL);
+  Engraver_group::derived_mark ();
+  scm_gc_mark (now_events_);
+  scm_gc_mark (accumulator_);
 }
 
-Recording_group_engraver::Recording_group_engraver()
+Recording_group_engraver::Recording_group_engraver ()
 {
+  accumulator_ = SCM_EOL;
+  now_events_ = SCM_EOL;
 }
 
 void
-Recording_group_engraver::start_translation_timestep ()
+Recording_group_engraver::add_music (SCM music, SCM success)
 {
-  Engraver_group_engraver::start_translation_timestep();
-  
-  scm_set_car_x (accumulator_, gh_cons (now_mom ().smobbed_copy (), SCM_EOL));
+  now_events_ = scm_cons (scm_cons (music, success), now_events_);
 }
 
 void
 Recording_group_engraver::stop_translation_timestep ()
 {
-  Engraver_group_engraver::stop_translation_timestep();
-  scm_set_cdr_x (accumulator_, gh_cons (gh_car (accumulator_), gh_cdr (accumulator_)));
-
-  scm_set_car_x (accumulator_, SCM_EOL);
+  accumulator_ = scm_acons (scm_cons (context ()->now_mom ().smobbed_copy (),
+                                     context ()->get_property ("instrumentTransposition")),
+                           now_events_,
+                           accumulator_);
+  now_events_ = SCM_EOL;
 }
 
 void
 Recording_group_engraver::finalize ()
 {
-  Engraver_group_engraver::finalize ();
-  SCM proc = get_property ("recordEventSequence");
-
-  if (gh_procedure_p (proc))
-    scm_call_2  (proc, daddy_context_->self_scm(), gh_cdr (accumulator_));
+  SCM proc = context ()->get_property ("recordEventSequence");
 
-  accumulator_ = SCM_EOL;
+  if (ly_is_procedure (proc))
+    scm_call_2 (proc, context ()->self_scm (), scm_cdr (accumulator_));
 }
 
 bool
-Recording_group_engraver::try_music (Music  *m)
+Recording_group_engraver::try_music (Music *m)
 {
   bool retval = Translator_group::try_music (m);
 
-  SCM seq = gh_cdar (accumulator_);
-  seq = gh_cons (gh_cons  (m->self_scm(), gh_bool2scm (retval)),
-                seq);
-  
-  scm_set_cdr_x  (gh_car (accumulator_), seq);
-
+  add_music (m->self_scm (), ly_bool2scm (retval));
   return retval;
 }
 
+void
+recording_engraver (Translator_group *tg)
+{
+  Recording_group_engraver *rg = dynamic_cast<Recording_group_engraver *> (tg);
+  rg->stop_translation_timestep ();
+}
+
+void
+Recording_group_engraver::fetch_precomputable_methods (Translator_group_void_method ptrs[])
+{
+  Translator_group::fetch_precomputable_methods (ptrs);
+  ptrs[STOP_TRANSLATION_TIMESTEP] = &recording_engraver;
+}
 
-ENTER_DESCRIPTION(Recording_group_engraver,
-                 "Engraver_group_engraver that records all music events "
-                 "for this context. Calls the procedure "
-                 "in @code{recordEventSequence} when finished.",
-                 "",
-                 "",
-                 "",
-                 "recordEventSequence",
-                 "");
+ADD_TRANSLATOR_GROUP (Recording_group_engraver,
+                     "Engraver_group that records all music events "
+                     "for this context. Calls the procedure "
+                     "in @code{recordEventSequence} when finished.",
+                     "",
+                     "",
+                     "recordEventSequence",
+                     "");