[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Added transition lines for lyrics (issue 565750043 by address@hidden
From: |
hanwenn |
Subject: |
Re: Added transition lines for lyrics (issue 565750043 by address@hidden) |
Date: |
Tue, 24 Mar 2020 14:46:29 -0700 |
LGTM
Can you verify that the patch description is correct? The current
version doesn't use -> anymore.
https://codereview.appspot.com/565750043/diff/569570043/lily/vowel-transition.cc
File lily/vowel-transition.cc (right):
https://codereview.appspot.com/565750043/diff/569570043/lily/vowel-transition.cc#newcode37
lily/vowel-transition.cc:37: SCM num_length = me->get_property
("minimum-length");
num suggests a number.
minimum_length ?
https://codereview.appspot.com/565750043/diff/569570043/lily/vowel-transition.cc#newcode137
lily/vowel-transition.cc:137: w += -d * r->item_drul_[d]->extent
(r->item_drul_[d], X_AXIS)[-d];
this still looks strange, but if it's problem, it'll be contained within
the vowel-transition code, which is acceptable.
https://codereview.appspot.com/565750043/
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), (continued)
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), lemzwerg, 2020/03/11
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), davidgrant . no, 2020/03/11
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), lemzwerg, 2020/03/11
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), davidgrant . no, 2020/03/12
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), lemzwerg, 2020/03/12
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), davidgrant . no, 2020/03/13
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), hanwenn, 2020/03/15
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), hanwenn, 2020/03/15
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), davidgrant . no, 2020/03/20
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden),
hanwenn <=
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), davidgrant . no, 2020/03/25
- Re: Added transition lines for lyrics (issue 565750043 by address@hidden), hanwenn, 2020/03/27