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

[Sandro Santilli]

On Mon, May 05, 2008 at 10:02:30AM +0000, Sandro Santilli wrote:
> CVSROOT:      /sources/gnash
> Module name:  gnash
> Changes by:   Sandro Santilli <strk>  08/05/05 10:02:30
> 
> Modified files:
>       .              : ChangeLog 
>       server         : PropertyList.cpp PropertyList.h as_object.cpp 
>                        as_object.h 
>       server/asobj   : ClassHierarchy.cpp 
> 
> Log message:
>               * server/PropertyList.{cpp,h}: rename addDestructiveGetterSetter
>                 to addDestructiveGetter, as we're not using the setter anyway,
>                 add a flagIfMissing to avoid a duplicated scan.
>               * server/as_object.{cpp,h}: drop the 'setter' parameter to
>                 init_destructive_property.
>               * server/asobj/ClassHierarchy.cpp: don't create a useless
>                 setter for all declarative properties.

Running an empty swf reduces GC load with this patch:

        from: 162 registered GcResources
          to: 146 registered GcResources 

        (with GNASH_GC_TRIGGER_THRESHOLD=0)

Can be done better by avoiding as_function as a whole and work with
C function pointers. I'm working on that (digesting ClassHierarchy
isn't a walk in the park)

--strk;