X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fparser.yy;h=4bf3c9d191e98d5585404c45d16b2aa3858fb2ed;hb=3ee551fa4f2a2c3736a06f9f85f29b9f7bd89818;hp=1d54ba97601290bdb5da524b8be090286f9080f0;hpb=cc676c5aadd45985251b5d60fa23eed1ed98f6e6;p=lilypond.git

diff --git a/lily/parser.yy b/lily/parser.yy
index 1d54ba9760..4bf3c9d191 100644
--- a/lily/parser.yy
+++ b/lily/parser.yy
@@ -3,12 +3,13 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 1997--2006 Han-Wen Nienhuys <hanwen@xs4all.nl>
+  (c) 1997--2007 Han-Wen Nienhuys <hanwen@xs4all.nl>
                  Jan Nieuwenhuizen <janneke@gnu.org>
 */
 
 %{
 
+#define YYDEBUG 1
 #define YYERROR_VERBOSE 1
 #define YYPARSE_PARAM my_lily_parser
 #define YYLEX_PARAM my_lily_parser
@@ -74,13 +75,11 @@ using namespace std;
 #include "context-def.hh"
 #include "dimensions.hh"
 #include "file-path.hh"
-#include "input-smob.hh"
 #include "input.hh"
 #include "international.hh"
 #include "lily-guile.hh"
 #include "lily-lexer.hh"
 #include "lily-parser.hh"
-#include "lilypond-input-version.hh"
 #include "main.hh"
 #include "misc.hh"
 #include "music.hh"
@@ -108,7 +107,7 @@ using namespace std;
 
 %{
 
-#define MY_MAKE_MUSIC(x)  make_music_by_name (ly_symbol2scm (x))
+#define MY_MAKE_MUSIC(x, spot)  make_music_with_input (ly_symbol2scm (x), spot)
 
 /* ES TODO:
 - Don't use lily module, create a new module instead.
@@ -132,12 +131,12 @@ SCM get_next_unique_lyrics_context_id ();
 #endif
 
 
-
-SCM make_music_relative (Pitch start, SCM music);
+static Music *make_music_with_input (SCM name, Input where);
+SCM make_music_relative (Pitch start, SCM music, Input loc);
 SCM run_music_function (Lily_parser *, SCM expr);
 SCM get_first_context_id (SCM type, Music *m);
 SCM make_chord_elements (SCM pitch, SCM dur, SCM modification_list);
-SCM make_chord_step (int step, int alter);
+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);
@@ -157,7 +156,6 @@ void set_music_properties (Music *p, SCM a);
 /* Keyword tokens with plain escaped name.  */
 %token ACCEPTS "\\accepts"
 %token ADDLYRICS "\\addlyrics"
-%token ADDQUOTE "\\addquote"
 %token ALIAS "\\alias"
 %token ALTERNATIVE "\\alternative"
 %token BOOK "\\book"
@@ -250,13 +248,19 @@ If we give names, Bison complains.
 %token CHORDMODIFIERS
 %token LYRIC_MARKUP
 %token MULTI_MEASURE_REST
-%token SCM_T
 
 
 %token <i> DIGIT
 %token <i> E_UNSIGNED
 %token <i> UNSIGNED
 
+/* Artificial tokens, for more generic function syntax */
+%token <i> EXPECT_MARKUP;
+%token <i> EXPECT_MUSIC;
+%token <i> EXPECT_SCM;
+/* After the last argument. */
+%token <i> EXPECT_NO_MORE_ARGS;
+
 %token <scm> BOOK_IDENTIFIER
 %token <scm> CHORDMODIFIER_PITCH
 %token <scm> CHORD_MODIFIER
@@ -278,22 +282,6 @@ If we give names, Bison complains.
 %token <scm> MARKUP_HEAD_SCM0_SCM1_SCM2
 %token <scm> MARKUP_IDENTIFIER
 %token <scm> MUSIC_FUNCTION
-%token <scm> MUSIC_FUNCTION_MARKUP 
-%token <scm> MUSIC_FUNCTION_MARKUP_MARKUP 
-%token <scm> MUSIC_FUNCTION_MARKUP_MARKUP_MUSIC 
-%token <scm> MUSIC_FUNCTION_MARKUP_MUSIC 
-%token <scm> MUSIC_FUNCTION_MARKUP_MUSIC_MUSIC 
-%token <scm> MUSIC_FUNCTION_MUSIC 
-%token <scm> MUSIC_FUNCTION_MUSIC_MUSIC 
-%token <scm> MUSIC_FUNCTION_SCM 
-%token <scm> MUSIC_FUNCTION_SCM_MUSIC 
-%token <scm> MUSIC_FUNCTION_SCM_MUSIC_MUSIC 
-%token <scm> MUSIC_FUNCTION_SCM_SCM_MUSIC_MUSIC 
-%token <scm> MUSIC_FUNCTION_SCM_SCM 
-%token <scm> MUSIC_FUNCTION_SCM_SCM_MUSIC 
-%token <scm> MUSIC_FUNCTION_SCM_SCM_SCM 
-%token <scm> MUSIC_FUNCTION_SCM_SCM_SCM_MUSIC 
-%token <scm> MUSIC_FUNCTION_SCM_SCM_SCM_SCM_MUSIC 
 %token <scm> MUSIC_IDENTIFIER
 %token <scm> NOTENAME_PITCH
 %token <scm> NUMBER_IDENTIFIER
@@ -301,7 +289,7 @@ If we give names, Bison complains.
 %token <scm> REAL
 %token <scm> RESTNAME
 %token <scm> SCM_IDENTIFIER
-%token <scm> SCM_T
+%token <scm> SCM_TOKEN
 %token <scm> SCORE_IDENTIFIER
 %token <scm> STRING
 %token <scm> STRING_IDENTIFIER
@@ -312,7 +300,7 @@ If we give names, Bison complains.
 %type <book> book_body
 
 %type <i> bare_unsigned
-%type <i> figured_bass_alteration
+%type <scm> figured_bass_alteration
 %type <i> dots
 %type <i> exclamations
 %type <i> optional_rest
@@ -362,10 +350,7 @@ If we give names, Bison complains.
 %type <scm> absolute_pitch
 %type <scm> assignment_id
 %type <scm> bare_number
-%type <scm> music_function_event
-%type <scm> music_function_chord_body
-%type <scm> music_function_musicless_prefix
-%type <scm> music_function_musicless_function
+%type <scm> unsigned_number
 %type <scm> bass_figure
 %type <scm> figured_bass_modification
 %type <scm> br_bass_figure
@@ -387,6 +372,11 @@ If we give names, Bison complains.
 %type <scm> figure_spec
 %type <scm> fraction
 %type <scm> full_markup
+%type <scm> function_scm_argument
+%type <scm> function_arglist
+%type <scm> function_arglist_music_last
+%type <scm> function_arglist_nonmusic_last
+%type <scm> function_arglist_nonmusic
 %type <scm> identifier_init
 %type <scm> lilypond_header
 %type <scm> lilypond_header_body
@@ -403,6 +393,9 @@ If we give names, Bison complains.
 %type <scm> mode_changing_head
 %type <scm> mode_changing_head_with_context
 %type <scm> multiplied_duration
+%type <scm> music_function_identifier_musicless_prefix
+%type <scm> music_function_event
+%type <scm> music_function_chord_body
 %type <scm> new_chord
 %type <scm> new_lyrics
 %type <scm> number_expression
@@ -417,6 +410,7 @@ If we give names, Bison complains.
 %type <scm> pitch_also_in_chords
 %type <scm> post_events
 %type <scm> property_operation
+%type <scm> property_path
 %type <scm> scalar
 %type <scm> script_abbreviation
 %type <scm> simple_chord_elements
@@ -428,7 +422,6 @@ If we give names, Bison complains.
 %type <scm> step_number
 %type <scm> step_numbers 
 %type <scm> string
-%type <scm> function_scm_argument
 
 %type <score> score_block
 %type <score> score_body
@@ -504,14 +497,14 @@ toplevel_expression:
 	;
 
 embedded_scm:
-	SCM_T
+	SCM_TOKEN
 	| SCM_IDENTIFIER
 	;
 
 
 lilypond_header_body:
 	{
-		$$ = get_header(PARSER);
+		$$ = get_header (PARSER);
 		PARSER->lexer_->add_scope ($$);
 	}
 	| lilypond_header_body assignment  {
@@ -535,15 +528,6 @@ assignment_id:
 
 assignment:
 	assignment_id '=' identifier_init  {
-		if (! is_regular_identifier ($1))
-		{
-#if 0
-			/* no longer valid with dashes in \paper{} block. */ 
-			@1.warning (_ ("identifier should have alphabetic characters only"));
-#endif
-		}
-
-
 	        PARSER->lexer_->set_identifier ($1, $3);
 
 /*
@@ -616,11 +600,11 @@ context_def_spec_block:
 context_def_spec_body:
 	/**/ {
 		$$ = Context_def::make_scm ();
-		unsmob_context_def ($$)->set_spot (@$);
+		unsmob_context_def ($$)->origin ()->set_spot (@$);
 	}
 	| CONTEXT_DEF_IDENTIFIER {
 		$$ = $1;
-		unsmob_context_def ($$)->set_spot (@$);
+		unsmob_context_def ($$)->origin ()->set_spot (@$);
 	}
 	| context_def_spec_body GROBDESCRIPTIONS embedded_scm {
 		Context_def*td = unsmob_context_def ($$);
@@ -652,14 +636,15 @@ book_block:
 book_body:
 	{
 		$$ = new Book;
-		$$->set_spot (@$);
+		$$->origin ()->set_spot (@$);
 		$$->paper_ = dynamic_cast<Output_def*> (unsmob_output_def (PARSER->lexer_->lookup_identifier ("$defaultpaper"))->clone ());
 		$$->paper_->unprotect ();
 		$$->header_ = PARSER->lexer_->lookup_identifier ("$defaultheader"); 
 	}
 	| BOOK_IDENTIFIER {
 		$$ = unsmob_book ($1);
-		$$->set_spot (@$);
+		$$->protect ();
+		$$->origin ()->set_spot (@$);
 	}
 	| book_body paper_block {
 		$$->paper_ = $2;
@@ -700,11 +685,12 @@ score_body:
 		// pass ownernship to C++ again.
 		$$ = unsmob_score (score);
 		$$->protect ();
-		$$->set_spot (@$);
+		$$->origin ()->set_spot (@$);
 	}
 	| SCORE_IDENTIFIER {
 		$$ = unsmob_score ($1);
-		$$->set_spot (@$);
+		$$->protect ();
+		$$->origin ()->set_spot (@$);
 	}
 	| score_body object_id_setting {
 		$$->user_key_ = ly_scm2string ($2);
@@ -740,6 +726,7 @@ paper_block:
 		if ($$->lookup_variable (ly_symbol2scm ("is-paper")) != SCM_BOOL_T)
 		{
 			PARSER->parser_error (@1, _ ("need \\paper for paper block"));
+			$1->unprotect ();
 			$$ = get_paper (PARSER);
 		}
 	}
@@ -788,9 +775,11 @@ output_def_body:
 	}
 	| output_def_head_with_mode_switch '{' OUTPUT_DEF_IDENTIFIER 	{
 		$1->unprotect ();
+
 		Output_def *o = unsmob_output_def ($3);
 		o->input_origin_.set_spot (@$);
 		$$ = o;
+		$$->protect ();
 		PARSER->lexer_->remove_scope ();
 		PARSER->lexer_->add_scope (o->scope_);
 	}
@@ -800,15 +789,6 @@ output_def_body:
 	| output_def_body context_def_spec_block	{
 		assign_context_def ($$, $2);
 	}
-	| output_def_body tempo_event  {
-		/*
-			junk this ? there already is tempo stuff in
-			music.
-		*/
-		int m = scm_to_int (unsmob_music($2)->get_property ("metronome-count"));
-		Duration *d = unsmob_duration (unsmob_music($2)->get_property ("tempo-unit"));
-		set_tempo ($$, d->get_length (), m);
-	}
 	| output_def_body error {
 
 	}
@@ -816,11 +796,8 @@ output_def_body:
 
 tempo_event:
 	TEMPO steno_duration '=' bare_unsigned	{
-		Music *m = MY_MAKE_MUSIC ("MetronomeChangeEvent");
-		m->set_property ("tempo-unit", $2);
-		m->set_property ("metronome-count", scm_from_int ( $4));
-		$$ = m->unprotect ();
-	}
+		$$ = MAKE_SYNTAX ("tempo", @$, $2, scm_int2num ($4));
+	}				
 	;
 
 /*
@@ -848,7 +825,7 @@ music_list:
 
 	}
 	| music_list error {
-		Music *m = MY_MAKE_MUSIC("Music");
+		Music *m = MY_MAKE_MUSIC("Music", @$);
 		// ugh. code dup 
 		m->set_property ("error-found", SCM_BOOL_T);
 		SCM s = $$;
@@ -879,9 +856,9 @@ alternative_music:
 
 
 repeated_music:
-	REPEAT simple_string bare_unsigned music alternative_music
+	REPEAT simple_string unsigned_number music alternative_music
 	{
-		$$ = MAKE_SYNTAX ("repeat", @$, $2, scm_int2num ($3), $4, $5);
+		$$ = MAKE_SYNTAX ("repeat", @$, $2, $3, $4, $5);
 	}
 	;
 
@@ -942,72 +919,48 @@ function_scm_argument:
 	| simple_string
 	;
 
-/*
-TODO: use code generation for this
-*/
-music_function_musicless_function:
-	MUSIC_FUNCTION {
-		$$ = scm_list_2 ($1, make_input (@$));
-	}
-	| MUSIC_FUNCTION_SCM function_scm_argument {
-		$$ = scm_list_3 ($1, make_input (@$), $2);
-	}
-	| MUSIC_FUNCTION_MARKUP full_markup {
-		$$ = scm_list_3 ($1, make_input (@$), $2);
-	}
-	| MUSIC_FUNCTION_SCM_SCM function_scm_argument function_scm_argument {
-		$$ = scm_list_4 ($1, make_input (@$), $2, $3);
-	}
-	| MUSIC_FUNCTION_SCM_SCM_SCM function_scm_argument function_scm_argument function_scm_argument {
-		$$ = scm_list_5 ($1, make_input (@$), $2, $3, $4);
-	}
-	| MUSIC_FUNCTION_MARKUP_MARKUP full_markup full_markup {
-		$$ = scm_list_4 ($1, make_input (@$), $2, $3);
+/* An argument list. If a function \foo expects scm scm music, then the lexer expands \foo into the token sequence:
+ MUSIC_FUNCTION EXPECT_MUSIC EXPECT_SCM EXPECT_SCM
+and this rule returns the reversed list of arguments. */
+
+function_arglist_music_last:
+	EXPECT_MUSIC function_arglist music {
+		$$ = scm_cons ($3, $2);
 	}
 	;
 
-/*
-TODO: use code generation for this
-*/
-music_function_musicless_prefix:
-	MUSIC_FUNCTION_MUSIC {
-		$$ = scm_list_2 ($1, make_input (@$));
+function_arglist_nonmusic_last:
+	EXPECT_MARKUP function_arglist full_markup {
+		$$ = scm_cons ($3, $2);
 	}
-	| MUSIC_FUNCTION_SCM_MUSIC function_scm_argument { 
-		$$ = scm_list_3 ($1, make_input (@$), $2);
+	| EXPECT_SCM function_arglist function_scm_argument {
+		$$ = scm_cons ($3, $2);
 	}
-	| MUSIC_FUNCTION_SCM_SCM_MUSIC function_scm_argument function_scm_argument {
-		$$ = scm_list_4 ($1, make_input (@$), $2, $3);
+	;
+
+function_arglist_nonmusic: EXPECT_NO_MORE_ARGS {
+		$$ = SCM_EOL;
 	}
-	| MUSIC_FUNCTION_SCM_SCM_SCM_MUSIC function_scm_argument function_scm_argument function_scm_argument {
-		$$ = scm_list_5 ($1, make_input (@$), $2, $3, $4);
+	| EXPECT_MARKUP function_arglist_nonmusic full_markup {
+		$$ = scm_cons ($3, $2);
 	}
-	| MUSIC_FUNCTION_SCM_SCM_SCM_SCM_MUSIC function_scm_argument function_scm_argument function_scm_argument function_scm_argument {
-		$$ = scm_list_n ($1, make_input (@$), $2, $3, $4, $5, SCM_UNDEFINED);
+	| EXPECT_SCM function_arglist_nonmusic function_scm_argument {
+		$$ = scm_cons ($3, $2);
 	}
-	| MUSIC_FUNCTION_MARKUP_MUSIC full_markup {
-		$$ = scm_list_3 ($1, make_input (@$), $2);
+	;
+
+function_arglist: EXPECT_NO_MORE_ARGS {
+		/* This is for 0-ary functions, so they don't need to
+		   read a lookahead token */
+		$$ = SCM_EOL;
 	}
+	| function_arglist_music_last
+	| function_arglist_nonmusic_last
 	;
 
 generic_prefix_music_scm:
-	music_function_musicless_function {
-		$$ = $1;
-	}
-	| music_function_musicless_prefix music {
-		$$ = ly_append2 ($1, scm_list_1 ($2));
-	}
-	| MUSIC_FUNCTION_MUSIC_MUSIC music music {
-		$$ = scm_list_4 ($1, make_input (@$), $2, $3);
-	}
-	| MUSIC_FUNCTION_SCM_MUSIC_MUSIC function_scm_argument music music {
-		$$ = scm_list_5 ($1, make_input (@$), $2, $3, $4);
-	}
-	| MUSIC_FUNCTION_SCM_SCM_MUSIC_MUSIC function_scm_argument function_scm_argument music music {
-		$$ = scm_list_n ($1, make_input (@$), $2, $3, $4, $5, SCM_UNDEFINED);
-	}
-	| MUSIC_FUNCTION_MARKUP_MUSIC_MUSIC full_markup music music {
-		$$ = scm_list_5 ($1, make_input (@$), $2, $3, $4);
+	MUSIC_FUNCTION function_arglist {
+		$$ = ly_append2 (scm_list_2 ($1, make_input (@$)), scm_reverse_x ($2, SCM_EOL));
 	}
 	;
 
@@ -1024,7 +977,7 @@ prefix_composite_music:
 	generic_prefix_music_scm {
 		$$ = run_music_function (PARSER, $1);
 	}
-	| CONTEXT    simple_string optional_id optional_context_mod music {
+	| CONTEXT simple_string optional_id optional_context_mod music {
 		$$ = MAKE_SYNTAX ("context-specification", @$, $2, $3, $5, $4, SCM_BOOL_F);
 	}
 	| NEWCONTEXT simple_string optional_id optional_context_mod music {
@@ -1126,11 +1079,11 @@ mode_changing_head_with_context:
 relative_music:
 	RELATIVE absolute_pitch music {
 		Pitch start = *unsmob_pitch ($2);
-		$$ = make_music_relative (start, $3);
+		$$ = make_music_relative (start, $3, @$);
 	}
 	| RELATIVE composite_music {
 		Pitch middle_c (0, 0, 0);
-		$$ = make_music_relative (middle_c, $2);
+		$$ = make_music_relative (middle_c, $2, @$);
 	}
 	;
 
@@ -1170,6 +1123,16 @@ context_change:
 	}
 	;
 
+
+property_path:
+	embedded_scm {
+		$$ = scm_cons ($1, SCM_EOL);
+	}
+	| property_path embedded_scm {
+		$$ = scm_cons ($2, $1);
+	}
+	;
+
 property_operation:
 	STRING '=' scalar {
 		$$ = scm_list_3 (ly_symbol2scm ("assign"),
@@ -1179,13 +1142,10 @@ property_operation:
 		$$ = scm_list_2 (ly_symbol2scm ("unset"),
 			scm_string_to_symbol ($2));
 	}
-	| OVERRIDE simple_string embedded_scm '=' embedded_scm {
-		$$ = scm_list_4 (ly_symbol2scm ("push"),
-			scm_string_to_symbol ($2), $5, $3);
-	}
-	| OVERRIDE simple_string embedded_scm embedded_scm '=' embedded_scm {
-		$$ = scm_list_5 (ly_symbol2scm ("push"),
-				scm_string_to_symbol ($2), $6, $4, $3);
+	| OVERRIDE simple_string property_path '=' embedded_scm {
+		$$ = scm_append (scm_list_2 (scm_list_3 (ly_symbol2scm ("push"),
+							scm_string_to_symbol ($2), $5),
+					     $3));
 	}
 	| REVERT simple_string embedded_scm {
 		$$ = scm_list_3 (ly_symbol2scm ("pop"),
@@ -1231,17 +1191,12 @@ context_prop_spec:
 	;
 
 simple_music_property_def:
-	OVERRIDE context_prop_spec embedded_scm '=' scalar {
-		$$ = scm_list_5 (scm_car ($2),
-			ly_symbol2scm ("OverrideProperty"),
-			scm_cadr ($2),
-			$5, $3);
-	}
-	| OVERRIDE context_prop_spec embedded_scm  embedded_scm '=' scalar {
-		$$ = scm_list_n (scm_car ($2),			
-			ly_symbol2scm ("OverrideProperty"),
-			scm_cadr ($2),
-			$6, $4, $3, SCM_UNDEFINED);
+	OVERRIDE context_prop_spec property_path '=' scalar {
+		$$ = scm_append (scm_list_2 (scm_list_n (scm_car ($2),			
+				ly_symbol2scm ("OverrideProperty"),
+				scm_cadr ($2),
+				$5, SCM_UNDEFINED),
+				$3));
 	}
 	| REVERT context_prop_spec embedded_scm {
 		$$ = scm_list_4 (scm_car ($2),
@@ -1377,9 +1332,8 @@ chord_body_element:
 		SCM check = $4;
 		SCM post = $5;
 
-		Music *n = MY_MAKE_MUSIC ("NoteEvent");
+		Music *n = MY_MAKE_MUSIC ("NoteEvent", @$);
 		n->set_property ("pitch", $1);
-		n->set_spot (@$);
 		if (q % 2)
 			n->set_property ("cautionary", SCM_BOOL_T);
 		if (ex % 2 || q % 2)
@@ -1398,10 +1352,9 @@ chord_body_element:
 		$$ = n->unprotect ();
 	}
 	| DRUM_PITCH post_events {
-		Music *n = MY_MAKE_MUSIC ("NoteEvent");
+		Music *n = MY_MAKE_MUSIC ("NoteEvent", @$);
 		n->set_property ("duration", $2);
 		n->set_property ("drum-type", $1);
-		n->set_spot (@$);
 
 		if (scm_is_pair ($2)) {
 			SCM arts = scm_reverse_x ($2, SCM_EOL);
@@ -1414,24 +1367,38 @@ chord_body_element:
 	}
 	;
 
+music_function_identifier_musicless_prefix: MUSIC_FUNCTION {
+		SCM sig = scm_object_property (yylval.scm, ly_symbol2scm ("music-function-signature"));
+		if (scm_is_pair (sig) && to_boolean (scm_memq (ly_music_p_proc, scm_cdr (scm_reverse (sig)))))
+		{
+			PARSER->parser_error (@$, "Music function applied to event may not have a Music argument, except as the last argument.");
+		}
+	}
+	;
+
 music_function_chord_body:
-	music_function_musicless_function {
-		$$ = $1;
+	/* We could allow chord functions to have multiple music arguments,
+	   but it's more consistent with music_function_event if we
+	   prohibit it here too */
+	music_function_identifier_musicless_prefix EXPECT_MUSIC function_arglist_nonmusic chord_body_element {
+		$$ = ly_append2 (scm_list_2 ($1, make_input (@$)), scm_reverse_x ($3, scm_list_1 ($4)));
 	}
-	| music_function_musicless_prefix chord_body_element {
-		$$ = ly_append2 ($1, scm_list_1 ($2));
+	| music_function_identifier_musicless_prefix function_arglist_nonmusic {
+		$$ = ly_append2 (scm_list_2 ($1, make_input (@$)), scm_reverse_x ($2, SCM_EOL));
 	}
 	;
 
 music_function_event:
-	music_function_musicless_prefix post_event {
-		$$ = ly_append2 ($1, scm_list_1 ($2));
+	/* Post-events can only have the last argument as music, without this
+	   restriction we get a shift/reduce conflict from e.g.
+	   c8-\partcombine c8 -. */
+	music_function_identifier_musicless_prefix EXPECT_MUSIC function_arglist_nonmusic post_event {
+		$$ = ly_append2 (scm_list_2 ($1, make_input (@$)), scm_reverse_x ($3, scm_list_1 ($4)));
 	}
-	| music_function_musicless_function {
-		$$ = $1;
+	| music_function_identifier_musicless_prefix function_arglist_nonmusic {
+		$$ = ly_append2 (scm_list_2 ($1, make_input (@$)), scm_reverse_x ($2, SCM_EOL));
 	}
 	;
-	
 
 command_element:
 	command_event {
@@ -1441,15 +1408,13 @@ command_element:
 		$$ = MAKE_SYNTAX ("skip-music", @$, $2);
 	}
 	| E_BRACKET_OPEN {
-		Music *m = MY_MAKE_MUSIC ("LigatureEvent");
+		Music *m = MY_MAKE_MUSIC ("LigatureEvent", @$);
 		m->set_property ("span-direction", scm_from_int (START));
-		m->set_spot (@$);
 		$$ = m->unprotect();
 	}
 	| E_BRACKET_CLOSE {
-		Music *m = MY_MAKE_MUSIC ("LigatureEvent");
+		Music *m = MY_MAKE_MUSIC ("LigatureEvent", @$);
 		m->set_property ("span-direction", scm_from_int (STOP));
-		m->set_spot (@$);
 		$$ = m->unprotect ();
 	}
 	| E_BACKSLASH {
@@ -1489,22 +1454,22 @@ command_element:
 
 command_event:
 	E_TILDE {
-		$$ = MY_MAKE_MUSIC ("PesOrFlexaEvent")->unprotect ();
+		$$ = MY_MAKE_MUSIC ("PesOrFlexaEvent", @$)->unprotect ();
 	}
 	| MARK DEFAULT  {
-		Music *m = MY_MAKE_MUSIC ("MarkEvent");
+		Music *m = MY_MAKE_MUSIC ("MarkEvent", @$);
 		$$ = m->unprotect ();
 	}
 	| tempo_event {
 		$$ = $1;
 	}
 	| KEY DEFAULT {
-		Music *key = MY_MAKE_MUSIC ("KeyChangeEvent");
+		Music *key = MY_MAKE_MUSIC ("KeyChangeEvent", @$);
 		$$ = key->unprotect ();
 	}
 	| KEY NOTENAME_PITCH SCM_IDENTIFIER 	{
 
-		Music *key = MY_MAKE_MUSIC ("KeyChangeEvent");
+		Music *key = MY_MAKE_MUSIC ("KeyChangeEvent", @$);
 		if (scm_ilength ($3) > 0)
 		{		
 			key->set_property ("pitch-alist", $3);
@@ -1539,12 +1504,12 @@ post_event:
 	| HYPHEN {
 		if (!PARSER->lexer_->is_lyric_state ())
 			PARSER->parser_error (@1, _ ("have to be in Lyric mode for lyrics"));
-		$$ = MY_MAKE_MUSIC ("HyphenEvent")->unprotect ();
+		$$ = MY_MAKE_MUSIC ("HyphenEvent", @$)->unprotect ();
 	}
 	| EXTENDER {
 		if (!PARSER->lexer_->is_lyric_state ())
 			PARSER->parser_error (@1, _ ("have to be in Lyric mode for lyrics"));
-		$$ = MY_MAKE_MUSIC ("ExtenderEvent")->unprotect ();
+		$$ = MY_MAKE_MUSIC ("ExtenderEvent", @$)->unprotect ();
 	}
 	| script_dir direction_reqd_event {
 		if ($1)
@@ -1567,9 +1532,8 @@ post_event:
 
 string_number_event:
 	E_UNSIGNED {
-		Music *s = MY_MAKE_MUSIC ("StringNumberEvent");
+		Music *s = MY_MAKE_MUSIC ("StringNumberEvent", @$);
 		s->set_property ("string-number", scm_from_int ($1));
-		s->set_spot (@$);
 		$$ = s->unprotect ();
 	}
 	;
@@ -1614,20 +1578,19 @@ direction_less_event:
 		if (unsmob_music (predefd))
 		{
 			m = unsmob_music (predefd)->clone ();
+			m->set_spot (@$);
 		}
 		else
 		{
-			m = MY_MAKE_MUSIC ("Music");
+			m = MY_MAKE_MUSIC ("Music", @$);
 		}
-		m->set_spot (@$);
 		$$ = m->unprotect ();
 	}
 	| EVENT_IDENTIFIER	{
 		$$ = $1;
 	}
 	| tremolo_type  {
-               Music *a = MY_MAKE_MUSIC ("TremoloEvent");
-               a->set_spot (@$);
+               Music *a = MY_MAKE_MUSIC ("TremoloEvent", @$);
                a->set_property ("tremolo-type", scm_from_int ($1));
                $$ = a->unprotect ();
         }
@@ -1639,7 +1602,7 @@ direction_reqd_event:
 	}
 	| script_abbreviation {
 		SCM s = PARSER->lexer_->lookup_identifier ("dash" + ly_scm2string ($1));
-		Music *a = MY_MAKE_MUSIC ("ArticulationEvent");
+		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"));
@@ -1650,7 +1613,7 @@ direction_reqd_event:
 octave_check:
 	/**/ { $$ = SCM_EOL; }
 	| '='  { $$ = scm_from_int (0); }
-	| '=' sub_quotes { $$ = scm_from_int ($2); }
+	| '=' sub_quotes { $$ = scm_from_int (-$2); }
 	| '=' sup_quotes { $$ = scm_from_int ($2); }
 	;
 
@@ -1702,7 +1665,7 @@ steno_tonic_pitch:
 		$$ = p.smobbed_copy ();
 	}
 	| TONICNAME_PITCH sub_quotes	 {
-		Pitch p =* unsmob_pitch ($1);
+		Pitch p = *unsmob_pitch ($1);
 
 		p = p.transposed (Pitch (-$2,0,0));
 		$$ = p.smobbed_copy ();
@@ -1722,47 +1685,44 @@ pitch_also_in_chords:
 
 gen_text_def:
 	full_markup {
-		Music *t = MY_MAKE_MUSIC ("TextScriptEvent");
+		Music *t = MY_MAKE_MUSIC ("TextScriptEvent", @$);
 		t->set_property ("text", $1);
-		t->set_spot (@$);
 		$$ = t->unprotect ();
 	}
 	| string {
-		Music *t = MY_MAKE_MUSIC ("TextScriptEvent");
+		Music *t = MY_MAKE_MUSIC ("TextScriptEvent", @$);
 		t->set_property ("text",
 			make_simple_markup ($1));
-		t->set_spot (@$);
 		$$ = t->unprotect ();
 	}
 	| DIGIT {
-		Music *t = MY_MAKE_MUSIC ("FingeringEvent");
+		Music *t = MY_MAKE_MUSIC ("FingeringEvent", @$);
 		t->set_property ("digit", scm_from_int ($1));
-		t->set_spot (@$);
 		$$ = t->unprotect ();
 	}
 	;
 
 script_abbreviation:
 	'^'		{
-		$$ = scm_makfrom0str ("Hat");
+		$$ = scm_from_locale_string ("Hat");
 	}
 	| '+'		{
-		$$ = scm_makfrom0str ("Plus");
+		$$ = scm_from_locale_string ("Plus");
 	}
 	| '-' 		{
-		$$ = scm_makfrom0str ("Dash");
+		$$ = scm_from_locale_string ("Dash");
 	}
  	| '|'		{
-		$$ = scm_makfrom0str ("Bar");
+		$$ = scm_from_locale_string ("Bar");
 	}
 	| ANGLE_CLOSE	{
-		$$ = scm_makfrom0str ("Larger");
+		$$ = scm_from_locale_string ("Larger");
 	}
 	| '.' 		{
-		$$ = scm_makfrom0str ("Dot");
+		$$ = scm_from_locale_string ("Dot");
 	}
 	| '_' {
-		$$ = scm_makfrom0str ("Underscore");
+		$$ = scm_from_locale_string ("Underscore");
 	}
 	;
 
@@ -1867,20 +1827,18 @@ bass_number:
 	;
 
 figured_bass_alteration:
-	'-' 	{ $$ = -2; }
-	| '+'	{ $$ = 2; }
-	| '!'	{ $$ = 0; }
+	'-' 	{ $$ = ly_rational2scm (FLAT_ALTERATION); }
+	| '+'	{ $$ = ly_rational2scm (SHARP_ALTERATION); }
+	| '!'	{ $$ = scm_from_int (0); }
 	;
 
 bass_figure:
 	FIGURE_SPACE {
-		Music *bfr = MY_MAKE_MUSIC ("BassFigureEvent");
-		$$ = bfr->self_scm ();
-		bfr->unprotect ();
-		bfr->set_spot (@1);
+		Music *bfr = MY_MAKE_MUSIC ("BassFigureEvent", @$);
+		$$ = bfr->unprotect ();
 	}
 	| bass_number  {
-		Music *bfr = MY_MAKE_MUSIC ("BassFigureEvent");
+		Music *bfr = MY_MAKE_MUSIC ("BassFigureEvent", @$);
 		$$ = bfr->self_scm ();
 
 		if (scm_is_number ($1))
@@ -1889,7 +1847,6 @@ bass_figure:
 			bfr->set_property ("text", $1);
 
 		bfr->unprotect ();
-		bfr->set_spot (@1);
 	}
 	| bass_figure ']' {
 		$$ = $1;
@@ -1897,11 +1854,11 @@ bass_figure:
 	}
 	| bass_figure figured_bass_alteration {
 		Music *m = unsmob_music ($1);
-		if ($2) {
+		if (scm_to_double ($2)) {
 			SCM salter = m->get_property ("alteration");
-			int alter = scm_is_number (salter) ? scm_to_int (salter) : 0;
+			SCM alter = scm_is_number (salter) ? salter : scm_from_int (0);
 			m->set_property ("alteration",
-				scm_from_int (alter + $2));
+					 scm_sum (alter, $2));
 		} else {
 			m->set_property ("alteration", scm_from_int (0));
 		}
@@ -1974,9 +1931,9 @@ simple_element:
 
 		Music *n = 0;
 		if ($6)
-			n = MY_MAKE_MUSIC ("RestEvent");
+			n = MY_MAKE_MUSIC ("RestEvent", @$);
 		else
-			n = MY_MAKE_MUSIC ("NoteEvent");
+			n = MY_MAKE_MUSIC ("NoteEvent", @$);
 		
 		n->set_property ("pitch", $1);
 		n->set_property ("duration", $5);
@@ -1992,11 +1949,10 @@ simple_element:
 		if ($2 % 2 || $3 % 2)
 			n->set_property ("force-accidental", SCM_BOOL_T);
 
-		n->set_spot (@$);
 		$$ = n->unprotect ();
 	}
 	| DRUM_PITCH optional_notemode_duration {
-		Music *n = MY_MAKE_MUSIC ("NoteEvent");
+		Music *n = MY_MAKE_MUSIC ("NoteEvent", @$);
 		n->set_property ("duration", $2);
 		n->set_property ("drum-type", $1);
 
@@ -2006,24 +1962,22 @@ simple_element:
 		Music *ev = 0;
  		if (ly_scm2string ($1) == "s") {
 			/* Space */
-			ev = MY_MAKE_MUSIC ("SkipEvent");
+			ev = MY_MAKE_MUSIC ("SkipEvent", @$);
 		  }
 		else {
-			ev = MY_MAKE_MUSIC ("RestEvent");
+			ev = MY_MAKE_MUSIC ("RestEvent", @$);
 		
 		    }
 		ev->set_property ("duration", $2);
-		ev->set_spot (@$);
  		$$ = ev->unprotect ();
 	}
 	| lyric_element optional_notemode_duration 	{
 		if (!PARSER->lexer_->is_lyric_state ())
 			PARSER->parser_error (@1, _ ("have to be in Lyric mode for lyrics"));
 
-		Music *levent = MY_MAKE_MUSIC ("LyricEvent");
+		Music *levent = MY_MAKE_MUSIC ("LyricEvent", @$);
 		levent->set_property ("text", $1);
 		levent->set_property ("duration",$2);
-		levent->set_spot (@$);
 		$$= levent->unprotect ();
 	}
 	;
@@ -2113,10 +2067,10 @@ step_number:
 		$$ = make_chord_step ($1, 0);
         }
 	| bare_unsigned '+' {
-		$$ = make_chord_step ($1, SHARP);
+		$$ = make_chord_step ($1, SHARP_ALTERATION);
 	}
 	| bare_unsigned CHORD_MINUS {
-		$$ = make_chord_step ($1, FLAT);
+		$$ = make_chord_step ($1, FLAT_ALTERATION);
 	}
 	;	
 
@@ -2184,6 +2138,14 @@ bare_unsigned:
 	}
 	;
 
+unsigned_number:
+	bare_unsigned  { $$ = scm_from_int ($1); }
+	| NUMBER_IDENTIFIER {
+		$$ = $1;
+	}
+	;
+	
+
 exclamations:
 		{ $$ = 0; }
 	| exclamations '!'	{ $$ ++; }
@@ -2342,11 +2304,9 @@ markup:
 %%
 
 void
-Lily_parser::set_yydebug (bool )
+Lily_parser::set_yydebug (bool x)
 {
-#if 0
-	yydebug = 1;
-#endif
+	yydebug = x;
 }
 
 void
@@ -2372,17 +2332,26 @@ Lily_lexer::try_special_identifiers (SCM *destination, SCM sid)
 		*destination = sid;
 		return STRING_IDENTIFIER;
 	} else if (unsmob_book (sid)) {
-		*destination = unsmob_book (sid)->clone ()->self_scm ();
+		Book *book =  unsmob_book (sid)->clone ();
+		*destination = book->self_scm ();
+		book->unprotect ();
+
 		return BOOK_IDENTIFIER;
 	} else if (scm_is_number (sid)) {
 		*destination = sid;
 		return NUMBER_IDENTIFIER;
 	} else if (unsmob_context_def (sid)) {
-		*destination = unsmob_context_def (sid)->clone_scm ();
+		Context_def *def= unsmob_context_def (sid)->clone ();
+
+		*destination = def->self_scm ();
+		def->unprotect ();
+		
 		return CONTEXT_DEF_IDENTIFIER;
 	} else if (unsmob_score (sid)) {
 		Score *score = new Score (*unsmob_score (sid));
 		*destination = score->self_scm ();
+
+		score->unprotect ();
 		return SCORE_IDENTIFIER;
 	} else if (Music *mus = unsmob_music (sid)) {
 		mus = mus->clone ();
@@ -2393,6 +2362,7 @@ Lily_lexer::try_special_identifiers (SCM *destination, SCM sid)
 		bool is_event = scm_memq (ly_symbol2scm ("event"), mus->get_property ("types"))
 			!= SCM_BOOL_F;
 
+		mus->unprotect ();
 		return is_event ? EVENT_IDENTIFIER : MUSIC_IDENTIFIER;
 	} else if (unsmob_duration (sid)) {
 		*destination = unsmob_duration (sid)->smobbed_copy ();
@@ -2400,8 +2370,9 @@ Lily_lexer::try_special_identifiers (SCM *destination, SCM sid)
 	} else if (unsmob_output_def (sid)) {
 		Output_def *p = unsmob_output_def (sid);
 		p = p->clone ();
-
+	
 		*destination = p->self_scm ();
+		p->unprotect ();
 		return OUTPUT_DEF_IDENTIFIER;
 	} else if (Text_interface::is_markup (sid)) {
 		*destination = sid;
@@ -2416,7 +2387,7 @@ Lily_lexer::try_special_identifiers (SCM *destination, SCM sid)
 SCM
 get_next_unique_context_id ()
 {
-	return scm_makfrom0str ("$uniqueContextId");
+	return scm_from_locale_string ("$uniqueContextId");
 }
 
 
@@ -2426,7 +2397,7 @@ get_next_unique_lyrics_context_id ()
 	static int new_context_count;
 	char s[128];
 	snprintf (s, sizeof (s)-1, "uniqueContext%d", new_context_count++);
-	return scm_makfrom0str (s);
+	return scm_from_locale_string (s);
 }
 
 
@@ -2469,6 +2440,13 @@ is_regular_identifier (SCM id)
   return v;
 }
 
+Music *
+make_music_with_input (SCM name, Input where)
+{
+       Music *m = make_music_by_name (name);
+       m->set_spot (where);
+       return m;
+}
 
 SCM
 get_first_context_id (SCM type, Music *m)
@@ -2499,15 +2477,15 @@ void
 set_music_properties (Music *p, SCM a)
 {
   for (SCM k = a; scm_is_pair (k); k = scm_cdr (k))
- 	p->internal_set_property (scm_caar (k), scm_cdar (k));
+ 	p->set_property (scm_caar (k), scm_cdar (k));
 }
 
 
 SCM
-make_chord_step (int step, int alter)
+make_chord_step (int step, Rational alter)
 {
 	if (step == 7)
-		alter += FLAT;
+		alter += FLAT_ALTERATION;
 
 	while (step < 0)
 		step += 7;
@@ -2533,9 +2511,9 @@ ly_input_procedure_p (SCM x)
 }
 
 SCM
-make_music_relative (Pitch start, SCM music)
+make_music_relative (Pitch start, SCM music, Input loc)
 {
-	Music *relative = MY_MAKE_MUSIC ("RelativeOctaveMusic");
+	Music *relative = MY_MAKE_MUSIC ("RelativeOctaveMusic", loc);
  	relative->set_property ("element", music);
 	
 	Music *m = unsmob_music (music);