|
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:30:45 +0000 |
Still cherry-picking stuff into the 2.20 branch, so we don't want extensive departures (in particular large convert-ly orgies) in master yet. Once 2.20.1 is out, this can get back on the slate, but I'd actually want to get 2.21.1 (pretty much right after 2.20.1, however) without such changes in it as well.
[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: Wed Sep 27, 2017 07:04 AM 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] |