X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fparser.yy;h=da164e48c4fe2ef46337ef55b09afad2bec762b8;hb=8864a7cc89abc6d910896aa4681e89fd71c5e4f1;hp=d98780c374c6d65bffa3a6d2009527c3c1bdfbdc;hpb=83e6304ae3cc708a8e4d0462249ca8babae265a3;p=lilypond.git

diff --git a/lily/parser.yy b/lily/parser.yy
index d98780c374..da164e48c4 100644
--- a/lily/parser.yy
+++ b/lily/parser.yy
@@ -1,7 +1,8 @@
+/* -*- mode: c++; c-file-style: "linux" -*- */
 /*
   This file is part of LilyPond, the GNU music typesetter.
 
-  Copyright (C) 1997--2011 Han-Wen Nienhuys <hanwen@xs4all.nl>
+  Copyright (C) 1997--2012 Han-Wen Nienhuys <hanwen@xs4all.nl>
                  Jan Nieuwenhuizen <janneke@gnu.org>
 
   LilyPond is free software: you can redistribute it and/or modify
@@ -18,15 +19,20 @@
   along with LilyPond.  If not, see <http://www.gnu.org/licenses/>.
 */
 
+/* Mode and indentation are at best a rough approximation based on TAB
+ * formatting (reasonable for compatibility with unspecific editor
+ * modes as Bison modes are hard to find) and need manual correction
+ * frequently.  Without a reasonably dependable way of formatting a
+ * Bison file sensibly, there is little point in trying to fix the
+ * inconsistent state of indentation.
+ */
+
 %{
 
 #define YYDEBUG 1
 #define YYERROR_VERBOSE 1
-#define YYPARSE_PARAM my_lily_parser
-#define YYLEX_PARAM my_lily_parser
-#define PARSER ((Lily_parser *) my_lily_parser)
 
-#define yyerror PARSER->parser_error
+#define yyerror Lily_parser::parser_error
 
 /* We use custom location type: Input objects */
 #define YYLTYPE Input
@@ -36,8 +42,11 @@
 
 %}
 
+%parse-param {Lily_parser *parser}
+%lex-param {Lily_parser *parser}
+
 /* We use SCMs to do strings, because it saves us the trouble of
-deleting them.  Let's hope that a stack overflow doesnt trigger a move
+deleting them.  Let's hope that a stack overflow doesn't trigger a move
 of the parse stack onto the heap. */
 
 %left PREC_BOT
@@ -135,15 +144,21 @@ using namespace std;
 #include "text-interface.hh"
 #include "warn.hh"
 
+void
+Lily_parser::parser_error (Input const *i, Lily_parser *parser, string s)
+{
+	parser->parser_error (*i, s);
+}
+
 #define MYBACKUP(Token, Value, Location)				\
 do									\
 	if (yychar == YYEMPTY)						\
 	{								\
 		if (Token)						\
-			PARSER->lexer_->push_extra_token (Token, Value); \
-		PARSER->lexer_->push_extra_token (BACKUP);		\
+			parser->lexer_->push_extra_token (Token, Value); \
+		parser->lexer_->push_extra_token (BACKUP);		\
 	} else {							\
-		PARSER->parser_error					\
+		parser->parser_error					\
 			(Location, _("Too much lookahead"));		\
 	}								\
 while (0)
@@ -153,11 +168,11 @@ while (0)
 do									\
 	if (yychar == YYEMPTY)						\
 	{								\
-		PARSER->lexer_->push_extra_token (Token, Value);	\
-		PARSER->lexer_->push_extra_token (REPARSE,		\
+		parser->lexer_->push_extra_token (Token, Value);	\
+		parser->lexer_->push_extra_token (REPARSE,		\
 						  Pred);		\
 	} else {							\
-		PARSER->parser_error					\
+		parser->parser_error					\
 			(Location, _("Too much lookahead"));		\
 	}								\
 while (0)
@@ -186,11 +201,11 @@ while (0)
   scm_apply_0 (proc, args)
 /* Syntactic Sugar. */
 #define MAKE_SYNTAX(name, location, ...)	\
-  LOWLEVEL_MAKE_SYNTAX (ly_lily_module_constant (name), scm_list_n (PARSER->self_scm (), make_input (location) , ##__VA_ARGS__, SCM_UNDEFINED));
+  LOWLEVEL_MAKE_SYNTAX (ly_lily_module_constant (name), scm_list_n (parser->self_scm (), make_input (location) , ##__VA_ARGS__, SCM_UNDEFINED))
 #define START_MAKE_SYNTAX(name, ...)					\
 	scm_list_n (ly_lily_module_constant (name) , ##__VA_ARGS__, SCM_UNDEFINED)
 #define FINISH_MAKE_SYNTAX(start, location, ...)			\
-	LOWLEVEL_MAKE_SYNTAX (scm_car (start), scm_cons2 (PARSER->self_scm (), make_input (location), scm_append_x (scm_list_2 (scm_cdr (start), scm_list_n (__VA_ARGS__, SCM_UNDEFINED)))))
+	LOWLEVEL_MAKE_SYNTAX (scm_car (start), scm_cons2 (parser->self_scm (), make_input (location), scm_append_x (scm_list_2 (scm_cdr (start), scm_list_n (__VA_ARGS__, SCM_UNDEFINED)))))
 
 SCM get_next_unique_context_id ();
 SCM get_next_unique_lyrics_context_id ();
@@ -208,14 +223,12 @@ static Music *make_music_with_input (SCM name, Input where);
 SCM check_scheme_arg (Lily_parser *parser, Input loc,
 		      SCM arg, SCM args, SCM pred);
 SCM loc_on_music (Input loc, SCM arg);
-SCM get_first_context_id (SCM type, Music *m);
-SCM make_chord_elements (SCM pitch, SCM dur, SCM modification_list);
+SCM make_chord_elements (Input loc, SCM pitch, SCM dur, SCM modification_list);
 SCM make_chord_step (int step, Rational alter);
 SCM make_simple_markup (SCM a);
 bool is_duration (int t);
 bool is_regular_identifier (SCM id);
-bool ly_input_procedure_p (SCM x);
-int yylex (YYSTYPE *s, YYLTYPE *loc, void *v);
+int yylex (YYSTYPE *s, YYLTYPE *loc, Lily_parser *parser);
 void set_music_properties (Music *p, SCM a);
 
 %}
@@ -346,7 +359,6 @@ If we give names, Bison complains.
 %token <scm> FRACTION
 %token <scm> LYRICS_STRING
 %token <scm> LYRIC_ELEMENT
-%token <scm> LYRIC_ELEMENT_P
 %token <scm> LYRIC_MARKUP_IDENTIFIER
 %token <scm> MARKUP_FUNCTION
 %token <scm> MARKUP_LIST_FUNCTION
@@ -456,7 +468,6 @@ If we give names, Bison complains.
 %type <scm> embedded_scm_bare
 %type <scm> embedded_scm_bare_arg
 %type <scm> embedded_scm_closed
-%type <scm> embedded_scm_chord_body
 %type <scm> event_function_event
 %type <scm> figure_list
 %type <scm> figure_spec
@@ -501,7 +512,6 @@ If we give names, Bison complains.
 %type <scm> multiplied_duration
 %type <scm> music_function_event
 %type <scm> music_function_chord_body
-%type <scm> music_function_chord_body_arglist
 %type <scm> new_chord
 %type <scm> new_lyrics
 %type <scm> number_expression
@@ -548,11 +558,11 @@ prec levels in different prods */
 start_symbol:
 	lilypond
 	| EMBEDDED_LILY {
-		SCM nn = PARSER->lexer_->lookup_identifier ("pitchnames");
-		PARSER->lexer_->push_note_state (alist_to_hashq (nn));
+		SCM nn = parser->lexer_->lookup_identifier ("pitchnames");
+		parser->lexer_->push_note_state (alist_to_hashq (nn));
 	} embedded_lilypond {
-		PARSER->lexer_->pop_state ();
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("parseStringResult"), $3);
+		parser->lexer_->pop_state ();
+		parser->lexer_->set_identifier (ly_symbol2scm ("parseStringResult"), $3);
  	}
 	;
 
@@ -562,49 +572,49 @@ lilypond:	/* empty */ { }
 	| lilypond assignment {
 	}
 	| lilypond error {
-		PARSER->error_level_ = 1;
+		parser->error_level_ = 1;
 	}
 	| lilypond INVALID	{
-		PARSER->error_level_ = 1;
+		parser->error_level_ = 1;
 	}
 	;
 
 
 toplevel_expression:
 	lilypond_header {
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("$defaultheader"), $1);
+		parser->lexer_->set_identifier (ly_symbol2scm ("$defaultheader"), $1);
 	}
 	| book_block {
 		Book *book = $1;
-		SCM proc = PARSER->lexer_->lookup_identifier ("toplevel-book-handler");
-		scm_call_2 (proc, PARSER->self_scm (), book->self_scm ());
+		SCM proc = parser->lexer_->lookup_identifier ("toplevel-book-handler");
+		scm_call_2 (proc, parser->self_scm (), book->self_scm ());
 		book->unprotect ();
 	}
 	| bookpart_block {
 		Book *bookpart = $1;
-		SCM proc = PARSER->lexer_->lookup_identifier ("toplevel-bookpart-handler");
-		scm_call_2 (proc, PARSER->self_scm (), bookpart->self_scm ());
+		SCM proc = parser->lexer_->lookup_identifier ("toplevel-bookpart-handler");
+		scm_call_2 (proc, parser->self_scm (), bookpart->self_scm ());
 		bookpart->unprotect ();
 	}
 	| score_block {
 		Score *score = $1;
 
-		SCM proc = PARSER->lexer_->lookup_identifier ("toplevel-score-handler");
-		scm_call_2 (proc, PARSER->self_scm (), score->self_scm ());
+		SCM proc = parser->lexer_->lookup_identifier ("toplevel-score-handler");
+		scm_call_2 (proc, parser->self_scm (), score->self_scm ());
 		score->unprotect ();
 	}
 	| composite_music {
 		Music *music = unsmob_music ($1);
-		SCM proc = PARSER->lexer_->lookup_identifier ("toplevel-music-handler");
-		scm_call_2 (proc, PARSER->self_scm (), music->self_scm ());
+		SCM proc = parser->lexer_->lookup_identifier ("toplevel-music-handler");
+		scm_call_2 (proc, parser->self_scm (), music->self_scm ());
 	}
 	| full_markup {
-		SCM proc = PARSER->lexer_->lookup_identifier ("toplevel-text-handler");
-		scm_call_2 (proc, PARSER->self_scm (), scm_list_1 ($1));
+		SCM proc = parser->lexer_->lookup_identifier ("toplevel-text-handler");
+		scm_call_2 (proc, parser->self_scm (), scm_list_1 ($1));
 	}
 	| full_markup_list {
-		SCM proc = PARSER->lexer_->lookup_identifier ("toplevel-text-handler");
-		scm_call_2 (proc, PARSER->self_scm (), $1);
+		SCM proc = parser->lexer_->lookup_identifier ("toplevel-text-handler");
+		scm_call_2 (proc, parser->self_scm (), $1);
 	}
 	| output_def {
 		SCM id = SCM_EOL;
@@ -617,7 +627,7 @@ toplevel_expression:
 		else if ($1->c_variable ("is-layout") == SCM_BOOL_T)
 			id = ly_symbol2scm ("$defaultlayout");
 
-		PARSER->lexer_->set_identifier (id, od->self_scm ());
+		parser->lexer_->set_identifier (id, od->self_scm ());
 		od->unprotect();
 	}
 	;
@@ -625,7 +635,7 @@ toplevel_expression:
 embedded_scm_bare:
 	SCM_TOKEN
 	{
-		$$ = PARSER->lexer_->eval_scm ($1);
+		$$ = parser->lexer_->eval_scm ($1);
 	}
 	| SCM_IDENTIFIER
 	;
@@ -691,18 +701,18 @@ embedded_lilypond:
 				  scm_cons2 ($1, $2, scm_reverse_x ($3, SCM_EOL)));
 	}
 	| error {
-		PARSER->error_level_ = 1;
+		parser->error_level_ = 1;
 	}
 	| INVALID embedded_lilypond {
-		PARSER->error_level_ = 1;
+		parser->error_level_ = 1;
 	}
 	;
 
 
 lilypond_header_body:
 	{
-		$$ = get_header (PARSER);
-		PARSER->lexer_->add_scope ($$);
+		$$ = get_header (parser);
+		parser->lexer_->add_scope ($$);
 	}
 	| lilypond_header_body assignment  {
 
@@ -711,7 +721,7 @@ lilypond_header_body:
 
 lilypond_header:
 	HEADER '{' lilypond_header_body '}'	{
-		$$ = PARSER->lexer_->remove_scope ();
+		$$ = parser->lexer_->remove_scope ();
 	}
 	;
 
@@ -725,11 +735,11 @@ assignment_id:
 
 assignment:
 	assignment_id '=' identifier_init  {
-	        PARSER->lexer_->set_identifier ($1, $3);
+	        parser->lexer_->set_identifier ($1, $3);
 	}
 	| assignment_id property_path '=' identifier_init {
 		SCM path = scm_cons (scm_string_to_symbol ($1), $2);
-		PARSER->lexer_->set_identifier (path, $4);
+		parser->lexer_->set_identifier (path, $4);
 	;
 /*
  TODO: devise standard for protection in parser.
@@ -764,16 +774,7 @@ identifier_init:
 		$$ = $1;
 	}
 	| music  {
-		/* Hack: Create event-chord around standalone events.
-		   Prevents the identifier from being interpreted as a post-event. */
-		Music *mus = unsmob_music ($1);
-		bool is_event = mus &&
-			(scm_memq (ly_symbol2scm ("event"), mus->get_property ("types"))
-				!= SCM_BOOL_F);
-		if (!is_event)
-			$$ = $1;
-		else
-			$$ = MAKE_SYNTAX ("event-chord", @$, scm_list_1 ($1));
+		$$ = $1;
 	}
 	| post_event_nofinger {
 		$$ = $1;
@@ -823,7 +824,7 @@ context_def_spec_body:
 				td->add_context_mod (scm_car (p));
 			}
 		} else {
-			PARSER->parser_error (@2, "context-mod expected");
+			parser->parser_error (@2, _ ("not a context mod"));
 		}
 	}
 	| context_def_spec_body context_mod {
@@ -843,8 +844,8 @@ context_def_spec_body:
 book_block:
 	BOOK '{' book_body '}' 	{
 		$$ = $3;
-		pop_paper (PARSER);
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("$current-book"), SCM_BOOL_F);
+		pop_paper (parser);
+		parser->lexer_->set_identifier (ly_symbol2scm ("$current-book"), SCM_BOOL_F);
 	}
 	;
 
@@ -854,50 +855,50 @@ book_block:
 book_body:
 	{
 		$$ = new Book;
-		init_papers (PARSER);
+		init_papers (parser);
 		$$->origin ()->set_spot (@$);
-		$$->paper_ = dynamic_cast<Output_def*> (unsmob_output_def (PARSER->lexer_->lookup_identifier ("$defaultpaper"))->clone ());
+		$$->paper_ = dynamic_cast<Output_def*> (unsmob_output_def (parser->lexer_->lookup_identifier ("$defaultpaper"))->clone ());
 		$$->paper_->unprotect ();
-		push_paper (PARSER, $$->paper_);
-		$$->header_ = PARSER->lexer_->lookup_identifier ("$defaultheader");
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("$current-book"), $$->self_scm ());
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("book-output-suffix"), SCM_BOOL_F);
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("book-filename"), SCM_BOOL_F);
+		push_paper (parser, $$->paper_);
+		$$->header_ = parser->lexer_->lookup_identifier ("$defaultheader");
+		parser->lexer_->set_identifier (ly_symbol2scm ("$current-book"), $$->self_scm ());
+		parser->lexer_->set_identifier (ly_symbol2scm ("book-output-suffix"), SCM_BOOL_F);
+		parser->lexer_->set_identifier (ly_symbol2scm ("book-filename"), SCM_BOOL_F);
 	}
 	| BOOK_IDENTIFIER {
 		$$ = unsmob_book ($1);
 		$$->protect ();
 		$$->origin ()->set_spot (@$);
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("$current-book"), $1);
+		parser->lexer_->set_identifier (ly_symbol2scm ("$current-book"), $1);
 	}
 	| book_body paper_block {
 		$$->paper_ = $2;
 		$2->unprotect ();
-		set_paper (PARSER, $2);
+		set_paper (parser, $2);
 	}
 	| book_body bookpart_block {
 		Book *bookpart = $2;
-		SCM proc = PARSER->lexer_->lookup_identifier ("book-bookpart-handler");
+		SCM proc = parser->lexer_->lookup_identifier ("book-bookpart-handler");
 		scm_call_2 (proc, $$->self_scm (), bookpart->self_scm ());
 		bookpart->unprotect ();
 	}
 	| book_body score_block {
 		Score *score = $2;
-		SCM proc = PARSER->lexer_->lookup_identifier ("book-score-handler");
+		SCM proc = parser->lexer_->lookup_identifier ("book-score-handler");
 		scm_call_2 (proc, $$->self_scm (), score->self_scm ());
 		score->unprotect ();
 	}
 	| book_body composite_music {
 		Music *music = unsmob_music ($2);
-		SCM proc = PARSER->lexer_->lookup_identifier ("book-music-handler");
-		scm_call_3 (proc, PARSER->self_scm (), $$->self_scm (), music->self_scm ());
+		SCM proc = parser->lexer_->lookup_identifier ("book-music-handler");
+		scm_call_3 (proc, parser->self_scm (), $$->self_scm (), music->self_scm ());
 	}
 	| book_body full_markup {
-		SCM proc = PARSER->lexer_->lookup_identifier ("book-text-handler");
+		SCM proc = parser->lexer_->lookup_identifier ("book-text-handler");
 		scm_call_2 (proc, $$->self_scm (), scm_list_1 ($2));
 	}
 	| book_body full_markup_list {
-		SCM proc = PARSER->lexer_->lookup_identifier ("book-text-handler");
+		SCM proc = parser->lexer_->lookup_identifier ("book-text-handler");
 		scm_call_2 (proc, $$->self_scm (), $2);
 	}
 	| book_body lilypond_header {
@@ -914,7 +915,7 @@ book_body:
 bookpart_block:
 	BOOKPART '{' bookpart_body '}' {
 		$$ = $3;
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("$current-bookpart"), SCM_BOOL_F);
+		parser->lexer_->set_identifier (ly_symbol2scm ("$current-bookpart"), SCM_BOOL_F);
 	}
 	;
 
@@ -922,13 +923,13 @@ bookpart_body:
 	{
 		$$ = new Book;
 		$$->origin ()->set_spot (@$);
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("$current-bookpart"), $$->self_scm ());
+		parser->lexer_->set_identifier (ly_symbol2scm ("$current-bookpart"), $$->self_scm ());
 	}
 	| BOOK_IDENTIFIER {
 		$$ = unsmob_book ($1);
 		$$->protect ();
 		$$->origin ()->set_spot (@$);
-		PARSER->lexer_->set_identifier (ly_symbol2scm ("$current-bookpart"), $1);
+		parser->lexer_->set_identifier (ly_symbol2scm ("$current-bookpart"), $1);
 	}
 	| bookpart_body paper_block {
 		$$->paper_ = $2;
@@ -936,21 +937,21 @@ bookpart_body:
 	}
 	| bookpart_body score_block {
 		Score *score = $2;
-		SCM proc = PARSER->lexer_->lookup_identifier ("bookpart-score-handler");
+		SCM proc = parser->lexer_->lookup_identifier ("bookpart-score-handler");
 		scm_call_2 (proc, $$->self_scm (), score->self_scm ());
 		score->unprotect ();
 	}
 	| bookpart_body composite_music {
 		Music *music = unsmob_music ($2);
-		SCM proc = PARSER->lexer_->lookup_identifier ("bookpart-music-handler");
-		scm_call_3 (proc, PARSER->self_scm (), $$->self_scm (), music->self_scm ());
+		SCM proc = parser->lexer_->lookup_identifier ("bookpart-music-handler");
+		scm_call_3 (proc, parser->self_scm (), $$->self_scm (), music->self_scm ());
 	}
 	| bookpart_body full_markup {
-		SCM proc = PARSER->lexer_->lookup_identifier ("bookpart-text-handler");
+		SCM proc = parser->lexer_->lookup_identifier ("bookpart-text-handler");
 		scm_call_2 (proc, $$->self_scm (), scm_list_1 ($2));
 	}
 	| bookpart_body full_markup_list {
-		SCM proc = PARSER->lexer_->lookup_identifier ("bookpart-text-handler");
+		SCM proc = parser->lexer_->lookup_identifier ("bookpart-text-handler");
 		scm_call_2 (proc, $$->self_scm (), $2);
 	}
 	| bookpart_body lilypond_header {
@@ -973,7 +974,7 @@ score_body:
 	music {
 		SCM m = $1;
 		SCM scorify = ly_lily_module_constant ("scorify-music");
-		SCM score = scm_call_2 (scorify, m, PARSER->self_scm ());
+		SCM score = scm_call_2 (scorify, m, parser->self_scm ());
 
 		// pass ownernship to C++ again.
 		$$ = unsmob_score (score);
@@ -991,7 +992,7 @@ score_body:
 	| score_body output_def {
 		if ($2->lookup_variable (ly_symbol2scm ("is-paper")) == SCM_BOOL_T)
 		{
-			PARSER->parser_error (@2, _("\\paper cannot be used in \\score, use \\layout instead"));
+			parser->parser_error (@2, _("\\paper cannot be used in \\score, use \\layout instead"));
 
 		}
 		else
@@ -1015,9 +1016,9 @@ paper_block:
 		$$ = $1;
 		if ($$->lookup_variable (ly_symbol2scm ("is-paper")) != SCM_BOOL_T)
 		{
-			PARSER->parser_error (@1, _ ("need \\paper for paper block"));
+			parser->parser_error (@1, _ ("need \\paper for paper block"));
 			$1->unprotect ();
-			$$ = get_paper (PARSER);
+			$$ = get_paper (parser);
 		}
 	}
 	;
@@ -1027,33 +1028,33 @@ output_def:
 	output_def_body '}' {
 		$$ = $1;
 
-		PARSER->lexer_->remove_scope ();
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->remove_scope ();
+		parser->lexer_->pop_state ();
 	}
 	;
 
 output_def_head:
 	PAPER {
-		$$ = get_paper (PARSER);
+		$$ = get_paper (parser);
 		$$->input_origin_ = @$;
-		PARSER->lexer_->add_scope ($$->scope_);
+		parser->lexer_->add_scope ($$->scope_);
 	}
 	| MIDI    {
-		Output_def *p = get_midi (PARSER);
+		Output_def *p = get_midi (parser);
 		$$ = p;
-		PARSER->lexer_->add_scope (p->scope_);
+		parser->lexer_->add_scope (p->scope_);
 	}
 	| LAYOUT 	{
-		Output_def *p = get_layout (PARSER);
+		Output_def *p = get_layout (parser);
 
-		PARSER->lexer_->add_scope (p->scope_);
+		parser->lexer_->add_scope (p->scope_);
 		$$ = p;
 	}
 	;
 
 output_def_head_with_mode_switch:
 	output_def_head {
-		PARSER->lexer_->push_initial_state ();
+		parser->lexer_->push_initial_state ();
 		$$ = $1;
 	}
 	;
@@ -1070,8 +1071,8 @@ output_def_body:
 		o->input_origin_.set_spot (@$);
 		$$ = o;
 		$$->protect ();
-		PARSER->lexer_->remove_scope ();
-		PARSER->lexer_->add_scope (o->scope_);
+		parser->lexer_->remove_scope ();
+		parser->lexer_->add_scope (o->scope_);
 	}
 	| output_def_body assignment  {
 
@@ -1171,9 +1172,9 @@ simple_music:
 	;
 
 context_modification:
-        WITH { PARSER->lexer_->push_initial_state (); } '{' context_mod_list '}'
+        WITH { parser->lexer_->push_initial_state (); } '{' context_mod_list '}'
         {
-                PARSER->lexer_->pop_state ();
+                parser->lexer_->pop_state ();
                 $$ = $4;
         }
         | WITH CONTEXT_MOD_IDENTIFIER
@@ -1189,7 +1190,7 @@ context_modification:
 		if (unsmob_context_mod ($2))
 			$$ = $2;
 		else {
-			PARSER->parser_error (@2, "context-mod expected");
+			parser->parser_error (@2, _ ("not a context mod"));
 			$$ = Context_mod ().smobbed_copy ();
 		}
 	}
@@ -1217,6 +1218,13 @@ context_mod_list:
                  if (md)
                      unsmob_context_mod ($1)->add_context_mods (md->get_mods ());
         }
+	| context_mod_list embedded_scm {
+		Context_mod *md = unsmob_context_mod ($2);
+		if (md)
+			unsmob_context_mod ($1)->add_context_mods (md->get_mods ());
+		else
+			parser->parser_error (@2, _ ("not a context mod"));
+        }
         ;
 
 composite_music:
@@ -1258,7 +1266,7 @@ function_arglist_skip:
 	{
 		$$ = scm_cons ($1, $3);
 	} %prec FUNCTION_ARGLIST
-	| EXPECT_SCM EXPECT_DURATION function_arglist_skip
+	| EXPECT_OPTIONAL EXPECT_SCM function_arglist_skip
 	{
 		$$ = scm_cons ($1, $3);
 	} %prec FUNCTION_ARGLIST
@@ -1274,19 +1282,19 @@ function_arglist_nonbackup:
 	}
 	| EXPECT_OPTIONAL EXPECT_SCM function_arglist embedded_scm_arg_closed
 	{
-		$$ = check_scheme_arg (PARSER, @4, $4, $3, $2);
+		$$ = check_scheme_arg (parser, @4, $4, $3, $2);
 	}
 	| EXPECT_OPTIONAL EXPECT_SCM function_arglist_closed bare_number_closed
 	{
-		$$ = check_scheme_arg (PARSER, @4, $4, $3, $2);
+		$$ = check_scheme_arg (parser, @4, $4, $3, $2);
 	}
 	| EXPECT_OPTIONAL EXPECT_SCM function_arglist_closed FRACTION
 	{
-		$$ = check_scheme_arg (PARSER, @4, $4, $3, $2);
+		$$ = check_scheme_arg (parser, @4, $4, $3, $2);
 	}
 	| EXPECT_OPTIONAL EXPECT_SCM function_arglist_closed post_event_nofinger
 	{
-		$$ = check_scheme_arg (PARSER, @4, $4, $3, $2);
+		$$ = check_scheme_arg (parser, @4, $4, $3, $2);
 	}
 	| EXPECT_OPTIONAL EXPECT_SCM function_arglist_closed '-' UNSIGNED
 	{
@@ -1300,19 +1308,19 @@ function_arglist_nonbackup:
 			if (scm_is_true (scm_call_1 ($2, $$)))
 				$$ = scm_cons ($$, $3);
 			else
-				$$ = check_scheme_arg (PARSER, @4, n, $3, $2);
+				$$ = check_scheme_arg (parser, @4, n, $3, $2);
 		}
 		
 	}
 	| EXPECT_OPTIONAL EXPECT_SCM function_arglist_closed '-' REAL
 	{
-		$$ = check_scheme_arg (PARSER, @4,
+		$$ = check_scheme_arg (parser, @4,
 				       scm_difference ($5, SCM_UNDEFINED),
 				       $3, $2);
 	}
 	| EXPECT_OPTIONAL EXPECT_SCM function_arglist_closed '-' NUMBER_IDENTIFIER
 	{
-		$$ = check_scheme_arg (PARSER, @4,
+		$$ = check_scheme_arg (parser, @4,
 				       scm_difference ($5, SCM_UNDEFINED),
 				       $3, $2);
 	}
@@ -1361,10 +1369,10 @@ function_arglist_backup:
 		// well: see how we do this at the mandatory argument
 		// point.
 		if (scm_is_true (scm_call_1 ($2, $4)))
-			$$ = scm_cons ($3, $2);
+			$$ = scm_cons ($4, $3);
 		else {
 			$$ = scm_cons (loc_on_music (@3, $1), $3);
-			MYBACKUP (LYRIC_MARKUP_IDENTIFIER, $4, @4);
+			MYBACKUP (LYRICS_STRING, $4, @4);
 		}
 	}
 	| EXPECT_OPTIONAL EXPECT_SCM function_arglist_closed_keep UNSIGNED
@@ -1424,7 +1432,7 @@ function_arglist_backup:
 			else {
 				$$ = scm_cons (loc_on_music (@3, $1), $3);
 				MYBACKUP (UNSIGNED, $5, @5);
-				PARSER->lexer_->push_extra_token ('-');
+				parser->lexer_->push_extra_token ('-');
 			}
 		}
 		
@@ -1463,17 +1471,17 @@ function_arglist_backup:
 	}
 	| function_arglist_backup REPARSE embedded_scm_arg_closed
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $1, $2);
 	}
 	| function_arglist_backup REPARSE bare_number
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $1, $2);
 	}
 	| function_arglist_backup REPARSE fraction
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $1, $2);
 	}
 	;
@@ -1487,22 +1495,22 @@ function_arglist_common:
 	function_arglist_bare
 	| EXPECT_SCM function_arglist_optional embedded_scm_arg
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $2, $1);
 	}
 	| EXPECT_SCM function_arglist_closed_optional bare_number
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $2, $1);
 	}
 	| EXPECT_SCM function_arglist_closed_optional fraction
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $2, $1);
 	}
 	| EXPECT_SCM function_arglist_closed_optional post_event_nofinger
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $2, $1);
 	}
 	| function_arglist_common_minus
@@ -1521,12 +1529,12 @@ function_arglist_common_lyric:
 		// something looking like a duration or post event
 		// follows, we take the event, otherwise the string).
 		SCM lyric_event = MAKE_SYNTAX ("lyric-event", @3, $3,
-					       PARSER->default_duration_.smobbed_copy ());
+					       parser->default_duration_.smobbed_copy ());
 		if (scm_is_true (scm_call_1 ($1, $3)))
 			if (scm_is_true (scm_call_1 ($1, lyric_event)))
 			{
 				$$ = $2;
-				MYREPARSE (@3, $1, LYRIC_ELEMENT_P, $3);
+				MYREPARSE (@3, $1, LYRICS_STRING, $3);
 			} else {
 				$$ = scm_cons ($3, $2);
 			}
@@ -1537,14 +1545,14 @@ function_arglist_common_lyric:
 		} else {
 			// This is going to flag a syntax error, we
 			// know the predicate to be false.
-			check_scheme_arg (PARSER, @3,
+			check_scheme_arg (parser, @3,
 					  $3, $2, $1);
 		}
 	}
 	| function_arglist_common_lyric REPARSE lyric_element_arg
 	{
 		// This should never be false
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $1, $2);
 	}
 	;
@@ -1563,7 +1571,7 @@ function_arglist_common_minus:
 			if (scm_is_true (scm_call_1 ($1, $$)))
 				$$ = scm_cons ($$, $2);
 			else
-				$$ = check_scheme_arg (PARSER, @3, n, $2, $1);
+				$$ = check_scheme_arg (parser, @3, n, $2, $1);
 		}
 		
 	}
@@ -1576,11 +1584,11 @@ function_arglist_common_minus:
 	| EXPECT_SCM function_arglist_closed_optional '-' NUMBER_IDENTIFIER
 	{
 		SCM n = scm_difference ($4, SCM_UNDEFINED);
-		$$ = check_scheme_arg (PARSER, @4, n, $2, $1);
+		$$ = check_scheme_arg (parser, @4, n, $2, $1);
 	}
 	| function_arglist_common_minus REPARSE bare_number
 	{
-		$$ = check_scheme_arg (PARSER, @3, $3, $1, $2);
+		$$ = check_scheme_arg (parser, @3, $3, $1, $2);
 	}
 	;
 
@@ -1593,12 +1601,12 @@ function_arglist_closed_common:
 	function_arglist_bare
 	| EXPECT_SCM function_arglist_optional embedded_scm_arg_closed
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $2, $1);
 	}
 	| EXPECT_SCM function_arglist_closed_optional bare_number
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $2, $1);
 	}
 	| EXPECT_SCM function_arglist_closed_optional '-' UNSIGNED
@@ -1613,35 +1621,35 @@ function_arglist_closed_common:
 			if (scm_is_true (scm_call_1 ($1, $$)))
 				$$ = scm_cons ($$, $2);
 			else
-				$$ = check_scheme_arg (PARSER, @3, n, $2, $1);
+				$$ = check_scheme_arg (parser, @3, n, $2, $1);
 		}
 		
 	}
 	| EXPECT_SCM function_arglist_closed_optional '-' REAL
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       scm_difference ($4, SCM_UNDEFINED),
 				       $2, $1);
 	}
 	| EXPECT_SCM function_arglist_closed_optional '-' NUMBER_IDENTIFIER
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       scm_difference ($4, SCM_UNDEFINED),
 				       $2, $1);
 	}
 	| EXPECT_SCM function_arglist_closed_optional post_event_nofinger
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $2, $1);
 	}
 	| EXPECT_SCM function_arglist_closed_optional fraction
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $2, $1);
 	}
 	| EXPECT_SCM function_arglist_optional lyric_element
 	{
-		$$ = check_scheme_arg (PARSER, @3,
+		$$ = check_scheme_arg (parser, @3,
 				       $3, $2, $1);
 	}
 	;
@@ -1763,7 +1771,7 @@ mode_changed_music:
 		{
 		  $$ = $2;
 		}
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->pop_state ();
 	}
 	| mode_changing_head_with_context optional_context_mod grouped_music_list {
                 Context_mod *ctxmod = unsmob_context_mod ($2);
@@ -1775,82 +1783,82 @@ mode_changed_music:
 		{
 		  $$ = MAKE_SYNTAX ("unrelativable-music", @$, $$);
 		}
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->pop_state ();
 	}
 	;
 
 mode_changing_head:
 	NOTEMODE {
-		SCM nn = PARSER->lexer_->lookup_identifier ("pitchnames");
-		PARSER->lexer_->push_note_state (alist_to_hashq (nn));
+		SCM nn = parser->lexer_->lookup_identifier ("pitchnames");
+		parser->lexer_->push_note_state (alist_to_hashq (nn));
 
 		$$ = ly_symbol2scm ("notes");
 	}
 	| DRUMMODE
 		{
-		SCM nn = PARSER->lexer_->lookup_identifier ("drumPitchNames");
-		PARSER->lexer_->push_note_state (alist_to_hashq (nn));
+		SCM nn = parser->lexer_->lookup_identifier ("drumPitchNames");
+		parser->lexer_->push_note_state (alist_to_hashq (nn));
 
 		$$ = ly_symbol2scm ("drums");
 	}
 	| FIGUREMODE {
-		PARSER->lexer_->push_figuredbass_state ();
+		parser->lexer_->push_figuredbass_state ();
 
 		$$ = ly_symbol2scm ("figures");
 	}
 	| CHORDMODE {
-		SCM nn = PARSER->lexer_->lookup_identifier ("chordmodifiers");
-		PARSER->lexer_->chordmodifier_tab_ = alist_to_hashq (nn);
-		nn = PARSER->lexer_->lookup_identifier ("pitchnames");
-		PARSER->lexer_->push_chord_state (alist_to_hashq (nn));
+		SCM nn = parser->lexer_->lookup_identifier ("chordmodifiers");
+		parser->lexer_->chordmodifier_tab_ = alist_to_hashq (nn);
+		nn = parser->lexer_->lookup_identifier ("pitchnames");
+		parser->lexer_->push_chord_state (alist_to_hashq (nn));
 		$$ = ly_symbol2scm ("chords");
 
 	}
 	| LYRICMODE
-		{ PARSER->lexer_->push_lyric_state ();
+		{ parser->lexer_->push_lyric_state ();
 		$$ = ly_symbol2scm ("lyrics");
 	}
 	;
 
 mode_changing_head_with_context:
 	DRUMS {
-		SCM nn = PARSER->lexer_->lookup_identifier ("drumPitchNames");
-		PARSER->lexer_->push_note_state (alist_to_hashq (nn));
+		SCM nn = parser->lexer_->lookup_identifier ("drumPitchNames");
+		parser->lexer_->push_note_state (alist_to_hashq (nn));
 
 		$$ = ly_symbol2scm ("DrumStaff");
 	}
 	| FIGURES {
-		PARSER->lexer_->push_figuredbass_state ();
+		parser->lexer_->push_figuredbass_state ();
 
 		$$ = ly_symbol2scm ("FiguredBass");
 	}
 	| CHORDS {
-		SCM nn = PARSER->lexer_->lookup_identifier ("chordmodifiers");
-		PARSER->lexer_->chordmodifier_tab_ = alist_to_hashq (nn);
-		nn = PARSER->lexer_->lookup_identifier ("pitchnames");
-		PARSER->lexer_->push_chord_state (alist_to_hashq (nn));
+		SCM nn = parser->lexer_->lookup_identifier ("chordmodifiers");
+		parser->lexer_->chordmodifier_tab_ = alist_to_hashq (nn);
+		nn = parser->lexer_->lookup_identifier ("pitchnames");
+		parser->lexer_->push_chord_state (alist_to_hashq (nn));
 		$$ = ly_symbol2scm ("ChordNames");
 	}
 	| LYRICS
-		{ PARSER->lexer_->push_lyric_state ();
+		{ parser->lexer_->push_lyric_state ();
 		$$ = ly_symbol2scm ("Lyrics");
 	}
 	;
 
 new_lyrics:
-	ADDLYRICS { PARSER->lexer_->push_lyric_state (); }
+	ADDLYRICS { parser->lexer_->push_lyric_state (); }
 	/*cont */
 	composite_music {
 	/* Can also use music at the expensive of two S/Rs similar to
            \repeat \alternative */
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->pop_state ();
 
 		$$ = scm_cons ($3, SCM_EOL);
 	}
 	| new_lyrics ADDLYRICS {
-		PARSER->lexer_->push_lyric_state ();
+		parser->lexer_->push_lyric_state ();
 	} composite_music {
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->pop_state ();
 		$$ = scm_cons ($4, $1);
 	}
 	;
@@ -1860,9 +1868,9 @@ re_rhythmed_music:
 		$$ = MAKE_SYNTAX ("add-lyrics", @$, $1, scm_reverse_x ($2, SCM_EOL));
 	} %prec COMPOSITE
 	| LYRICSTO simple_string {
-		PARSER->lexer_->push_lyric_state ();
+		parser->lexer_->push_lyric_state ();
 	} music {
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->pop_state ();
 		$$ = MAKE_SYNTAX ("lyric-combine", @$, $2, $4);
 	}
 	;
@@ -1932,7 +1940,7 @@ context_mod:
 	   if (ly_symbol2scm ("consists") != $1)
 	   {
 	     $$ = SCM_EOL;
-             PARSER->parser_error (@1, _ ("only \\consists takes non-string argument."));
+             parser->parser_error (@1, _ ("only \\consists takes non-string argument."));
 	   }
 	   else
 	   {
@@ -1986,7 +1994,7 @@ simple_music_property_def:
 
 music_property_def:
 	simple_music_property_def {
-		$$ = LOWLEVEL_MAKE_SYNTAX (ly_lily_module_constant ("property-operation"), scm_cons2 (PARSER->self_scm (), make_input (@$), $1));
+		$$ = LOWLEVEL_MAKE_SYNTAX (ly_lily_module_constant ("property-operation"), scm_cons2 (parser->self_scm (), make_input (@$), $1));
 	}
 	;
 
@@ -2027,10 +2035,13 @@ scalar_closed:
 
 
 event_chord:
-	/* TODO: Create a special case that avoids the creation of
-	   EventChords around simple_elements that have no post_events?
-	 */
-	simple_chord_elements post_events	{
+	simple_element post_events {
+		// Let the rhythmic music iterator sort this mess out.
+		if (scm_is_pair ($2))
+			unsmob_music ($1)->set_property ("articulations",
+							 scm_reverse_x ($2, SCM_EOL));
+	}
+	| simple_chord_elements post_events	{
 		SCM elts = ly_append2 ($1, scm_reverse_x ($2, SCM_EOL));
 
 		Input i;
@@ -2043,8 +2054,6 @@ event_chord:
 		Input i;
 		i.set_location (@1, @3);
 		$$ = MAKE_SYNTAX ("repetition-chord", i,
-				  PARSER->lexer_->chord_repetition_.last_chord_,
-				  PARSER->lexer_->chord_repetition_.repetition_function_,
 				  $2, scm_reverse_x ($3, SCM_EOL));
 	}
 	| MULTI_MEASURE_REST optional_notemode_duration post_events {
@@ -2054,13 +2063,7 @@ event_chord:
 				  scm_reverse_x ($3, SCM_EOL));
 	}
 	| command_element
-	/* note chord elements are memorized into
-	   PARSER->lexer_->chord_repetition_ so that the chord repetition
-	   mechanism copy them when a chord repetition symbol is found
-	*/
-	| note_chord_element	{
-		PARSER->lexer_->chord_repetition_.last_chord_ = $$;
-	}
+	| note_chord_element
 	;
 
 
@@ -2092,7 +2095,8 @@ chord_body:
 chord_body_elements:
 	/* empty */ 		{ $$ = SCM_EOL; }
 	| chord_body_elements chord_body_element {
-		$$ = scm_cons ($2, $1);
+		if (!SCM_UNBNDP ($2))
+			$$ = scm_cons ($2, $1);
 	}
 	;
 
@@ -2134,43 +2138,24 @@ chord_body_element:
 		$$ = n->unprotect ();
 	}
 	| music_function_chord_body
-	;
-
-/* We can't accept a music argument, not even a closed one,
- * immediately before chord_body_elements, otherwise a function \fun
- * with a signature of two music arguments can't be sorted out
- * properly in a construct like
- * <\fun { c } \fun { c } c>
- * The second call could be interpreted either as a chord constituent
- * or a music expression.
- */
-
-music_function_chord_body_arglist:
-	function_arglist_bare
-	| EXPECT_SCM music_function_chord_body_arglist embedded_scm_chord_body
 	{
-		$$ = check_scheme_arg (PARSER, @3,
-				       $3, $2, $1);
-	}
-	;
+		Music *m = unsmob_music ($1);
 
-embedded_scm_chord_body:
-	embedded_scm_bare_arg
-	| SCM_FUNCTION music_function_chord_body_arglist {
-		$$ = MAKE_SYNTAX ("music-function", @$,
-					 $1, $2);
+		while (m && m->is_mus_type ("music-wrapper-music")) {
+			$$ = m->get_property ("element");
+			m = unsmob_music ($$);
+		}
+
+		if (!(m && m->is_mus_type ("rhythmic-event"))) {
+			parser->parser_error (@$, _ ("not a rhythmic event"));
+			$$ = SCM_UNDEFINED;
+		}
 	}
-	| bare_number
-	| fraction
-	| lyric_element
-	| chord_body_element
 	;
 
 music_function_chord_body:
-	MUSIC_FUNCTION music_function_chord_body_arglist {
-		$$ = MAKE_SYNTAX ("music-function", @$,
-					 $1, $2);
-	}
+	music_function_call
+	| MUSIC_IDENTIFIER
 	;
 
 // Event functions may only take closed arglists, otherwise it would
@@ -2210,7 +2195,7 @@ command_element:
 		$$ = MAKE_SYNTAX ("voice-separator", @$);
 	}
 	| '|'      {
-		SCM pipe = PARSER->lexer_->lookup_identifier ("pipeSymbol");
+		SCM pipe = parser->lexer_->lookup_identifier ("pipeSymbol");
 
 		Music *m = unsmob_music (pipe);
 		if (m)
@@ -2257,13 +2242,13 @@ post_event_nofinger:
 		}
 	}
 	| HYPHEN {
-		if (!PARSER->lexer_->is_lyric_state ())
-			PARSER->parser_error (@1, _ ("have to be in Lyric mode for lyrics"));
+		if (!parser->lexer_->is_lyric_state ())
+			parser->parser_error (@1, _ ("have to be in Lyric mode for lyrics"));
 		$$ = MY_MAKE_MUSIC ("HyphenEvent", @$)->unprotect ();
 	}
 	| EXTENDER {
-		if (!PARSER->lexer_->is_lyric_state ())
-			PARSER->parser_error (@1, _ ("have to be in Lyric mode for lyrics"));
+		if (!parser->lexer_->is_lyric_state ())
+			parser->parser_error (@1, _ ("have to be in Lyric mode for lyrics"));
 		$$ = MY_MAKE_MUSIC ("ExtenderEvent", @$)->unprotect ();
 	}
 	| script_dir direction_reqd_event {
@@ -2345,7 +2330,7 @@ direction_less_char:
 
 direction_less_event:
 	direction_less_char {
-		SCM predefd = PARSER->lexer_->lookup_identifier_symbol ($1);
+		SCM predefd = parser->lexer_->lookup_identifier_symbol ($1);
 		Music *m = 0;
 		if (unsmob_music (predefd))
 		{
@@ -2374,11 +2359,11 @@ direction_reqd_event:
 		$$ = $1;
 	}
 	| script_abbreviation {
-		SCM s = PARSER->lexer_->lookup_identifier ("dash" + ly_scm2string ($1));
+		SCM s = parser->lexer_->lookup_identifier ("dash" + ly_scm2string ($1));
 		Music *a = MY_MAKE_MUSIC ("ArticulationEvent", @$);
 		if (scm_is_string (s))
 			a->set_property ("articulation-type", s);
-		else PARSER->parser_error (@1, _ ("expecting string as script definition"));
+		else parser->parser_error (@1, _ ("expecting string as script definition"));
 		$$ = a->unprotect ();
 	}
 	;
@@ -2517,12 +2502,12 @@ duration_length:
 
 optional_notemode_duration:
 	{
-		Duration dd = PARSER->default_duration_;
+		Duration dd = parser->default_duration_;
 		$$ = dd.smobbed_copy ();
 	}
 	| multiplied_duration	{
 		$$ = $1;
-		PARSER->default_duration_ = *unsmob_duration ($$);
+		parser->default_duration_ = *unsmob_duration ($$);
 	}
 	;
 
@@ -2530,7 +2515,7 @@ steno_duration:
 	bare_unsigned dots		{
 		int len = 0;
 		if (!is_duration ($1))
-			PARSER->parser_error (@1, _f ("not a duration: %d", $1));
+			parser->parser_error (@1, _f ("not a duration: %d", $1));
 		else
 			len = intlog2 ($1);
 
@@ -2581,7 +2566,7 @@ tremolo_type:
 	}
 	| ':' bare_unsigned {
 		if (!is_duration ($2))
-			PARSER->parser_error (@2, _f ("not a duration: %d", $2));
+			parser->parser_error (@2, _f ("not a duration: %d", $2));
 		$$ = $2;
 	}
 	;
@@ -2699,8 +2684,8 @@ optional_rest:
 
 simple_element:
 	pitch exclamations questions octave_check optional_notemode_duration optional_rest {
-		if (!PARSER->lexer_->is_note_state ())
-			PARSER->parser_error (@1, _ ("have to be in Note mode for notes"));
+		if (!parser->lexer_->is_note_state ())
+			parser->parser_error (@1, _ ("have to be in Note mode for notes"));
 
 		Music *n = 0;
 		if ($6)
@@ -2747,12 +2732,9 @@ simple_element:
 	;
 
 simple_chord_elements:
-	simple_element	{
-		$$ = scm_list_1 ($1);
-	}
-	| new_chord {
-                if (!PARSER->lexer_->is_chord_state ())
-                        PARSER->parser_error (@1, _ ("have to be in Chord mode for chords"));
+	new_chord {
+                if (!parser->lexer_->is_chord_state ())
+                        parser->parser_error (@1, _ ("have to be in Chord mode for chords"));
                 $$ = $1;
 	}
 	| figure_spec optional_notemode_duration {
@@ -2771,51 +2753,47 @@ lyric_element:
 	| LYRICS_STRING {
 		$$ = $1;
 	}
-	| LYRIC_ELEMENT_P
 	;
 
 lyric_element_arg:
 	lyric_element
 	| lyric_element multiplied_duration post_events {
-		SCM lyric_event = MAKE_SYNTAX ("lyric-event", @$, $1, $2);
-		$$ = MAKE_SYNTAX ("event-chord", @$,
-				  scm_cons (lyric_event,
-					    scm_reverse_x ($3, SCM_EOL)));
+		$$ = MAKE_SYNTAX ("lyric-event", @$, $1, $2);
+		if (scm_is_pair ($3))
+			unsmob_music ($$)->set_property
+				("articulations", scm_reverse_x ($3, SCM_EOL));
 	}
 	| lyric_element post_event post_events {
-		SCM lyric_event =
-			MAKE_SYNTAX ("lyric-event", @$, $1,
-				     PARSER->default_duration_.smobbed_copy ());
-		$$ = MAKE_SYNTAX ("event-chord", @$,
-				  scm_cons2 (lyric_event, $2,
-					     scm_reverse_x ($3, SCM_EOL)));
-					     
+		$$ = MAKE_SYNTAX ("lyric-event", @$, $1,
+				  parser->default_duration_.smobbed_copy ());
+		unsmob_music ($$)->set_property
+			("articulations", scm_cons ($2, scm_reverse_x ($3, SCM_EOL)));
 	}
 	| LYRIC_ELEMENT optional_notemode_duration post_events {
-		SCM lyric_event = MAKE_SYNTAX ("lyric-event", @$, $1, $2);
-		$$ = MAKE_SYNTAX ("event-chord", @$,
-				  scm_cons (lyric_event,
-					    scm_reverse_x ($3, SCM_EOL)));
+		$$ = MAKE_SYNTAX ("lyric-event", @$, $1, $2);
+		if (scm_is_pair ($3))
+			unsmob_music ($$)->set_property
+				("articulations", scm_reverse_x ($3, SCM_EOL));
 	}
 	;
 
 
 lyric_element_music:
 	lyric_element optional_notemode_duration post_events {
-		SCM lyric_event = MAKE_SYNTAX ("lyric-event", @$, $1, $2);
-		$$ = MAKE_SYNTAX ("event-chord", @$,
-				  scm_cons (lyric_event,
-					    scm_reverse_x ($3, SCM_EOL)));
+		$$ = MAKE_SYNTAX ("lyric-event", @$, $1, $2);
+		if (scm_is_pair ($3))
+			unsmob_music ($$)->set_property
+				("articulations", scm_reverse_x ($3, SCM_EOL));
 	}
 	;
 
 new_chord:
 	steno_tonic_pitch optional_notemode_duration   {
-		$$ = make_chord_elements ($1, $2, SCM_EOL);
+		$$ = make_chord_elements (@$, $1, $2, SCM_EOL);
 	}
 	| steno_tonic_pitch optional_notemode_duration chord_separator chord_items {
 		SCM its = scm_reverse_x ($4, SCM_EOL);
-		$$ = make_chord_elements ($1, $2, scm_cons ($3, its));
+		$$ = make_chord_elements (@$, $1, $2, scm_cons ($3, its));
 	}
 	;
 
@@ -2922,10 +2900,10 @@ number_factor:
 bare_number:
 	bare_number_closed
 	| UNSIGNED NUMBER_IDENTIFIER	{
-		$$ = scm_product ($1, $2)
+		$$ = scm_product ($1, $2);
 	}
 	| REAL NUMBER_IDENTIFIER	{
-		$$ = scm_product ($1, $2)
+		$$ = scm_product ($1, $2);
 	}
 	;
 
@@ -2965,10 +2943,10 @@ lyric_markup:
 		$$ = $1;
 	}
 	| LYRIC_MARKUP
-		{ PARSER->lexer_->push_markup_state (); }
+		{ parser->lexer_->push_markup_state (); }
 	markup_top {
 		$$ = $3;
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->pop_state ();
 	}
 	;
 
@@ -2977,10 +2955,10 @@ full_markup_list:
 		$$ = $1;
 	}
 	| MARKUPLIST
-		{ PARSER->lexer_->push_markup_state (); }
+		{ parser->lexer_->push_markup_state (); }
 	markup_list {
 		$$ = $3;
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->pop_state ();
 	}
 	;
 
@@ -2989,10 +2967,10 @@ full_markup:
 		$$ = $1;
 	}
 	| MARKUP
-		{ PARSER->lexer_->push_markup_state (); }
+		{ parser->lexer_->push_markup_state (); }
 	markup_top {
 		$$ = $3;
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->pop_state ();
 	}
 	;
 
@@ -3016,7 +2994,7 @@ markup_scm:
 		else if (Text_interface::is_markup_list ($1))
 			MYBACKUP (MARKUPLIST_IDENTIFIER, $1, @1);
 		else {
-			PARSER->parser_error (@1, _("markup expected"));
+			parser->parser_error (@1, _ ("not a markup"));
 			MYBACKUP (MARKUP_IDENTIFIER, scm_string (SCM_EOL), @1);
 		}
 	} BACKUP
@@ -3076,7 +3054,7 @@ markup_command_basic_arguments:
 	  $$ = scm_cons ($3, $2);
 	}
 	| EXPECT_SCM markup_command_list_arguments embedded_scm_closed {
-	  $$ = check_scheme_arg (PARSER, @3, $3, $2, $1);
+	  $$ = check_scheme_arg (parser, @3, $3, $2, $1);
 	}
 	| EXPECT_NO_MORE_ARGS {
 	  $$ = SCM_EOL;
@@ -3119,13 +3097,13 @@ simple_markup:
 		$$ = $1;
 	}
 	| SCORE {
-		SCM nn = PARSER->lexer_->lookup_identifier ("pitchnames");
-		PARSER->lexer_->push_note_state (alist_to_hashq (nn));
+		SCM nn = parser->lexer_->lookup_identifier ("pitchnames");
+		parser->lexer_->push_note_state (alist_to_hashq (nn));
 	} '{' score_body '}' {
 		Score * sc = $4;
 		$$ = scm_list_2 (ly_lily_module_constant ("score-markup"), sc->self_scm ());
 		sc->unprotect ();
-		PARSER->lexer_->pop_state ();
+		parser->lexer_->pop_state ();
 	}
 	| MARKUP_FUNCTION markup_command_basic_arguments {
 		$$ = scm_cons ($1, scm_reverse_x ($2, SCM_EOL));
@@ -3157,7 +3135,7 @@ Lily_parser::set_yydebug (bool x)
 void
 Lily_parser::do_yyparse ()
 {
-	yyparse ((void*)this);
+	yyparse (this);
 }
 
 
@@ -3208,9 +3186,7 @@ Lily_lexer::try_special_identifiers (SCM *destination, SCM sid)
 		unsmob_music (*destination)->
 			set_property ("origin", make_input (last_input_));
 
-		bool is_event = scm_memq (ly_symbol2scm ("event"), mus->get_property ("types"))
-			!= SCM_BOOL_F;
-
+		bool is_event = mus->is_mus_type ("post-event");
 		mus->unprotect ();
 		return is_event ? EVENT_IDENTIFIER : MUSIC_IDENTIFIER;
 	} else if (unsmob_pitch (sid)) {
@@ -3255,7 +3231,7 @@ get_next_unique_lyrics_context_id ()
 	return scm_from_locale_string (s);
 }
 
-SCM check_scheme_arg (Lily_parser *my_lily_parser, Input loc,
+SCM check_scheme_arg (Lily_parser *parser, Input loc,
 		      SCM arg, SCM args, SCM pred)
 {
 	args = scm_cons (arg, args);
@@ -3305,19 +3281,6 @@ make_music_with_input (SCM name, Input where)
        return m;
 }
 
-SCM
-get_first_context_id (SCM type, Music *m)
-{
-	SCM id = m->get_property ("context-id");
-	if (SCM_BOOL_T == scm_equal_p (m->get_property ("context-type"), type)
-	    && scm_is_string (m->get_property ("context-id"))
-	    && scm_c_string_length (id) > 0)
-	{
-		return id;
-	}
-	return SCM_EOL;
-}
-
 SCM
 make_simple_markup (SCM a)
 {
@@ -3352,39 +3315,23 @@ make_chord_step (int step, Rational alter)
 
 
 SCM
-make_chord_elements (SCM pitch, SCM dur, SCM modification_list)
+make_chord_elements (Input loc, SCM pitch, SCM dur, SCM modification_list)
 {
 	SCM chord_ctor = ly_lily_module_constant ("construct-chord-elements");
-	return scm_call_3 (chord_ctor, pitch, dur, modification_list);
-}
-
-SCM
-try_unpack_lyrics (SCM pred, SCM arg)
-{
-	if (Music *m = unsmob_music (arg))
-		if (m->is_mus_type ("lyric-event")) {
-			SCM text = m->get_property ("text");
-			if (scm_is_true (scm_call_1 (pred, text)))
-					return text;
-			}
-	return SCM_UNDEFINED;
-}	
-
-/* Todo: actually also use apply iso. call too ...  */
-bool
-ly_input_procedure_p (SCM x)
-{
-	return ly_is_procedure (x)
-		|| (scm_is_pair (x) && ly_is_procedure (scm_car (x)));
+	SCM res = scm_call_3 (chord_ctor, pitch, dur, modification_list);
+	for (SCM s = res; scm_is_pair (s); s = scm_cdr (s))
+	{
+		unsmob_music (scm_car (s))->set_spot (loc);
+	}
+	return res;
 }
 
 int
-yylex (YYSTYPE *s, YYLTYPE *loc, void *v)
+yylex (YYSTYPE *s, YYLTYPE *loc, Lily_parser *parser)
 {
-	Lily_parser *pars = (Lily_parser*) v;
-	Lily_lexer *lex = pars->lexer_;
+	Lily_lexer *lex = parser->lexer_;
 
-	lex->lexval_ = (void*) s;
+	lex->lexval_ = s;
 	lex->lexloc_ = loc;
 	lex->prepare_for_next_token ();
 	return lex->yylex ();