[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by addres
From: |
lilyliska |
Subject: |
Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden) |
Date: |
Sun, 26 Apr 2015 13:49:33 +0000 |
On 2015/04/26 02:35:34, Carl wrote:
baseMoment does have another use. It is used to define the major
beaming groups
for automatic beaming.
...
And this use of baseMoment is important for subdivision; subdivision
is only
possible at units of baseMoment.
Thanks,
Carl
OK, so I "withdraw" my suggestions. Was a nice experience to make
changes to a C++ file that had working and visible effects, though ;-)
So, I repeat: LGTM
https://codereview.appspot.com/226700043/
- Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden), (continued)
Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden), Carl . D . Sorensen, 2015/04/25
Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden), lilyliska, 2015/04/25
Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden), lilyliska, 2015/04/25
Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden), Carl . D . Sorensen, 2015/04/25
Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden), lemzwerg, 2015/04/26
Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden),
lilyliska <=
Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden), tdanielsmusic, 2015/04/26