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

Issue 6846107: Remove "selective" contextmods. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 5 months ago by dak
Modified:
11 years, 5 months ago
Reviewers:
CC:
lilypond-devel_gnu.org
Base URL:
http://git.savannah.gnu.org/gitweb/?p=lilypond.git/trunk/
Visibility:
Public.

Description

Remove "selective" contextmods. This reverts those parts of commit 70365334614c31a82e9a3860c9eb9334cdc2879a that have been responsible for allowing context-mods to contain modifications restricted to certain context types. It also removes use of this feature from context-mod-from-music which is used for harvesting music into context modifications. As a result, any overrides in music used inside of context mods are interpreted without looking at their actually targeted contexts. It turned out that this "feature" offered more confusion than benefits.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -19 lines) Patch
M lily/context-property.cc View 1 chunk +4 lines, -12 lines 0 comments Download
M scm/lily-library.scm View 2 chunks +5 lines, -7 lines 0 comments Download

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