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

Issue 4309052: Don't set 'tieMelismaBusy until old waiting heads have been cleared (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 1 month ago by Trevor Daniels
Modified:
13 years ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Don't set 'tieMelismaBusy until old waiting heads have been cleared - fixes 742

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -15 lines) Patch
M lily/tie-engraver.cc View 11 chunks +15 lines, -15 lines 0 comments Download

Messages

Total messages: 7
Trevor Daniels
This is a suggested fix to issue 742: Delayed ties influence lyrics (even with tieWaitForNote ...
13 years, 1 month ago (2011-03-26 13:34:57 UTC) #1
Graham Percival (old account)
LGTM
13 years, 1 month ago (2011-03-26 14:42:23 UTC) #2
Graham Percival (old account)
Time for complaints is over; please push.
13 years, 1 month ago (2011-03-28 14:02:49 UTC) #3
t.daniels_treda.co.uk
percival.music.ca@gmail.com wrote Monday, March 28, 2011 3:02 PM > Time for complaints is over; please ...
13 years, 1 month ago (2011-03-28 16:10:21 UTC) #4
t.daniels_treda.co.uk
percival.music.ca@gmail.com wrote Monday, March 28, 2011 3:02 PM > Time for complaints is over; please ...
13 years, 1 month ago (2011-03-28 16:10:22 UTC) #5
Neil Puttock
LGTM.
13 years, 1 month ago (2011-03-28 20:06:16 UTC) #6
Trevor Daniels
13 years, 1 month ago (2011-03-29 08:06:09 UTC) #7
Pushed.
3adffbb190d18e769f111594fdb629f4c0c6322e
Sign in to reply to this message.

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