]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/parser.yy
release: 1.3.146
[lilypond.git] / lily / parser.yy
index 2cf1bf94a3803c3aae1aea74a1b1a601abf7b730..33a75d50c89654c2728ae87f3cafb1746b9fa6cb 100644 (file)
@@ -5,7 +5,7 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997--2000 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c)  1997--2001 Han-Wen Nienhuys <hanwen@cs.uu.nl>
            Jan Nieuwenhuizen <janneke@gnu.org>
 */
 
 #include "repeated-music.hh"
 #include "lilypond-input-version.hh"
 #include "grace-music.hh"
-#include "auto-change-music.hh"
 #include "part-combine-music.hh"
 #include "scm-hash.hh"
-
+#include "auto-change-iterator.hh"
+#include "un-relativable-music.hh"
 #include "chord.hh"
 
 bool
 is_duration_b (int t)
 {
-  return t && t == 1 << intlog2(t);
+  return t && t == 1 << intlog2 (t);
 }
 
 
@@ -73,7 +73,7 @@ print_lilypond_versions (ostream &os)
 }
 
 
-// needed for bison.simple's malloc() and free()
+// needed for bison.simple's malloc () and free ()
 #include <malloc.h>
 
 #ifndef NDEBUG
@@ -84,7 +84,8 @@ print_lilypond_versions (ostream &os)
 
 #define YYPARSE_PARAM my_lily_parser_l
 #define YYLEX_PARAM my_lily_parser_l
-#define THIS ((My_lily_parser *) my_lily_parser_l)
+#define THIS\
+       ((My_lily_parser *) my_lily_parser_l)
 
 #define yyerror THIS->parser_error
 #define ARRAY_SIZE(a,s)   if (a.size () != s) THIS->parser_error (_f ("Expecting %d arguments", s))
@@ -95,16 +96,14 @@ print_lilypond_versions (ostream &os)
 %union {
 
     Link_array<Request> *reqvec;
-    String * string;
+
+    String *string; // needed by the lexer as temporary scratch area.
     Music *music;
     Score *score;
     Scope *scope;
     Scheme_hash_table *scmhash;
-    Musical_req* musreq;
     Music_output_def * outputdef;
 
-    Midi_def* midi;
-    Real real;
     Request * request;
 
     /* We use SCMs to do strings, because it saves us the trouble of
@@ -135,10 +134,10 @@ yylex (YYSTYPE *s,  void * v_l)
 
 /* tokens which are not keywords */
 %token AUTOCHANGE
+%token ALIAS
 %token APPLY
 %token ARPEGGIO
 %token DYNAMICSCRIPT
-%token TEXTSCRIPT
 %token ACCEPTS
 %token ALTERNATIVE
 %token BAR
@@ -167,10 +166,11 @@ yylex (YYSTYPE *s,  void * v_l)
 %token KEY
 %token LYRICS
 %token MARK
-%token MEASURES
+%token MULTI_MEASURE_REST
 %token MIDI
 %token MM_T
-%token MUSICAL_PITCH
+%token PITCH
+%token DEFAULT
 %token NAME
 %token PITCHNAMES
 %token NOTES
@@ -199,10 +199,11 @@ yylex (YYSTYPE *s,  void * v_l)
 %token TRANSLATOR
 %token TRANSPOSE
 %token TYPE
+%token UNSET
 %token CONTEXT
 
 /* escaped */
-%token E_CHAR E_EXCLAMATION E_SMALLER E_BIGGER 
+%token E_CHAR E_EXCLAMATION E_SMALLER E_BIGGER E_OPEN E_CLOSE
 %token CHORD_BASS CHORD_COLON CHORD_MINUS CHORD_CARET 
 
 %type <i>      exclamations questions dots
@@ -222,10 +223,10 @@ yylex (YYSTYPE *s,  void * v_l)
 %token <scm>   MUSIC_IDENTIFIER TRANSLATOR_IDENTIFIER
 %token <scm>   STRING_IDENTIFIER SCM_IDENTIFIER 
 %token <scm>   RESTNAME
-%token <scm>   STRING 
+%token <scm>   STRING   
 %token <scm>   SCM_T
 %token <i>     UNSIGNED
-%token <real>   REAL
+%token <scm>   REAL
 
 %type <outputdef> output_def
 %type <scmhash>        lilypond_header lilypond_header_body
@@ -240,13 +241,13 @@ yylex (YYSTYPE *s,  void * v_l)
 
 %type <scm>    identifier_init 
 
-%type <scm> steno_duration optional_notemode_duration
-%type <scm> entered_notemode_duration explicit_duration
+%type <scm> steno_duration optional_notemode_duration multiplied_duration
+%type <scm>  explicit_duration
        
 %type <reqvec>  pre_requests post_requests
 %type <request> gen_text_def
-%type <scm>   steno_musical_pitch musical_pitch absolute_musical_pitch
-%type <scm>   steno_tonic_pitch
+%type <scm>   steno_pitch pitch absolute_pitch
+%type <scm>   explicit_pitch steno_tonic_pitch
 
 %type <scm>    chord_additions chord_subtractions chord_notes chord_step
 %type <music>  chord
@@ -265,6 +266,7 @@ yylex (YYSTYPE *s,  void * v_l)
 %type <request>        extender_req
 %type <request> hyphen_req
 %type <scm>    string bare_number number_expression
+
 %type <score>  score_block score_body
 
 %type <scm>    translator_spec_block translator_spec_body
@@ -347,7 +349,7 @@ notenames_body:
                if (!unsmob_pitch (pt))
                        THIS->parser_error ("Need pitch object.");
                else
-                       scm_hashq_set_x (tab, gh_caar(s), pt);
+                       scm_hashq_set_x (tab, gh_caar (s), pt);
          }
 
          $$ = tab;
@@ -361,7 +363,7 @@ lilypond_header_body:
                Scope *sc = new Scope ($$);
                THIS->lexer_p_-> scope_l_arr_.push (sc);
        }
-       | lilypond_header_body assignment semicolon { 
+       | lilypond_header_body assignment  { 
 
        }
        ;
@@ -446,7 +448,7 @@ translator_spec_body:
                $$ = unsmob_translator_def ($1)->clone_scm ();
                unsmob_translator_def ($$)-> set_spot (THIS->here_input ());
        }
-       | TYPE STRING semicolon {
+       | TYPE STRING   {
                $$ = Translator_def::make_scm ();
                Translator_def*td =  unsmob_translator_def ($$);
                td->translator_group_type_ = $2;
@@ -460,41 +462,35 @@ translator_spec_body:
                        ->add_push_property (scm_string_to_symbol ($2), $4, $6);
        }
        | translator_spec_body STRING REVERT embedded_scm  {
-         unsmob_translator_def($$)->add_pop_property (
+         unsmob_translator_def ($$)->add_pop_property (
                scm_string_to_symbol ($2), $4);
        }
-       | translator_spec_body STRING '=' identifier_init semicolon     { 
-               SCM v = gh_int2scm (0);
-               if (gh_string_p ($4) || gh_number_p ($4) || gh_boolean_p ($4))
-                       v = $4;
-               else 
-                       THIS->parser_error (_("Wrong type for property value"));
-
-               /* ugh*/
-               unsmob_translator_def($$)->add_property_assign ($2, v);
-       }
-       | translator_spec_body NAME STRING semicolon {
+       | translator_spec_body NAME STRING  {
                unsmob_translator_def ($$)->type_name_ = $3;
        }
-       | translator_spec_body CONSISTS STRING semicolon {
+       | 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_ = gh_cons ($3, td->type_aliases_);
+       }
        | translator_spec_body ELEMENTDESCRIPTIONS embedded_scm {
                for (SCM p = $3; gh_pair_p (p); p = gh_cdr (p))
                        unsmob_translator_def ($$)
                        ->add_property_assign (scm_symbol_to_string (gh_caar (p)), gh_cdar (p));
 
        }
-       | translator_spec_body CONSISTSEND STRING semicolon {
+       | translator_spec_body CONSISTSEND STRING  {
                unsmob_translator_def ($$)->add_last_element ( $3);
        }
-       | translator_spec_body ACCEPTS STRING semicolon {
+       | translator_spec_body ACCEPTS STRING  {
                unsmob_translator_def ($$)->set_acceptor ($3,true);
        }
-       | translator_spec_body DENIES STRING semicolon {
+       | translator_spec_body DENIES STRING  {
                unsmob_translator_def ($$)->set_acceptor ($3,false);
        }
-       | translator_spec_body REMOVE STRING semicolon {
+       | translator_spec_body REMOVE STRING  {
                unsmob_translator_def ($$)->remove_element ($3);
        }
        ;
@@ -532,7 +528,7 @@ score_body:
                $$->set_spot (THIS->here_input ());
        }
        | score_body lilypond_header    {
-               scm_unprotect_object ( $1->self_scm ()); 
+               scm_unprotect_object ($2->self_scm ()); 
                $$->header_p_ = $2;
        }
        | score_body output_def {
@@ -550,7 +546,7 @@ score_body:
 output_def:
        music_output_def_body '}' {
                $$ = $1;
-               THIS-> lexer_p_-> scope_l_arr_.pop();
+               THIS-> lexer_p_-> scope_l_arr_.pop ();
        }
        ;
 
@@ -580,18 +576,18 @@ music_output_def_body:
        }
        | PAPER '{' MUSIC_OUTPUT_DEF_IDENTIFIER         {
                Music_output_def *p = unsmob_music_output_def ($3);
-               p = p->clone();
+               p = p->clone ();
                THIS->lexer_p_->scope_l_arr_.push (p->scope_p_);
                $$ = p;
        }
        | MIDI '{' MUSIC_OUTPUT_DEF_IDENTIFIER  {
                Music_output_def *p = unsmob_music_output_def ($3);
-               p = p->clone();
+               p = p->clone ();
 
                THIS->lexer_p_->scope_l_arr_.push (p->scope_p_);
                $$ = p;
        }
-       | music_output_def_body assignment semicolon {
+       | music_output_def_body assignment  {
 
        }
        | music_output_def_body translator_spec_block   {
@@ -600,7 +596,7 @@ music_output_def_body:
        | music_output_def_body STYLESHEET embedded_scm {
                dynamic_cast<Paper_def*> ($$)-> style_sheet_ = $3;
        }
-       | music_output_def_body tempo_request semicolon {
+       | music_output_def_body tempo_request  {
                /*
                        junk this ? there already is tempo stuff in
                        music.
@@ -629,8 +625,6 @@ Music_list: /* empty */ {
                SCM s = $$;
                SCM c = gh_cons ($2->self_scm (), SCM_EOL);
                scm_unprotect_object ($2->self_scm ()); /* UGH */
-
-       
                if (gh_pair_p (gh_cdr (s)))
                        gh_set_cdr_x (gh_cdr (s), c); /* append */
                else
@@ -661,7 +655,7 @@ Alternative_music:
 
 
 Repeated_music:
-       REPEAT STRING bare_unsigned Music Alternative_music
+       REPEAT string bare_unsigned Music Alternative_music
        {
                Music_sequence* alts = dynamic_cast <Music_sequence*> ($5);
                if (alts && $3 < alts->length_i ())
@@ -729,20 +723,20 @@ Simple_music:
                SCM pred = $2;
                if (!gh_symbol_p ($3))
                {
-                       THIS->parser_error (_("Second argument must be a symbol")); 
+                       THIS->parser_error (_ ("Second argument must be a symbol")); 
                }
                /*hould check # args */
                if (!gh_procedure_p (pred))
                {
-                       THIS->parser_error (_("First argument must be a procedure taking 1 argument"));
+                       THIS->parser_error (_ ("First argument must be a procedure taking 1 argument"));
                }
 
-         Music *m = new Music (SCM_EOL);
-         m->set_mus_property ("predicate", pred);
-         m->set_mus_property ("symbol", $3);
-         m->set_mus_property ("value",  $5);
-         m->set_mus_property ("iterator-ctor",
-                       Output_property_music_iterator::constructor_cxx_function);
+               Music *m = new Music (SCM_EOL);
+               m->set_mus_property ("predicate", pred);
+               m->set_mus_property ("symbol", $3);
+               m->set_mus_property ("value",  $5);
+               m->set_mus_property ("iterator-ctor",
+               Output_property_music_iterator::constructor_cxx_function);
 
                $$ = m;
        }
@@ -751,14 +745,6 @@ Simple_music:
        }
        | property_def
        | translator_change
-       | Simple_music '*' bare_unsigned '/' bare_unsigned      {
-               $$ = $1;
-               $$->compress (Moment($3, $5 ));
-       }
-       | Simple_music '*' bare_unsigned                 {
-               $$ = $1;
-               $$->compress (Moment ($3, 1));
-       }
        ;
 
 
@@ -774,8 +760,9 @@ Composite_music:
                $$ = csm;
        }
        | AUTOCHANGE STRING Music       {
-               Auto_change_music * chm = new Auto_change_music (SCM_EOL);
+               Music * chm = new Music_wrapper (SCM_EOL);
                chm->set_mus_property ("element", $3->self_scm ());
+               chm->set_mus_property ("iterator-ctor", Auto_change_iterator::constructor_cxx_function);
 
                scm_unprotect_object ($3->self_scm ());
                chm->set_mus_property ("what", $2); 
@@ -789,7 +776,7 @@ Composite_music:
                scm_unprotect_object ($2->self_scm ());
 
        }
-       | CONTEXT STRING '=' STRING Music {
+       | CONTEXT string '=' string Music {
                Context_specced_music *csm =  new Context_specced_music (SCM_EOL);
                csm->set_mus_property ("element", $5->self_scm ());
                scm_unprotect_object ($5->self_scm ());
@@ -822,7 +809,7 @@ Composite_music:
        | Repeated_music                { $$ = $1; }
        | Simultaneous_music            { $$ = $1; }
        | Sequential_music              { $$ = $1; }
-       | TRANSPOSE musical_pitch Music {
+       | TRANSPOSE pitch Music {
                $$ = new Transposed_music (SCM_EOL);
                Music *p = $3;
                Pitch pit = *unsmob_pitch ($2);
@@ -860,7 +847,10 @@ Composite_music:
                { THIS->lexer_p_->push_chord_state (); }
        Music
                {
-                 $$ = $3;
+                 Music * chm = new Un_relativable_music ;
+                 chm->set_mus_property ("element", $3->self_scm ());
+                 $$ = chm;
+
                  THIS->lexer_p_->pop_state ();
        }
        | LYRICS
@@ -876,7 +866,7 @@ Composite_music:
        ;
 
 relative_music:
-       RELATIVE absolute_musical_pitch Music {
+       RELATIVE absolute_pitch Music {
                Music * p = $3;
                Pitch pit = *unsmob_pitch ($2);
                $$ = new Relative_octave_music (SCM_EOL);
@@ -908,8 +898,8 @@ part_combined_music:
                p->set_mus_property ("one", $3->self_scm ());
                p->set_mus_property ("two", $4->self_scm ());  
 
-               scm_unprotect_object ($3->self_scm());
-               scm_unprotect_object ($4->self_scm());  
+               scm_unprotect_object ($3->self_scm ());
+               scm_unprotect_object ($4->self_scm ());  
 
 
                $$ = p;
@@ -948,6 +938,22 @@ property_def:
 
                csm-> set_mus_property ("context-type", $2);
        }
+       | PROPERTY STRING '.' STRING UNSET {
+               Music *t = new Music (SCM_EOL);
+
+               t->set_mus_property ("iterator-ctor",
+                       Property_unset_iterator::constructor_cxx_function);
+               t->set_mus_property ("symbol", scm_string_to_symbol ($4));
+
+               Context_specced_music *csm = new Context_specced_music (SCM_EOL);
+               csm->set_mus_property ("element", t->self_scm ());
+               scm_unprotect_object (t->self_scm ());
+
+               $$ = csm;
+               $$->set_spot (THIS->here_input ());
+
+               csm-> set_mus_property ("context-type", $2);
+       }
        | PROPERTY STRING '.' STRING SET embedded_scm '=' embedded_scm {
                Music *t = new Music (SCM_EOL);
                t->set_mus_property ("iterator-ctor",
@@ -1007,12 +1013,12 @@ scalar:
 
 request_chord:
        pre_requests simple_element post_requests       {
-               Music_sequence *l = dynamic_cast<Music_sequence*>($2);
+               Music_sequence *l = dynamic_cast<Music_sequence*> ($2);
                if (l) {
-                       for (int i=0; i < $1->size(); i++)
-                               l->append_music ($1->elem(i));
-                       for (int i=0; i < $3->size(); i++)
-                               l->append_music ($3->elem(i));
+                       for (int i=0; i < $1->size (); i++)
+                               l->append_music ($1->elem (i));
+                       for (int i=0; i < $3->size (); i++)
+                               l->append_music ($3->elem (i));
                        }
                else
                        programming_error ("Need Sequence to add music to");
@@ -1029,7 +1035,7 @@ command_element:
                $$-> set_spot (THIS->here_input ());
                $1-> set_spot (THIS->here_input ());
        }
-       | BAR STRING ';'                        {
+       | BAR STRING                    {
                Music *t = new Music (SCM_EOL);
 
                t->set_mus_property ("iterator-ctor",
@@ -1046,13 +1052,13 @@ command_element:
 
                csm->set_mus_property ("context-type", ly_str02scm ("Score"));
        }
-       | PARTIAL duration_length ';'   {
+       | PARTIAL duration_length       {
                Music * p = new Music (SCM_EOL);
                p->set_mus_property ("symbol", ly_symbol2scm ( "measurePosition"));
                p->set_mus_property ("iterator-ctor",
                        Property_iterator::constructor_cxx_function);
 
-               Moment m = - unsmob_duration($2)->length_mom ();
+               Moment m = - unsmob_duration ($2)->length_mom ();
                p->set_mus_property ("value", m.smobbed_copy ());
 
                Context_specced_music * sp = new Context_specced_music (SCM_EOL);
@@ -1062,14 +1068,14 @@ command_element:
                $$ =sp ;
                sp-> set_mus_property ("context-type", ly_str02scm ( "Score"));
        }
-       | CLEF STRING ';' {
+       | CLEF STRING  {
                SCM func = scm_eval2 (ly_symbol2scm ("clef-name-to-properties"), SCM_EOL);
                SCM result = gh_call1 (func, $2);
 
                SCM l = SCM_EOL;
                for (SCM s = result ; gh_pair_p (s); s = gh_cdr (s)) {
                        Music * p = new Music (SCM_EOL);
-                       set_music_properties(p, gh_car (s));
+                       set_music_properties (p, gh_car (s));
                        l = gh_cons (p->self_scm (), l);
                        scm_unprotect_object (p->self_scm ());
                }
@@ -1081,9 +1087,9 @@ command_element:
                scm_unprotect_object (seq->self_scm ());
 
                $$ =sp ;
-               sp-> set_mus_property("context-type", ly_str02scm("Staff"));
+               sp-> set_mus_property ("context-type", ly_str02scm ("Staff"));
        }
-       | TIME_T bare_unsigned '/' bare_unsigned ';' {
+       | TIME_T bare_unsigned '/' bare_unsigned  {
                Music * p = new Music (SCM_EOL);
                p->set_mus_property ("symbol",
                        ly_symbol2scm ( "timeSignatureFraction"));
@@ -1098,13 +1104,18 @@ command_element:
                scm_unprotect_object (p->self_scm ());
 
                $$ = sp;
+
+/*
+ TODO: should make alias TimingContext for Score
+*/
+
                sp-> set_mus_property ("context-type", ly_str02scm ( "Score"));
        }
        ;
 
 command_req:
        shorthand_command_req   { $$ = $1; }
-       | verbose_command_req semicolon { $$ = $1; }
+       | verbose_command_req   { $$ = $1; }
        ;
 
 shorthand_command_req:
@@ -1122,13 +1133,13 @@ shorthand_command_req:
        }
        | '['           {
                Span_req*b= new Span_req;
-               b->set_span_dir(START);
+               b->set_span_dir (START);
                b->set_mus_property ("span-type", ly_str02scm ("beam"));
                $$ =b;
        }
        | ']'           {
                Span_req*b= new Span_req;
-               b->set_span_dir( STOP);
+               b->set_span_dir ( STOP);
                b->set_mus_property ("span-type", ly_str02scm ("beam"));
                $$ = b;
        }
@@ -1139,31 +1150,32 @@ shorthand_command_req:
 
 
 verbose_command_req:
-       COMMANDSPANREQUEST bare_int STRING {
+       COMMANDSPANREQUEST bare_int STRING { /*TODO: junkme */
                Span_req * sp_p = new Span_req;
-               sp_p-> set_span_dir ( Direction($2));
+               sp_p-> set_span_dir ( Direction ($2));
                sp_p->set_mus_property ("span-type",$3);
                sp_p->set_spot (THIS->here_input ());
                $$ = sp_p;
        }
-       | MARK  {
+       | MARK DEFAULT  {
                Mark_req * m = new Mark_req;
                $$ = m;
        }
-       | MARK STRING {
+       | MARK scalar {
                Mark_req *m = new Mark_req;
                m->set_mus_property ("label", $2);
                $$ = m;
 
        }
-       | MARK bare_unsigned {
-               Mark_req *m = new Mark_req;
-               m->set_mus_property ("label",  gh_int2scm ($2));
-               $$ = m;
-       }
-       | PENALTY bare_int      {
+       | PENALTY SCM_T         {
+
+               
                Break_req * b = new Break_req;
-               b->set_mus_property ("penalty", gh_double2scm ( $2 / 100.0));
+               SCM s = $2;
+               if (!gh_number_p (s))
+                       s  =gh_int2scm (0);
+
+               b->set_mus_property ("penalty", s);
                b->set_spot (THIS->here_input ());
                $$ = b;
        }
@@ -1176,7 +1188,7 @@ verbose_command_req:
        | tempo_request {
                $$ = $1;
        }
-       | KEY {
+       | KEY DEFAULT {
                Key_change_req *key_p= new Key_change_req;
                $$ = key_p;
        }
@@ -1184,7 +1196,7 @@ verbose_command_req:
                Key_change_req *key_p= new Key_change_req;
                
                key_p->set_mus_property ("pitch-alist", $3);
              ((Music* )key_p)->transpose (* unsmob_pitch ($2));
+ ((Music* )key_p)->transpose (* unsmob_pitch ($2));
                $$ = key_p; 
        }
        ;
@@ -1244,31 +1256,28 @@ verbose_request:
                d->set_spot (THIS->here_input ());
                $$ = d;
        }
-       | TEXTSCRIPT embedded_scm {
-               Text_script_req *t = new Text_script_req;
-               t->set_mus_property ("text", $2);
-               t->set_spot (THIS->here_input ());
-               $$ = t;
-       }
        | SPANREQUEST bare_int STRING {
                Span_req * sp_p = new Span_req;
-               sp_p->set_span_dir( Direction($2));
+               sp_p->set_span_dir ( Direction ($2));
                sp_p->set_mus_property ("span-type", $3);
                sp_p->set_spot (THIS->here_input ());
                $$ = sp_p;
        }
-       | tremolo_type  {
-               Tremolo_req* a = new Tremolo_req;
-               a->set_spot (THIS->here_input ());
-               a->set_mus_property ("tremolo-type", gh_int2scm ($1));
-               $$ = a;
-       }
+       | tremolo_type  {
+               Tremolo_req* a = new Tremolo_req;
+               a->set_spot (THIS->here_input ());
+               a->set_mus_property ("tremolo-type", gh_int2scm ($1));
+               $$ = a;
+        }
        | SCRIPT STRING         { 
                Articulation_req * a = new Articulation_req;
                a->set_mus_property ("articulation-type", $2);
                a->set_spot (THIS->here_input ());
                $$ = a;
        }
+       /*
+duh, junk this syntax from the parser, if possible. 
+       */
        | ARPEGGIO {
                Arpeggio_req *a = new Arpeggio_req;
                a->set_spot (THIS->here_input ());
@@ -1299,7 +1308,7 @@ sub_quotes:
        }
        ;
 
-steno_musical_pitch:
+steno_pitch:
        NOTENAME_PITCH  {
                $$ = $1;
        }
@@ -1339,15 +1348,22 @@ steno_tonic_pitch:
        }
        ;
 
-musical_pitch:
-       steno_musical_pitch {
+pitch:
+       steno_pitch {
                $$ = $1;
        }
-       | MUSICAL_PITCH embedded_scm {
-               if (!unsmob_pitch ($2))
+       | explicit_pitch {
+               $$ = $1;
+       }
+       ;
+
+explicit_pitch:
+       PITCH embedded_scm {
+               $$ = $2;
+               if (!unsmob_pitch ($2)) {
                        THIS->parser_error (_f ("Expecting musical-pitch value", 3));
-                Pitch m;
-               $$ = m.smobbed_copy ();
+                        $$ = Pitch ().smobbed_copy ();
+               }
        }
        ;
 
@@ -1356,7 +1372,7 @@ explicit_duration:
                $$ = $2;
                if (!unsmob_duration ($2))
                {
-                       THIS->parser_error (_("Must have duration object"));
+                       THIS->parser_error (_ ("Must have duration object"));
                        $$ = Duration ().smobbed_copy ();
                }
        }
@@ -1388,7 +1404,12 @@ close_request_parens:
        '('     {
                Span_req* s= new Span_req;
                $$ = s;
-               s->set_mus_property ("span-type", ly_str02scm( "slur"));
+               s->set_mus_property ("span-type", ly_str02scm ( "slur"));
+       }
+       | E_OPEN        {
+               Span_req* s= new Span_req;
+               $$ = s;
+               s->set_mus_property ("span-type", ly_str02scm ( "phrasing-slur"));
        }
        | E_SMALLER {
                Span_req*s =new Span_req;
@@ -1420,7 +1441,12 @@ open_request_parens:
        | ')'   {
                Span_req* s= new Span_req;
                $$ = s;
-               s->set_mus_property ("span-type", ly_str02scm( "slur"));
+               s->set_mus_property ("span-type", ly_str02scm ( "slur"));
+       }
+       | E_CLOSE       {
+               Span_req* s= new Span_req;
+               $$ = s;
+               s->set_mus_property ("span-type", ly_str02scm ( "phrasing-slur"));
        }
        ;
 
@@ -1441,7 +1467,7 @@ gen_text_def:
                String ds = to_str ($1);
                Text_script_req* t = new Text_script_req;
 
-               t->set_mus_property ("text",  ly_str02scm (ds.ch_C()));
+               t->set_mus_property ("text",  ly_str02scm (ds.ch_C ()));
                t->set_mus_property ("text-type" , ly_symbol2scm ("finger"));
                t->set_spot (THIS->here_input ());
                $$ = t;
@@ -1450,10 +1476,10 @@ gen_text_def:
 
 script_abbreviation:
        '^'             {
-               $$ = gh_str02scm  ("hat");
+               $$ = gh_str02scm ("hat");
        }
        | '+'           {
-               $$ = gh_str02scm("plus");
+               $$ = gh_str02scm ("plus");
        }
        | '-'           {
                $$ = gh_str02scm ("dash");
@@ -1485,37 +1511,31 @@ pre_requests:
        }
        ;
 
-absolute_musical_pitch:
-       steno_musical_pitch     {
+absolute_pitch:
+       steno_pitch     {
                $$ = $1;
        }
        ;
 
 duration_length:
-       steno_duration {
+       multiplied_duration {
                $$ = $1;
        }
-       | duration_length '*' bare_unsigned {
-               $$ = unsmob_duration ($$)->compressed ( $3) .smobbed_copy ();
-       }
-       | duration_length '/' bare_unsigned {
-               $$ = unsmob_duration ($$)->compressed (Moment (1,$3)).smobbed_copy ();
-       }
-       ;
-
-entered_notemode_duration:
-       steno_duration  {
-               THIS->set_last_duration (unsmob_duration ($1));
-       }
+       | explicit_duration {
+               $$ = $1;
+       }       
        ;
 
 optional_notemode_duration:
        {
                $$ = THIS->default_duration_.smobbed_copy ();
        }
-       | entered_notemode_duration {
+       | multiplied_duration   {
                $$ = $1;
        }
+       | explicit_duration {
+               $$ = $1;
+       }       
        ;
 
 steno_duration:
@@ -1527,11 +1547,30 @@ steno_duration:
                        l =  intlog2 ($1);
 
                $$ = Duration (l, $2).smobbed_copy ();
+
+               THIS->set_last_duration (unsmob_duration ($$));
        }
        | DURATION_IDENTIFIER dots      {
                Duration *d =unsmob_duration ($1);
                Duration k (d->duration_log (),d->dot_count () + $2);
-               $$ = k.smobbed_copy ();         
+               $$ = k.smobbed_copy ();
+
+               THIS->set_last_duration (unsmob_duration ($$));
+       }
+       ;
+
+
+
+
+multiplied_duration:
+       steno_duration {
+               $$ = $1;
+       }
+       | multiplied_duration '*' bare_unsigned {
+               $$ = unsmob_duration ($$)->compressed ( $3) .smobbed_copy ();
+       }
+       | multiplied_duration '/' bare_unsigned {
+               $$ = unsmob_duration ($$)->compressed (Moment (1,$3)).smobbed_copy ();
        }
        ;
 
@@ -1558,7 +1597,7 @@ tremolo_type:
 
 
 simple_element:
-       musical_pitch exclamations questions optional_notemode_duration {
+       pitch exclamations questions optional_notemode_duration {
                if (!THIS->lexer_p_->note_state_b ())
                        THIS->parser_error (_ ("Have to be in Note mode for notes"));
 
@@ -1591,24 +1630,24 @@ FIXME: location is one off, since ptich & duration don't contain origin refs.
                      Skip_req * skip_p = new Skip_req;
                      skip_p->set_mus_property ("duration" ,$2);
 
-                     skip_p->set_spot (THIS->here_input());
+                     skip_p->set_spot (THIS->here_input ());
                        e = skip_p->self_scm ();
                    }
                  else
                    {
                      Rest_req * rest_req_p = new Rest_req;
                      rest_req_p->set_mus_property ("duration", $2);
-                     rest_req_p->set_spot (THIS->here_input());
+                     rest_req_p->set_spot (THIS->here_input ());
                        e = rest_req_p->self_scm ();
                    }
                  Simultaneous_music* velt_p = new Request_chord (SCM_EOL);
                velt_p-> set_mus_property ("elements", gh_list (e,SCM_UNDEFINED));
-                 velt_p->set_spot (THIS->here_input());
+                 velt_p->set_spot (THIS->here_input ());
 
 
                  $$ = velt_p;
        }
-       | MEASURES optional_notemode_duration   {
+       | MULTI_MEASURE_REST optional_notemode_duration         {
                Skip_req * sk = new Skip_req;
                sk->set_mus_property ("duration", $2);
                Span_req *sp1 = new Span_req;
@@ -1623,7 +1662,7 @@ FIXME: location is one off, since ptich & duration don't contain origin refs.
                rqc1->set_mus_property ("elements", gh_list (sp1->self_scm (), SCM_UNDEFINED));
                Request_chord * rqc2 = new Request_chord (SCM_EOL);
                rqc2->set_mus_property ("elements", gh_list (sk->self_scm (), SCM_UNDEFINED));;
-               Request_chord * rqc3 = new Request_chord(SCM_EOL);
+               Request_chord * rqc3 = new Request_chord (SCM_EOL);
                rqc3->set_mus_property ("elements", gh_list (sp2->self_scm (), SCM_UNDEFINED));;
 
                SCM ms = gh_list (rqc1->self_scm (), rqc2->self_scm (), rqc3->self_scm (), SCM_UNDEFINED);
@@ -1631,22 +1670,12 @@ FIXME: location is one off, since ptich & duration don't contain origin refs.
                $$ = new Sequential_music (SCM_EOL);
                $$->set_mus_property ("elements", ms);
        }
-       | STRING { 
-               THIS->remember_spot ();
-       } 
-       /* cont */
-       optional_notemode_duration      {
-               if (!THIS->lexer_p_->lyric_state_b ()) {
-                       THIS->pop_spot ().error (_ ("Have to be in Lyric mode for lyrics"));
-                       THIS->error_level_i_  = 1;
-                       THIS->parser_error (_ ("Giving up"));
-               } 
-               else
-                       THIS->pop_spot ();
+       | STRING optional_notemode_duration     {
+
                Lyric_req* lreq_p = new Lyric_req;
                 lreq_p->set_mus_property ("text", $1);
-               lreq_p->set_mus_property ("duration",$3);
-               lreq_p->set_spot (THIS->here_input());
+               lreq_p->set_mus_property ("duration",$2);
+               lreq_p->set_spot (THIS->here_input ());
                Simultaneous_music* velt_p = new Request_chord (SCM_EOL);
                velt_p->set_mus_property ("elements", gh_list (lreq_p->self_scm (), SCM_UNDEFINED));
 
@@ -1785,49 +1814,39 @@ bare_number:
        UNSIGNED        {
                $$ = gh_int2scm ($1);
        }
-       | DIGIT         {
-               $$ = gh_int2scm ($1);
-       }
        | REAL          {
-               $$ = gh_double2scm ($1);
+               $$ = $1;
        }
        | NUMBER_IDENTIFIER             {
                $$ = $1;
        }
        | REAL CM_T     {
-               $$ = gh_double2scm ($1 CM);
+               $$ = gh_double2scm (gh_scm2double ($1) CM );
        }
        | REAL PT_T     {
-               $$ = gh_double2scm ($1 PT);
+               $$ = gh_double2scm (gh_scm2double ($1) PT);
        }
        | REAL IN_T     {
-               $$ = gh_double2scm ($1 INCH);
+               $$ = gh_double2scm (gh_scm2double ($1) INCH);
        }
        | REAL MM_T     {
-               $$ = gh_double2scm ($1 MM);
+               $$ = gh_double2scm (gh_scm2double ($1) MM);
        }
        | REAL CHAR_T   {
-               $$ = gh_double2scm ($1 CHAR);
+               $$ = gh_double2scm (gh_scm2double ($1) CHAR);
        }
        ;
 
 
 bare_unsigned:
-       bare_number {
-               if (scm_integer_p ($1) == SCM_BOOL_T) {
-                       $$ = gh_scm2int ($1);
-
-               } else {
-                       THIS->parser_error (_("need integer number arg"));
-                       $$ = 0;
-               }
-               if ($$ < 0) {
-                       THIS->parser_error (_("Must be positive integer"));
-                       $$ = -$$;
-                       }
-
+       UNSIGNED {
+                       $$ = $1;
+       }
+       | DIGIT {
+               $$ = $1;
        }
        ;
+
 bare_int:
        bare_number {
                if (scm_integer_p ($1) == SCM_BOOL_T)
@@ -1836,7 +1855,7 @@ bare_int:
                        $$ = k;
                } else
                {
-                       THIS->parser_error (_("need integer number arg"));
+                       THIS->parser_error (_ ("need integer number arg"));
                        $$ = 0;
                }
        }
@@ -1870,10 +1889,6 @@ questions:
        ;
 
 
-semicolon:
-       ';'
-       ;
-
 %%
 
 void