[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Implement MeasureAttachedSpanner (issue 571180043 by address@hidden)
|
From: |
Carl . D . Sorensen |
|
Subject: |
Re: Implement MeasureAttachedSpanner (issue 571180043 by address@hidden) |
|
Date: |
Fri, 15 Nov 2019 11:21:09 -0800 |
I have not reviewed the code, but this looks like a worthwhile addition.
Thanks for doing it!
I think the name should be changed from MeasureAttachedSpanner to
BarAttachedSpanner. A measure is the interval between bar lines. The
spanner is attached to the bar line. While it requires some work, I
think it's worth making the change to be more clear in our terminology.
Thanks,
Carl
https://codereview.appspot.com/571180043/