Re: Case of 'c' in partcombine

[Charles Winston]

> On May 25, 2017, at 11:29 AM, Phil Holmes <address@hidden> wrote:
> 
> ----- 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?

Yes that would definitely help!

Charles