[Gnash-commit] gnash ChangeLog server/as_object.cpp testsuite/...

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  08/03/11 08:01:36

Modified files:
        .              : ChangeLog 
        server         : as_object.cpp 
        testsuite/actionscript.all: MovieClip.as 

Log message:
                * server/as_object.cpp: mark reachable resources from 'super'
                  (may live longer then a function call, if that's the user
                  intention).
                * testsuite/actionscript.all/MovieClip.as: check 'constructor'
                  and '__constructor__' members of builtin class instances.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.5865&r2=1.5866
http://cvs.savannah.gnu.org/viewcvs/gnash/server/as_object.cpp?cvsroot=gnash&r1=1.103&r2=1.104
http://cvs.savannah.gnu.org/viewcvs/gnash/testsuite/actionscript.all/MovieClip.as?cvsroot=gnash&r1=1.118&r2=1.119

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.5865
retrieving revision 1.5866
diff -u -b -r1.5865 -r1.5866
--- ChangeLog   10 Mar 2008 23:29:38 -0000      1.5865
+++ ChangeLog   11 Mar 2008 08:01:35 -0000      1.5866
@@ -1,3 +1,11 @@
+2008-03-10 Sandro Santilli <address@hidden>
+
+       * server/as_object.cpp: mark reachable resources from 'super'
+         (may live longer then a function call, if that's the user
+         intention).
+       * testsuite/actionscript.all/MovieClip.as: check 'constructor'
+         and '__constructor__' members of builtin class instances.
+
 2008-03-10 Bastiaan Jacques <address@hidden>
 
        * server/StringPredicates.h: Make case insensitive compare work with

Index: server/as_object.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/as_object.cpp,v
retrieving revision 1.103
retrieving revision 1.104
diff -u -b -r1.103 -r1.104
--- server/as_object.cpp        10 Mar 2008 23:23:10 -0000      1.103
+++ server/as_object.cpp        11 Mar 2008 08:01:36 -0000      1.104
@@ -100,6 +100,15 @@
                return as_value();
        }
 
+protected:
+
+       virtual void markReachableResources() const
+       {
+               if ( _ctor ) _ctor->setReachable();
+               if ( _proto ) _proto->setReachable();
+               markAsFunctionReachable();
+       }
+
 private:
 
        as_function* _ctor;

Index: testsuite/actionscript.all/MovieClip.as
===================================================================
RCS file: /sources/gnash/gnash/testsuite/actionscript.all/MovieClip.as,v
retrieving revision 1.118
retrieving revision 1.119
diff -u -b -r1.118 -r1.119
--- testsuite/actionscript.all/MovieClip.as     10 Mar 2008 17:18:48 -0000      
1.118
+++ testsuite/actionscript.all/MovieClip.as     11 Mar 2008 08:01:36 -0000      
1.119
@@ -20,7 +20,7 @@
 // compile this test case with Ming makeswf, and then
 // execute it like this gnash -1 -r 0 -v out.swf
 
-rcsid="$Id: MovieClip.as,v 1.118 2008/03/10 17:18:48 strk Exp $";
+rcsid="$Id: MovieClip.as,v 1.119 2008/03/11 08:01:36 strk Exp $";
 
 #include "check.as"
 
@@ -28,19 +28,19 @@
 endOfTest = function() 
 {
 #if OUTPUT_VERSION <= 5
-       check_totals(197); // SWF5
+       check_totals(199); // SWF5
 #endif
 
 #if OUTPUT_VERSION == 6
-       check_totals(611); // SWF6
+       check_totals(613); // SWF6
 #endif
 
 #if OUTPUT_VERSION == 7
-       check_totals(628); // SWF7
+       check_totals(630); // SWF7
 #endif
 
 #if OUTPUT_VERSION >= 8
-       check_totals(629); // SWF8+
+       check_totals(631); // SWF8+
 #endif
 
        play();
@@ -776,6 +776,8 @@
 
 var cl = new MovieClip();
 check_equals(cl.__proto__.constructor, MovieClip);
+check_equals(cl.constructor, MovieClip);
+check_equals(cl.__constructor__, MovieClip);
 check(cl instanceOf MovieClip);
 check(cl instanceOf Object);
 check_equals(typeof(cl), "object");
@@ -1404,7 +1406,7 @@
 xcheck_equals(propinspect, 20);
 
 createEmptyMovieClip('mc', 10);
-mc._x = 30;
+mc._x = 1;
 ret = mc.addProperty('_x', get, set);
 check_equals(ret, true);
 getCalls=0; setCalls=0;