Updates:
        Labels: -Patch-needs_work Patch-review

Comment #11 on issue 2470 by [email protected]: Patch: introduce completionUnit
http://code.google.com/p/lilypond/issues/detail?id=2470

Looking at the first comparative picture makes clear that the beams are broken here whenever a quarter note intervenes, namely when different circumstances make it impossible to continue. The usual 6/4 beaming exceptions in scm/time-signature-settings.scm state
    ;;   use defaults, but end beams with 32nd or finer each 1/4 beat
    ((6 . 4) .
             ((beamExceptions . ((end .  (((1 . 16) . (4 4 4 4 4 4))))))))

which, among other things, means that the comment does not match the code. One should likely add a triplet exception like on 4/4, and make either the code match the comment, or the comment match the code, depending on what the desired behavior actually is (my guess would be that the code is correct here).

In other words: I agree with Pál's statement that the beaming is an independent problem (for example, see how the first triplets in the unchanged code are beamed together contraintuitively), and consequently set the patch back to "review".

Reply via email to