[Top][All Lists]

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

Re: Extra check in define-event-class patch

From: Mike Solomon
Subject: Re: Extra check in define-event-class patch
Date: Fri, 13 Aug 2010 10:12:07 +0200
User-agent: Microsoft-Entourage/

On 8/12/10 7:20 PM, "Trevor Daniels" <address@hidden> wrote:

> Mike Solomon wrote Thursday, August 12, 2010 10:16 AM
>>    I added an extra symbol check to my event class patch:
>> Also, I have a regression test that creates a spanner engraver
>> from the
>> scheme end using this function (said regtest could also be turned
>> into a bit
>> of documentation for the contributor's guide if people feel that'd
>> be
>> helpful - I think it would given a recent thread on frogs
>> concerning
>> engravers & their complexity), but I don't know if I should submit
>> it as
>> part of this patch set or as a new issue on Reitveld.  Lemme know
>> :-)
> If the regression test tests the new function I'd make
> it part of the same Reitveld issue.  If your patch makes
> it to the distribution we'd want to install both together.
> Trevor

Done - thank you for the suggestion.  Attached is the patch series, which
can also be found on .  Thank you!


Attachment: 0003-Adds-a-regression-test-showing-ly-define-event-class.patch
Description: Binary data

Attachment: 0002-Symbol-checking.patch
Description: Binary data

Attachment: 0001-Create-ly-define-event-class.patch
Description: Binary data

reply via email to

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