]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/parser.yy
release: 1.3.146
[lilypond.git] / lily / parser.yy
index 32dc7fc492ecd0715393856670d21c9730714b72..33a75d50c89654c2728ae87f3cafb1746b9fa6cb 100644 (file)
 #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 PITCH
+%token DEFAULT
 %token NAME
 %token PITCHNAMES
 %token NOTES
@@ -199,6 +199,7 @@ yylex (YYSTYPE *s,  void * v_l)
 %token TRANSLATOR
 %token TRANSPOSE
 %token TYPE
+%token UNSET
 %token CONTEXT
 
 /* escaped */
@@ -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,8 +241,8 @@ 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
@@ -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);
        }
        ;
@@ -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.
@@ -727,12 +723,12 @@ 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);
@@ -749,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));
-       }
        ;
 
 
@@ -788,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 ());
@@ -859,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
@@ -907,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;
@@ -947,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",
@@ -1006,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");
@@ -1028,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",
@@ -1045,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);
@@ -1061,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 ());
                }
@@ -1080,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"));
@@ -1097,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:
@@ -1121,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;
        }
@@ -1140,12 +1152,12 @@ shorthand_command_req:
 verbose_command_req:
        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;
        }
@@ -1155,9 +1167,15 @@ verbose_command_req:
                $$ = 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;
        }
@@ -1170,7 +1188,7 @@ verbose_command_req:
        | tempo_request {
                $$ = $1;
        }
-       | KEY {
+       | KEY DEFAULT {
                Key_change_req *key_p= new Key_change_req;
                $$ = key_p;
        }
@@ -1178,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; 
        }
        ;
@@ -1238,15 +1256,9 @@ 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;
@@ -1257,7 +1269,6 @@ verbose_request:
                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);
@@ -1361,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 ();
                }
        }
@@ -1393,12 +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"));
+               s->set_mus_property ("span-type", ly_str02scm ( "phrasing-slur"));
        }
        | E_SMALLER {
                Span_req*s =new Span_req;
@@ -1430,12 +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"));
+               s->set_mus_property ("span-type", ly_str02scm ( "phrasing-slur"));
        }
        ;
 
@@ -1456,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;
@@ -1465,10 +1476,10 @@ gen_text_def:
 
 script_abbreviation:
        '^'             {
-               $$ = gh_str02scm  ("hat");
+               $$ = gh_str02scm ("hat");
        }
        | '+'           {
-               $$ = gh_str02scm("plus");
+               $$ = gh_str02scm ("plus");
        }
        | '-'           {
                $$ = gh_str02scm ("dash");
@@ -1507,31 +1518,19 @@ absolute_pitch:
        ;
 
 duration_length:
-       steno_duration {
+       multiplied_duration {
                $$ = $1;
        }
        | explicit_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));
-       }
        ;
 
 optional_notemode_duration:
        {
                $$ = THIS->default_duration_.smobbed_copy ();
        }
-       | entered_notemode_duration {
+       | multiplied_duration   {
                $$ = $1;
        }
        | explicit_duration {
@@ -1548,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 ();
        }
        ;
 
@@ -1612,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;
@@ -1644,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);
@@ -1652,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));
 
@@ -1806,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)
@@ -1857,7 +1855,7 @@ bare_int:
                        $$ = k;
                } else
                {
-                       THIS->parser_error (_("need integer number arg"));
+                       THIS->parser_error (_ ("need integer number arg"));
                        $$ = 0;
                }
        }
@@ -1891,10 +1889,6 @@ questions:
        ;
 
 
-semicolon:
-       ';'
-       ;
-
 %%
 
 void