[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Uses single algorithm for side-position spacing. (issue 6827072)
From: |
k-ohara5a5a |
Subject: |
Re: Uses single algorithm for side-position spacing. (issue 6827072) |
Date: |
Thu, 29 Nov 2012 09:24:43 +0000 |
It was Box_quarantine that seemed a confusing name. (Why 'quarantine'?
Is there something special about 40 boxes?) I take back my suggestion.
http://codereview.appspot.com/6827072/diff/7041/lily/box-quarantine.cc
File lily/box-quarantine.cc (right):
http://codereview.appspot.com/6827072/diff/7041/lily/box-quarantine.cc#newcode63
lily/box-quarantine.cc:63: TODO: not sure if this loop causes infinite
behavior...
The while(dirty) loop runs 2366 times for the last chord in
'fingering-collision.ly' but that's an extreme case.
http://codereview.appspot.com/6827072/diff/7041/lily/fingering-column.cc
File lily/fingering-column.cc (left):
http://codereview.appspot.com/6827072/diff/7041/lily/fingering-column.cc#oldcode63
lily/fingering-column.cc:63: fingerings[i]->translate_axis
(-fingerings[i]->extent (common[Y_AXIS], Y_AXIS).length () / 2, Y_AXIS);
This shifted all the fingerings down to align them horizontally to their
note-heads, and is missing from the new code
http://codereview.appspot.com/6827072/
- Re: Uses single algorithm for side-position spacing. (issue 6827072), (continued)
Re: Uses single algorithm for side-position spacing. (issue 6827072), k-ohara5a5a, 2012/11/17
Re: Uses single algorithm for side-position spacing. (issue 6827072), k-ohara5a5a, 2012/11/17
Re: Uses single algorithm for side-position spacing. (issue 6827072), k-ohara5a5a, 2012/11/28
Re: Uses single algorithm for side-position spacing. (issue 6827072), k-ohara5a5a, 2012/11/29
Re: Uses single algorithm for side-position spacing. (issue 6827072),
k-ohara5a5a <=
Re: Uses single algorithm for side-position spacing. (issue 6827072), k-ohara5a5a, 2012/11/30
Re: Uses single algorithm for side-position spacing. (issue 6827072), k-ohara5a5a, 2012/11/30