X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fsimple-music-iterator.cc;h=05fcc293f4211d6c003f0f7387f5ad3f386df3ba;hb=4b83959939a2c4ad3a44322d1e27a27e7c5a91d8;hp=c4a78e9583a8f9048b2ba200a233ad4d1a7959f4;hpb=45bae7c4e36d595cf545c85a53ff14c95bf4cf9c;p=lilypond.git

diff --git a/lily/simple-music-iterator.cc b/lily/simple-music-iterator.cc
index c4a78e9583..05fcc293f4 100644
--- a/lily/simple-music-iterator.cc
+++ b/lily/simple-music-iterator.cc
@@ -1,13 +1,13 @@
-/*   
-  simple-music-iterator.cc --  implement Simple_music_iterator
-  
+/*
+  simple-music-iterator.cc -- implement Simple_music_iterator
+
   source file of the GNU LilyPond music typesetter
-  
-  (c) 2000 Han-Wen Nienhuys <hanwen@cs.uu.nl>
-  
- */
+
+  (c) 2000--2007 Han-Wen Nienhuys <hanwen@xs4all.nl>
+*/
 
 #include "simple-music-iterator.hh"
+
 #include "music.hh"
 #include "input.hh"
 
@@ -17,48 +17,37 @@ Simple_music_iterator::Simple_music_iterator ()
   last_processed_mom_ = -1;
 }
 
-Simple_music_iterator::Simple_music_iterator (Simple_music_iterator const &src)
-  : Music_iterator (src)
-{
-  last_processed_mom_ = src.last_processed_mom_;
-}
-
 bool
 Simple_music_iterator::ok ()const
 {
-  return last_processed_mom_ < music_length_mom ();
+  return last_processed_mom_ < music_get_length ();
 }
 
+/*
+  TODO: remove last_processed_mom_, and the complete shit.  We should
+  only process a simple-music once, and that is at its start.
+
+  Engravers can detect and event the end-moments to be processed as
+  well.
+*/
 Moment
 Simple_music_iterator::pending_moment ()const
 {
-  if (music_l_)
+  if (last_processed_mom_ < Moment (0))
     return Moment (0);
   else
-    return music_length_mom ();
-}
-
-void
-Simple_music_iterator::skip (Moment m)
-{
-  music_l_ = 0;
-  last_processed_mom_ = m;
+    return music_get_length ();
 }
 
 void
 Simple_music_iterator::process (Moment m)
 {
-#if 0
   /*
-    try_music () causes trouble for base classes
-   */
-  if (music_l_)
-    {
-      bool b = try_music (music_l_);
-      if (!b)
-	music_l_->origin ()->warning (_f ("Junking music: `%s'",
-					  classname (music_l_)));
-    }
-#endif
-  skip (m);
+    don't do report_event (), since it would make the function useless for
+    base classes
+  */
+
+  last_processed_mom_ = m;
 }
+
+IMPLEMENT_CTOR_CALLBACK (Simple_music_iterator);