Re: [frogs] My First Patch: issue #1063

[Carl Sorensen <c_sorensen@xxxxxxx>]

On 5/20/10 5:08 AM, "Rodolfo Zitellini" <xhero.gm@xxxxxxxxx> wrote:

>> Wow -- this is the first time I've ever heard "clement" used in a sentence.
>> In the U.S., all I hear is "inclement"; I've even heard discussions about
>> whether "clement" is a word or not.  Thanks for the education!
>> 
> Well I have to admit I am not 100% sure it is correct, but I looked it
> up in the dictionary before using it :)

I checked the dictionary after I read it, and you are absolutely correct.

> 
>> 
>> If MyGrob's default value for 'myProp is 1, and I do
>> 
>> \override MyGrob #'myProp = #10
>> \override MyGrob #'myProp = #2
>> \revert MyGrob #'myProp
>> 
>> the value of 'myProp for MyGrob should be 1, not 10.
>> 
> 
> This works with my patch too. I have noticed, using the debugger, that
> every \override generates a call to execute_revert_property() before
> calling execute_override_property() on the same property - so you end
> up with only two copies (new and old) in the list all the times (this
> seems to be done for "normal" and nested properties).

Great!  I'll test the patch as soon as I can.

Thanks,

Carl


---
----
Join the Frogs!