]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/chord.cc
release: 1.3.114
[lilypond.git] / lily / chord.cc
index 6f4e565ace04c0e4977a06c9ca619edbc0c7c42c..2aa4b0532eba2cae3b9bd43a6061908d0e4236fe 100644 (file)
@@ -46,12 +46,6 @@ ly_remove_member (SCM s, SCM list)
   return gh_reverse (removed);
 }
 
-SCM
-ly_last (SCM list)
-{
-  return gh_car (scm_last_pair (list));
-}
-
 /* tail add */
 SCM
 ly_snoc (SCM s, SCM list)
@@ -78,65 +72,6 @@ ly_split_list (SCM s, SCM list)
   return gh_cons (gh_reverse (before), after);
 }
 
-
-/* Construct from list of pitches and requests:
-
-  (PITCHES . (INVERSION . BASS))
-
-
-  Note, the pitches here, are all inclusive.
-  We must identify tonic, filter-out (and maybe detect) inversion and bass. */
-
-SCM
-Chord::pitches_and_requests_to_chord (SCM pitches,
-                                     SCM tonic_req,
-                                     SCM inversion_req,
-                                     SCM bass_req,
-                                     bool find_inversion_b)
-{
-  pitches = scm_sort_list (pitches,
-                          scm_eval2 (ly_symbol2scm ("Pitch::less_p"),
-                                     SCM_EOL));
-
-                          
-  if (bass_req != SCM_EOL)
-    {
-      assert (unsmob_pitch (gh_car (pitches))->notename_i_
-             == unsmob_pitch (bass_req)->notename_i_);
-      pitches = gh_cdr (pitches);
-    }
-    
-  if (inversion_req != SCM_EOL)
-    {
-      assert (unsmob_pitch (gh_car (pitches))->notename_i_
-             == unsmob_pitch (inversion_req)->notename_i_);
-      /* huh ? */
-      assert (tonic_req != SCM_EOL);
-      
-      SCM tonic = member_notename (tonic_req, pitches);
-      if (tonic != SCM_EOL)
-       pitches = rebuild_insert_inversion (pitches); //, tonic);
-    }
-  else if (find_inversion_b)
-    {
-      SCM tonic = (tonic_req != SCM_EOL)
-       ? member_notename (pitches, tonic_req)
-       : guess_tonic (pitches);
-       
-      if (tonic != SCM_EOL)
-       {
-         inversion_req = gh_car (pitches);
-         pitches = rebuild_insert_inversion (pitches); //, tonic);
-       }
-    }
-
-  if (tonic_req != SCM_EOL)
-      assert (unsmob_pitch (gh_car (pitches))->notename_i_
-             == unsmob_pitch (tonic_req)->notename_i_);
-
-  return gh_cons (pitches, gh_cons (inversion_req, bass_req));
-}
-
 /*
   JUNKME. 
   do something smarter.
@@ -188,20 +123,14 @@ Chord::lower_step (SCM tonic, SCM pitches, SCM step)
       if (gh_equal_p (step_scm (tonic, gh_car (i)), step)
          || gh_scm2int (step) == 0)
        {
-#if 0
-         Pitch x = *unsmob_pitch (p);
-         x.alteration_i_--;
-         p = x.smobbed_copy ();
-#else
          p = Pitch::transpose (p, Pitch (0, 0, -1).smobbed_copy ());
-#endif
        }
       lowered = gh_cons (p, lowered);
     }
   return gh_reverse (lowered);
 }
 
-/* Return member that has same notename, disregarding octave or accidentals */
+/* Return member that has same notename, disregarding octave or alterations */
 SCM
 Chord::member_notename (SCM p, SCM pitches)
 {
@@ -226,26 +155,54 @@ Chord::member_notename (SCM p, SCM pitches)
   return member;
 }
 
-int
-Chord::step_i (Pitch tonic, Pitch p)
+/* Return member that has same notename and alteration, disregarding octave */
+SCM
+Chord::member_pitch (SCM p, SCM pitches)
 {
-  int i = p.notename_i_ - tonic.notename_i_
-    + (p.octave_i ()  - tonic.octave_i () ) * 7;
-  while (i < 0)
-    i += 7;
-  i++;
-  return i;
+  /* If there's an exact match, make sure to return that */
+  SCM member = gh_member (p, pitches);
+  if (member == SCM_BOOL_F)
+    {
+      for (SCM i = pitches; gh_pair_p (i); i = gh_cdr (i))
+       {
+         if (unsmob_pitch (p)->notename_i_
+             == unsmob_pitch (gh_car (i))->notename_i_
+             && unsmob_pitch (p)->alteration_i_
+             == unsmob_pitch (gh_car (i))->alteration_i_)
+           {
+             member = gh_car (i);
+             break;
+           }
+       }
+    }
+  return member;
 }
 
 SCM
 Chord::step_scm (SCM tonic, SCM p)
 {
-  return gh_int2scm (step_i (*unsmob_pitch (tonic), *unsmob_pitch (p)));
+  /* De Pitch intervaas is nog beetje sleutelgat? */
+  int i = unsmob_pitch (p)->notename_i_
+    - unsmob_pitch (tonic)->notename_i_
+    + (unsmob_pitch (p)->octave_i_
+       - unsmob_pitch (tonic)->octave_i_ ) * 7;
+  while (i < 0)
+    i += 7;
+  i++;
+  return gh_int2scm (i);
 }
 
 /*
-  docme
- */
+  Assuming that PITCHES is a chord, with tonic (CAR PITCHES), find
+  missing thirds, only considering notenames.  Eg, for
+
+    PITCHES = c gis d'
+
+  return
+  
+    MISSING = e b'
+
+*/
 SCM
 Chord::missing_thirds (SCM pitches)
 {
@@ -294,139 +251,44 @@ Chord::missing_thirds (SCM pitches)
   return lower_step (tonic, missing, gh_int2scm (7));
 }
 
-
-/*
- Mangle
-
- (PITCHES . (INVERSION . BASS))
- into list of pitches.
- For normal chord entry, inversion and bass pitches are retained in
- specific *_requests */
-
+/* Return PITCHES with PITCH added not as lowest note */
 SCM
-Chord::to_pitches (SCM chord)
+Chord::add_above_tonic (SCM pitch, SCM pitches)
 {
-  SCM pitches = gh_car (chord);
-  SCM modifiers = gh_cdr (chord);
-  SCM inversion = gh_car (modifiers);
-  SCM bass = gh_cdr (modifiers);
-  if (inversion != SCM_EOL)
-    {
-      Pitch inversion_pitch = *unsmob_pitch (inversion);
-      SCM i = pitches;
-      for (; gh_pair_p (i); i = gh_cdr (i))
-       {
-         Pitch p = *unsmob_pitch (gh_car (i));
-         if ((p.notename_i_ == inversion_pitch.notename_i_)
-             && (p.alteration_i_ == inversion_pitch.alteration_i_))
-           break;
-       }
-      if (gh_pair_p (i))
-       pitches = rebuild_with_bass (pitches, gh_car (i));
-      else
-       warning (_f ("invalid inversion pitch: not part of chord: %s",
-                    unsmob_pitch (inversion)->str ()));
-    }
-
-  if (bass != SCM_EOL)
-    {
-      pitches = gh_cons (bass, pitches);
-      pitches = rebuild_with_bass (pitches, bass);
-    }
-  return pitches;
-}
-
-/*
-  This routine tries to guess tonic in a possibly inversed chord, ie
-  <e g c'> should produce: C.
-  This is only used for chords that are entered as simultaneous notes,
-  chords entered in \chord mode are fully defined.
- */
-
-SCM
-Chord::guess_tonic (SCM pitches)
-{
-  return gh_car (scm_sort_list (pitches, Pitch::less_p_proc)); 
-} 
-
-SCM
-Chord::rebuild_from_base (SCM pitches, SCM base)
-{
-  SCM split = ly_split_list (base, pitches);
-  SCM before = gh_car (split);
-  SCM after = gh_cdr (split);
-
-  SCM last = Pitch (0, 0, -5).smobbed_copy ();
-  SCM rebuilt = SCM_EOL;
-  rebuilt = gh_cons (base, rebuilt);
-  for (SCM i = gh_append2 (after, before); gh_pair_p (i); i = gh_cdr (i))
-    {
-      SCM p = gh_car (i);
-      if (Pitch::less_p (p, last) == SCM_BOOL_T)
-       {
-         // UHUHUrg
-         p = Pitch (unsmob_pitch (last)->octave_i_,
-                            unsmob_pitch (p)->notename_i_,
-                            unsmob_pitch (p)->alteration_i_).smobbed_copy ();
-         if (Pitch::less_p (p, last))
-           p = Pitch::transpose (p, Pitch (1, 0, 0).smobbed_copy ());
-       }
-      rebuilt = gh_cons (p, rebuilt);
-      last = p;
-    }
-
-  return gh_reverse (rebuilt);
-}
-
-SCM
-Chord::rebuild_insert_inversion (SCM pitches) //, SCM tonic)
-{
-  SCM inversion = gh_car (pitches);
-  pitches = gh_cdr (pitches);
-  SCM tonic = gh_car (pitches);
-  pitches = rebuild_from_base (pitches, tonic);
+  /* Should we maybe first make sure that PITCH is below tonic? */
   if (pitches != SCM_EOL)
-    {
-      // UHUHUrg
-      inversion = Pitch (unsmob_pitch (gh_car (pitches))->octave_i_-1,
-                                unsmob_pitch (inversion)->notename_i_,
-                                unsmob_pitch (inversion)->alteration_i_).smobbed_copy ();
-      while (Pitch::less_p (inversion, gh_car (pitches)) == SCM_BOOL_T)
-       inversion = Pitch::transpose (inversion, Pitch (1, 0, 0).smobbed_copy ());
-    }
-  pitches = gh_cons (inversion, pitches);
-  return scm_sort_list (pitches,
-                       scm_eval2 (ly_symbol2scm ("Pitch::less_p"),
-                                  SCM_EOL));
+    while (Pitch::less_p (pitch, gh_car (pitches)) == SCM_BOOL_T)
+      pitch = Pitch::transpose (pitch, Pitch (1, 0, 0).smobbed_copy ());
+   
+  pitches = gh_cons (pitch, pitches);
+  return scm_sort_list (pitches, Pitch::less_p_proc);
 }
 
+/* Return PITCHES with PITCH added as lowest note */
 SCM
-Chord::rebuild_with_bass (SCM pitches, SCM bass)
+Chord::add_below_tonic (SCM pitch, SCM pitches)
 {
-  pitches = ly_remove_member (bass, pitches);
-  // is lowering fine, or should others be raised?
   if (pitches != SCM_EOL)
-    while (Pitch::less_p (gh_car (pitches), bass) == SCM_BOOL_T)
-      bass = Pitch::transpose (bass, Pitch (-1, 0, 0).smobbed_copy ());
-  return gh_cons (bass, pitches);
+    while (Pitch::less_p (gh_car (pitches), pitch) == SCM_BOOL_T)
+      pitch = Pitch::transpose (pitch, Pitch (-1, 0, 0).smobbed_copy ());
+  return gh_cons (pitch, pitches);
 }
 
 
 
-/*********************************/
-/* Parser stuff */
-
-/* Construct from parser output:
+/*
+  Parser stuff 
+  
+  Construct from parser output:
 
-  (PITCHES . (INVERSION . BASS)) */
+  PITCHES is the plain chord, it does not include bass or inversion
+  
+  Part of Chord:: namespace for now, because we do lots of
+  chord-manipulating stuff.
+*/
 SCM
-Chord::tonic_add_sub_inversion_bass_to_scm (SCM tonic, SCM add, SCM sub,
-                                           SCM inversion, SCM bass)
+Chord::tonic_add_sub_to_pitches (SCM tonic, SCM add, SCM sub)
 {
-  SCM less = scm_eval2 (ly_symbol2scm ("Pitch::less_p"), SCM_EOL);
-
   /* urg: catch dim modifier: 3rd, 5th, 7th, .. should be lowered */
   bool dim_b = false;
   for (SCM i = add; gh_pair_p (i); i = gh_cdr (i))
@@ -440,12 +302,12 @@ Chord::tonic_add_sub_inversion_bass_to_scm (SCM tonic, SCM add, SCM sub,
     }
   add = transpose_pitches (tonic, add);
   add = lower_step (tonic, add, gh_int2scm (7));
-  add = scm_sort_list (add, less);
+  add = scm_sort_list (add, Pitch::less_p_proc);
   add = ly_unique (add);
   
   sub = transpose_pitches (tonic, sub);
   sub = lower_step (tonic, sub, gh_int2scm (7));
-  sub = scm_sort_list (sub, less);
+  sub = scm_sort_list (sub, Pitch::less_p_proc);
   
   /* default chord includes upto 5: <1, 3, 5>   */
   add = gh_cons (tonic, add);
@@ -470,21 +332,21 @@ Chord::tonic_add_sub_inversion_bass_to_scm (SCM tonic, SCM add, SCM sub,
   /* if additions include any 3, don't add third */
   SCM third = gh_cadr (base_pitches (tonic));
   if (member_notename (third, add) != SCM_BOOL_F)
-    missing = ly_remove_member (third, missing);
+    missing = scm_delete (third, missing);
 
   /* if additions include any 4, assume sus4 and don't add third implicitely
      C-sus (4) = c f g (1 4 5) */
   SCM sus = Pitch::transpose (tonic, Pitch (0, 3, 0).smobbed_copy ());
   if (member_notename (sus, add) != SCM_BOOL_F)
-    missing = ly_remove_member (third, missing);
+    missing = scm_delete (third, missing);
   
   /* if additions include some 5, don't add fifth */
   if (member_notename (fifth, add) != SCM_BOOL_F)
-    missing = ly_remove_member (fifth, missing);
+    missing = scm_delete (fifth, missing);
     
   /* complete the list of thirds to be added */
   add = gh_append2 (missing, add);
-  add = scm_sort_list (add, less);
+  add = scm_sort_list (add, Pitch::less_p_proc);
   
   SCM pitches = SCM_EOL;
   /* Add all that aren't subtracted */
@@ -493,82 +355,67 @@ Chord::tonic_add_sub_inversion_bass_to_scm (SCM tonic, SCM add, SCM sub,
       SCM p = gh_car (i);
       SCM s = member_notename (p, sub);
       if (s != SCM_BOOL_F)
-       sub = ly_remove_member (s, sub);
+       sub = scm_delete (s, sub);
       else
        pitches = gh_cons (p, pitches);
     }
-  pitches = scm_sort_list (pitches, less);
+  pitches = scm_sort_list (pitches, Pitch::less_p_proc);
   
   for (SCM i = sub; gh_pair_p (i); i = gh_cdr (i))
     warning (_f ("invalid subtraction: not part of chord: %s",
                 unsmob_pitch (gh_car (i))->str ()));
 
-  return gh_cons (pitches, gh_cons (inversion, bass));
+  return pitches;
 }
 
 
-/*
-  junk me
-
-  snapnie
-  
-  Een chord invoer bestaat uit een naam.  Maar, we willen een aantal
-  pitch-requests doen, zodat na het parsen van een chord geen verschil
-  meer is met een gewoon accoord.  Die vertaalslag is behoorlijk
-  harig, hoe wil je dit junken?  Nouja, cleanup lijkt me aardige
-  eerste stap enniewee.
-
-
-  --Het lijkt me dat dit in het paarse gedeelte moet.
-
-  Zo-en-zo, lijktme dat je ipv. Inversion_req een (inversion . #t) aan
-  de betreffende Noot_req kan hangen
-*/
-
+/* --Het lijkt me dat dit in het paarse gedeelte moet. */
 Simultaneous_music *
 Chord::get_chord (SCM tonic, SCM add, SCM sub, SCM inversion, SCM bass, SCM dur)
 {
-  SCM chord = tonic_add_sub_inversion_bass_to_scm (tonic, add, sub,
-                                                  inversion, bass);
-                                                  
-  Tonic_req* t = new Tonic_req;
-  t->set_mus_property ("pitch",  tonic);
-  SCM l = gh_cons (t->self_scm (), SCM_EOL);
-
-  SCM modifiers = gh_cdr (chord);
-  inversion = gh_car (modifiers);
-  bass = gh_cdr (modifiers);
-
-  //urg
+  SCM pitches = tonic_add_sub_to_pitches (tonic, add, sub);
+  SCM list = SCM_EOL;
   if (inversion != SCM_EOL)
     {
-      Inversion_req* i = new Inversion_req;
-      i->set_mus_property ("pitch",  inversion);
-      l = gh_cons (i->self_scm (), l);
-      scm_unprotect_object (i->self_scm ());
+      /* If inversion requested, check first if the note is part of chord */
+      SCM s = member_pitch (inversion, pitches);
+      if (s != SCM_BOOL_F)
+       {
+         /* Then, delete and add as base note, ie: the inversion */
+         pitches = scm_delete (s, pitches);
+         Note_req* n = new Note_req;
+         n->set_mus_property ("pitch", gh_car (add_below_tonic (s, pitches)));
+         n->set_mus_property ("duration", dur);
+         n->set_mus_property ("inversion", SCM_BOOL_T);
+         list = gh_cons (n->self_scm (), list);
+         scm_unprotect_object (n->self_scm ());
+       }
+      else
+       warning (_f ("invalid inversion pitch: not part of chord: %s",
+                    unsmob_pitch (inversion)->str ()));
     }
 
+  /* Bass is easy, just add if requested */
   if (bass != SCM_EOL)
     {
-      Bass_req* b = new Bass_req;
-      b->set_mus_property ("pitch", bass);
-
-      l = gh_cons (b->self_scm (), l);
-      scm_unprotect_object (b->self_scm ());      
+      Note_req* n = new Note_req;
+      n->set_mus_property ("pitch", gh_car (add_below_tonic (bass, pitches)));
+      n->set_mus_property ("duration", dur);
+      n->set_mus_property ("bass", SCM_BOOL_T);
+      list = gh_cons (n->self_scm (), list);
+      scm_unprotect_object (n->self_scm ());
     }
-
-  SCM pitches = Chord::to_pitches (chord);
+  
   for (SCM i = pitches; gh_pair_p (i); i = gh_cdr (i))
     {
       Note_req* n = new Note_req;
       n->set_mus_property ("pitch", gh_car (i));
       n->set_mus_property ("duration", dur);
-      l = gh_cons (n->self_scm (), l);
-
+      list = gh_cons (n->self_scm (), list);
       scm_unprotect_object (n->self_scm ());
     }
 
-  Simultaneous_music*v = new Request_chord (l);
+  Simultaneous_music*v = new Request_chord (list);
 
   return v;
 }