Re: [Gnash-commit] gnash ChangeLog server/PropertyList.cpp server/...

[Sandro Santilli]

On Thu, Oct 26, 2006 at 10:32:19PM +0000, Sandro Santilli wrote:

Hello strk :)

> Log message:
>               * server/PropertyList.{cpp,h}: changed container to store
>                 pointers to the new polymorphic Property class; added pointer
>                 to as_object owner for calls to getter/setter properties.
>               * testsuite/server/PropertyListTest.cpp: fixed test to work
>                 with new interface (requirement for an as_object in ctor)
>               * server/as_object.h: fix constructors to properly initialize
>                 the PropertyList.
>               * server/as_member.h: removed, as obsoleted

A short explanation:

The whole point is enabling getter/setter methods and try to use
a single lookup table for all members.
This as an ActionScript feature, but might possibly also be used
for standard properties like _x,_y,_xscale,_yscale etc (to be tested).
Note that the VariableText thing could probably also be implemented 
with the GetterSetter interface.

One of the side-effects of the change is that we now use std::string
and std::map rather then tu_stringi and stringi_hash, which seems a good
thing. The next step would be updating some of as_object interface to
directly take a std::string (or whatever we should use for UTF8 - suggestions
welcome).

The new architecture is probably not much memory efficient, but I've done
my best to provide memory management correctness.

--strk;