[Laszlo-dev] For Review: Change 20081015-bargull-Q0K Summary: placement for contextmenus

[André Bargull]

I've added "dynamic" because of two reasons:

1) It's needed for contextmenus to work in swf9
=> the setter look-up in "LzNode#__LZapplyArgs(..)" tried to poke at 
"$lzc$set_ignoreplacement", which resulted in a ReferenceError.

2) It's now possible to define contextmenus in LZX (LPP-834). Let's say 
a user defined a <contextmenu> and added an <attribute> to it. If the 
contextmenu class was non-dynamic, this would result in a 
ReferenceError, too (e.g. again at __LZapplyArgs()).



On 10/16/2008 6:00 PM, P T Withington wrote:
> On 2008-10-15, at 09:20EDT, André Bargull wrote:
> 
>> Added "dynamic" modifier so setter look-up works (e.g required for 
>> "ignoreplacement").
> 
> Can you explain to me more about why you need 'dynamic'?  I think we 
> want to minimize the use of dynamic so we can get better compile-time 
> checking.    At least in some cases, rather than using dynamic on a 
> class, the class should just have a table as a property for holding 
> whatever is being stored dynamically on the instance.  Clearly, we'd 
> need to check the performance trade-off, but I think it would be safer 
> to do things that way.