|From:||Auto mailings of changes to Lily Issues via Testlilyissues-auto|
|Subject:||[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #4603 change all occurences of ‘partcombine’ to ‘partCombine’.|
|Date:||Sat, 22 Dec 2018 13:26:13 -0000|
Passes make, make check and a full make doc.
[issues:#4603] change all occurences of ‘partcombine’ to ‘partCombine’.
Created: Sun Sep 13, 2015 03:17 PM UTC by pkx166h
Last Updated: Fri Dec 21, 2018 05:51 PM UTC
Owner: Valentin Villenave
On 13/08/15 21:42, Malte Meyn wrote:
the case of the ‘c’ in partcombine is inconsistent, confusing me every
time I use \partcombine(Apart|Chords|…):
\partcombine, \partcombineApart, … but
I would suggest to change all occurences of ‘partcombine’ to ‘partCombine’.
Why not change ‘partCombine’ to ‘partcombine’? Because I looked at the
engravers which use the more ‘natural’ underscore instead of camelCase
for spaces. And in the same way as ‘Cue_clef_engraver’ suggests that
‘cue clef’ are two words (resulting in ‘cueClefGlyph’, not
‘cueclefGlyph’), ‘Part_combine_engraver’ does.
I think that this would be rather easy to change.
New Rietveld - http://codereview.appspot.com/369930043
Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
_______________________________________________ Testlilyissues-auto mailing list address@hidden https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
|[Prev in Thread]||Current Thread||[Next in Thread]|