[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Lilypond-auto] Issue 3683 in lilypond: fingering collides
From: |
lilypond |
Subject: |
Re: [Lilypond-auto] Issue 3683 in lilypond: fingering collides |
Date: |
Thu, 28 Nov 2013 21:34:13 +0000 |
Updates:
Status: Started
Cc: address@hidden
Labels: Patch-new
Comment #2 on issue 3683 by address@hidden: fingering collides
http://code.google.com/p/lilypond/issues/detail?id=3683
We can try addressing the efficiency complaint at
<http://code.google.com/p/lilypond/issues/detail?id=2527#c61>
but leave the recursive_add_support() in place.
(Note that the intervening issue 3365 makes remove_duplicates() more
expensive.)
This is a bit more risky than a simple revert, if recursive_add_support()
actually does add large number of duplicates, as Mike feared when he first
introduced the code. Mike, any memory of how you thought the duplicates
might become significant?
Here the 5 is supported by the 4, and recursive_add_support() also makes
the 5 also be supported by the 1.
http://codereview.appspot.com/2642041/
Attachments:
3683.preview.png 1.7 KB
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
- [Lilypond-auto] Issue 3683 in lilypond: fingering collides, lilypond, 2013/11/28
- Re: [Lilypond-auto] Issue 3683 in lilypond: fingering collides, lilypond, 2013/11/28
- Re: [Lilypond-auto] Issue 3683 in lilypond: fingering collides,
lilypond <=
- Re: [Lilypond-auto] Issue 3683 in lilypond: fingering collides, lilypond, 2013/11/28
- Re: [Lilypond-auto] Issue 3683 in lilypond: fingering collides, lilypond, 2013/11/28
- Re: [Lilypond-auto] Issue 3683 in lilypond: fingering collides, lilypond, 2013/11/29
- Re: [Lilypond-auto] Issue 3683 in lilypond: fingering collides, lilypond, 2013/11/29