]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/parser.yy
* mf/GNUmakefile: always trace pfa fonts.
[lilypond.git] / lily / parser.yy
index 7dccfe006b23040f09682c435841b8bd0d61b649..2f9d9b37c232daa0163a66877b4a4e2be722c178 100644 (file)
@@ -5,12 +5,12 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997--2002 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c)  1997--2003 Han-Wen Nienhuys <hanwen@cs.uu.nl>
            Jan Nieuwenhuizen <janneke@gnu.org>
 */
 
 /*
-  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
 
@@ -42,13 +48,17 @@ TODO:
 * The rules for who is protecting what are very shady. Uniformise
   this.
 
-* There are too many lexical modes. 
-
+* There are too many lexical modes?
 
 */
 
 #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"
@@ -66,12 +76,47 @@ TODO:
 #include "lilypond-input-version.hh"
 #include "scm-hash.hh"
 #include "auto-change-iterator.hh"
-#include "chord.hh"
 #include "ly-modules.hh"
 #include "music-sequence.hh"
 #include "input-smob.hh"
 #include "event.hh"
 #include "text-item.hh"
+#include "music-list.hh"
+
+
+#define MY_MAKE_MUSIC(x)  make_music_by_name (ly_symbol2scm (x))
+
+
+
+#define YYERROR_VERBOSE 1
+
+My_lily_parser* my_lily_parser;
+#define YYPARSE_PARAM my_lily_parser
+#define YYLEX_PARAM my_lily_parser
+#define THIS\
+       ((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);
+}
+
+
 
 bool
 regular_identifier_b (SCM id)
@@ -114,9 +159,38 @@ set_music_properties (Music *p, SCM a)
        }
 }
 
+SCM
+make_chord_step (int step, int alter)
+{
+       if (step == 7)
+               alter--;
+
+       /* ugh: fucks up above 13 */
+       Pitch m(step > 7 ? 1 : 0,(step - 1) % 7, alter);
+       return m.smobbed_copy ();
+}
 
 
-#define MY_MAKE_MUSIC(x)  make_music_by_name (ly_symbol2scm (x))
+SCM
+make_chord (SCM pitch, SCM dur, SCM modification_list)
+{
+       static SCM chord_ctor;
+       if (!chord_ctor)
+               chord_ctor= scm_c_eval_string ("construct-chord");
+       SCM ch=  scm_call_3 (chord_ctor, pitch, dur, modification_list);
+       scm_gc_protect_object (ch);
+       return ch;
+}
+
+/*
+  Todo: actually also use apply iso. call too ... 
+*/
+bool
+ly_input_procedure_p (SCM x)
+{
+       return gh_procedure_p (x)
+               || (gh_pair_p (x) && gh_procedure_p (gh_car (x)));
+}
 
 Music* 
 set_property_music (SCM sym, SCM value)
@@ -127,24 +201,6 @@ set_property_music (SCM sym, SCM value)
        return p;
 }
 
-
-// needed for bison.simple's malloc () and free ()
-
-// #include <malloc.h>
-#include <stdlib.h>
-
-
-
-#define YYERROR_VERBOSE 1
-
-My_lily_parser* my_lily_parser;
-#define YYPARSE_PARAM my_lily_parser
-#define YYLEX_PARAM my_lily_parser
-#define THIS\
-       ((My_lily_parser *) my_lily_parser)
-
-#define yyerror THIS->parser_error
-
 %}
 
 /* We use SCMs to do strings, because it saves us the trouble of
@@ -178,70 +234,73 @@ yylex (YYSTYPE *s,  void * v)
 
 %pure_parser
 
-/* tokens which are not keywords */
-%token AUTOCHANGE
-%token ALIAS
-%token APPLYCONTEXT 
-%token APPLY
+
 %token ACCEPTS
+%token ADDLYRICS
+%token ALIAS
 %token ALTERNATIVE
+%token APPLY
+%token APPLYCONTEXT
+%token APPLYOUTPUT
+%token AUTOCHANGE
 %token BAR
 %token BREATHE
-%token CHORDMODIFIERS
+%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 */
@@ -253,8 +312,10 @@ yylex (YYSTYPE *s,  void * v)
 %token FIGURE_SPACE
 
 %type <i>      exclamations questions dots optional_rest
-%type <i>      bass_number bass_mod
-%type <scm>    br_bass_figure bass_figure figure_list figure_spec
+%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
 %token <scm>   TONICNAME_PITCH
@@ -262,7 +323,9 @@ yylex (YYSTYPE *s,  void * v)
 %token <scm>   DURATION_IDENTIFIER
 %token <scm>    FRACTION
 %token <id>    IDENTIFIER
+%token <scm>   CHORDNAMES
 
+%token <scm> CHORD_MODIFIER
 
 %token <scm>   SCORE_IDENTIFIER
 %token <scm>   MUSIC_OUTPUT_DEF_IDENTIFIER
@@ -282,6 +345,7 @@ yylex (YYSTYPE *s,  void * v)
 %token <scm> MARKUP_HEAD_SCM0
 %token <scm> MARKUP_HEAD_SCM0_MARKUP1
 %token <scm> MARKUP_HEAD_SCM0_SCM1 
+%token <scm> MARKUP_HEAD_SCM0_SCM1_SCM2 
 %token <scm> MARKUP_HEAD_SCM0_SCM1_MARKUP2
 
 %token <scm> MARKUP_IDENTIFIER MARKUP_HEAD_LIST0
@@ -289,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
@@ -303,18 +366,15 @@ 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
+
 %type <scm>  embedded_scm scalar
 %type <music>  Music Sequential_music Simultaneous_music 
 %type <music>  relative_music re_rhythmed_music part_combined_music
@@ -322,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
 
@@ -399,12 +459,8 @@ notenames_body:
          SCM tab = scm_make_vector (gh_int2scm (i), SCM_EOL);
          for (SCM s = $1; gh_pair_p (s); s = ly_cdr (s)) {
                SCM pt = ly_cdar (s);
-               if (!unsmob_pitch (pt))
-                       THIS->parser_error ("Need pitch object.");
-               else
-                       scm_hashq_set_x (tab, ly_caar (s), pt);
+               scm_hashq_set_x (tab, ly_caar (s), pt);
          }
-
          $$ = tab;
        }
        ;
@@ -483,9 +539,6 @@ identifier_init:
                $$ = $1->self_scm ();
                scm_gc_unprotect_object ($$);
        }
-       | verbose_duration {
-               $$ = $1;
-       }
        | number_expression {
                $$ = $1;
        }
@@ -515,6 +568,9 @@ translator_spec_body:
                td->translator_group_type_ = $2;
                td->set_spot (THIS->here_input ());
        }
+       | translator_spec_body DESCRIPTION string  {
+               unsmob_translator_def ($$)->description_ = $3;
+       }
        | translator_spec_body STRING '=' embedded_scm                  {
                unsmob_translator_def ($$)->add_property_assign ($2, $4);
        }
@@ -531,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($$);
@@ -550,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);
@@ -683,7 +739,7 @@ music_output_def_body:
 
 tempo_event:
        TEMPO steno_duration '=' bare_unsigned  {
-               $$ = MY_MAKE_MUSIC("TempoEvent");
+               $$ = MY_MAKE_MUSIC("MetronomeChangeEvent");
                $$->set_mus_property ("tempo-unit", $2);
                $$->set_mus_property ("metronome-count", gh_int2scm ( $4));
        }
@@ -758,17 +814,30 @@ Repeated_music:
                r->set_mus_property ("repeat-count", gh_int2scm (times >? 1));
 
                r-> set_mus_property ("elements",alts);
-               if (gh_equal_p ($2, scm_makfrom0str ("tremolo")))
-               {
-               /*
-               we can not get durations and other stuff correct down the line, so we have to
-               add to the duration log here.
-               */
-                       SCM func = scm_primitive_eval (ly_symbol2scm ("shift-duration-log"));
-                       if (($3 % 3) == 0)
-                         gh_call3 (func, r->self_scm (), gh_int2scm(-intlog2 ($3*2/3)),gh_int2scm(1));
-                       else
-                         gh_call3 (func, r->self_scm (), gh_int2scm(-intlog2 ($3)), gh_int2scm(0));
+               if (gh_equal_p ($2, scm_makfrom0str ("tremolo"))) {
+                       /*
+                       we can not get durations and other stuff correct down the line, so we have to
+                       add to the duration log here.
+                       */
+                       static SCM func;
+
+                       if (!func)
+                               func = scm_primitive_eval (ly_symbol2scm ("shift-duration-log"));
+
+                       int dots = ($3 % 3) ? 0 : 1;
+                       int shift = -intlog2 ((dots) ? ($3*2/3) : $3);
+
+                       Sequential_music * seq = dynamic_cast<Sequential_music*> ($4);
+                       
+                       if (seq) {
+                               int list_len =scm_ilength (seq->music_list ());
+                               if (list_len != 2)
+                                       seq->origin ()->warning ("Chord tremolo must have 2 elements.");
+                               shift -= 1;
+                               r->compress (Moment (Rational (1,list_len)));
+                               }
+                       gh_call3 (func, r->self_scm (), gh_int2scm(shift),gh_int2scm(dots));
+
                }
                r->set_spot (*$4->origin ());
 
@@ -796,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());
@@ -805,32 +874,20 @@ Simultaneous_music:
 
 Simple_music:
        event_chord             { $$ = $1; }
+       | APPLYOUTPUT embedded_scm {
+               if (!ly_input_procedure_p ($2))
+                       THIS->parser_error (_ ("\\applycontext takes function argument"));
+               $$ = MY_MAKE_MUSIC ("ApplyOutputEvent");
+               $$->set_mus_property ("procedure", $2);
+               $$->set_spot (THIS->here_input());
+       }
        | APPLYCONTEXT embedded_scm {
-               if (!gh_procedure_p ($2))
-                       THIS->parser_error (_ ("\applycontext takes function argument"));
+               if (!ly_input_procedure_p ($2))
+                       THIS->parser_error (_ ("\\applycontext takes function argument"));
                $$ = MY_MAKE_MUSIC ("ApplyContext");
                $$->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);
        }
@@ -839,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");
@@ -846,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;
@@ -857,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);
 
@@ -908,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 ();
        }
@@ -922,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 ());
@@ -947,13 +1039,21 @@ Composite_music:
                scm_gc_unprotect_object (p->self_scm ());
        }
        | APPLY embedded_scm Music  {
-               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");
+               if (!ly_input_procedure_p ($2))
+                       {
+                       THIS->parser_error (_ ("\\apply takes function argument"));
+                       $$ = $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 (); }
@@ -992,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:
@@ -1004,8 +1108,10 @@ relative_music:
                $$->set_mus_property ("element", p->self_scm ());
                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 ());
        }
        ;
 
@@ -1021,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 ());
@@ -1035,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;
@@ -1063,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 {
                
@@ -1077,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
@@ -1099,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
@@ -1127,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 {
@@ -1150,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));
        }
        ;
 
@@ -1159,20 +1265,43 @@ scalar:
         string          { $$ = $1; }
         | bare_int      { $$ = gh_int2scm ($1); }
         | embedded_scm  { $$ = $1; }
+       | full_markup {  $$ = $1; }
+       | DIGIT { $$ = gh_int2scm ($1); }
         ;
 
 
-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
@@ -1195,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",
@@ -1233,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));
@@ -1277,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 ();
@@ -1288,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 ;
@@ -1322,26 +1470,11 @@ 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");
        }
        | E_TILDE {
-               $$ = MY_MAKE_MUSIC("PorrectusEvent");
+               $$ = MY_MAKE_MUSIC("PesOrFlexaEvent");
        }
        ;
 
@@ -1369,11 +1502,18 @@ verbose_command_req:
                $$ = key;
        }
        | KEY NOTENAME_PITCH SCM_IDENTIFIER     {
+
                Music *key= MY_MAKE_MUSIC("KeyChangeEvent");
-               
-               key->set_mus_property ("pitch-alist", $3);
-               ((Music*)key)->transpose (* unsmob_pitch ($2));
-               $$ = key; 
+               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;
        }
        ;
 
@@ -1386,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));
@@ -1405,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  {
@@ -1454,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;
@@ -1480,15 +1654,13 @@ steno_pitch:
        }
        | NOTENAME_PITCH sup_quotes     {
                Pitch p = *unsmob_pitch ($1);
-               p.octave_ +=  $2;
+               p = p.transposed (Pitch ($2,0,0));
                $$ = p.smobbed_copy ();
        }
        | NOTENAME_PITCH sub_quotes      {
                Pitch p =* unsmob_pitch ($1);
-
-               p.octave_ +=  -$2;
+               p = p.transposed (Pitch (-$2,0,0));
                $$ = p.smobbed_copy ();
-
        }
        ;
 
@@ -1502,15 +1674,14 @@ steno_tonic_pitch:
        }
        | TONICNAME_PITCH sup_quotes    {
                Pitch p = *unsmob_pitch ($1);
-               p.octave_ +=  $2;
+               p = p.transposed (Pitch ($2,0,0));
                $$ = p.smobbed_copy ();
        }
        | TONICNAME_PITCH sub_quotes     {
                Pitch p =* unsmob_pitch ($1);
 
-               p.octave_ +=  -$2;
+               p = p.transposed (Pitch (-$2,0,0));
                $$ = p.smobbed_copy ();
-
        }
        ;
 
@@ -1518,9 +1689,6 @@ pitch:
        steno_pitch {
                $$ = $1;
        }
-       | explicit_pitch {
-               $$ = $1;
-       }
        ;
 
 pitch_also_in_chords:
@@ -1528,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 {
@@ -1566,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;
@@ -1598,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());
@@ -1627,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 ());
@@ -1684,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     {
@@ -1704,9 +1824,6 @@ duration_length:
        multiplied_duration {
                $$ = $1;
        }
-       | verbose_duration {
-               $$ = $1;
-       }       
        ;
 
 optional_notemode_duration:
@@ -1722,10 +1839,6 @@ optional_notemode_duration:
 
                THIS->beam_check ($$);
        }
-       | verbose_duration {
-               $$ = $1;
-               THIS->default_duration_ = *unsmob_duration ($$);
-       }       
        ;
 
 steno_duration:
@@ -1798,8 +1911,13 @@ tremolo_type:
                BASS FIGURES
 *****************************************************************/
 bass_number:
-       DIGIT
-       | UNSIGNED 
+       DIGIT   {
+               $$ = scm_number_to_string (gh_int2scm ($1), gh_int2scm (10));
+       }
+       | UNSIGNED {
+               $$ = scm_number_to_string (gh_int2scm ($1), gh_int2scm (10));
+       }
+       | STRING { $$ =  $1; }
        ;
 
 bass_mod:
@@ -1818,7 +1936,7 @@ bass_figure:
                Music *bfr = MY_MAKE_MUSIC("BassFigureEvent");
                $$ = bfr->self_scm();
 
-               bfr->set_mus_property ("figure", gh_int2scm ($1));
+               bfr->set_mus_property ("figure", $1);
 
                scm_gc_unprotect_object ($$);
        }
@@ -1945,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);
@@ -1957,111 +2078,90 @@ simple_element:
 
                $$= velt;
        }
-       | chord {
+       | new_chord {
                THIS->pop_spot ();
 
-               if (!THIS->lexer_->chord_state_b ())
-                       THIS->parser_error (_ ("Have to be in Chord mode for chords"));
-               $$ = $1;
+                if (!THIS->lexer_->chord_state_b ())
+                        THIS->parser_error (_ ("Have to be in Chord mode for chords"));
+                $$ = unsmob_music ($1);
        }
        ;
 
-
-chord:
-       steno_tonic_pitch optional_notemode_duration chord_additions chord_subtractions chord_inversion chord_bass {
-                $$ = Chord::get_chord ($1, $3, $4, $5, $6, $2);
-               $$->set_spot (THIS->here_input ());
-        };
-
-chord_additions: 
-       {
-               $$ = SCM_EOL;
-       } 
-       | CHORD_COLON chord_notes {
-               $$ = $2;
-       }
+lyric_element:
+       full_markup { $$ = $1; }
+       | STRING {  $$ = $1 ; }
        ;
 
-chord_notes:
-       chord_step {
-               $$ = $1;
+new_chord:
+       steno_tonic_pitch optional_notemode_duration   {
+               $$ = make_chord ($1, $2, SCM_EOL);
        }
-       | chord_notes '.' chord_step {
-               $$ = gh_append2 ($$, $3);
+       | steno_tonic_pitch optional_notemode_duration chord_separator chord_items {
+               SCM its = scm_reverse_x ($4, SCM_EOL);
+               $$ = make_chord ($1, $2, gh_cons ($3, its));
        }
        ;
 
-chord_subtractions: 
-       {
-               $$ = SCM_EOL;
-       } 
-       | CHORD_CARET chord_notes {
-               $$ = $2;
+chord_items:
+       /**/ {
+               $$ = SCM_EOL;           
+       }
+       | chord_items chord_item {
+               $$ = gh_cons ($2, $$);
        }
        ;
 
-
-chord_inversion:
-       {
-               $$ = SCM_EOL;
+chord_separator:
+       CHORD_COLON {
+               $$ = ly_symbol2scm ("chord-colon");
        }
-       | CHORD_SLASH steno_tonic_pitch {
-               $$ = $2;
+       | CHORD_CARET {
+               $$ = ly_symbol2scm ("chord-caret"); 
        }
-       ;
-
-chord_bass:
-       {
-               $$ = SCM_EOL;
+       | CHORD_SLASH steno_tonic_pitch {
+               $$ = scm_list_n (ly_symbol2scm ("chord-slash"), $2, SCM_UNDEFINED); 
        }
        | CHORD_BASS steno_tonic_pitch {
-               $$ = $2;
+               $$ = scm_list_n (ly_symbol2scm ("chord-bass"), $2, SCM_UNDEFINED); 
        }
        ;
 
-chord_step:
-       chord_note {
-               $$ = scm_cons ($1, SCM_EOL);
+chord_item:
+       chord_separator {
+               $$ = $1;
        }
-       | CHORDMODIFIER_PITCH {
-               $$ = scm_cons (unsmob_pitch ($1)->smobbed_copy (), SCM_EOL);
+       | step_numbers {
+               $$ = scm_reverse_x ($1, SCM_EOL);
        }
-       | CHORDMODIFIER_PITCH chord_note { /* Ugh. */
-               $$ = scm_list_n (unsmob_pitch ($1)->smobbed_copy (),
-                       $2, SCM_UNDEFINED);
+       | CHORD_MODIFIER  {
+               $$ = $1;
        }
        ;
 
-chord_note:
-       bare_unsigned {
-                Pitch m;
-               m.notename_ = ($1 - 1) % 7;
-               m.octave_ = $1 > 7 ? 1 : 0;
-               m.alteration_ = 0;
+step_numbers:
+       step_number { $$ = gh_cons ($1, SCM_EOL); } 
+       | step_numbers '.' step_number {
+               $$ = gh_cons ($3, $$);
+       }
+       ;
 
-               $$ = m.smobbed_copy ();
+step_number:
+       bare_unsigned {
+               $$ = make_chord_step ($1, 0);
         } 
        | bare_unsigned '+' {
-               Pitch m;
-               m.notename_ = ($1 - 1) % 7;
-               m.octave_ = $1 > 7 ? 1 : 0;
-               m.alteration_ = 1;
-
-
-               $$ = m.smobbed_copy ();
+               $$ = make_chord_step ($1, 1);
        }
        | bare_unsigned CHORD_MINUS {
-               Pitch m;
-               m.notename_ = ($1 - 1) % 7;
-               m.octave_ = $1 > 7 ? 1 : 0;
-               m.alteration_ = -1;
-
-               $$ = m.smobbed_copy ();
+               $$ = make_chord_step ($1,-1);
        }
-        ;
+        ;      
 
 /*
        UTILITIES
+
+TODO: should deprecate in favor of Scheme?
+
  */
 number_expression:
        number_expression '+' number_term {
@@ -2175,7 +2275,11 @@ full_markup:
                  THIS->lexer_->pop_state ();
                }
        ;
-       
+
+
+/*
+This should be done more dynamically if possible.
+*/ 
 markup:
        STRING {
                $$ = make_simple_markup ($1);
@@ -2201,6 +2305,9 @@ markup:
        | MARKUP_HEAD_SCM0_SCM1_MARKUP2 embedded_scm embedded_scm markup {
                $$ = scm_list_n ($1, $2, $3, $4, SCM_UNDEFINED);
        }
+       | MARKUP_HEAD_SCM0_SCM1_SCM2 embedded_scm embedded_scm embedded_scm {
+               $$ = scm_list_n ($1, $2, $3, $4, SCM_UNDEFINED);
+       }
        | MARKUP_IDENTIFIER {
                $$ = $1;
        }
@@ -2208,7 +2315,7 @@ markup:
        ;
 
 markup_list:
-       '<' markup_list_body '>' { $$ = scm_reverse_x ($2, SCM_EOL); }
+       chord_open markup_list_body chord_close { $$ = scm_reverse_x ($2, SCM_EOL); }
        ;
 
 markup_line:
@@ -2274,14 +2381,10 @@ 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"));
-}
+
 /*
-It is a little strange, to have this function in this file, but
+
+It is a little strange to have this function in this file, but
 otherwise, we have to import music classes into the lexer.
 
 */
@@ -2316,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;
        }