* lily/include/music.hh (class Music): include SCM init argument.

[lilypond.git] / lily / music-sequence.cc

diff --git a/lily/music-sequence.cc b/lily/music-sequence.cc
index 1d967a3dde51731c1480fe88e7d64418c80d2b3e..35db3471c8a55e058d765973fe2702379e9fe82d 100644 (file)
--- a/lily/music-sequence.cc
+++ b/lily/music-sequence.cc
@@ -3,47 +3,30 @@
   
   source file of the GNU LilyPond music typesetter
   
-  (c) 1998--2003 Han-Wen Nienhuys <hanwen@cs.uu.nl>
-  
- */
+  (c) 1998--2004 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+*/
+
 #include "music-list.hh"
 #include "warn.hh"
 #include "pitch.hh"
 #include "input.hh"
 
-SCM
-Music_sequence::music_list ()const
-{
-  return get_mus_property ("elements");
-}
-
-/*
-  Ugh this sucks. Linear. do not use.
- */
-void
-Music_sequence::append_music (Music *m)
+Music_sequence::Music_sequence (SCM x)
+  : Music (x)
 {
-  set_mus_property ("elements",
-                   gh_append2 (music_list (), gh_cons (m->self_scm (), SCM_EOL)));
-  scm_gc_unprotect_object (m->self_scm ());
 }
 
-Music_sequence::Music_sequence ( )
-  : Music ()
-{
-}
-
-void
-Music_sequence::transpose (Pitch rq)
+SCM
+Music_sequence::music_list () const
 {
-  transpose_list (music_list (), rq);
+  return get_property ("elements");
 }
 
 void
-Music_sequence::transpose_list (SCM l,  Pitch rq)
+transpose_music_list (SCM lst, Pitch rq)
 {
-  for (SCM s = l; gh_pair_p (s);  s = ly_cdr (s))
-    unsmob_music (ly_car (s))->transpose (rq);    
+  for (SCM s = lst; scm_is_pair (s);  s = scm_cdr (s))
+    unsmob_music (scm_car (s))->transpose (rq);    
 }
 
 Moment
@@ -52,9 +35,9 @@ Music_sequence::cumulative_length (SCM l)
   Moment cumulative;
   Moment last_len; 
 
-  for (SCM s = l; gh_pair_p (s);  s = ly_cdr (s))
+  for (SCM s = l; scm_is_pair (s);  s = scm_cdr (s))
     {
-      Moment l = unsmob_music (ly_car (s))->get_length ();
+      Moment l = unsmob_music (scm_car (s))->get_length ();
       if (last_len.grace_part_ && l.main_part_)
        {
          last_len.grace_part_ = Rational (0);
@@ -69,20 +52,13 @@ Music_sequence::cumulative_length (SCM l)
   return  cumulative;
 }
 
-Pitch
-Music_sequence::to_relative_octave (Pitch p)
-{
-  return do_relative_octave (p, false);
-}
-
-
 Moment
 Music_sequence::maximum_length (SCM l)
 {
   Moment dur = 0;
-  for (SCM s = l; gh_pair_p (s);  s = ly_cdr (s))
+  for (SCM s = l; scm_is_pair (s);  s = scm_cdr (s))
     {
-      Music * m = unsmob_music (ly_car (s));
+      Music * m = unsmob_music (scm_car (s));
       Moment l = m->get_length ();
       dur = dur >? l;
     }
@@ -90,22 +66,16 @@ Music_sequence::maximum_length (SCM l)
   return dur;
 }
 
-
 Pitch
-Music_sequence::do_relative_octave (Pitch p, bool ret_first)
+music_list_to_relative (SCM l,Pitch p, bool ret_first)
 {
-  Pitch first;
-  int count=0;
+  Pitch first = p;
+  int count = 0;
 
   Pitch last = p;
-  for (SCM s = music_list (); gh_pair_p (s);  s = ly_cdr (s))
+  for (SCM s = l; scm_is_pair (s);  s = scm_cdr (s))
     {
-      Music *m = unsmob_music (ly_car (s));
-      if (!m)
-       {
-         programming_error ("Music_sequence should only contain music!");
-       }
-      else
+      if (Music *m = unsmob_music (scm_car (s)))
        {
          last = m->to_relative_octave (last);
          if (!count ++)
@@ -113,28 +83,15 @@ Music_sequence::do_relative_octave (Pitch p, bool ret_first)
        }
     }
 
-  if (ret_first && first != last)
-    {
-      String str = _("Changing relative definition causes pitch change.");
-      str += "\nWas: " +  first.to_string ()
-       + "Will be: " + last.to_string () + "\n";
-      
-      origin()->warning (str);
-    }
-  return last;
+  return  (ret_first)?  first : last;
 }
 
-void
-Music_sequence::compress (Moment m)
-{
-  compress_list (music_list (), m);
-}
 
 void
-Music_sequence::compress_list (SCM l, Moment m)
+compress_music_list (SCM l, Moment m)
 {
-  for (SCM s = l; gh_pair_p (s);  s = ly_cdr (s))
-    unsmob_music (ly_car (s))->compress (m);
+  for (SCM s = l; scm_is_pair (s);  s = scm_cdr (s))
+    unsmob_music (scm_car (s))->compress (m);
 }
 
 ADD_MUSIC (Music_sequence);
@@ -144,9 +101,9 @@ Music_sequence::minimum_start (SCM l)
 {
   Moment m;
   
-  for (SCM s = l; gh_pair_p (s);  s = ly_cdr (s))
+  for (SCM s = l; scm_is_pair (s);  s = scm_cdr (s))
     {
-      m = m <? unsmob_music (ly_car (s))->start_mom ();
+      m = m <? unsmob_music (scm_car (s))->start_mom ();
     }
   return m;
 }
@@ -156,9 +113,9 @@ Music_sequence::first_start (SCM l)
 {
   Moment m;
   
-  for (SCM s = l; gh_pair_p (s);  s = ly_cdr (s))
+  for (SCM s = l; scm_is_pair (s);  s = scm_cdr (s))
     {
-      Music * mus = unsmob_music (ly_car (s));
+      Music * mus = unsmob_music (scm_car (s));
       Moment l = mus->get_length ();
       Moment s = mus->start_mom ();
       if (l.to_bool () || s.to_bool ())