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

Issue 251130043: Provide music functions property-{override,revert,set,unset} (Closed)

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

Description

Provide music functions property-{override,revert,set,unset} The previous music functions `override' and `revert' (intentionally colliding with the respective keywords) are removed. It would have been too confusing to have collisions for `set' and `unset' as well which are just too similar to existing Scheme constructs. A convert-ly rule for the previous calls override/revert is not attempted: they have not yet been documented and have been available only for one unstable version. Whoever happened to use them already is capable of replacing them. Provide check-context-path function

Patch Set 1 #

Patch Set 2 : property-override etc -> propertyOverride etc #

Patch Set 3 : Add forgotten (*location*) arguments to check-context-path #

Unified diffs Side-by-side diffs Delta from patch set Stats (+103 lines, -55 lines) Patch
M ly/music-functions-init.ly View 1 2 7 chunks +79 lines, -55 lines 0 comments Download
M scm/music-functions.scm View 1 chunk +24 lines, -0 lines 0 comments Download

Messages

Total messages: 5
dak
Well, the consideration for choosing a hyphenated name here instead of camel case (namely property-override ...
8 years, 9 months ago (2015-07-12 22:46:07 UTC) #1
dak
property-override etc -> propertyOverride etc
8 years, 9 months ago (2015-07-12 23:19:29 UTC) #2
benko.pal
that's fabulous, thanks!
8 years, 9 months ago (2015-07-13 06:15:25 UTC) #3
dak
On 2015/07/13 06:15:25, benko.pal wrote: > that's fabulous, thanks! I'd have considered this issue on ...
8 years, 9 months ago (2015-07-13 07:50:04 UTC) #4
dak
8 years, 9 months ago (2015-07-13 18:51:55 UTC) #5
Add forgotten (*location*) arguments to check-context-path
Sign in to reply to this message.

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