]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/parser.yy
Doc: NR Update information for modern-cautionary
[lilypond.git] / lily / parser.yy
index b89c561ea806d6e8404d69800fb7201f67c9177e..8c08819d7dd9b52aaea3568ae2bcdf8604671796 100644 (file)
@@ -2,7 +2,7 @@
 /*
   This file is part of LilyPond, the GNU music typesetter.
 
-  Copyright (C) 1997--2014 Han-Wen Nienhuys <hanwen@xs4all.nl>
+  Copyright (C) 1997--2015 Han-Wen Nienhuys <hanwen@xs4all.nl>
                  Jan Nieuwenhuizen <janneke@gnu.org>
 
   LilyPond is free software: you can redistribute it and/or modify
@@ -41,9 +41,7 @@
        do {                                                            \
                if (scm_is_eq (value, SCM_UNSPECIFIED))                 \
                        break;                                          \
-               SCM s = scm_call_2 (ly_lily_module_constant ("value->lily-string"), \
-                                   value,                              \
-                                   parser->self_scm ());               \
+               SCM s = Display::value_to_lily_string (value);          \
                char *p = scm_to_locale_string (s);                     \
                fputs (p, file);                                        \
                free (p);                                               \
@@ -130,6 +128,7 @@ using namespace std;
 #include "lily-guile.hh"
 #include "lily-lexer.hh"
 #include "lily-parser.hh"
+#include "ly-module.hh"
 #include "main.hh"
 #include "misc.hh"
 #include "music.hh"
@@ -139,6 +138,7 @@ using namespace std;
 #include "score.hh"
 #include "text-interface.hh"
 #include "warn.hh"
+#include "lily-imports.hh"
 
 void
 Lily_parser::parser_error (Input const *i, Lily_parser *parser, SCM *, const string &s)
@@ -190,15 +190,36 @@ Lily_parser::parser_error (Input const *i, Lily_parser *parser, SCM *, const str
 - Don't use lily module, create a new module instead.
 - delay application of the function
 */
-#define LOWLEVEL_MAKE_SYNTAX(proc, args)       \
-  scm_apply_0 (proc, args)
+
+static SCM
+syntax_call (void *arg)
+{
+       SCM sarg = reinterpret_cast <SCM> (arg);
+       return scm_apply_0 (scm_car (sarg), scm_cdr (sarg));
+}
+
+#define LOWLEVEL_MAKE_SYNTAX(location, args)                           \
+       scm_c_with_fluid                                                \
+               (Lily::f_location,                                      \
+                parser->lexer_->override_input (location).smobbed_copy (), \
+                syntax_call,                                           \
+                reinterpret_cast <void*> (args))
+
 /* Syntactic Sugar. */
 #define MAKE_SYNTAX(name, location, ...)                               \
-       LOWLEVEL_MAKE_SYNTAX (ly_lily_module_constant (name), scm_list_n (parser->self_scm (), make_input (parser->lexer_->override_input (location)), ##__VA_ARGS__, SCM_UNDEFINED))
+       LOWLEVEL_MAKE_SYNTAX (location,                                 \
+                             scm_list_n (Lily::name,                   \
+                                         ##__VA_ARGS__, SCM_UNDEFINED))
+
 #define START_MAKE_SYNTAX(name, ...)                                   \
-       scm_list_n (ly_lily_module_constant (name) , ##__VA_ARGS__, SCM_UNDEFINED)
+       scm_list_n (Lily::name,                                         \
+                   ##__VA_ARGS__, SCM_UNDEFINED)
+
 #define FINISH_MAKE_SYNTAX(start, location, ...)                       \
-       LOWLEVEL_MAKE_SYNTAX (scm_car (start), scm_cons2 (parser->self_scm (), make_input (parser->lexer_->override_input (location)), scm_append_x (scm_list_2 (scm_cdr (start), scm_list_n (__VA_ARGS__, SCM_UNDEFINED)))))
+       LOWLEVEL_MAKE_SYNTAX (location,                                 \
+                             scm_append_x                              \
+                             (scm_list_2 (start, scm_list_n            \
+                                          (__VA_ARGS__, SCM_UNDEFINED))))
 
 SCM get_next_unique_context_id ();
 SCM get_next_unique_lyrics_context_id ();
@@ -400,34 +421,34 @@ toplevel_expression:
        }
        | book_block {
                SCM proc = parser->lexer_->lookup_identifier ("toplevel-book-handler");
-               scm_call_2 (proc, parser->self_scm (), $1);
+               scm_call_1 (proc, $1);
        }
        | bookpart_block {
                SCM proc = parser->lexer_->lookup_identifier ("toplevel-bookpart-handler");
-               scm_call_2 (proc, parser->self_scm (), $1);
+               scm_call_1 (proc, $1);
        }
        | BOOK_IDENTIFIER {
                SCM proc = parser->lexer_->lookup_identifier
-                       (unsmob_book($1)->paper_
+                       (unsmob<Book>($1)->paper_
                         ? "toplevel-book-handler"
                         : "toplevel-bookpart-handler");
-               scm_call_2 (proc, parser->self_scm (), $1);
+               scm_call_1 (proc, $1);
        }
        | score_block {
                SCM proc = parser->lexer_->lookup_identifier ("toplevel-score-handler");
-               scm_call_2 (proc, parser->self_scm (), $1);
+               scm_call_1 (proc, $1);
        }
        | composite_music {
                SCM proc = parser->lexer_->lookup_identifier ("toplevel-music-handler");
-               scm_call_2 (proc, parser->self_scm (), $1);
+               scm_call_1 (proc, $1);
        }
        | full_markup {
                SCM proc = parser->lexer_->lookup_identifier ("toplevel-text-handler");
-               scm_call_2 (proc, parser->self_scm (), scm_list_1 ($1));
+               scm_call_1 (proc, 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_call_1 (proc, $1);
        }
        | SCM_TOKEN {
                // Evaluate and ignore #xxx, as opposed to \xxx
@@ -443,23 +464,34 @@ toplevel_expression:
                if (scm_is_pair (out))
                {
                        SCM proc = parser->lexer_->lookup_identifier ("toplevel-text-handler");
-                       scm_call_2 (proc, parser->self_scm (), out);
-               } else if (unsmob_score ($1))
+                       scm_call_1 (proc, out);
+               } else if (unsmob<Score> ($1))
                {
                        SCM proc = parser->lexer_->lookup_identifier ("toplevel-score-handler");
-                       scm_call_2 (proc, parser->self_scm (), $1);
+                       scm_call_1 (proc, $1);
+               } else if (Output_def * od = unsmob<Output_def> ($1)) {
+                       SCM id = SCM_EOL;
+
+                       if (to_boolean (od->c_variable ("is-paper")))
+                               id = ly_symbol2scm ("$defaultpaper");
+                       else if (to_boolean (od->c_variable ("is-midi")))
+                               id = ly_symbol2scm ("$defaultmidi");
+                       else if (to_boolean (od->c_variable ("is-layout")))
+                               id = ly_symbol2scm ("$defaultlayout");
+
+                       parser->lexer_->set_identifier (id, $1);
                } else if (!scm_is_eq ($1, SCM_UNSPECIFIED))
                        parser->parser_error (@1, _("bad expression type"));
        }
        | output_def {
                SCM id = SCM_EOL;
-               Output_def * od = unsmob_output_def ($1);
+               Output_def * od = unsmob<Output_def> ($1);
 
-               if (od->c_variable ("is-paper") == SCM_BOOL_T)
+               if (to_boolean (od->c_variable ("is-paper")))
                        id = ly_symbol2scm ("$defaultpaper");
-               else if (od->c_variable ("is-midi") == SCM_BOOL_T)
+               else if (to_boolean (od->c_variable ("is-midi")))
                        id = ly_symbol2scm ("$defaultmidi");
-               else if (od->c_variable ("is-layout") == SCM_BOOL_T)
+               else if (to_boolean (od->c_variable ("is-layout")))
                        id = ly_symbol2scm ("$defaultlayout");
 
                parser->lexer_->set_identifier (id, $1);
@@ -519,8 +551,8 @@ embedded_scm_arg:
 
 scm_function_call:
        SCM_FUNCTION function_arglist {
-               $$ = MAKE_SYNTAX ("music-function", @$,
-                                        $1, $2);
+               $$ = MAKE_SYNTAX (music_function, @$,
+                                 $1, $2);
        }
        ;
 
@@ -543,14 +575,14 @@ embedded_lilypond:
                // for empty rules, and the only token in the whole
                // production, EMBEDDED_LILY, is synthetic and also
                // contains no source location.
-               $$ = MAKE_SYNTAX ("void-music", @$);
+               $$ = MAKE_SYNTAX (void_music, @$);
        }
        | identifier_init_nonumber
        | embedded_lilypond_number
        | post_event post_events
        {
                $$ = scm_reverse_x ($2, SCM_EOL);
-               if (Music *m = unsmob_music ($1))
+               if (Music *m = unsmob<Music> ($1))
                {
                        if (m->is_mus_type ("post-event-wrapper"))
                                $$ = scm_append
@@ -572,11 +604,11 @@ embedded_lilypond:
        | multiplied_duration
        | music_embedded music_embedded music_list {
                $3 = scm_reverse_x ($3, SCM_EOL);
-               if (unsmob_music ($2))
+               if (unsmob<Music> ($2))
                        $3 = scm_cons ($2, $3);
-               if (unsmob_music ($1))
+               if (unsmob<Music> ($1))
                        $3 = scm_cons ($1, $3);
-               $$ = MAKE_SYNTAX ("sequential-music", @$, $3);
+               $$ = MAKE_SYNTAX (sequential_music, @$, $3);
        }
        | error {
                parser->error_level_ = 1;
@@ -636,7 +668,7 @@ identifier_init:
        | post_event_nofinger post_events
        {
                $$ = scm_reverse_x ($2, SCM_EOL);
-               if (Music *m = unsmob_music ($1))
+               if (Music *m = unsmob<Music> ($1))
                {
                        if (m->is_mus_type ("post-event-wrapper"))
                                $$ = scm_append
@@ -676,10 +708,10 @@ context_def_spec_block:
        CONTEXT '{' context_def_spec_body '}'
        {
                $$ = $3;
-               Context_def *td = unsmob_context_def ($$);
+               Context_def *td = unsmob<Context_def> ($$);
                if (!td) {
                        $$ = Context_def::make_scm ();
-                       td = unsmob_context_def ($$);
+                       td = unsmob<Context_def> ($$);
                }
                td->origin ()->set_spot (@$);
        }
@@ -706,41 +738,41 @@ context_def_spec_body:
        }
        | context_def_spec_body context_mod {
                if (!SCM_UNBNDP ($2)) {
-                       Context_def *td = unsmob_context_def ($$);
+                       Context_def *td = unsmob<Context_def> ($$);
                        if (!td) {
                                $$ = Context_def::make_scm ();
-                               td = unsmob_context_def ($$);
+                               td = unsmob<Context_def> ($$);
                        }
-                       unsmob_context_def ($$)->add_context_mod ($2);
+                       unsmob<Context_def> ($$)->add_context_mod ($2);
                }
        }
        | context_def_spec_body context_modification {
-                Context_def *td = unsmob_context_def ($$);
+                Context_def *td = unsmob<Context_def> ($$);
                if (!td) {
                        $$ = Context_def::make_scm ();
-                       td = unsmob_context_def ($$);
+                       td = unsmob<Context_def> ($$);
                }
-                SCM new_mods = unsmob_context_mod ($2)->get_mods ();
+                SCM new_mods = unsmob<Context_mod> ($2)->get_mods ();
                 for (SCM m = new_mods; scm_is_pair (m); m = scm_cdr (m)) {
                     td->add_context_mod (scm_car (m));
                 }
        }
        | context_def_spec_body context_mod_arg {
-               Context_def *td = unsmob_context_def ($1);
+               Context_def *td = unsmob<Context_def> ($1);
                if (scm_is_eq ($2, SCM_UNSPECIFIED))
                        ;
-               else if (!td && unsmob_context_def ($2))
+               else if (!td && unsmob<Context_def> ($2))
                        $$ = $2;
                else {
                        if (!td) {
                                $$ = Context_def::make_scm ();
-                               td = unsmob_context_def ($$);
+                               td = unsmob<Context_def> ($$);
                        }
-                       if (unsmob_music ($2)) {
+                       if (unsmob<Music> ($2)) {
                                SCM proc = parser->lexer_->lookup_identifier ("context-mod-music-handler");
-                               $2 = scm_call_2 (proc, parser->self_scm (), $2);
+                               $2 = scm_call_1 (proc, $2);
                        }
-                       if (Context_mod *cm = unsmob_context_mod ($2)) {
+                       if (Context_mod *cm = unsmob<Context_mod> ($2)) {
                                for (SCM m = cm->get_mods (); scm_is_pair (m); m = scm_cdr (m)) {
                                        td->add_context_mod (scm_car (m));
                                }
@@ -755,7 +787,7 @@ context_def_spec_body:
 book_block:
        BOOK '{' book_body '}'  {
                $$ = $3;
-               unsmob_book ($$)->origin ()->set_spot (@$);
+               unsmob<Book> ($$)->origin ()->set_spot (@$);
                pop_paper (parser);
                parser->lexer_->set_identifier (ly_symbol2scm ("$current-book"), SCM_BOOL_F);
        }
@@ -768,7 +800,7 @@ book_body:
        {
                Book *book = new Book;
                init_papers (parser);
-               book->paper_ = dynamic_cast<Output_def*> (unsmob_output_def (parser->lexer_->lookup_identifier ("$defaultpaper"))->clone ());
+               book->paper_ = dynamic_cast<Output_def*> (unsmob<Output_def> (parser->lexer_->lookup_identifier ("$defaultpaper"))->clone ());
                book->paper_->unprotect ();
                push_paper (parser, book->paper_);
                book->header_ = get_header (parser);
@@ -779,8 +811,8 @@ book_body:
                parser->lexer_->set_identifier (ly_symbol2scm ("$current-book"), $1);
        }
        | book_body paper_block {
-               unsmob_book ($1)->paper_ = unsmob_output_def ($2);
-               set_paper (parser, unsmob_output_def ($2));
+               unsmob<Book> ($1)->paper_ = unsmob<Output_def> ($2);
+               set_paper (parser, unsmob<Output_def> ($2));
        }
        | book_body bookpart_block {
                SCM proc = parser->lexer_->lookup_identifier ("book-bookpart-handler");
@@ -792,7 +824,7 @@ book_body:
        }
        | book_body composite_music {
                SCM proc = parser->lexer_->lookup_identifier ("book-music-handler");
-               scm_call_3 (proc, parser->self_scm (), $1, $2);
+               scm_call_2 (proc, $1, $2);
        }
        | book_body full_markup {
                SCM proc = parser->lexer_->lookup_identifier ("book-text-handler");
@@ -817,19 +849,30 @@ book_body:
                {
                        SCM proc = parser->lexer_->lookup_identifier ("book-text-handler");
                        scm_call_2 (proc, $1, out);
-               } else if (unsmob_score ($2))
+               } else if (unsmob<Score> ($2))
                {
                        SCM proc = parser->lexer_->lookup_identifier ("book-score-handler");
                        scm_call_2 (proc, $1, $2);
+               } else if (Output_def *od = unsmob<Output_def> ($2)) {
+                       SCM id = SCM_EOL;
+
+                       if (to_boolean (od->c_variable ("is-paper")))
+                               id = ly_symbol2scm ("$defaultpaper");
+                       else if (to_boolean (od->c_variable ("is-midi")))
+                               id = ly_symbol2scm ("$defaultmidi");
+                       else if (to_boolean (od->c_variable ("is-layout")))
+                               id = ly_symbol2scm ("$defaultlayout");
+
+                       parser->lexer_->set_identifier (id, $2);
                } else if (!scm_is_eq ($2, SCM_UNSPECIFIED))
                        parser->parser_error (@2, _("bad expression type"));
        }
        | book_body
        {
-               parser->lexer_->add_scope (unsmob_book ($1)->header_);
+               parser->lexer_->add_scope (unsmob<Book> ($1)->header_);
        } lilypond_header
        | book_body error {
-                Book *book = unsmob_book ($1);
+                Book *book = unsmob<Book> ($1);
                book->paper_ = 0;
                book->scores_ = SCM_EOL;
                book->bookparts_ = SCM_EOL;
@@ -839,7 +882,7 @@ book_body:
 bookpart_block:
        BOOKPART '{' bookpart_body '}' {
                $$ = $3;
-               unsmob_book ($$)->origin ()->set_spot (@$);
+               unsmob<Book> ($$)->origin ()->set_spot (@$);
                parser->lexer_->set_identifier (ly_symbol2scm ("$current-bookpart"), SCM_BOOL_F);
        }
        ;
@@ -854,7 +897,7 @@ bookpart_body:
                parser->lexer_->set_identifier (ly_symbol2scm ("$current-bookpart"), $1);
        }
        | bookpart_body paper_block {
-               unsmob_book ($$)->paper_ = unsmob_output_def ($2);
+               unsmob<Book> ($$)->paper_ = unsmob<Output_def> ($2);
        }
        | bookpart_body score_block {
                SCM proc = parser->lexer_->lookup_identifier ("bookpart-score-handler");
@@ -862,7 +905,7 @@ bookpart_body:
        }
        | bookpart_body composite_music {
                SCM proc = parser->lexer_->lookup_identifier ("bookpart-music-handler");
-               scm_call_3 (proc, parser->self_scm (), $1, $2);
+               scm_call_2 (proc, $1, $2);
        }
        | bookpart_body full_markup {
                SCM proc = parser->lexer_->lookup_identifier ("bookpart-text-handler");
@@ -887,22 +930,33 @@ bookpart_body:
                {
                        SCM proc = parser->lexer_->lookup_identifier ("bookpart-text-handler");
                        scm_call_2 (proc, $1, out);
-               } else if (unsmob_score ($2))
+               } else if (unsmob<Score> ($2))
                {
                        SCM proc = parser->lexer_->lookup_identifier ("bookpart-score-handler");
                        scm_call_2 (proc, $1, $2);
+               } else if (Output_def *od = unsmob<Output_def> ($2)) {
+                       SCM id = SCM_EOL;
+
+                       if (to_boolean (od->c_variable ("is-paper")))
+                               id = ly_symbol2scm ("$defaultpaper");
+                       else if (to_boolean (od->c_variable ("is-midi")))
+                               id = ly_symbol2scm ("$defaultmidi");
+                       else if (to_boolean (od->c_variable ("is-layout")))
+                               id = ly_symbol2scm ("$defaultlayout");
+
+                       parser->lexer_->set_identifier (id, $2);
                } else if (!scm_is_eq ($2, SCM_UNSPECIFIED))
                        parser->parser_error (@2, _("bad expression type"));
        }
        | bookpart_body
        {
-                Book *book = unsmob_book ($1);
+                Book *book = unsmob<Book> ($1);
                if (!ly_is_module (book->header_))
                        book->header_ = ly_make_module (false);
                parser->lexer_->add_scope (book->header_);
        } lilypond_header
        | bookpart_body error {
-                Book *book = unsmob_book ($1);
+                Book *book = unsmob<Book> ($1);
                book->paper_ = 0;
                book->scores_ = SCM_EOL;
        }
@@ -910,91 +964,102 @@ bookpart_body:
 
 score_block:
        SCORE '{' score_body '}'        {
-               unsmob_score ($3)->origin ()->set_spot (@$);
+               unsmob<Score> ($3)->origin ()->set_spot (@$);
                $$ = $3;
        }
        ;
 
-score_headers:
-       /* empty */
-       {
-               $$ = SCM_EOL;
-       }
-       | score_headers
-       {
-               if (!scm_is_pair ($1)
-                   || !ly_is_module (scm_car ($1)))
-                       $1 = scm_cons (ly_make_module (false), $1);
-               parser->lexer_->add_scope (scm_car ($1));
-       } lilypond_header
-       {
-               $$ = $1;
-       }
-       | score_headers output_def
-       {
-                Output_def *od = unsmob_output_def ($2);
-               if (od->lookup_variable (ly_symbol2scm ("is-paper")) == SCM_BOOL_T)
-               {
-                       parser->parser_error (@2, _("\\paper cannot be used in \\score, use \\layout instead"));
-
-               }
-               else
-               {
+score_body:
+       score_items {
+               if (!unsmob<Score> ($1)) {
+                       parser->parser_error (@1, _("Missing music in \\score"));
+                       $$ = (new Score)->unprotect ();
                        if (scm_is_pair ($1) && ly_is_module (scm_car ($1)))
-                               scm_set_cdr_x ($1, scm_cons ($2, scm_cdr ($1)));
-                       else
-                               $$ = scm_cons ($2, $1);
+                       {
+                               unsmob<Score> ($$)->set_header (scm_car ($1));
+                               $1 = scm_cdr ($1);
+                       }
+                       for (SCM p = scm_reverse_x ($1, SCM_EOL);
+                            scm_is_pair (p); p = scm_cdr (p))
+                       {
+                               unsmob<Score> ($$)->
+                                       add_output_def (unsmob<Output_def> (scm_car (p)));
+                       }
                }
        }
+       | score_body error {
+               unsmob<Score> ($$)->error_found_ = true;
+       }
        ;
 
-               
-
-score_body:
-       score_headers music {
-               SCM scorify = ly_lily_module_constant ("scorify-music");
-               $$ = scm_call_2 (scorify, $2, parser->self_scm ());
+score_item:
+       embedded_scm
+       | music
+       | output_def
+       ;
 
-               if (scm_is_pair ($1) && ly_is_module (scm_car ($1)))
-               {
-                       unsmob_score ($$)->set_header (scm_car ($1));
-                       $1 = scm_cdr ($1);
-               }
-               for (SCM p = scm_reverse_x ($1, SCM_EOL);
-                    scm_is_pair (p); p = scm_cdr (p))
-               {
-                       unsmob_score ($$)->
-                               add_output_def (unsmob_output_def (scm_car (p)));
-               }
-       }
-       | embedded_scm_active {
-               if (!unsmob_score ($1))
-               {
-                       $$ = (new Score)->unprotect ();
-                       parser->parser_error (@1, _("score expected"));
-               }
+score_items:
+       /* empty */
+       {
+               $$ = SCM_EOL;
        }
-       | score_body
+       | score_items score_item
        {
-                Score *score = unsmob_score ($1);
-               if (!ly_is_module (score->get_header ()))
-                       score->set_header (ly_make_module (false));
-               parser->lexer_->add_scope (score->get_header ());
-       } lilypond_header
-       | score_body output_def {
-                Output_def *od = unsmob_output_def ($2);
-               if (od->lookup_variable (ly_symbol2scm ("is-paper")) == SCM_BOOL_T)
-               {
-                       parser->parser_error (@2, _("\\paper cannot be used in \\score, use \\layout instead"));
-
+               Output_def *od = unsmob<Output_def> ($2);
+               if (od) {
+                       if (to_boolean (od->lookup_variable (ly_symbol2scm ("is-paper"))))
+                       {
+                               parser->parser_error (@2, _("\\paper cannot be used in \\score, use \\layout instead"));
+                               od = 0;
+                               $2 = SCM_UNSPECIFIED;
+                       }
+               } else if (!unsmob<Score> ($$)) {
+                       if (unsmob<Music> ($2)) {
+                               $2 = Lily::scorify_music ($2);
+                       }
+                       if (unsmob<Score> ($2))
+                       {
+                               $$ = $2;
+                               $2 = SCM_UNSPECIFIED;
+                       }
                }
-               else
-               {
-                       unsmob_score ($1)->add_output_def (od);
+               Score *score = unsmob<Score> ($$);
+               if (score && scm_is_pair ($1)) {
+                       if (ly_is_module (scm_car ($1)))
+                       {
+                               score->set_header (scm_car ($1));
+                               $1 = scm_cdr ($1);
+                       }
+                       for (SCM p = scm_reverse_x ($1, SCM_EOL);
+                            scm_is_pair (p); p = scm_cdr (p))
+                       {
+                               score->add_output_def (unsmob<Output_def> (scm_car (p)));
+                       }
                }
+               if (od) {
+                       if (score)
+                               score->add_output_def (od);
+                       else if (scm_is_pair ($$) && ly_is_module (scm_car ($$)))
+                               scm_set_cdr_x ($$, scm_cons ($2, scm_cdr ($$)));
+                       else
+                               $$ = scm_cons ($2, $$);
+               } else if (!scm_is_eq ($2, SCM_UNSPECIFIED))
+                       parser->parser_error (@2, _("Spurious expression in \\score"));
        }
-       | score_body error {
-               unsmob_score ($$)->error_found_ = true;
+       | score_items
+       {
+               if (Score *score = unsmob<Score> ($1)) {
+                       if (!ly_is_module (score->get_header ()))
+                               score->set_header (ly_make_module (false));
+                       parser->lexer_->add_scope (score->get_header ());
+               } else {
+                       if (!scm_is_pair ($1) || !ly_is_module (scm_car ($1)))
+                               $1 = scm_cons (ly_make_module (false), $1);
+                       parser->lexer_->add_scope (scm_car ($1));
+               }
+       } lilypond_header
+       {
+               $$ = $1;
        }
        ;
 
@@ -1005,9 +1070,9 @@ score_body:
 
 paper_block:
        output_def {
-                Output_def *od = unsmob_output_def ($1);
+                Output_def *od = unsmob<Output_def> ($1);
 
-               if (od->lookup_variable (ly_symbol2scm ("is-paper")) != SCM_BOOL_T)
+               if (!to_boolean (od->lookup_variable (ly_symbol2scm ("is-paper"))))
                {
                        parser->parser_error (@1, _ ("need \\paper for paper block"));
                        $$ = get_paper (parser)->unprotect ();
@@ -1065,7 +1130,7 @@ music_or_context_def:
 
 output_def_body:
        output_def_head_with_mode_switch '{' {
-               unsmob_output_def ($1)->input_origin_.set_spot (@$);
+               unsmob<Output_def> ($1)->input_origin_.set_spot (@$);
                // This is a stupid trick to mark the beginning of the
                // body for deciding whether to allow
                // embedded_scm_active to have an output definition
@@ -1082,7 +1147,7 @@ output_def_body:
                // definitions.
                if (scm_is_pair ($1))
                {
-                       Output_def *o = unsmob_output_def ($2);
+                       Output_def *o = unsmob<Output_def> ($2);
                        if (o) {
                                o->input_origin_.set_spot (@$);
                                $1 = o->self_scm ();
@@ -1092,15 +1157,14 @@ output_def_body:
                        } else
                                $1 = scm_car ($1);
                }
-               if (unsmob_context_def ($2))
-                       assign_context_def (unsmob_output_def ($1), $2);
+               if (unsmob<Context_def> ($2))
+                       assign_context_def (unsmob<Output_def> ($1), $2);
                // Seems unlikely, but let's be complete:
-               else if (unsmob_music ($2))
+               else if (unsmob<Music> ($2))
                {
                        SCM proc = parser->lexer_->lookup_identifier
                                ("output-def-music-handler");
-                       scm_call_3 (proc, parser->self_scm (),
-                                   $1, $2);
+                       scm_call_2 (proc, $1, $2);
                } else if (!scm_is_eq ($2, SCM_UNSPECIFIED))
                        parser->parser_error (@2, _("bad expression type"));
                $$ = $1;
@@ -1120,14 +1184,13 @@ output_def_body:
        } music_or_context_def
        {
                parser->lexer_->pop_state ();
-               if (unsmob_context_def ($3))
-                       assign_context_def (unsmob_output_def ($1), $3);
+               if (unsmob<Context_def> ($3))
+                       assign_context_def (unsmob<Output_def> ($1), $3);
                else {
 
                        SCM proc = parser->lexer_->lookup_identifier
                                     ("output-def-music-handler");
-                       scm_call_3 (proc, parser->self_scm (),
-                                   $1, $3);
+                       scm_call_2 (proc, $1, $3);
                }
                $$ = $1;
        }
@@ -1138,13 +1201,13 @@ output_def_body:
 
 tempo_event:
        TEMPO steno_duration '=' tempo_range    {
-               $$ = MAKE_SYNTAX ("tempo", @$, SCM_EOL, $2, $4);
+               $$ = MAKE_SYNTAX (tempo, @$, SCM_EOL, $2, $4);
        }
        | TEMPO scalar steno_duration '=' tempo_range   {
-               $$ = MAKE_SYNTAX ("tempo", @$, $2, $3, $5);
+               $$ = MAKE_SYNTAX (tempo, @$, $2, $3, $5);
        }
        | TEMPO scalar {
-               $$ = MAKE_SYNTAX ("tempo", @$, $2);
+               $$ = MAKE_SYNTAX (tempo, @$, $2);
        } %prec ':'
        ;
 
@@ -1156,7 +1219,7 @@ music_list:
                $$ = SCM_EOL;
        }
        | music_list music_embedded {
-               if (unsmob_music ($2))
+               if (unsmob<Music> ($2))
                        $$ = scm_cons ($2, $1);
        }
        | music_list error {
@@ -1177,13 +1240,17 @@ braced_music_list:
 
 music: music_assign
        | lyric_element_music
-       | pitch_or_music
+       | pitch_as_music
+       ;
+
+pitch_as_music:
+       pitch_or_music
        {
                $$ = make_music_from_simple (parser, @1, $1);
-                if (!unsmob_music ($$))
+                if (!unsmob<Music> ($$))
                {
                         parser->parser_error (@1, _ ("music expected"));
-                       $$ = MAKE_SYNTAX ("void-music", @$);
+                       $$ = MAKE_SYNTAX (void_music, @$);
                }
        }
        ;
@@ -1191,7 +1258,7 @@ music:    music_assign
 music_embedded:
        music
        {
-               if (unsmob_music ($1)->is_mus_type ("post-event")) {
+               if (unsmob<Music> ($1)->is_mus_type ("post-event")) {
                        parser->parser_error (@1, _ ("unexpected post-event"));
                        $$ = SCM_UNSPECIFIED;
                }
@@ -1208,7 +1275,7 @@ music_embedded:
        {
                Music *n = MY_MAKE_MUSIC ("NoteEvent", @$);
 
-               parser->default_duration_ = *unsmob_duration ($1);
+               parser->default_duration_ = *unsmob<Duration> ($1);
                n->set_property ("duration", $1);
 
                if (scm_is_pair ($2))
@@ -1223,7 +1290,7 @@ music_embedded_backup:
        {
                if (scm_is_eq ($1, SCM_UNSPECIFIED))
                        $$ = $1;
-               else if (Music *m = unsmob_music ($1)) {
+               else if (Music *m = unsmob<Music> ($1)) {
                        if (m->is_mus_type ("post-event")) {
                                parser->parser_error
                                        (@1, _ ("unexpected post-event"));
@@ -1250,29 +1317,29 @@ music_assign:
 repeated_music:
        REPEAT simple_string unsigned_number music
        {
-               $$ = MAKE_SYNTAX ("repeat", @$, $2, $3, $4, SCM_EOL);
+               $$ = MAKE_SYNTAX (repeat, @$, $2, $3, $4, SCM_EOL);
        }
        | REPEAT simple_string unsigned_number music ALTERNATIVE braced_music_list
        {
-               $$ = MAKE_SYNTAX ("repeat", @$, $2, $3, $4, $6);
+               $$ = MAKE_SYNTAX (repeat, @$, $2, $3, $4, $6);
        }
        ;
 
 sequential_music:
        SEQUENTIAL braced_music_list {
-               $$ = MAKE_SYNTAX ("sequential-music", @$, $2);
+               $$ = MAKE_SYNTAX (sequential_music, @$, $2);
        }
        | braced_music_list {
-               $$ = MAKE_SYNTAX ("sequential-music", @$, $1);
+               $$ = MAKE_SYNTAX (sequential_music, @$, $1);
        }
        ;
 
 simultaneous_music:
        SIMULTANEOUS braced_music_list {
-               $$ = MAKE_SYNTAX ("simultaneous-music", @$, $2);
+               $$ = MAKE_SYNTAX (simultaneous_music, @$, $2);
        }
        | DOUBLE_ANGLE_OPEN music_list DOUBLE_ANGLE_CLOSE       {
-               $$ = MAKE_SYNTAX ("simultaneous-music", @$, scm_reverse_x ($2, SCM_EOL));
+               $$ = MAKE_SYNTAX (simultaneous_music, @$, scm_reverse_x ($2, SCM_EOL));
        }
        ;
 
@@ -1302,11 +1369,11 @@ context_modification:
         }
        | WITH context_modification_arg
        {
-               if (unsmob_music ($2)) {
+               if (unsmob<Music> ($2)) {
                        SCM proc = parser->lexer_->lookup_identifier ("context-mod-music-handler");
-                       $2 = scm_call_2 (proc, parser->self_scm (), $2);
+                       $2 = scm_call_1 (proc, $2);
                }
-               if (unsmob_context_mod ($2))
+               if (unsmob<Context_mod> ($2))
                        $$ = $2;
                else {
                        parser->parser_error (@2, _ ("not a context mod"));
@@ -1336,32 +1403,115 @@ context_mod_list:
         }
         | context_mod_list context_mod  {
                if (!SCM_UNBNDP ($2))
-                       unsmob_context_mod ($1)->add_context_mod ($2);
+                       unsmob<Context_mod> ($1)->add_context_mod ($2);
         }
         | context_mod_list CONTEXT_MOD_IDENTIFIER {
-                 Context_mod *md = unsmob_context_mod ($2);
+                 Context_mod *md = unsmob<Context_mod> ($2);
                  if (md)
-                     unsmob_context_mod ($1)->add_context_mods (md->get_mods ());
+                     unsmob<Context_mod> ($1)->add_context_mods (md->get_mods ());
         }
        | context_mod_list context_mod_arg {
                if (scm_is_eq ($2, SCM_UNSPECIFIED))
                        ;
-               else if (unsmob_music ($2)) {
+               else if (unsmob<Music> ($2)) {
                        SCM proc = parser->lexer_->lookup_identifier ("context-mod-music-handler");
-                       $2 = scm_call_2 (proc, parser->self_scm (), $2);
+                       $2 = scm_call_1 (proc, $2);
                }
-               if (unsmob_context_mod ($2))
-                       unsmob_context_mod ($$)->add_context_mods
-                               (unsmob_context_mod ($2)->get_mods ());
+               if (unsmob<Context_mod> ($2))
+                       unsmob<Context_mod> ($$)->add_context_mods
+                               (unsmob<Context_mod> ($2)->get_mods ());
                else {
                        parser->parser_error (@2, _ ("not a context mod"));
                }
         }
         ;
 
-composite_music:
-       complex_music
+context_prefix:
+       CONTEXT symbol optional_id optional_context_mod {
+                Context_mod *ctxmod = unsmob<Context_mod> ($4);
+                SCM mods = SCM_EOL;
+                if (ctxmod)
+                        mods = ctxmod->get_mods ();
+               $$ = START_MAKE_SYNTAX (context_specification, $2, $3, mods, SCM_BOOL_F);
+       }
+       | NEWCONTEXT symbol optional_id optional_context_mod {
+                Context_mod *ctxmod = unsmob<Context_mod> ($4);
+                SCM mods = SCM_EOL;
+                if (ctxmod)
+                        mods = ctxmod->get_mods ();
+               $$ = START_MAKE_SYNTAX (context_specification, $2, $3, mods, SCM_BOOL_T);
+       }
+       ;
+
+new_lyrics:
+       ADDLYRICS lyric_mode_music {
+               $$ = scm_list_1 ($2);
+       }
+       | new_lyrics ADDLYRICS lyric_mode_music {
+               $$ = scm_cons ($3, $1);
+       }
+       ;
+
+/* basic_music is basically the same as composite_music but with
+ * context-prefixed music and lyricized music explicitly removed.  The
+ * reason is that in a sequence
+ *
+ *   \new Staff \new Voice { ... } \addlyrics { ... } \addlyrics { ... }
+ *
+ * we need to group both \addlyrics together (as they go with the same
+ * voice) but then combine them with \new Voice { ... }, meaning that
+ * combining \new Voice { ... } needs higher priority than
+ * { ... } \addlyrics, and *not* have \new Staff \new Voice { ... }
+ * combine before combining \new Voice { ... } \addlyrics: only one
+ * layer of context-prefixed music should assemble before \addlyrics
+ * is integrated.  Total mess, and we sort this mess out with explicit
+ * rules preferring a single context-prefix.
+ */
+
+basic_music:
+       music_function_call
+       | repeated_music
        | music_bare
+       | LYRICSTO simple_string lyric_mode_music {
+               $$ = MAKE_SYNTAX (lyric_combine, @$, $2, SCM_EOL, $3);
+       }
+       | LYRICSTO symbol '=' simple_string lyric_mode_music
+       {
+               $$ = MAKE_SYNTAX (lyric_combine, @$, $3, $2, $4);
+       }
+       ;
+
+contextable_music:
+       basic_music
+       | pitch_as_music
+       | event_chord
+       ;
+
+contexted_basic_music:
+       context_prefix contextable_music new_lyrics
+       {
+               Input i;
+               i.set_location (@1, @2);
+               $$ = FINISH_MAKE_SYNTAX ($1, i, $2);
+               $$ = MAKE_SYNTAX (add_lyrics, @$, $$, scm_reverse_x ($3, SCM_EOL));
+       } %prec COMPOSITE
+       | context_prefix contextable_music
+       {
+               $$ = FINISH_MAKE_SYNTAX ($1, @$, $2);
+       } %prec COMPOSITE
+       | context_prefix contexted_basic_music
+       {
+               $$ = FINISH_MAKE_SYNTAX ($1, @$, $2);
+       }
+       ;
+
+composite_music:
+       basic_music %prec COMPOSITE
+       | contexted_basic_music
+       | basic_music new_lyrics
+       {
+               $$ = MAKE_SYNTAX (add_lyrics, @$, $1, scm_reverse_x ($2, SCM_EOL));
+       } %prec COMPOSITE
        ;
 
 music_bare:
@@ -1449,8 +1599,7 @@ symbol_list_rev:
 symbol_list_part:
        symbol_list_element
        {
-               SCM sym_l_p = ly_lily_module_constant ("symbol-list?");
-               $$ = try_string_variants (sym_l_p, $1);
+               $$ = try_string_variants (Lily::symbol_list_p, $1);
                if (SCM_UNBNDP ($$)) {
                        parser->parser_error (@1, _("not a symbol"));
                        $$ = SCM_EOL;
@@ -1483,7 +1632,6 @@ function_arglist_nonbackup:
                        $$ = check_scheme_arg (parser, @4, t->unprotect (),
                                               $3, $2, n);
                }
-               
        }
        | EXPECT_OPTIONAL EXPECT_SCM function_arglist_nonbackup '-' REAL
        {
@@ -1700,7 +1848,7 @@ function_arglist_backup:
                        $$ = scm_cons ($4, $3);
                else {
                        $$ = scm_cons (loc_on_music (@3, $1), $3);
-                       MYBACKUP (LYRIC_ELEMENT, $4, @4);
+                       MYBACKUP (SCM_IDENTIFIER, $4, @4);
                }
        }
        | EXPECT_OPTIONAL EXPECT_SCM function_arglist_backup UNSIGNED
@@ -1760,7 +1908,6 @@ function_arglist_backup:
                                parser->lexer_->push_extra_token (@4, '-');
                        }
                }
-               
        }
        | EXPECT_OPTIONAL EXPECT_SCM function_arglist_backup '-' REAL
        {
@@ -2037,7 +2184,6 @@ function_arglist_common_reparse:
                        else
                                MYREPARSE (@4, $1, SCM_ARG, $4);
                }
-               
        }
        | EXPECT_SCM function_arglist_optional '-' REAL
        {
@@ -2066,8 +2212,8 @@ function_arglist_skip_backup:
 
 music_function_call:
        MUSIC_FUNCTION function_arglist {
-               $$ = MAKE_SYNTAX ("music-function", @$,
-                                        $1, $2);
+               $$ = MAKE_SYNTAX (music_function, @$,
+                                 $1, $2);
        }
        ;
 
@@ -2096,38 +2242,11 @@ lyric_mode_music:
        | MUSIC_IDENTIFIER
        ;
 
-complex_music:
-       music_function_call
-       | repeated_music                { $$ = $1; }
-       | re_rhythmed_music     { $$ = $1; }
-       | complex_music_prefix music
-       {
-               $$ = FINISH_MAKE_SYNTAX ($1, @$, $2);
-       }
-       ;
-
-complex_music_prefix:
-       CONTEXT symbol optional_id optional_context_mod {
-                Context_mod *ctxmod = unsmob_context_mod ($4);
-                SCM mods = SCM_EOL;
-                if (ctxmod)
-                        mods = ctxmod->get_mods ();
-               $$ = START_MAKE_SYNTAX ("context-specification", $2, $3, mods, SCM_BOOL_F);
-       }
-       | NEWCONTEXT symbol optional_id optional_context_mod {
-                Context_mod *ctxmod = unsmob_context_mod ($4);
-                SCM mods = SCM_EOL;
-                if (ctxmod)
-                        mods = ctxmod->get_mods ();
-               $$ = START_MAKE_SYNTAX ("context-specification", $2, $3, mods, SCM_BOOL_T);
-       }
-       ;
-
 mode_changed_music:
        mode_changing_head grouped_music_list {
-               if ($1 == ly_symbol2scm ("chords"))
+               if (scm_is_eq ($1, ly_symbol2scm ("chords")))
                {
-                 $$ = MAKE_SYNTAX ("unrelativable-music", @$, $2);
+                 $$ = MAKE_SYNTAX (unrelativable_music, @$, $2);
                }
                else
                {
@@ -2136,14 +2255,14 @@ mode_changed_music:
                parser->lexer_->pop_state ();
        }
        | mode_changing_head_with_context optional_context_mod grouped_music_list {
-                Context_mod *ctxmod = unsmob_context_mod ($2);
+                Context_mod *ctxmod = unsmob<Context_mod> ($2);
                 SCM mods = SCM_EOL;
                 if (ctxmod)
                         mods = ctxmod->get_mods ();
-               $$ = MAKE_SYNTAX ("context-specification", @$, $1, SCM_EOL, mods, SCM_BOOL_T, $3);
-               if ($1 == ly_symbol2scm ("ChordNames"))
+               $$ = MAKE_SYNTAX (context_specification, @$, $1, SCM_EOL, mods, SCM_BOOL_T, $3);
+               if (scm_is_eq ($1, ly_symbol2scm ("ChordNames")))
                {
-                 $$ = MAKE_SYNTAX ("unrelativable-music", @$, $$);
+                 $$ = MAKE_SYNTAX (unrelativable_music, @$, $$);
                }
                parser->lexer_->pop_state ();
        }
@@ -2207,27 +2326,9 @@ mode_changing_head_with_context:
        }
        ;
 
-new_lyrics:
-       ADDLYRICS lyric_mode_music {
-               $$ = scm_list_1 ($2);
-       }
-       | new_lyrics ADDLYRICS lyric_mode_music {
-               $$ = scm_cons ($3, $1);
-       }
-       ;
-
-re_rhythmed_music:
-       composite_music new_lyrics {
-               $$ = MAKE_SYNTAX ("add-lyrics", @$, $1, scm_reverse_x ($2, SCM_EOL));
-       } %prec COMPOSITE
-       | LYRICSTO simple_string lyric_mode_music {
-               $$ = MAKE_SYNTAX ("lyric-combine", @$, $2, $3);
-       }
-       ;
-
 context_change:
        CHANGE symbol '=' simple_string  {
-               $$ = MAKE_SYNTAX ("context-change", @$, $2, $4);
+               $$ = MAKE_SYNTAX (context_change, @$, $2, $4);
        }
        ;
 
@@ -2315,7 +2416,7 @@ revert_arg_part:
        | revert_arg_backup BACKUP SCM_ARG symbol_list_part
        {
                $$ = scm_append_x (scm_list_2 ($4, $3));
-       }               
+       }
        ;
 
 context_def_mod:
@@ -2412,43 +2513,6 @@ context_prop_spec:
        }
        ;
 
-simple_music_property_def:
-       OVERRIDE grob_prop_path '=' scalar {
-               if (SCM_UNBNDP ($2))
-                       $$ = SCM_UNDEFINED;
-               else {
-                       $$ = scm_list_5 (scm_car ($2),
-                                        ly_symbol2scm ("OverrideProperty"),
-                                        scm_cadr ($2),
-                                        $4,
-                                        scm_cddr ($2));
-               }
-       }
-       | REVERT simple_revert_context revert_arg {
-               $$ = scm_list_4 ($2,
-                                ly_symbol2scm ("RevertProperty"),
-                                scm_car ($3),
-                                scm_cdr ($3));
-       }
-       | SET context_prop_spec '=' scalar {
-               if (SCM_UNBNDP ($2))
-                       $$ = SCM_UNDEFINED;
-               else
-                       $$ = scm_list_4 (scm_car ($2),
-                                        ly_symbol2scm ("PropertySet"),
-                                        scm_cadr ($2),
-                                        $4);
-       }
-       | UNSET context_prop_spec {
-               if (SCM_UNBNDP ($2))
-                       $$ = SCM_UNDEFINED;
-               else
-                       $$ = scm_list_3 (scm_car ($2),
-                                        ly_symbol2scm ("PropertyUnset"),
-                                        scm_cadr ($2));
-       }
-       ;
-
 
 // This is all quite awkward for the sake of substantial backward
 // compatibility while at the same time allowing a more "natural" form
@@ -2483,11 +2547,43 @@ simple_revert_context:
        ;
 
 music_property_def:
-       simple_music_property_def {
-               if (SCM_UNBNDP ($1))
-                       $$ = MAKE_SYNTAX ("void-music", @1);
+       OVERRIDE grob_prop_path '=' scalar {
+               if (SCM_UNBNDP ($2))
+                       $$ = MAKE_SYNTAX (void_music, @$);
+               else {
+                       $$ = MAKE_SYNTAX (property_operation, @$,
+                                         scm_car ($2),
+                                         ly_symbol2scm ("OverrideProperty"),
+                                         scm_cadr ($2),
+                                         $4,
+                                         scm_cddr ($2));
+               }
+       }
+       | REVERT simple_revert_context revert_arg {
+               $$ = MAKE_SYNTAX (property_operation, @$,
+                                 $2,
+                                 ly_symbol2scm ("RevertProperty"),
+                                 scm_car ($3),
+                                 scm_cdr ($3));
+       }
+       | SET context_prop_spec '=' scalar {
+               if (SCM_UNBNDP ($2))
+                       $$ = MAKE_SYNTAX (void_music, @$);
                else
-                       $$ = LOWLEVEL_MAKE_SYNTAX (ly_lily_module_constant ("property-operation"), scm_cons2 (parser->self_scm (), make_input (@$), $1));
+                       $$ = MAKE_SYNTAX (property_operation, @$,
+                                         scm_car ($2),
+                                         ly_symbol2scm ("PropertySet"),
+                                         scm_cadr ($2),
+                                         $4);
+       }
+       | UNSET context_prop_spec {
+               if (SCM_UNBNDP ($2))
+                       $$ = MAKE_SYNTAX (void_music, @$);
+               else
+                       $$ = MAKE_SYNTAX (property_operation, @$,
+                                         scm_car ($2),
+                                         ly_symbol2scm ("PropertyUnset"),
+                                         scm_cadr ($2));
        }
        ;
 
@@ -2520,8 +2616,7 @@ symbol:
        {
                // This is a bit of overkill but makes the same
                // routine responsible for all symbol interpretations.
-               $$ = try_string_variants (ly_lily_module_constant ("symbol?"),
-                                         $1);
+               $$ = try_string_variants (Guile_user::symbol_p, $1);
                if (SCM_UNBNDP ($$))
                {
                        parser->parser_error (@1, (_ ("symbol expected")));
@@ -2555,20 +2650,20 @@ event_chord:
        simple_element post_events {
                // Let the rhythmic music iterator sort this mess out.
                if (scm_is_pair ($2)) {
-                       unsmob_music ($$)->set_property ("articulations",
+                       unsmob<Music> ($$)->set_property ("articulations",
                                                         scm_reverse_x ($2, SCM_EOL));
                }
        } %prec ':'
        | CHORD_REPETITION optional_notemode_duration post_events {
                Input i;
                i.set_location (@1, @3);
-               $$ = MAKE_SYNTAX ("repetition-chord", i,
+               $$ = MAKE_SYNTAX (repetition_chord, i,
                                  $2, scm_reverse_x ($3, SCM_EOL));
        } %prec ':'
        | MULTI_MEASURE_REST optional_notemode_duration post_events {
                Input i;
                i.set_location (@1, @3);
-               $$ = MAKE_SYNTAX ("multi-measure-rest", i, $2,
+               $$ = MAKE_SYNTAX (multi_measure_rest, i, $2,
                                  scm_reverse_x ($3, SCM_EOL));
        } %prec ':'
        | tempo_event
@@ -2579,13 +2674,13 @@ event_chord:
 note_chord_element:
        chord_body optional_notemode_duration post_events
        {
-               Music *m = unsmob_music ($1);
-               SCM dur = unsmob_duration ($2)->smobbed_copy ();
+               Music *m = unsmob<Music> ($1);
+               SCM dur = unsmob<Duration> ($2)->smobbed_copy ();
                SCM es = m->get_property ("elements");
                SCM postevs = scm_reverse_x ($3, SCM_EOL);
 
                for (SCM s = es; scm_is_pair (s); s = scm_cdr (s))
-                 unsmob_music (scm_car (s))->set_property ("duration", dur);
+                 unsmob<Music> (scm_car (s))->set_property ("duration", dur);
                es = ly_append2 (es, postevs);
 
                m-> set_property ("elements", es);
@@ -2597,7 +2692,11 @@ note_chord_element:
 chord_body:
        ANGLE_OPEN chord_body_elements ANGLE_CLOSE
        {
-               $$ = MAKE_SYNTAX ("event-chord", @$, scm_reverse_x ($2, SCM_EOL));
+               $$ = MAKE_SYNTAX (event_chord, @$, scm_reverse_x ($2, SCM_EOL));
+       }
+       | FIGURE_OPEN figure_list FIGURE_CLOSE
+       {
+               $$ = MAKE_SYNTAX (event_chord, @$, scm_reverse_x ($2, SCM_EOL));
        }
        ;
 
@@ -2610,7 +2709,7 @@ chord_body_elements:
        ;
 
 chord_body_element:
-       pitch exclamations questions octave_check post_events
+       pitch_or_tonic_pitch exclamations questions octave_check post_events
        {
                bool q = to_boolean ($3);
                bool ex = to_boolean ($2);
@@ -2648,11 +2747,11 @@ chord_body_element:
        }
        | music_function_chord_body
        {
-               Music *m = unsmob_music ($1);
+               Music *m = unsmob<Music> ($1);
 
                while (m && m->is_mus_type ("music-wrapper-music")) {
                        $$ = m->get_property ("element");
-                       m = unsmob_music ($$);
+                       m = unsmob<Music> ($$);
                }
 
                if (!(m && m->is_mus_type ("rhythmic-event"))) {
@@ -2670,8 +2769,8 @@ music_function_chord_body:
 
 event_function_event:
        EVENT_FUNCTION function_arglist {
-               $$ = MAKE_SYNTAX ("music-function", @$,
-                                        $1, $2);
+               $$ = MAKE_SYNTAX (music_function, @$,
+                                 $1, $2);
        }
        ;
 
@@ -2681,7 +2780,7 @@ post_events:
        }
        | post_events post_event {
                $$ = $1;
-               if (Music *m = unsmob_music ($2))
+               if (Music *m = unsmob<Music> ($2))
                {
                        if (m->is_mus_type ("post-event-wrapper"))
                        {
@@ -2705,12 +2804,12 @@ post_event_nofinger:
        }
        | script_dir music_function_call {
                $$ = $2;
-               if (!unsmob_music ($2)->is_mus_type ("post-event")) {
+               if (!unsmob<Music> ($2)->is_mus_type ("post-event")) {
                        parser->parser_error (@2, _ ("post-event expected"));
                        $$ = SCM_UNSPECIFIED;
                } else if (!SCM_UNBNDP ($1))
                {
-                       unsmob_music ($$)->set_property ("direction", $1);
+                       unsmob<Music> ($$)->set_property ("direction", $1);
                }
        }
        | HYPHEN {
@@ -2726,7 +2825,7 @@ post_event_nofinger:
        | script_dir direction_reqd_event {
                if (!SCM_UNBNDP ($1))
                {
-                       Music *m = unsmob_music ($2);
+                       Music *m = unsmob<Music> ($2);
                        m->set_property ("direction", $1);
                }
                $$ = $2;
@@ -2734,7 +2833,7 @@ post_event_nofinger:
        | script_dir direction_less_event {
                if (!SCM_UNBNDP ($1))
                {
-                       Music *m = unsmob_music ($2);
+                       Music *m = unsmob<Music> ($2);
                        m->set_property ("direction", $1);
                }
                $$ = $2;
@@ -2742,13 +2841,13 @@ post_event_nofinger:
        | '^' fingering
        {
                $$ = $2;
-               unsmob_music ($$)->set_property ("direction", scm_from_int (UP));
+               unsmob<Music> ($$)->set_property ("direction", scm_from_int (UP));
        }
        | '_' fingering
        {
                $$ = $2;
-               unsmob_music ($$)->set_property ("direction", scm_from_int (DOWN));
-       }                       
+               unsmob<Music> ($$)->set_property ("direction", scm_from_int (DOWN));
+       }
        ;
 
 post_event:
@@ -2776,7 +2875,7 @@ direction_less_event:
                a->set_property ("tremolo-type", $1);
                $$ = a->unprotect ();
         }
-       | event_function_event  
+       | event_function_event
        ;
 
 direction_reqd_event:
@@ -2790,7 +2889,7 @@ direction_reqd_event:
                        a->set_property ("articulation-type", s);
                        $$ = a->unprotect ();
                } else {
-                       Music *original = unsmob_music (s);
+                       Music *original = unsmob<Music> (s);
                        if (original && original->is_mus_type ("post-event")) {
                                Music *a = original->clone ();
                                a->set_spot (parser->lexer_->override_input (@$));
@@ -2839,8 +2938,8 @@ steno_pitch:
        NOTENAME_PITCH quotes {
                 if (!scm_is_eq (SCM_INUM0, $2))
                 {
-                        Pitch p = *unsmob_pitch ($1);
-                        p = p.transposed (Pitch (scm_to_int ($2),0,0));
+                        Pitch p = *unsmob<Pitch> ($1);
+                        p = p.transposed (Pitch (scm_to_int ($2), 0));
                         $$ = p.smobbed_copy ();
                 }
        }
@@ -2854,8 +2953,8 @@ steno_tonic_pitch:
        TONICNAME_PITCH quotes {
                 if (!scm_is_eq (SCM_INUM0, $2))
                 {
-                        Pitch p = *unsmob_pitch ($1);
-                        p = p.transposed (Pitch (scm_to_int ($2),0,0));
+                        Pitch p = *unsmob<Pitch> ($1);
+                        p = p.transposed (Pitch (scm_to_int ($2), 0));
                         $$ = p.smobbed_copy ();
                 }
        }
@@ -2866,13 +2965,18 @@ pitch:
        | PITCH_IDENTIFIER quotes {
                 if (!scm_is_eq (SCM_INUM0, $2))
                 {
-                        Pitch p = *unsmob_pitch ($1);
-                        p = p.transposed (Pitch (scm_to_int ($2),0,0));
+                        Pitch p = *unsmob<Pitch> ($1);
+                        p = p.transposed (Pitch (scm_to_int ($2), 0));
                         $$ = p.smobbed_copy ();
                 }
        }
        ;
 
+pitch_or_tonic_pitch:
+       pitch
+       | steno_tonic_pitch
+       ;
+
 gen_text_def:
        full_markup {
                Music *t = MY_MAKE_MUSIC ("TextScriptEvent", @$);
@@ -2887,7 +2991,7 @@ gen_text_def:
        }
        | embedded_scm
        {
-               Music *m = unsmob_music ($1);
+               Music *m = unsmob<Music> ($1);
                if (m && m->is_mus_type ("post-event"))
                        $$ = $1;
                else if (Text_interface::is_markup ($1)) {
@@ -2909,25 +3013,25 @@ fingering:
 
 script_abbreviation:
        '^'             {
-               $$ = scm_from_locale_string ("Hat");
+               $$ = scm_from_ascii_string ("Hat");
        }
        | '+'           {
-               $$ = scm_from_locale_string ("Plus");
+               $$ = scm_from_ascii_string ("Plus");
        }
        | '-'           {
-               $$ = scm_from_locale_string ("Dash");
+               $$ = scm_from_ascii_string ("Dash");
        }
        | '!'           {
-               $$ = scm_from_locale_string ("Bang");
+               $$ = scm_from_ascii_string ("Bang");
        }
        | ANGLE_CLOSE   {
-               $$ = scm_from_locale_string ("Larger");
+               $$ = scm_from_ascii_string ("Larger");
        }
        | '.'           {
-               $$ = scm_from_locale_string ("Dot");
+               $$ = scm_from_ascii_string ("Dot");
        }
        | '_' {
-               $$ = scm_from_locale_string ("Underscore");
+               $$ = scm_from_ascii_string ("Underscore");
        }
        ;
 
@@ -2949,7 +3053,7 @@ maybe_notemode_duration:
        } %prec ':'
        | multiplied_duration   {
                $$ = $1;
-               parser->default_duration_ = *unsmob_duration ($$);
+               parser->default_duration_ = *unsmob<Duration> ($$);
        }
 ;
 
@@ -2972,7 +3076,7 @@ steno_duration:
                }
        }
        | DURATION_IDENTIFIER dots      {
-               Duration *d = unsmob_duration ($1);
+               Duration *d = unsmob<Duration> ($1);
                Duration k (d->duration_log (),
                             d->dot_count () + scm_to_int ($2));
                k = k.compressed (d->factor ());
@@ -2986,12 +3090,12 @@ multiplied_duration:
                $$ = $1;
        }
        | multiplied_duration '*' UNSIGNED {
-               $$ = unsmob_duration ($$)->compressed (scm_to_int ($3)).smobbed_copy ();
+               $$ = unsmob<Duration> ($$)->compressed (scm_to_int ($3)).smobbed_copy ();
        }
        | multiplied_duration '*' FRACTION {
                Rational  m (scm_to_int (scm_car ($3)), scm_to_int (scm_cdr ($3)));
 
-               $$ = unsmob_duration ($$)->compressed (m).smobbed_copy ();
+               $$ = unsmob<Duration> ($$)->compressed (m).smobbed_copy ();
        }
        ;
 
@@ -3061,10 +3165,10 @@ bass_figure:
        }
        | bass_figure ']' {
                $$ = $1;
-               unsmob_music ($1)->set_property ("bracket-stop", SCM_BOOL_T);
+               unsmob<Music> ($1)->set_property ("bracket-stop", SCM_BOOL_T);
        }
        | bass_figure figured_bass_alteration {
-               Music *m = unsmob_music ($1);
+               Music *m = unsmob<Music> ($1);
                if (scm_to_double ($2)) {
                        SCM salter = m->get_property ("alteration");
                        SCM alter = scm_is_number (salter) ? salter : scm_from_int (0);
@@ -3075,7 +3179,7 @@ bass_figure:
                }
        }
        | bass_figure figured_bass_modification  {
-               Music *m = unsmob_music ($1);
+               Music *m = unsmob<Music> ($1);
                m->set_property ($2, SCM_BOOL_T);
        }
        ;
@@ -3102,7 +3206,7 @@ br_bass_figure:
        }
        | '[' bass_figure {
                $$ = $2;
-               unsmob_music ($$)->set_property ("bracket-start", SCM_BOOL_T);
+               unsmob<Music> ($$)->set_property ("bracket-start", SCM_BOOL_T);
        }
        ;
 
@@ -3115,13 +3219,6 @@ figure_list:
        }
        ;
 
-figure_spec:
-       FIGURE_OPEN figure_list FIGURE_CLOSE {
-               $$ = scm_reverse_x ($2, SCM_EOL);
-       }
-       ;
-
-
 optional_rest:
        /**/   { $$ = SCM_BOOL_F; }
        | REST { $$ = SCM_BOOL_T; }
@@ -3143,20 +3240,20 @@ pitch_or_music:
                                n = MY_MAKE_MUSIC ("RestEvent", @$);
                        else
                                n = MY_MAKE_MUSIC ("NoteEvent", @$);
-                       
+
                        n->set_property ("pitch", $1);
                        if (SCM_UNBNDP ($5))
                                n->set_property ("duration",
                                                 parser->default_duration_.smobbed_copy ());
                        else
                                n->set_property ("duration", $5);
-                       
+
                        if (scm_is_number ($4))
                        {
                                int q = scm_to_int ($4);
                                n->set_property ("absolute-octave", scm_from_int (q-1));
                        }
-                       
+
                        if (to_boolean ($3))
                                n->set_property ("cautionary", SCM_BOOL_T);
                        if (to_boolean ($2) || to_boolean ($3))
@@ -3167,9 +3264,11 @@ pitch_or_music:
                        $$ = n->unprotect ();
                }
        } %prec ':'
-       | simple_chord_elements post_events {
+       | new_chord post_events {
+               if (!parser->lexer_->is_chord_state ())
+                        parser->parser_error (@1, _ ("have to be in Chord mode for chords"));
                if (scm_is_pair ($2)) {
-                       if (unsmob_pitch ($1))
+                       if (unsmob<Pitch> ($1))
                                $1 = make_chord_elements (@1,
                                                          $1,
                                                          parser->default_duration_.smobbed_copy (),
@@ -3177,9 +3276,9 @@ pitch_or_music:
 
                        SCM elts = ly_append2 ($1, scm_reverse_x ($2, SCM_EOL));
 
-                       $$ = MAKE_SYNTAX ("event-chord", @1, elts);
-               } else if (!unsmob_pitch ($1))
-                       $$ = MAKE_SYNTAX ("event-chord", @1, $1);
+                       $$ = MAKE_SYNTAX (event_chord, @1, elts);
+               } else if (!unsmob<Pitch> ($1))
+                       $$ = MAKE_SYNTAX (event_chord, @1, $1);
                // A mere pitch drops through.
        } %prec ':'
        ;
@@ -3207,22 +3306,6 @@ simple_element:
        }
        ;
 
-// Can return a single pitch rather than a list.
-simple_chord_elements:
-       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 {
-               for (SCM s = $1; scm_is_pair (s); s = scm_cdr (s))
-               {
-                       unsmob_music (scm_car (s))->set_property ("duration", $2);
-               }
-               $$ = $1;
-       }
-       ;
-
 lyric_element:
        full_markup {
                if (!parser->lexer_->is_lyric_state ())
@@ -3239,9 +3322,9 @@ lyric_element:
 
 lyric_element_music:
        lyric_element optional_notemode_duration post_events {
-               $$ = MAKE_SYNTAX ("lyric-event", @$, $1, $2);
+               $$ = MAKE_SYNTAX (lyric_event, @$, $1, $2);
                if (scm_is_pair ($3))
-                       unsmob_music ($$)->set_property
+                       unsmob<Music> ($$)->set_property
                                ("articulations", scm_reverse_x ($3, SCM_EOL));
        } %prec ':'
        ;
@@ -3446,11 +3529,11 @@ full_markup:
 
 markup_top:
        markup_list {
-               $$ = scm_list_2 (ly_lily_module_constant ("line-markup"),  $1);
+               $$ = scm_list_2 (Lily::line_markup,  $1);
        }
        | markup_head_1_list simple_markup
        {
-               $$ = scm_car (MAKE_SYNTAX ("composed-markup-list",
+               $$ = scm_car (MAKE_SYNTAX (composed_markup_list,
                                           @2, $1, scm_list_1 ($2)));
        }
        | simple_markup {
@@ -3471,7 +3554,7 @@ markup_scm:
                }
        } BACKUP
        ;
-                       
+
 
 markup_list:
        markup_composed_list {
@@ -3495,21 +3578,21 @@ markup_uncomposed_list:
                SCM nn = parser->lexer_->lookup_identifier ("pitchnames");
                parser->lexer_->push_note_state (nn);
        } '{' score_body '}' {
-               Score *sc = unsmob_score ($4);
+               Score *sc = unsmob<Score> ($4);
                sc->origin ()->set_spot (@$);
                if (sc->defs_.empty ()) {
                        Output_def *od = get_layout (parser);
                        sc->add_output_def (od);
                        od->unprotect ();
                }
-               $$ = scm_list_1 (scm_list_2 (ly_lily_module_constant ("score-lines-markup-list"), $4));
+               $$ = scm_list_1 (scm_list_2 (Lily::score_lines_markup_list, $4));
                parser->lexer_->pop_state ();
        }
        ;
 
 markup_composed_list:
        markup_head_1_list markup_uncomposed_list {
-               $$ = MAKE_SYNTAX ("composed-markup-list",
+               $$ = MAKE_SYNTAX (composed_markup_list,
                                  @2, $1, $2);
        }
        ;
@@ -3578,14 +3661,14 @@ simple_markup:
                SCM nn = parser->lexer_->lookup_identifier ("pitchnames");
                parser->lexer_->push_note_state (nn);
        } '{' score_body '}' {
-               Score *sc = unsmob_score ($4);
+               Score *sc = unsmob<Score> ($4);
                sc->origin ()->set_spot (@$);
                if (sc->defs_.empty ()) {
                        Output_def *od = get_layout (parser);
                        sc->add_output_def (od);
                        od->unprotect ();
                }
-               $$ = scm_list_2 (ly_lily_module_constant ("score-markup"), $4);
+               $$ = scm_list_2 (Lily::score_markup, $4);
                parser->lexer_->pop_state ();
        }
        | MARKUP_FUNCTION markup_command_basic_arguments {
@@ -3600,7 +3683,7 @@ simple_markup:
 markup:
        markup_head_1_list simple_markup
        {
-               $$ = scm_car (MAKE_SYNTAX ("composed-markup-list",
+               $$ = scm_car (MAKE_SYNTAX (composed_markup_list,
                                           @2, $1, scm_list_1 ($2)));
        }
        | simple_markup {
@@ -3619,12 +3702,19 @@ Lily_parser::set_yydebug (bool x)
 SCM
 Lily_parser::do_yyparse ()
 {
-        SCM retval = SCM_UNDEFINED;
-       yyparse (this, &retval);
-        return retval;
+       return scm_c_with_fluid (Lily::f_parser,
+                                self_scm (),
+                                do_yyparse_trampoline,
+                                static_cast <void *>(this));
 }
 
-
+SCM
+Lily_parser::do_yyparse_trampoline (void *parser)
+{
+       SCM retval = SCM_UNDEFINED;
+       yyparse (static_cast <Lily_parser *>(parser), &retval);
+       return retval;
+}
 
 
 
@@ -3637,8 +3727,8 @@ otherwise, we have to import music classes into the lexer.
 int
 Lily_lexer::try_special_identifiers (SCM *destination, SCM sid)
 {
-       if (unsmob_book (sid)) {
-               Book *book =  unsmob_book (sid)->clone ();
+       if (unsmob<Book> (sid)) {
+               Book *book =  unsmob<Book> (sid)->clone ();
                *destination = book->self_scm ();
                book->unprotect ();
 
@@ -3646,30 +3736,30 @@ Lily_lexer::try_special_identifiers (SCM *destination, SCM sid)
        } else if (scm_is_number (sid)) {
                *destination = sid;
                return NUMBER_IDENTIFIER;
-       } else if (unsmob_context_def (sid))
+       } else if (unsmob<Context_def> (sid))
        {
-               *destination = unsmob_context_def (sid)->clone ()->unprotect ();
+               *destination = unsmob<Context_def> (sid)->clone ()->unprotect ();
                return SCM_IDENTIFIER;
-        } else if (unsmob_context_mod (sid)) {
-                *destination = unsmob_context_mod (sid)->smobbed_copy ();
+        } else if (unsmob<Context_mod> (sid)) {
+                *destination = unsmob<Context_mod> (sid)->smobbed_copy ();
                 return CONTEXT_MOD_IDENTIFIER;
-       } else if (Music *mus = unsmob_music (sid)) {
+       } else if (Music *mus = unsmob<Music> (sid)) {
                mus = mus->clone ();
                *destination = mus->self_scm ();
                bool is_event = mus->is_mus_type ("post-event");
                mus->unprotect ();
                return is_event ? EVENT_IDENTIFIER : MUSIC_IDENTIFIER;
-       } else if (unsmob_pitch (sid)) {
-               *destination = unsmob_pitch (sid)->smobbed_copy ();
+       } else if (unsmob<Pitch> (sid)) {
+               *destination = unsmob<Pitch> (sid)->smobbed_copy ();
                return PITCH_IDENTIFIER;
-       } else if (unsmob_duration (sid)) {
-               *destination = unsmob_duration (sid)->smobbed_copy ();
+       } else if (unsmob<Duration> (sid)) {
+               *destination = unsmob<Duration> (sid)->smobbed_copy ();
                return DURATION_IDENTIFIER;
-       } else if (unsmob_output_def (sid)) {
-               *destination = unsmob_output_def (sid)->clone ()->unprotect ();
+       } else if (unsmob<Output_def> (sid)) {
+               *destination = unsmob<Output_def> (sid)->clone ()->unprotect ();
                return SCM_IDENTIFIER;
-       } else if (unsmob_score (sid)) {
-               *destination = unsmob_score (sid)->clone ()->unprotect ();
+       } else if (unsmob<Score> (sid)) {
+               *destination = unsmob<Score> (sid)->clone ()->unprotect ();
                return SCM_IDENTIFIER;
        }
 
@@ -3679,7 +3769,7 @@ Lily_lexer::try_special_identifiers (SCM *destination, SCM sid)
 SCM
 get_next_unique_context_id ()
 {
-       return scm_from_locale_string ("$uniqueContextId");
+       return scm_from_ascii_string ("$uniqueContextId");
 }
 
 
@@ -3689,7 +3779,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_from_locale_string (s);
+       return scm_from_ascii_string (s);
 }
 
 // check_scheme_arg checks one argument with a given predicate for use
@@ -3718,7 +3808,7 @@ SCM check_scheme_arg (Lily_parser *parser, Input loc,
                        return args;
        }
        scm_set_cdr_x (scm_last_pair (args), SCM_EOL);
-       MAKE_SYNTAX ("argument-error", loc, scm_length (args), pred,
+       MAKE_SYNTAX (argument_error, loc, scm_length (args), pred,
                     SCM_UNBNDP (disp) ? arg : disp);
        scm_set_cdr_x (scm_last_pair (args), SCM_BOOL_F);
        return args;
@@ -3726,7 +3816,7 @@ SCM check_scheme_arg (Lily_parser *parser, Input loc,
 
 SCM loc_on_music (Input loc, SCM arg)
 {
-       if (Music *m = unsmob_music (arg))
+       if (Music *m = unsmob<Music> (arg))
        {
                m = m->clone ();
                m->set_spot (loc);
@@ -3805,7 +3895,7 @@ is_regular_identifier (SCM id, bool multiple)
 SCM
 make_music_from_simple (Lily_parser *parser, Input loc, SCM simple)
 {
-       if (unsmob_music (simple))
+       if (unsmob<Music> (simple))
                return simple;
        if (parser->lexer_->is_note_state ()) {
                if (scm_is_symbol (simple)) {
@@ -3814,7 +3904,7 @@ make_music_from_simple (Lily_parser *parser, Input loc, SCM simple)
                        n->set_property ("drum-type", simple);
                        return n->unprotect ();
                }
-               if (unsmob_pitch (simple)) {
+               if (unsmob<Pitch> (simple)) {
                        Music *n = MY_MAKE_MUSIC ("NoteEvent", loc);
                        n->set_property ("duration", parser->default_duration_.smobbed_copy ());
                        n->set_property ("pitch", simple);
@@ -3823,12 +3913,12 @@ make_music_from_simple (Lily_parser *parser, Input loc, SCM simple)
                return simple;
        } else if (parser->lexer_->is_lyric_state ()) {
                if (Text_interface::is_markup (simple))
-                       return MAKE_SYNTAX ("lyric-event", loc, simple,
+                       return MAKE_SYNTAX (lyric_event, loc, simple,
                                            parser->default_duration_.smobbed_copy ());
        } else if (parser->lexer_->is_chord_state ()) {
-               if (unsmob_pitch (simple))
+               if (unsmob<Pitch> (simple))
                        return MAKE_SYNTAX
-                               ("event-chord",
+                               (event_chord,
                                 loc,
                                 make_chord_elements (loc, simple,
                                                      parser->default_duration_.smobbed_copy (),
@@ -3879,11 +3969,10 @@ make_chord_step (SCM step_scm, Rational alter)
 SCM
 make_chord_elements (Input loc, SCM pitch, SCM dur, SCM modification_list)
 {
-       SCM chord_ctor = ly_lily_module_constant ("construct-chord-elements");
-       SCM res = scm_call_3 (chord_ctor, pitch, dur, modification_list);
+       SCM res = Lily::construct_chord_elements (pitch, dur, modification_list);
        for (SCM s = res; scm_is_pair (s); s = scm_cdr (s))
        {
-               unsmob_music (scm_car (s))->set_spot (loc);
+               unsmob<Music> (scm_car (s))->set_spot (loc);
        }
        return res;
 }