]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/parser.yy
* mf/GNUmakefile: always trace pfa fonts.
[lilypond.git] / lily / parser.yy
index cb9d8f14cec7b6ec25228aeb148e073e934b0117..2f9d9b37c232daa0163a66877b4a4e2be722c178 100644 (file)
@@ -10,7 +10,7 @@
 */
 
 /*
-  Two shift/reduce problems:
+  Four shift/reduce problems:
 
 1.     foo = bar.
 
        "bar" -> String -> string-assignment
 
 
+Similar problem for
+
+ * \markup identifier.
+ * \markup { }
+
+
 2.  \repeat
        \repeat .. \alternative
 
@@ -47,10 +53,12 @@ TODO:
 */
 
 #include <ctype.h>
+#include <stdio.h>
 #include <stdlib.h>
+#include <stdio.h>
 
 
-
+#include "scm-option.hh"
 #include "translator-def.hh"
 #include "lily-guile.hh"
 #include "misc.hh"
@@ -90,7 +98,23 @@ My_lily_parser* my_lily_parser;
 
 #define yyerror THIS->parser_error
 
+/*
+  Add symbols to the  TAGS field of a music object. 
+*/
 
+void
+tag_music (Music*m,  SCM tag, Input ip)
+{
+       SCM tags = m->get_mus_property ("tags");
+       if (gh_symbol_p (tag))
+               tags = scm_cons (tag, tags);
+       else if (gh_list_p (tag))
+               tags = gh_append2 (tag, tags);
+       else
+               ip.warning (_("Tag must be symbol or list of symbols."));
+
+       m->set_mus_property ("tags", tags);
+}
 
 
 
@@ -210,72 +234,73 @@ yylex (YYSTYPE *s,  void * v)
 
 %pure_parser
 
-/* tokens which are not keywords */
-%token AUTOCHANGE
+
+%token ACCEPTS
+%token ADDLYRICS
 %token ALIAS
+%token ALTERNATIVE
+%token APPLY
 %token APPLYCONTEXT
 %token APPLYOUTPUT
-%token APPLY
-%token ACCEPTS
-%token ALTERNATIVE
+%token AUTOCHANGE
 %token BAR
 %token BREATHE
 %token CHORDMODIFIERS  
 %token CHORDS
+%token LESSLESS
+%token MOREMORE
 %token CLEF
+%token COMMANDSPANREQUEST
 %token CONSISTS
-%token DURATION
-%token SEQUENTIAL
-%token GROBDESCRIPTIONS
-%token SIMULTANEOUS
 %token CONSISTSEND
+%token CONTEXT
+%token DEFAULT
 %token DENIES
 %token DESCRIPTION
 %token EXTENDER
 %token FIGURES FIGURE_OPEN FIGURE_CLOSE
 %token FIGURE_BRACKET_CLOSE FIGURE_BRACKET_OPEN
 %token GRACE 
+%token ACCIACCATURA
+%token APPOGGIATURA 
+%token GROBDESCRIPTIONS
 %token HEADER
 %token HYPHEN
 %token INVALID
 %token KEY
 %token LYRICS
 %token MARK
-%token MULTI_MEASURE_REST
 %token MIDI
-%token PITCH
-%token DEFAULT
+%token MULTI_MEASURE_REST
 %token NAME
-%token PITCHNAMES
+%token NEWCONTEXT
 %token NOTES
+%token OCTAVE
 %token ONCE
+%token OVERRIDE SET REVERT 
 %token PAPER
+%token PARTCOMBINE
 %token PARTIAL
+%token PITCHNAMES
 %token PROPERTY
-%token OVERRIDE SET REVERT 
 %token RELATIVE
 %token REMOVE
 %token REPEAT
-%token ADDLYRICS
-%token PARTCOMBINE
+%token REST
 %token SCM_T
 %token SCORE
+%token SEQUENTIAL
+%token SIMULTANEOUS
 %token SKIP
 %token SPANREQUEST
-%token COMMANDSPANREQUEST
+%token TAG
 %token TEMPO
-%token OUTPUTPROPERTY
-%token TIME_T
 %token TIMES
+%token TIME_T
 %token TRANSLATOR
 %token TRANSPOSE
 %token TYPE
 %token UNSET
-%token CONTEXT
-%token REST
-
-%token CHORD_OPEN
-%token CHORD_CLOSE
 
 
 /* escaped */
@@ -288,6 +313,8 @@ yylex (YYSTYPE *s,  void * v)
 
 %type <i>      exclamations questions dots optional_rest
 %type <i>       bass_mod
+%type <scm>    grace_head
+%type <scm>    lyric_element
 %type <scm>    bass_number br_bass_figure bass_figure figure_list figure_spec
 %token <i>     DIGIT
 %token <scm>   NOTENAME_PITCH
@@ -326,8 +353,7 @@ yylex (YYSTYPE *s,  void * v)
 
 %type <outputdef> output_def
 %type <scm>    lilypond_header lilypond_header_body
-%type <music>  open_event_parens close_event_parens open_event close_event
-%type <music> event_with_dir event_that_take_dir verbose_event
+%type <music>  open_event close_event
 %type <i>      sub_quotes sup_quotes
 %type <music>  simple_element  event_chord command_element Simple_music  Composite_music 
 %type <music>  Repeated_music
@@ -340,16 +366,11 @@ yylex (YYSTYPE *s,  void * v)
 %type <music> note_chord_element chord_body chord_body_element
 %type <scm>  chord_body_elements 
 %type <scm> steno_duration optional_notemode_duration multiplied_duration
-%type <scm>  verbose_duration
        
-%type <scm>  pre_events post_events
-%type <music> gen_text_def
+%type <scm>   post_events 
+%type <music> gen_text_def direction_less_event direction_reqd_event
 %type <scm>   steno_pitch pitch absolute_pitch pitch_also_in_chords
-%type <scm>   explicit_pitch steno_tonic_pitch
-
-/* %type <scm> chord_additions chord_subtractions chord_notes chord_step */ 
-/* %type <music>       chord */
-/* %type <scm> chord_note chord_inversion chord_bass */
+%type <scm>    steno_tonic_pitch
 %type <scm>    duration_length fraction
 
 %type <scm> new_chord step_number chord_items chord_item chord_separator step_numbers
@@ -361,11 +382,11 @@ yylex (YYSTYPE *s,  void * v)
 %type <scm> Music_list
 %type <outputdef>  music_output_def_body
 %type <music> shorthand_command_req
-%type <music>  post_event 
+%type <music>  post_event tagged_post_event
 %type <music> command_req verbose_command_req
 %type <music>  extender_req
 %type <music> hyphen_req
-%type <music> string_event
+%type <music> string_number_event
 %type <scm>    string bare_number number_expression number_term number_factor 
 %type <score>  score_block score_body
 
@@ -518,9 +539,6 @@ identifier_init:
                $$ = $1->self_scm ();
                scm_gc_unprotect_object ($$);
        }
-       | verbose_duration {
-               $$ = $1;
-       }
        | number_expression {
                $$ = $1;
        }
@@ -569,14 +587,14 @@ translator_spec_body:
                scm_string_to_symbol ($2), $4);
        }
        | translator_spec_body NAME STRING  {
-               unsmob_translator_def ($$)->type_name_ = $3;
+               unsmob_translator_def ($$)->type_name_ = scm_string_to_symbol ($3);
        }
        | translator_spec_body CONSISTS STRING  {
                unsmob_translator_def ($$)->add_element ($3);
        }
        | translator_spec_body ALIAS STRING  {
                Translator_def*td = unsmob_translator_def ($$);
-               td->type_aliases_ = scm_cons ($3, td->type_aliases_);
+               td->type_aliases_ = scm_cons (scm_string_to_symbol ($3), td->type_aliases_);
        }
        | translator_spec_body GROBDESCRIPTIONS embedded_scm {
                Translator_def*td = unsmob_translator_def($$);
@@ -588,10 +606,10 @@ translator_spec_body:
                unsmob_translator_def ($$)->add_last_element ( $3);
        }
        | translator_spec_body ACCEPTS STRING  {
-               unsmob_translator_def ($$)->set_acceptor ($3,true);
+               unsmob_translator_def ($$)->set_acceptor (scm_string_to_symbol ($3), true);
        }
        | translator_spec_body DENIES STRING  {
-               unsmob_translator_def ($$)->set_acceptor ($3,false);
+               unsmob_translator_def ($$)->set_acceptor (scm_string_to_symbol ($3), false);
        }
        | translator_spec_body REMOVE STRING  {
                unsmob_translator_def ($$)->remove_element ($3);
@@ -847,7 +865,7 @@ Simultaneous_music:
                $$->set_spot(THIS->here_input());
 
        }
-       | '<' Music_list '>'    {
+       | simul_open Music_list simul_close     {
                $$ = MY_MAKE_MUSIC("SimultaneousMusic");
                $$->set_mus_property ("elements", ly_car ($2));
                $$->set_spot(THIS->here_input());
@@ -870,25 +888,6 @@ Simple_music:
                $$->set_mus_property ("procedure", $2);
                $$->set_spot (THIS->here_input());
        }
-       | OUTPUTPROPERTY embedded_scm embedded_scm '=' embedded_scm     {
-               SCM pred = $2;
-               if (!gh_symbol_p ($3))
-               {
-                       THIS->parser_error (_ ("Second argument must be a symbol")); 
-               }
-               /* Should check # args */
-               if (!gh_procedure_p (pred))
-               {
-                       THIS->parser_error (_ ("First argument must be a procedure taking one argument"));
-               }
-
-               Music*m = MY_MAKE_MUSIC("OutputPropertySetMusic");
-               m->set_mus_property ("predicate", pred);
-               m->set_mus_property ("grob-property", $3);
-               m->set_mus_property ("grob-value",  $5);
-
-               $$ = m;
-       }
        | MUSIC_IDENTIFIER {
                $$ = unsmob_music ($1);
        }
@@ -897,6 +896,13 @@ Simple_music:
        ;
 
 
+grace_head:
+       GRACE  { $$ = scm_makfrom0str ("Grace"); } 
+       | ACCIACCATURA { $$ = scm_makfrom0str ("Acciaccatura"); }
+       | APPOGGIATURA { $$ = scm_makfrom0str ("Appoggiatura"); }
+       ;
+       
+
 Composite_music:
        CONTEXT STRING Music    {
                Music*csm =MY_MAKE_MUSIC("ContextSpeccedMusic");
@@ -904,7 +910,7 @@ Composite_music:
                csm->set_mus_property ("element", $3->self_scm ());
                scm_gc_unprotect_object ($3->self_scm ());
 
-               csm->set_mus_property ("context-type",$2);
+               csm->set_mus_property ("context-type", scm_string_to_symbol ($2));
                csm->set_mus_property ("context-id", scm_makfrom0str (""));
 
                $$ = csm;
@@ -915,20 +921,31 @@ Composite_music:
                chm->set_mus_property ("iterator-ctor", Auto_change_iterator::constructor_proc);
 
                scm_gc_unprotect_object ($3->self_scm ());
-               chm->set_mus_property ("what", $2); 
+               chm->set_mus_property ("what", scm_string_to_symbol ($2));
 
                $$ = chm;
                chm->set_spot (*$3->origin ());
        }
-       | GRACE Music {
+       | grace_head Music {
 #if 1
        /*
                The other version is for easier debugging  of
                Sequential_music_iterator in combination with grace notes.
        */
 
-               SCM start = THIS->lexer_->lookup_identifier ("startGraceMusic");
-               SCM stop = THIS->lexer_->lookup_identifier ("stopGraceMusic");
+/*
+
+TODO: should distinguish between both grace types in the
+basic music objects too, since the meaning is different.
+
+*/
+
+               String start_str = "start" + ly_scm2string ($1) + "Music"; 
+               String stop_str = "stop" + ly_scm2string ($1) + "Music"; 
+               
+               SCM start = THIS->lexer_->lookup_identifier (start_str);
+               SCM stop = THIS->lexer_->lookup_identifier (stop_str);
+
                Music *startm = unsmob_music (start);
                Music *stopm = unsmob_music (stop);
 
@@ -966,11 +983,28 @@ Composite_music:
                csm->set_mus_property ("element", $5->self_scm ());
                scm_gc_unprotect_object ($5->self_scm ());
 
-               csm->set_mus_property ("context-type", $2);
+               csm->set_mus_property ("context-type", scm_string_to_symbol ($2));
                csm->set_mus_property ("context-id", $4);
 
                $$ = csm;
        }
+       | NEWCONTEXT string Music {
+               static int new_context_count;
+
+               Music * csm = MY_MAKE_MUSIC("ContextSpeccedMusic");
+
+               csm->set_mus_property ("element", $3->self_scm ());
+               scm_gc_unprotect_object ($3->self_scm ());
+
+               csm->set_mus_property ("context-type", scm_string_to_symbol ($2));
+
+               char s[1024];
+               snprintf (s, 1024, "uniqueContext%d", new_context_count ++);
+               
+               SCM new_id = scm_makfrom0str (s);
+               csm->set_mus_property ("context-id", new_id);
+               $$ = csm;
+       }
        | TIMES {
                THIS->push_spot ();
        }
@@ -980,9 +1014,9 @@ Composite_music:
        {
                int n = gh_scm2int (ly_car ($3)); int d = gh_scm2int (ly_cdr ($3));
                Music *mp = $4;
-       $$= MY_MAKE_MUSIC("TimeScaledMusic");
-               $$->set_spot (THIS->pop_spot ());
 
+               $$= MY_MAKE_MUSIC("TimeScaledMusic");
+               $$->set_spot (THIS->pop_spot ());
 
                $$->set_mus_property ("element", mp->self_scm ());
                scm_gc_unprotect_object (mp->self_scm ());
@@ -1006,15 +1040,20 @@ Composite_music:
        }
        | APPLY embedded_scm Music  {
                if (!ly_input_procedure_p ($2))
+                       {
                        THIS->parser_error (_ ("\\apply takes function argument"));
-               
-               SCM ret = gh_call1 ($2, $3->self_scm ());
-               Music *m = unsmob_music (ret);
-               if (!m) {
-                       THIS->parser_error ("\\apply must return a Music");
-                       m = MY_MAKE_MUSIC("Music");
+                       $$ = $3;
+                       }
+               else
+                       {
+                       SCM ret = gh_call1 ($2, $3->self_scm ());
+                       Music *m = unsmob_music (ret);
+                       if (!m) {
+                               THIS->parser_error ("\\apply must return a Music");
+                               m = MY_MAKE_MUSIC("Music");
+                               }
+                       $$ = m;
                        }
-               $$ = m;
        }
        | NOTES
                { THIS->lexer_->push_note_state (); }
@@ -1053,7 +1092,11 @@ Composite_music:
        }
        | relative_music        { $$ = $1; }
        | re_rhythmed_music     { $$ = $1; } 
-       | part_combined_music   { $$ = $1; } 
+       | part_combined_music   { $$ = $1; }
+       | TAG embedded_scm Music {
+               tag_music ($3, $2, THIS->here_input ());
+               $$ = $3;
+       }
        ;
 
 relative_music:
@@ -1066,8 +1109,9 @@ relative_music:
                scm_gc_unprotect_object (p->self_scm ());
 
 
-               $$->set_mus_property ("last-pitch", p->to_relative_octave (pit).smobbed_copy ());
-
+               Pitch retpitch = p->to_relative_octave (pit);
+               if (lily_1_8_relative)
+                       $$->set_mus_property ("last-pitch", retpitch.smobbed_copy ());
        }
        ;
 
@@ -1083,8 +1127,8 @@ re_rhythmed_music:
 
 part_combined_music:
        PARTCOMBINE STRING Music Music {
-       Music * p= MY_MAKE_MUSIC("PartCombineMusic");
-               p->set_mus_property ("what", $2);
+               Music * p= MY_MAKE_MUSIC("PartCombineMusic");
+               p->set_mus_property ("what", scm_string_to_symbol ($2));
                p->set_mus_property ("elements", gh_list ($3->self_scm (),$4->self_scm (), SCM_UNDEFINED));  
 
                scm_gc_unprotect_object ($3->self_scm ());
@@ -1097,7 +1141,7 @@ part_combined_music:
 translator_change:
        TRANSLATOR STRING '=' STRING  {
                Music*t= MY_MAKE_MUSIC("TranslatorChange");
-               t-> set_mus_property ("change-to-type", $2);
+               t-> set_mus_property ("change-to-type", scm_string_to_symbol ($2));
                t-> set_mus_property ("change-to-id", $4);
 
                $$ = t;
@@ -1125,7 +1169,7 @@ simple_property_def:
                $$ = csm;
                $$->set_spot (THIS->here_input ());
 
-               csm-> set_mus_property ("context-type", $2);
+               csm-> set_mus_property ("context-type", scm_string_to_symbol ($2));
        }
        | PROPERTY STRING '.' STRING UNSET {
                
@@ -1139,7 +1183,7 @@ simple_property_def:
                $$ = csm;
                $$->set_spot (THIS->here_input ());
 
-               csm-> set_mus_property ("context-type", $2);
+               csm-> set_mus_property ("context-type", scm_string_to_symbol ($2));
        }
        | PROPERTY STRING '.' STRING SET embedded_scm '=' embedded_scm {
                bool autobeam
@@ -1161,7 +1205,7 @@ simple_property_def:
                $$ = csm;
                $$->set_spot (THIS->here_input ());
 
-               csm-> set_mus_property ("context-type", $2);
+               csm-> set_mus_property ("context-type", scm_string_to_symbol ($2));
        }
        | PROPERTY STRING '.' STRING OVERRIDE
                embedded_scm '=' embedded_scm
@@ -1189,7 +1233,7 @@ simple_property_def:
                $$ = csm;
                $$->set_spot (THIS->here_input ());
 
-               csm-> set_mus_property ("context-type", $2);
+               csm-> set_mus_property ("context-type", scm_string_to_symbol ($2));
 
        }
        | PROPERTY STRING '.' STRING REVERT embedded_scm {
@@ -1212,7 +1256,7 @@ simple_property_def:
                $$ = csm;
                $$->set_spot (THIS->here_input ());
 
-               csm-> set_mus_property ("context-type", $2);
+               csm-> set_mus_property ("context-type", scm_string_to_symbol ($2));
        }
        ;
 
@@ -1226,17 +1270,38 @@ scalar:
         ;
 
 
-event_chord:
-       pre_events {
+
+/*
+This is a trick:
+
+Adding pre_events to the simple_element
+makes the choice between
+
+  string:  STRING
+
+and
+
+  simple_element: STRING
+
+a single shift/reduction conflict.
+
+nevertheless, this is not very clean, and we should find a different
+solution.  
+
+*/
+pre_events: {
                THIS->push_spot ();
-       } /*cont */ simple_element post_events  {
-               SCM elts = $3-> get_mus_property ("elements");
+       }
+       ;
 
-               elts = gh_append3 (elts, scm_reverse_x ($1, SCM_EOL),
-                                  scm_reverse_x ($4, SCM_EOL));
+event_chord:
+       pre_events simple_element post_events   {
+               SCM elts = $2-> get_mus_property ("elements");
 
-               $3-> set_mus_property ("elements", elts);
-               $$ = $3;
+               elts = gh_append2 (elts, scm_reverse_x ($3, SCM_EOL));
+
+               $2->set_mus_property ("elements", elts);
+               $$ = $2;
        }
        | command_element
        | note_chord_element
@@ -1259,8 +1324,20 @@ note_chord_element:
        }
        ;
 
+chord_open: '<'
+       ;
+
+chord_close: '>'
+       ;
+
+simul_open: LESSLESS
+       ;
+
+simul_close: MOREMORE
+       ;
+
 chord_body:
-       CHORD_OPEN chord_body_elements CHORD_CLOSE
+       chord_open chord_body_elements chord_close
        {
                $$ = MY_MAKE_MUSIC("EventChord");
                $$->set_mus_property ("elements",
@@ -1297,11 +1374,18 @@ command_element:
        command_req {
                $$ = MY_MAKE_MUSIC("EventChord");
                $$->set_mus_property ("elements", scm_cons ($1->self_scm (), SCM_EOL));
-         scm_gc_unprotect_object ($1->self_scm());
+               scm_gc_unprotect_object ($1->self_scm());
 
                $$-> set_spot (THIS->here_input ());
                $1-> set_spot (THIS->here_input ());
        }
+       | OCTAVE { THIS->push_spot (); }
+         pitch {
+               Music *l = MY_MAKE_MUSIC("RelativeOctaveCheck");
+               $$ = l;
+               $$->set_spot (THIS->pop_spot ());
+               $$->set_mus_property ("pitch", $3);
+       }
        | E_LEFTSQUARE {
                Music *l = MY_MAKE_MUSIC("LigatureEvent");
                l->set_mus_property ("span-direction", gh_int2scm (START));
@@ -1341,7 +1425,7 @@ command_element:
                $$ = csm;
                $$->set_spot (THIS->here_input ());
 
-               csm->set_mus_property ("context-type", scm_makfrom0str ("Timing"));
+               csm->set_mus_property ("context-type", ly_symbol2scm ("Timing"));
        }
        | PARTIAL duration_length       {
                Moment m = - unsmob_duration ($2)->get_length ();
@@ -1352,7 +1436,7 @@ command_element:
                scm_gc_unprotect_object (p->self_scm ());
 
                $$ =sp ;
-               sp-> set_mus_property ("context-type", scm_makfrom0str ("Timing"));
+               sp-> set_mus_property ("context-type", ly_symbol2scm ("Timing"));
        }
        | CLEF STRING  {
                static SCM proc ;
@@ -1386,21 +1470,6 @@ shorthand_command_req:
        | hyphen_req {
                $$ = $1;
        }
-       | '~'   {
-               $$ = MY_MAKE_MUSIC("TieEvent");
-       }
-       | '['           {
-               Music *b= MY_MAKE_MUSIC("BeamEvent");
-               b->set_mus_property ("span-direction", gh_int2scm (START));
-               $$ = b;
-
-               THIS->last_beam_start_ = b->self_scm ();
-       }
-       | ']'           {
-               Music *b= MY_MAKE_MUSIC("BeamEvent");
-               b->set_mus_property ("span-direction", gh_int2scm (STOP));
-               $$ = b;
-       }
        | BREATHE {
                $$ = MY_MAKE_MUSIC("BreathingSignEvent");
        }
@@ -1433,11 +1502,16 @@ verbose_command_req:
                $$ = key;
        }
        | KEY NOTENAME_PITCH SCM_IDENTIFIER     {
+
                Music *key= MY_MAKE_MUSIC("KeyChangeEvent");
-               
-               key->set_mus_property ("pitch-alist", $3);
-               key->set_mus_property ("tonic", Pitch (0,0,0).smobbed_copy());
-               ((Music*)key)->transpose (* unsmob_pitch ($2));
+               if (scm_ilength ($3) > 0)
+               {               
+                       key->set_mus_property ("pitch-alist", $3);
+                       key->set_mus_property ("tonic", Pitch (0,0,0).smobbed_copy());
+                       ((Music*)key)->transpose (* unsmob_pitch ($2));
+               } else {
+                       THIS->parser_error (_("Second argument must be pitch list."));
+               }
 
                $$ = key;
        }
@@ -1452,16 +1526,39 @@ post_events:
                $$ = gh_cons ($2->self_scm(), $$);
                scm_gc_unprotect_object ($2->self_scm());
        }
+       | post_events tagged_post_event {
+               $2 -> set_spot (THIS->here_input ());
+               $$ = scm_cons ($2->self_scm(), $$);
+               scm_gc_unprotect_object ($2->self_scm());
+       }
+       ;
+
+
+tagged_post_event:
+       '-' TAG embedded_scm post_event {
+               tag_music ($4, $3, THIS->here_input ());
+               $$ = $4;
+       }
        ;
 
 post_event:
-       verbose_event
-       | event_with_dir
-       | close_event
-       | string_event
+       direction_less_event {
+               $$ = $1;
+       }
+       | script_dir direction_reqd_event {
+               if ($1)
+                       $2->set_mus_property ("direction", gh_int2scm ($1));
+               $$ = $2;
+       }
+       | script_dir direction_less_event {
+               if ($1)
+                       $2->set_mus_property ("direction", gh_int2scm ($1));
+               $$ = $2;
+       }
+       | string_number_event
        ;
 
-string_event:
+string_number_event:
        E_UNSIGNED {
                Music * s = MY_MAKE_MUSIC("StringNumberEvent");
                s->set_mus_property ("string-number",  gh_int2scm($1));
@@ -1471,47 +1568,44 @@ string_event:
        ;
 
 
-event_that_take_dir:
-       gen_text_def
-       | verbose_event
-       | close_event
-       | open_event
-       | '['  {
-               Music * m = MY_MAKE_MUSIC ("NewBeamEvent");
+direction_less_event:
+       '['  {
+
+
+/*
+
+TODO: should take all these defs out of the parser, adn make use
+configurable, i.e.
+
+
+(set-articulation '~ "trill")
+
+*/
+               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;
        }
-       | script_abbreviation {
-               SCM s = THIS->lexer_->lookup_identifier ("dash" + ly_scm2string ($1));
-               Music *a = MY_MAKE_MUSIC("ArticulationEvent");
-               if (gh_string_p (s))
-                       a->set_mus_property ("articulation-type", s);
-               else THIS->parser_error (_ ("Expecting string as script definition"));
-               $$ = a;
+       | close_event {
+               $$ = $1;
+               dynamic_cast<Music *> ($$)->set_mus_property ("span-direction", gh_int2scm (START));
        }
-       ;
-
-event_with_dir:
-       script_dir event_that_take_dir  {
-               $2->set_mus_property ("direction", gh_int2scm ($1));
-               $$ = $2;
+       | open_event {
+               $$ = $1;
+               dynamic_cast<Music *> ($$)->set_mus_property ("span-direction", gh_int2scm (STOP));
        }
-       ;
-       
-verbose_event:
-       EVENT_IDENTIFIER        {
+       | EVENT_IDENTIFIER      {
                $$ = unsmob_music ($1);
        }
        | tremolo_type  {
@@ -1520,8 +1614,22 @@ verbose_event:
                a->set_mus_property ("tremolo-type", gh_int2scm ($1));
                $$ = a;
         }
+       ;       
+       
+direction_reqd_event:
+       gen_text_def {
+               $$ = $1; 
+       }
+       | script_abbreviation {
+               SCM s = THIS->lexer_->lookup_identifier ("dash" + ly_scm2string ($1));
+               Music *a = MY_MAKE_MUSIC("ArticulationEvent");
+               if (gh_string_p (s))
+                       a->set_mus_property ("articulation-type", s);
+               else THIS->parser_error (_ ("Expecting string as script definition"));
+               $$ = a;
+       }
        ;
-
+       
 sup_quotes:
        '\'' {
                $$ = 1;
@@ -1581,9 +1689,6 @@ pitch:
        steno_pitch {
                $$ = $1;
        }
-       | explicit_pitch {
-               $$ = $1;
-       }
        ;
 
 pitch_also_in_chords:
@@ -1591,26 +1696,9 @@ pitch_also_in_chords:
        | steno_tonic_pitch
        ;
 
-explicit_pitch:
-       PITCH embedded_scm {
-               $$ = $2;
-               if (!unsmob_pitch ($2)) {
-                       THIS->parser_error (_f ("Expecting musical-pitch value", 3));
-                        $$ = Pitch ().smobbed_copy ();
-               }
-       }
-       ;
 
-verbose_duration:
-       DURATION embedded_scm   {
-               $$ = $2;
-               if (!unsmob_duration ($2))
-               {
-                       THIS->parser_error (_ ("Must have duration object"));
-                       $$ = Duration ().smobbed_copy ();
-               }
-       }
-       ;
+
+
 
 extender_req:
        EXTENDER {
@@ -1629,14 +1717,6 @@ hyphen_req:
        ;
 
 close_event:
-       close_event_parens {
-               $$ = $1;
-               dynamic_cast<Music *> ($$)->set_mus_property ("span-direction", gh_int2scm (START))
-;
-       }
-       ;
-close_event_parens:
        '('     {
                Music * s= MY_MAKE_MUSIC("SlurEvent");
                $$ = s;
@@ -1661,14 +1741,6 @@ close_event_parens:
 
 
 open_event:
-       open_event_parens {
-               $$ = $1;
-               dynamic_cast<Music *> ($$)->set_mus_property ("span-direction", gh_int2scm (STOP))
-;
-       }
-       ;
-
-open_event_parens:
        E_EXCLAMATION   {
                Music *s =  MY_MAKE_MUSIC("CrescendoEvent");
                s->set_spot (THIS->here_input());
@@ -1690,13 +1762,7 @@ open_event_parens:
        ;
 
 gen_text_def:
-       embedded_scm {
-               Music *t = MY_MAKE_MUSIC("TextScriptEvent");
-               t->set_mus_property ("text", $1);
-               t->set_spot (THIS->here_input ());
-               $$ = t;
-       }
-       | full_markup {
+       full_markup {
                Music *t = MY_MAKE_MUSIC("TextScriptEvent");
                t->set_mus_property ("text", $1);
                t->set_spot (THIS->here_input ());
@@ -1747,15 +1813,6 @@ script_dir:
        | '-'   { $$ = CENTER; }
        ;
 
-pre_events:
-       /* empty */ { 
-               $$ = SCM_EOL;
-       }
-       | pre_events open_event {
-               $$ = gh_cons ($2->self_scm(), $$);
-               scm_gc_unprotect_object ($2->self_scm());
-       }
-       ;
 
 absolute_pitch:
        steno_pitch     {
@@ -1767,9 +1824,6 @@ duration_length:
        multiplied_duration {
                $$ = $1;
        }
-       | verbose_duration {
-               $$ = $1;
-       }       
        ;
 
 optional_notemode_duration:
@@ -1785,10 +1839,6 @@ optional_notemode_duration:
 
                THIS->beam_check ($$);
        }
-       | verbose_duration {
-               $$ = $1;
-               THIS->default_duration_ = *unsmob_duration ($$);
-       }       
        ;
 
 steno_duration:
@@ -1867,7 +1917,7 @@ bass_number:
        | UNSIGNED {
                $$ = scm_number_to_string (gh_int2scm ($1), gh_int2scm (10));
        }
-       | STRING { $$ =  $1 }
+       | STRING { $$ =  $1; }
        ;
 
 bass_mod:
@@ -2013,8 +2063,11 @@ simple_element:
                scm_gc_protect_object (mus);
                $$ = unsmob_music (mus);
        }
-       | STRING optional_notemode_duration     {
+       
+       | lyric_element optional_notemode_duration      {
                Input i = THIS->pop_spot ();
+               if (!THIS->lexer_->lyric_state_b ())
+                       THIS->parser_error (_ ("Have to be in Lyric mode for lyrics"));
 
                Music * lreq = MY_MAKE_MUSIC("LyricEvent");
                 lreq->set_mus_property ("text", $1);
@@ -2034,6 +2087,11 @@ simple_element:
        }
        ;
 
+lyric_element:
+       full_markup { $$ = $1; }
+       | STRING {  $$ = $1 ; }
+       ;
+
 new_chord:
        steno_tonic_pitch optional_notemode_duration   {
                $$ = make_chord ($1, $2, SCM_EOL);
@@ -2101,6 +2159,9 @@ step_number:
 
 /*
        UTILITIES
+
+TODO: should deprecate in favor of Scheme?
+
  */
 number_expression:
        number_expression '+' number_term {
@@ -2254,7 +2315,7 @@ markup:
        ;
 
 markup_list:
-       CHORD_OPEN markup_list_body CHORD_CLOSE { $$ = scm_reverse_x ($2, SCM_EOL); }
+       chord_open markup_list_body chord_close { $$ = scm_reverse_x ($2, SCM_EOL); }
        ;
 
 markup_line:
@@ -2320,13 +2381,6 @@ My_lily_parser::beam_check (SCM dur)
 
 
 
-bool
-markup_p (SCM x)
-{
-       return gh_pair_p (x)
-               && SCM_BOOL_F != scm_object_property (gh_car (x), ly_symbol2scm ("markup-signature"));
-}
-
 
 /*
 
@@ -2365,7 +2419,7 @@ My_lily_lexer::try_special_identifiers (SCM * destination, SCM sid)
 
                *destination = p->self_scm();
                return MUSIC_OUTPUT_DEF_IDENTIFIER;
-       } else if (new_markup_p (sid)) {
+       } else if (Text_item::markup_p (sid)) {
                *destination = sid;
                return MARKUP_IDENTIFIER;
        }