lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Case of 'c' in partcombine


From: Phil Holmes
Subject: Re: Case of 'c' in partcombine
Date: Thu, 25 May 2017 16:29:42 +0100

----- Original Message ----- From: "Charles Winston" <address@hidden>
To: <address@hidden>
Sent: Thursday, May 25, 2017 3:40 PM
Subject: Case of 'c' in partcombine


Hey developers,

I am working on my first patch—it is resolving a very simple issue from the tracker. The case of the ‘c’ in partcombine is inconsistent and can result in some confusion. For example: \partcombine, \partcombineApart, and others like this use the lower-case ‘c' … but \partCombineTextsOnNote, \partCombineListener use the camelCase ‘C’. The suggestion is to change all instances of partcombine to the camelCase partCombine—not the other way around because the engraver treats “part” and “combine” as separate words: ‘Part_combine_engraver’.

I would love some guidance as to where to find all the instances of partcombine. I know it is a command that takes two different as arguments, and combines them on the same staff, duplicating events that are different and keeping only one copy of events that are the same in both voices. I see the Scheme file which performs this function, but I believe that I need to make changes in the parsing process where the parser reads the string “partcombine” input by the user as the command. However I’m not completely clear on where to find this in the source code. Would love some preliminary help!

Thanks,
Charles Winston

git grep would allow you to find every instance of 'partcombine'. Would that help?

--
Phil Holmes



reply via email to

[Prev in Thread] Current Thread [Next in Thread]