[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: barcheck failure with partial in middle of piece

From: Paul Morris
Subject: Re: barcheck failure with partial in middle of piece
Date: Fri, 8 Aug 2014 14:59:19 -0700 (PDT)

David Kastrup wrote
> More like you are expecting too much from |.  A bar check will check
> that you are at the end of a bar.  Putting it in parallel with \partial
> does not make all that much sense since \partial explicitly demands
> _not_ to be at the end of a bar.
> At any rate, I found some branch in my repository that appears to change
> the behavior to the one you'd have preferred in this case.  Probably the
> patch did not work in the particular case I had in mind or had some
> logical fallacy in it or failed some other test.
> Since it at least seems to cause more expected behavior in your usage,
> I've put it to review as
> Tracker issue: 4056
> (http://code.google.com/p/lilypond/issues/detail?id=4056)
> Rietveld issue: 119610044 (http://codereview.appspot.com/119610044)
> Issue description:
>   Let \partial in mid-piece work via a finalization hook    That
>   allows all other processing to complete before measurePosition is
>   adjusted.  In particular, this avoids problems when multiple
>   parallel  contexts invoke \partial and/or bar checks.      Also
>   contains commit:  Regtest for \partial in mid-piece

Thanks David!  Glad you already had a solution for this on the drawing
board.  Talk about turnaround time.  I knew you were good, but anticipating
and solving issues before they are reported is taking it to another level. 
:-)  Thanks also for making \partial work mid-piece in the first place.

Abraham, thanks for the pointer to your workaround.

Having looked into this some more, it seems that (one place) where I went
wrong was assuming that I needed to have matching \partial commands in each
parallel context, when it seems that a \partial only needs to be in one of
them.  If that's correct, would it be worth adding something about this to
the docs?  Although maybe that wouldn't be needed after David's new patch?



View this message in context: 
Sent from the Bugs mailing list archive at Nabble.com.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]