On Tue, 24 Apr 2012 09:21:17 -0700, <milimet...@gmail.com> wrote:
If you replace *all* the while(flip()) loops, you can remove the flip
function,
Well, there will be one more place: spacing-spanner.cc:291
There is:
while (flip (&d) != LEFT && rb);
By now I can't tell what the whole code is for. Could you give me a tip?
This is a loop of the same type you are replacing, except that it will end
early if rb is NULL. If you do not change it, a new programmer will find it
very strange. Without thinking about the meaning of the code, just analyzing
the mechanics, you can make an equivalent loop. I think that might be
for (LEFT_and_RIGHT (d))
{
...
if (!rb) break;
}
On 2012/04/24 04:10:36, Keith wrote:
Chords in seconds get a bit too much space, if they have ledgers.
Certainly not related to the rest of this patch.
As I guess, this bug should be filed as a separate issue, right?
Correct. It is not related to the loop structure.
On 22 April 2012 13:28, Han-Wen Nienhuys <hanw...@gmail.com> wrote:
Looks like a bug. Maybe you could work on a test/repro case?
Well, my understanding of this code is really, really vague, so it would
take me a lot of time to: understand this, learn how to write test cases
and then write one.
You are able to write test cases. I wrote the test case for this bug
http://code.google.com/p/lilypond/issues/detail?id=2493
by reading HanWen's mail and thinking of LilyPond input that would create the
situation. Next time you make a test-case for a bug, email it to
bug-lilyp...@gnu.org and somebody will put it on the bug tracker.
You certainly should compile LilyPond with your changes and see if it does, in
fact, fix issue 2493
Also, use the GNU spacing style if you can:
for (UP_and_DOWN (d))
There is a Python script scripts/auxiliar/fixxcc.py that standardizes the spacing. It is
a strange spacing style, but LilyPond uses it consistently. Consistent spacing probably
made it easier for you to find all the "while (flip (".
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
