Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1392)

Issue 248720043: Establish %parser in Lily_parser::do_yyparse (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 11 months ago by dak
Modified:
8 years, 10 months ago
Reviewers:
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Establish %parser in Lily_parser::do_yyparse Consequently, this does not need to be done any more in syntax constructors or Scheme expressions.

Patch Set 1 #

Patch Set 2 : Rebase on and adapt to current master #

Patch Set 3 : Fix an oversight #

Unified diffs Side-by-side diffs Delta from patch set Stats (+45 lines, -42 lines) Patch
M lily/include/lily-parser.hh View 2 chunks +2 lines, -1 line 0 comments Download
M lily/music-function.cc View 1 3 chunks +6 lines, -8 lines 0 comments Download
M lily/parse-scm.cc View 1 chunk +3 lines, -5 lines 0 comments Download
M lily/parser.yy View 3 chunks +14 lines, -7 lines 0 comments Download
M scm/ly-syntax-constructors.scm View 1 2 8 chunks +20 lines, -21 lines 0 comments Download

Messages

Total messages: 2
dak
Rebase on and adapt to current master
8 years, 11 months ago (2015-06-10 18:24:14 UTC) #1
dak
8 years, 11 months ago (2015-06-10 18:31:36 UTC) #2
Fix an oversight
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b