|
From: | reinhold . kainhofer |
Subject: | Re: Make accidental styles available as context mods. (issue 4819064) |
Date: | Mon, 05 Sep 2011 16:21:43 +0000 |
LGTM. http://codereview.appspot.com/4819064/diff/1/lily/parser.yy File lily/parser.yy (right): http://codereview.appspot.com/4819064/diff/1/lily/parser.yy#newcode670 lily/parser.yy:670: continue; What exactly is the reason for this hardcoded workaround? Somehow I don't get your comment in the patch description... http://codereview.appspot.com/4819064/diff/1/ly/context-mods-init.ly File ly/context-mods-init.ly (right): http://codereview.appspot.com/4819064/diff/1/ly/context-mods-init.ly#newcode30 ly/context-mods-init.ly:30: (cdr style-settings)))))) I don't see any way around the lambda functions displayed for make-accidental-style... http://codereview.appspot.com/4819064/
[Prev in Thread] | Current Thread | [Next in Thread] |