|
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: | Wed, 27 Sep 2017 07:04:52 +0000 |
David,
This had been sat at Waiting for a whiile, I kind of stopped 'noticing' why. Now I see we have 2.21 are you saying - in effect - that this patch as it stands (assuming it still applies) is too little or could we apply this and then keep the tracker open, or create a new tracker for the 'rest' of your observation/suggestions?
It seems a waste to let this patch go by-the-by if it is usable.
[issues:#4603] change all occurences of ‘partcombine’ to ‘partCombine’.
Status: Started
Labels: Postponed_until_2_21
Created: Sun Sep 13, 2015 03:17 PM UTC by pkx166h
Last Updated: Fri Aug 11, 2017 01:21 PM UTC
Owner: pkx166h
On 13/08/15 21:42, Malte Meyn wrote:
Hi list,
the case of the ‘c’ in partcombine is inconsistent, confusing me every
time I use \partcombine(Apart|Chords|…):\partcombine, \partcombineApart, … but
\partCombineTextsOnNote, \partCombineListenerI 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.
Cheers,
Malte
Response to Issue #4603: Syntax change from all instances "partcombine" to "partCombine" and convert-ly rule added to go along with.
http://codereview.appspot.com/323040043 - part 1 of 3
http://codereview.appspot.com/326870043 - part 2 of 3
http://codereview.appspot.com/327970043 - part 3 of 3
Note this will require both a convert.ly and a makelsr.py run to make sure that the patch passes all the tests.
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.
------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________ Testlilyissues-auto mailing list address@hidden https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Prev in Thread] | Current Thread | [Next in Thread] |