Re: [frogs] patch for issue 708

[ Thread Index | Date Index | More lilynet.net/frogs Archives ]


On Sun, May 24, 2009 at 7:14 AM, Carl D. Sorensen <c_sorensen@xxxxxxx> wrote:
>
>
>
> On 5/24/09 4:49 AM, "Neil Puttock" <n.puttock@xxxxxxxxx> wrote:
>
>> 2009/5/24 Carl D. Sorensen <c_sorensen@xxxxxxx>:
>>> Thanks,  Applied.
>>
>> Unfortunately, there are two serious flaws here:
>>
>> - keySignature alists which aren't backquoted (e.g., the example in
>> the bug tracker) will be ignored
>>
>> - entries of the form (notename . alteration) are mangled:
>>
>> \set Staff.keySignature = #'((0 . 2) (1 . 2)  (4 . 2))
>>
>> -> \set Staff.keySignature = #`(((0 . 2) . ,SEMI-SHARP)
>>                              ((2 . 4) . ,SHARP))
>>
>> Less seriously, the two conversion functions appear to be identical
>> apart from the different dictionaries for alterations.  Would it be
>> possible to use a single `fixKS' function with the dictionaries passed
>> as an argument to cut down on the duplication?
>
>
> Thanks for catching this, Neil.
>
> Andrew, I've reverted the patch.  Could you rewrite it to fix these issues?


Yes, I'll take a look at it. Thanks, Neil for catching those!
Won't be this week, but I'll keep you posted.

Andrew

---

----
Join the Frogs!


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/