]> git.donarmstrong.com Git - lilypond.git/commitdiff
* scm/define-music-types.scm (music-descriptions): idem.
authorHan-Wen Nienhuys <hanwen@xs4all.nl>
Tue, 26 Aug 2003 11:47:36 +0000 (11:47 +0000)
committerHan-Wen Nienhuys <hanwen@xs4all.nl>
Tue, 26 Aug 2003 11:47:36 +0000 (11:47 +0000)
* lily/beam-engraver.cc: move new-beam-event to beam-event

* lily/tie-engraver.cc (class Tie_engraver): move New_tie_engraver
to Tie_engraver

* lily/parser.yy (Composite_music): add \newcontext

* lily/text-item.cc: remove old-style markups.
(interpret_markup): C++ version of markup function
(brew_molecule): idem.

ChangeLog
lily/beam-engraver.cc
lily/new-fingering-engraver.cc
lily/new-tie-engraver.cc
lily/parser.yy
lily/text-item.cc
lily/tie-engraver.cc
ly/engraver-init.ly
scm/define-grob-properties.scm
scm/define-music-types.scm

index d0604419e6ca82baca88fbb286288ec79e77c027..98566424caa53d45877d6ac25cce54b52def6f1a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2003-08-26  Han-Wen Nienhuys  <hanwen@cs.uu.nl>
 
+       * scm/define-music-types.scm (music-descriptions): idem.
+
+       * lily/beam-engraver.cc: move new-beam-event to beam-event
+
+       * lily/tie-engraver.cc (class Tie_engraver): move New_tie_engraver
+       to Tie_engraver
+
        * input/regression/bar-number.ly: remove unnecessary old-relative
        options.
        
index a00668d785e9bbc5a010ca701b762ed791aa0e01..6bd0be9e6303900a769f705a8d91ce9b5fc8ebe1 100644 (file)
@@ -104,29 +104,6 @@ Beam_engraver::try_music (Music *m)
       beam_ = 0;
     }
   else if (m->is_mus_type ("beam-event"))
-    {
-      Direction d = to_dir (m->get_mus_property ("span-direction"));
-
-      if (d == STOP && !valid_end_moment())
-       return false;
-
-      if (d == START && !valid_start_moment ())
-       return false;
-         
-      if (d == STOP)
-       {
-         SCM m = get_property ("automaticMelismata");
-         SCM b = get_property ("autoBeaming");
-         if (to_boolean (m) && !to_boolean (b))
-           {
-             set_melisma (false);
-           }
-       }
-
-      evs_drul_[d ] = m;
-      return true;
-    }
-  else if (m->is_mus_type ("new-beam-event"))
     {
       Direction d = to_dir (m->get_mus_property ("span-direction"));
       if (d == START && !valid_start_moment ())
@@ -330,7 +307,7 @@ ENTER_DESCRIPTION(Beam_engraver,
 /* descr */       "Handles Beam events by engraving Beams.    If omitted, then notes will be "
 "printed with flags instead of beams.",
 /* creats*/       "Beam",
-/* accepts */     "beam-event new-beam-event abort-event",
+/* accepts */     "beam-event abort-event",
 /* acks  */      "stem-interface rest-interface",
 /* reads */       "beamMelismaBusy beatLength subdivideBeams",
 /* write */       "");
@@ -373,7 +350,7 @@ ENTER_DESCRIPTION(Grace_beam_engraver,
 " are at grace points in time. "
 ,
 /* creats*/       "Beam",
-/* accepts */     "beam-event abort-event new-beam-event",
+/* accepts */     "beam-event abort-event",
 /* acks  */      "stem-interface rest-interface",
 /* reads */       "beamMelismaBusy beatLength allowBeamBreak subdivideBeams",
 /* write */       "");
index 63a6c83a0a15f89f1933fdd1a47b7cbbfda20ce5..f91a24d07e571fbf92023bd0fddd2e4db3099a05 100644 (file)
@@ -300,9 +300,9 @@ New_fingering_engraver::New_fingering_engraver()
 }
 
 ENTER_DESCRIPTION(New_fingering_engraver,
-/* descr */       "Create fingering-scripts for notes in a New Chord.",
+/* descr */       "Create fingering-scripts for notes in a new chord.",
 /* creats*/       "Fingering",
-/* accepts */     "text-script-event",
+/* accepts */     "",
 /* acks  */       "rhythmic-head-interface stem-interface",
 /* reads */       "fingerHorizontalDirection",
 /* write */       "");
index 11b414cc46cef1baa2f1f59b41a19474deed247b..e69de29bb2d1d6434b8b29ae775ad8c2e48c5391 100644 (file)
@@ -1,190 +0,0 @@
-/*   
-  new-tie-engraver.cc --  implement Tie_engraver
-  
-  source file of the GNU LilyPond music typesetter
-  
-  (c) 1998--2003 Han-Wen Nienhuys <hanwen@cs.uu.nl>
-  
- */
-
-#include "event.hh"
-#include "tie.hh"
-#include "translator-group.hh"
-#include "spanner.hh"
-#include "tie-column.hh"
-#include "engraver.hh"
-#include "item.hh"
-#include "grob-pitch-tuple.hh"
-#include "warn.hh"
-#include "note-head.hh"
-
-/**
-   Manufacture ties.  Acknowledge noteheads, and put them into a
-   priority queue. If we have a TieEvent, connect the notes that finish
-   just at this time, and note that start at this time.
-
-   TODO: Remove the dependency on musical info. We should tie on the
-   basis of position and duration-log of the heads (not of the events).
-
-   TODO: support sparseTies.
-
-   TODO: melismata will fuck up now:
-
-   < { c8 ~ c8 }
-     { c16 c c c  } >
-
-   melisma is after the 2nd 8th note, but will now be signaled as
-   lasting till the 3rd 16th.
-*/
-class New_tie_engraver : public Engraver
-{
-  Music *event_;
-  Music *last_event_;
-  Link_array<Grob> now_heads_;
-  Link_array<Grob> heads_to_tie_;
-  Link_array<Grob> ties_;
-  
-  Spanner * tie_column_;
-  
-  
-protected:
-  virtual void stop_translation_timestep ();
-  virtual void acknowledge_grob (Grob_info);
-  virtual bool try_music (Music*);
-  virtual void process_acknowledged_grobs ();
-  void typeset_tie (Grob*);
-public:
-  TRANSLATOR_DECLARATIONS(New_tie_engraver);
-};
-
-
-
-New_tie_engraver::New_tie_engraver ()
-{
-  event_ = 0;
-  last_event_  = 0;
-  tie_column_ = 0;
-}
-
-
-bool
-New_tie_engraver::try_music (Music *mus)
-{
-  if (mus->is_mus_type ("new-tie-event"))
-    {
-      event_ = mus;
-    }
-  
-  if (event_)
-    {
-      SCM m = get_property ("automaticMelismata");
-      bool am = gh_boolean_p (m) &&gh_scm2bool (m);
-      if (am)
-       {
-  daddy_trans_->set_property ("tieMelismaBusy", m ? SCM_BOOL_T : SCM_BOOL_F);
-       }
-    }
-  return true;
-}
-
-void
-New_tie_engraver::acknowledge_grob (Grob_info i)
-{
-  if (Note_head::has_interface (i.grob_))
-    {
-      Grob * h  = i.grob_;
-      now_heads_.push (h);
-      for  (int  i=heads_to_tie_.size (); i--;)
-       {
-         Grob *th =  heads_to_tie_[i];
-         int staff_pos = gh_scm2int (h->get_grob_property ("staff-position"));
-         int left_staff_pos = gh_scm2int (th->get_grob_property ("staff-position"));
-         if (staff_pos == left_staff_pos)
-           {
-             Grob * p = new Spanner (get_property ("Tie"));
-             Tie::set_interface (p); // cannot remove yet!
-         
-             Tie::set_head (p, LEFT, th);
-             Tie::set_head (p, RIGHT, h);
-         
-             ties_.push (p);
-             announce_grob(p, last_event_->self_scm());
-           }
-       }
-    }
-}
-
-void
-New_tie_engraver::process_acknowledged_grobs ()
-{
-  if (ties_.size () > 1 && !tie_column_)
-    {
-      tie_column_ = new Spanner (get_property ("TieColumn"));
-      
-      for (int i = ties_.size (); i--;)
-       Tie_column::add_tie (tie_column_,ties_ [i]);
-
-      announce_grob(tie_column_, SCM_EOL);
-    }
-}
-
-
-void
-New_tie_engraver::stop_translation_timestep ()
-{
-
-  if (ties_.size ())
-    {
-      heads_to_tie_.clear ();
-      for (int i=0; i<  ties_.size (); i++)
-       {
-         typeset_tie (ties_[i]);
-       }
-
-      ties_.clear();
-      last_event_ = 0;
-      if (tie_column_)
-       {
-         typeset_grob (tie_column_);
-         tie_column_ =0;
-       }
-    }
-  
-  if (event_)
-    {
-      heads_to_tie_ = now_heads_;
-      last_event_ = event_;
-    }
-  event_ = 0;
-  now_heads_.clear ();
-}
-
-void
-New_tie_engraver::typeset_tie (Grob *her)
-{
-  if (! (Tie::head (her,LEFT) && Tie::head (her,RIGHT)))
-    warning (_ ("lonely tie"));
-
-  Direction d = LEFT;
-  Drul_array<Grob *> new_head_drul;
-  new_head_drul[LEFT] = Tie::head (her,LEFT);
-  new_head_drul[RIGHT] = Tie::head (her,RIGHT);  
-  do {
-    if (!Tie::head (her,d))
-      new_head_drul[d] = Tie::head (her, (Direction)-d);
-  } while (flip (&d) != LEFT);
-
-  index_set_cell (her->get_grob_property ("heads"), LEFT, new_head_drul[LEFT]->self_scm ());
-  index_set_cell (her->get_grob_property ("heads"), RIGHT, new_head_drul[RIGHT]->self_scm ());
-
-  typeset_grob (her);
-}
-
-
-ENTER_DESCRIPTION(New_tie_engraver,
-/* descr */       "Generate ties between noteheads of equal pitch.",
-/* creats*/       "Tie TieColumn",
-/* accepts */     "new-tie-event",
-/* acks  */      "rhythmic-head-interface",
-/* reads */       "tieMelismaBusy",
-/* write */       "");
index 239b8f68f18446b0a09517a9e8568040c35bc4da..fbb7684c9ac5d913f1fc70a0650798390098b70d 100644 (file)
@@ -1524,19 +1524,19 @@ configurable, i.e.
 (set-articulation '~ "trill")
 
 */
-               Music * m = MY_MAKE_MUSIC ("NewBeamEvent");
+               Music * m = MY_MAKE_MUSIC ("BeamEvent");
                m->set_spot (THIS->here_input());
                m->set_mus_property ("span-direction" , gh_int2scm (START));
                $$ = m;
        }
        | ']'  {
-               Music * m = MY_MAKE_MUSIC ("NewBeamEvent");
+               Music * m = MY_MAKE_MUSIC ("BeamEvent");
                m->set_spot (THIS->here_input());
                m->set_mus_property ("span-direction" , gh_int2scm (STOP));
                $$ = m;
        }
        | '~' {
-               Music * m = MY_MAKE_MUSIC ("NewTieEvent");
+               Music * m = MY_MAKE_MUSIC ("TieEvent");
                m->set_spot (THIS->here_input());
                $$ = m;
        }
index ac80893cf281ae4f701870a0f013a269bd7ad390..0bf318708eff267e93cb7d607c387d5cda4b4cdf 100644 (file)
@@ -40,12 +40,19 @@ Text_item::interpret_markup (SCM grob, SCM props, SCM markup)
       Box b = fm->text_dimension (ly_scm2string (markup));
       return Molecule (b, list).smobbed_copy();
     }
-  else
+  else if (gh_pair_p (markup))
     {
       SCM func = gh_car (markup);
       SCM args = gh_cdr (markup);
+      if (!markup_p (markup))
+       programming_error ("Markup head has no markup signature.");
+      
       return scm_apply_2 (func, grob, props, args);
     }
+  else
+    {
+      return SCM_EOL;
+    }
 }
 
 MAKE_SCHEME_CALLBACK(Text_item,brew_molecule,1);
index e02eb889de2fa5931aa981f72ef52f9a920d6611..ad6e6b8859c0c5c9641ec6beeed9dfe2d069a6fc 100644 (file)
@@ -1,5 +1,5 @@
 /*   
-  tie-engraver.cc --  implement Tie_engraver
+  new-tie-engraver.cc --  implement Tie_engraver
   
   source file of the GNU LilyPond music typesetter
   
@@ -7,7 +7,6 @@
   
  */
 
-
 #include "event.hh"
 #include "tie.hh"
 #include "translator-group.hh"
    TODO: Remove the dependency on musical info. We should tie on the
    basis of position and duration-log of the heads (not of the events).
 
-   New tie event happens at the time of the first note, the 
+   TODO: support sparseTies.
+
+   TODO: melismata will fuck up now:
+
+   < { c8 ~ c8 }
+     { c16 c c c  } >
+
+   melisma is after the 2nd 8th note, but will now be signaled as
+   lasting till the 3rd 16th.
 */
 class Tie_engraver : public Engraver
 {
-  Moment end_mom_;
-  Moment next_end_mom_;
-
   Music *event_;
-  
+  Music *last_event_;
   Link_array<Grob> now_heads_;
-  Link_array<Grob> stopped_heads_;
+  Link_array<Grob> heads_to_tie_;
   Link_array<Grob> ties_;
-
+  
   Spanner * tie_column_;
   
-  void set_melisma (bool);
   
 protected:
-  virtual void start_translation_timestep ();
   virtual void stop_translation_timestep ();
   virtual void acknowledge_grob (Grob_info);
   virtual bool try_music (Music*);
@@ -60,6 +62,7 @@ public:
 Tie_engraver::Tie_engraver ()
 {
   event_ = 0;
+  last_event_  = 0;
   tie_column_ = 0;
 }
 
@@ -71,143 +74,89 @@ Tie_engraver::try_music (Music *mus)
     {
       event_ = mus;
     }
-
+  
   if (event_)
     {
       SCM m = get_property ("automaticMelismata");
       bool am = gh_boolean_p (m) &&gh_scm2bool (m);
       if (am)
        {
-         set_melisma (true);
+  daddy_trans_->set_property ("tieMelismaBusy", m ? SCM_BOOL_T : SCM_BOOL_F);
        }
     }
   return true;
 }
 
-void
-Tie_engraver::set_melisma (bool m)
-{
-  daddy_trans_->set_property ("tieMelismaBusy", m ? SCM_BOOL_T : SCM_BOOL_F);
-}
-
 void
 Tie_engraver::acknowledge_grob (Grob_info i)
 {
   if (Note_head::has_interface (i.grob_))
     {
-      now_heads_.push (i.grob_);
+      Grob * h  = i.grob_;
+      now_heads_.push (h);
+      for  (int  i=heads_to_tie_.size (); i--;)
+       {
+         Grob *th =  heads_to_tie_[i];
+         int staff_pos = gh_scm2int (h->get_grob_property ("staff-position"));
+         int left_staff_pos = gh_scm2int (th->get_grob_property ("staff-position"));
+         if (staff_pos == left_staff_pos)
+           {
+             Grob * p = new Spanner (get_property ("Tie"));
+             Tie::set_interface (p); // cannot remove yet!
+         
+             Tie::set_head (p, LEFT, th);
+             Tie::set_head (p, RIGHT, h);
+         
+             ties_.push (p);
+             announce_grob(p, last_event_->self_scm());
+           }
+       }
     }
 }
 
-int
-head_pitch_compare (Grob  *const&a,Grob  *const&b)
-{
-  Music *m1 =unsmob_music (a->get_grob_property ("cause"));
-  Music *m2 =unsmob_music (b->get_grob_property ("cause"));  
-
-  return Pitch::compare (* unsmob_pitch (m1->get_mus_property ("pitch")),
-                        * unsmob_pitch (m2->get_mus_property ("pitch")));
-}
-
 void
 Tie_engraver::process_acknowledged_grobs ()
 {
-  if (event_)
+  if (ties_.size () > 1 && !tie_column_)
     {
-      now_heads_.sort (&head_pitch_compare);
-      /*
-       We could sort stopped_heads_ as well (and use a linear alg. in
-       stead of nested loop), but we'd have to use a stable sorting
-       algorithm, since the ordering of the stopped heads (of the
-       same pitch) is relevant.
-       */
-
-      SCM head_list = SCM_EOL;
+      tie_column_ = new Spanner (get_property ("TieColumn"));
       
-      for (int i = now_heads_.size(); i--;)
-       {
-         for (int j = stopped_heads_.size(); j--;)
-           {
-             int comp
-               = head_pitch_compare (now_heads_[i], stopped_heads_[j]);
+      for (int i = ties_.size (); i--;)
+       Tie_column::add_tie (tie_column_,ties_ [i]);
 
-             if (!comp)
-               {
-                 head_list  = gh_cons (gh_cons (stopped_heads_[j]->self_scm (),
-                                                now_heads_[i]->self_scm ()),
-                                       head_list);
+      announce_grob(tie_column_, SCM_EOL);
+    }
+}
 
-                 now_heads_.del (i);
-                 stopped_heads_.del (j);
-                 break ;
-               }
-           }
-       }
-     
-      SCM basic = get_property ("Tie");
-      SCM sparse = get_property ("sparseTies");
-      if (to_boolean (sparse))
-       {
-         int i = scm_ilength (head_list);
 
-         if (!i)
-           return;
-         
-         SCM pair = scm_list_ref (head_list, gh_int2scm (i/2));
-         
-         Spanner * p = new Spanner (basic);
+void
+Tie_engraver::stop_translation_timestep ()
+{
 
-         Tie::set_interface (p); // cannot remove.
-         Tie::set_head (p,LEFT, dynamic_cast<Item*> (unsmob_grob (ly_car (pair))));
-         Tie::set_head (p,RIGHT, dynamic_cast<Item*> (unsmob_grob (ly_cdr (pair))));
-         
-         ties_.push (p);
-         announce_grob(p, event_->self_scm());
-       }
-      else for (SCM s = head_list; gh_pair_p (s); s = ly_cdr (s))
+  if (ties_.size ())
+    {
+      heads_to_tie_.clear ();
+      for (int i=0; i<  ties_.size (); i++)
        {
-         Grob * p = new Spanner (basic);
-         Tie::set_interface (p); // cannot remove yet!
-         
-         Tie::set_head (p, LEFT, dynamic_cast<Item*> (unsmob_grob (ly_caar (s))));
-         Tie::set_head (p, RIGHT, dynamic_cast<Item*> (unsmob_grob (ly_cdar (s))));
-         
-         ties_.push (p);
-         announce_grob(p, event_->self_scm());
+         typeset_tie (ties_[i]);
        }
 
-      if (ties_.size () > 1 && !tie_column_)
+      ties_.clear();
+      last_event_ = 0;
+      if (tie_column_)
        {
-         tie_column_ = new Spanner (get_property ("TieColumn"));
-
-         for (int i = ties_.size (); i--;)
-           Tie_column::add_tie (tie_column_,ties_ [i]);
-         announce_grob(tie_column_, SCM_EOL);
+         typeset_grob (tie_column_);
+         tie_column_ =0;
        }
     }
-}
-
-
-void
-Tie_engraver::stop_translation_timestep ()
-{
-  event_ = 0;
-  now_heads_.clear ();
-
-  /*
-    we don't warn for no ties, since this happens naturally when you
-    use skipTypesetting.  */
   
-  for (int i=0; i<  ties_.size (); i++)
-   {
-      typeset_tie (ties_[i]);
-    }
-  ties_.clear ();
-  if (tie_column_)
+  if (event_)
     {
-      typeset_grob (tie_column_);
-      tie_column_ =0;
+      heads_to_tie_ = now_heads_;
+      last_event_ = event_;
     }
+  event_ = 0;
+  now_heads_.clear ();
 }
 
 void
@@ -231,60 +180,11 @@ Tie_engraver::typeset_tie (Grob *her)
   typeset_grob (her);
 }
 
-void
-Tie_engraver::start_translation_timestep ()
-{
-  SCM m = get_property ("automaticMelismata");
-  if (to_boolean (m))
-    {
-      set_melisma (false);
-    }
-
-  SCM grobs = get_property ("busyGrobs");
-  Moment now = now_mom();
-  stopped_heads_.clear ();
-  
-  for (; gh_pair_p (grobs); grobs = gh_cdr (grobs))
-    {
-      Grob * grob  = unsmob_grob (gh_cdar (grobs));
-      Moment end  =*unsmob_moment (gh_caar (grobs));
-      
-      /*
-       This is slightly ugh: we are now confunding the frontend
-       (iterators) and the backend (note heads) */
-      if (end > now)
-       break;
-      else if (end == now
-              && Note_head::has_interface (grob))
-       stopped_heads_.push (grob);
-    }
-
-
-  /*
-    
-    The list starts with entries that start earlier. By going through
-    it, we reverse the order, where as we'd like to use the `last'
-    heads first.
-
-    This makes  a difference for grace notes. If we have
-
-    c4 \grace c8 ~ c4
-
-    Then busyGrobs will have ((1/4 . gc8) (1/4 . c4)). 
-
-    We want stopped_heads_ to contain (c4 gc8), because we start with
-    it at the top.
-   */
-  stopped_heads_.reverse();
-
-  event_ = 0;
-}
-
 
 ENTER_DESCRIPTION(Tie_engraver,
 /* descr */       "Generate ties between noteheads of equal pitch.",
 /* creats*/       "Tie TieColumn",
 /* accepts */     "tie-event",
 /* acks  */      "rhythmic-head-interface",
-/* reads */       "sparseTies tieMelismaBusy",
+/* reads */       "tieMelismaBusy",
 /* write */       "");
index 3598f76bcccc94a00cf94fdf8324b44c9e0ad1ae..065a8e502afcb8172187326005fbb5ca7a72df1c 100644 (file)
@@ -189,7 +189,6 @@ VoiceContext = \translator {
        \consists "Cluster_spanner_engraver"
        \consists "Slur_engraver"
        \consists "Tie_engraver"
-       \consists "New_tie_engraver"
        \consists "Tuplet_engraver"
        \consists "A2_engraver"
 
index d16085b3965280947e5f8b05795449d6f07c096e..942ad999c56ec22ed3a126287ccc3da37095bb05 100644 (file)
@@ -223,10 +223,6 @@ style-sheet's normal size, -1 is smaller, +1 is bigger.")
 (grob-property-description 'font-series symbol? "partial font definition: medium, bold.")
 (grob-property-description 'font-shape symbol? "partial font definition: upright or italic.")
 
-(grob-property-description 'font-style symbol? "a precooked set of font
-definitions, eg. finger volta timesig mark script large Large
-dynamic.")
-
 (grob-property-description 'force-hshift number? "amount of
 collision_note_width that overides automatic collision settings. This
 is used by @ref{note-collision-interface}.")
@@ -434,7 +430,8 @@ the Nth element of the list gives the amount stem shortening of a note with N fl
 (grob-property-description 'style symbol? "a string determining what style of  glyph is typeset. Valid choices depend on the function that is reading this property. .")
 (grob-property-description 'support-head ly:grob? "the note head at
 one end of the stem.")
-(grob-property-description 'text markup? "Text markup.  See reference manual for more information.")
+(grob-property-description 'text markup? "Text markup.  See the
+notation manual for more information.")
 (grob-property-description 'thick-thickness number? "thickness, measured in stafflinethickness.")
 (grob-property-description 'thickness number? "thickness, measured in stafflinethickness.")
 (grob-property-description 'thin-kern number? "space after a hair-line.")
index d981db254eb57a6fa01605aaa089bcb52ca456e2..2bc5bc9a6de1c1aa24692e74cd008392e831d42d 100644 (file)
@@ -87,14 +87,6 @@ is an articulation (such as @code{-.}, @code{->}, @code{\\tenuto},
      . (
    (description .  "Starts or stops a beam.  Syntax for manual control:
 c8-[ c c-] c8")
-
-       (internal-class-name . "Event")
-       (types . (general-music event beam-event span-event))
-       ))
-    (NewBeamEvent
-     . (
-   (description .  "Starts or stops a beam.  Syntax for manual control:
-c8-[ c c-] c8")
        (internal-class-name . "Event")
        (types . (general-music event new-beam-event span-event))
        ))
@@ -572,12 +564,6 @@ For example, transposed music.")
        (types . (general-music script-event text-script-event event))
        )) 
     (TieEvent
-     . (
-       (description .  "A tie.  Entered as @code{~}.")
-       (internal-class-name . "Event")
-       (types . (general-music tie-event event))
-       ))
-    (NewTieEvent
      . (
        (description .  "A tie.  Entered as @var{note}-~.")
        (internal-class-name . "Event")