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

Issue 276260043: Disallow "Scheme_engraver" as an engraver class of its own (Closed)

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

Description

Disallow "Scheme_engraver" as an engraver class of its own It should not be registered as "Scheme_engraver", and no instances of it should be allowed to exist without a definition of their own. Also, instead of TRANSLATOR_DECLARATIONS_NO_LISTENER we use TRANSLATOR_FAMILY_DECLARATIONS for anything not pertaining to an individually documented translator.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -37 lines) Patch
M lily/include/scheme-engraver.hh View 2 chunks +3 lines, -4 lines 0 comments Download
M lily/include/translator.hh View 3 chunks +8 lines, -9 lines 0 comments Download
M lily/include/translator.icc View 1 chunk +6 lines, -3 lines 0 comments Download
M lily/scheme-engraver.cc View 3 chunks +4 lines, -15 lines 0 comments Download
M lily/translator-group.cc View 2 chunks +3 lines, -6 lines 0 comments Download

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