[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Adds stem acknowledgement to beam collision engraver to fix issue 79
From: |
address@hidden |
Subject: |
Re: Adds stem acknowledgement to beam collision engraver to fix issue 795. (issue4337045) |
Date: |
Thu, 31 Mar 2011 21:05:05 -0400 |
On Mar 31, 2011, at 6:39 PM, address@hidden wrote:
> Lots of changes to stem lengths, but it compiles and nothing makes me go
> "definitely ick".
>
> http://codereview.appspot.com/4337045/
I'm a fan of most of the changes in the regtests.
I'm amazed you got it to compile, though. Mine segfaulted, at which point I
changed it to disregard stems with beams. New patch set uploaded.
Cheers,
MS