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

Issue 4670056: Fix 1390: slashedGrace function (graces with slashed stems but no slur, e.g. when grace is tied) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 9 months ago by Reinhold
Modified:
12 years, 9 months ago
Reviewers:
markpolesky, James.Lowe
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Fix 1390: slashedGrace function (graces with slashed stems but no slur, e.g. when grace is tied)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -0 lines) Patch
A input/regression/grace-slashed-no-slur.ly View 1 chunk +11 lines, -0 lines 0 comments Download
M ly/grace-init.ly View 1 chunk +8 lines, -0 lines 0 comments Download
M ly/music-functions-init.ly View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 4
pkx166h
passes make and reg tests
12 years, 9 months ago (2011-07-11 21:17:15 UTC) #1
Keith
Looks good and works well for me.
12 years, 9 months ago (2011-07-16 00:54:58 UTC) #2
markpolesky_yahoo.com
> http://codereview.appspot.com/4670056/ Reinhold, I think the slashedGrace function needs to be described in the NR ...
12 years, 9 months ago (2011-07-23 16:20:31 UTC) #3
James.Lowe_datacore.com
12 years, 9 months ago (2011-07-23 22:16:53 UTC) #4
Hello
________________________________________
From: lilypond-devel-bounces+james.lowe=datacore.com@gnu.org
[lilypond-devel-bounces+james.lowe=datacore.com@gnu.org] on behalf of Mark
Polesky [markpolesky@yahoo.com]
Sent: 23 July 2011 17:20
To: reinhold.kainhofer@gmail.com
Cc: reply@codereview.appspotmail.com; lilypond-devel@gnu.org
Subject: Re: Fix 1390: slashedGrace function (graces with slashed stems but no 
slur, e.g. when grace is tied) (issue4670056)

> http://codereview.appspot.com/4670056/

Reinhold,

I think the slashedGrace function needs to be described in
the NR (1.2.6), and also preferably listed in changes.tely.

Thanks.
- Mark
-----------

Added as 

http://code.google.com/p/lilypond/issues/detail?id=1778


James
Sign in to reply to this message.

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