[Gnash-commit] gnash ChangeLog server/swf_function.cpp testsui...

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  08/03/07 21:07:51

Modified files:
        .              : ChangeLog 
        server         : swf_function.cpp 
        testsuite/actionscript.all: Function.as 

Log message:
        arguments.callee is not enumerable by default.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.5847&r2=1.5848
http://cvs.savannah.gnu.org/viewcvs/gnash/server/swf_function.cpp?cvsroot=gnash&r1=1.48&r2=1.49
http://cvs.savannah.gnu.org/viewcvs/gnash/testsuite/actionscript.all/Function.as?cvsroot=gnash&r1=1.67&r2=1.68

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.5847
retrieving revision 1.5848
diff -u -b -r1.5847 -r1.5848
--- ChangeLog   7 Mar 2008 20:03:06 -0000       1.5847
+++ ChangeLog   7 Mar 2008 21:07:50 -0000       1.5848
@@ -1,5 +1,12 @@
 2008-03-07 Sandro Santilli <address@hidden>
 
+       * server/swf_function.cpp: arguments.callee is not enumerable
+         by default.
+       * testsuite/actionscript.all/Function.as: test for default
+         enumerable properties of 'arguments' succeeds.
+
+2008-03-07 Sandro Santilli <address@hidden>
+
        * testsuite/actionscript.all/Function.as: expect a failure on
          composition of the 'arguments' object. Was already broken,
          used to succeed due to another bug (availability of a 'size'

Index: server/swf_function.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/swf_function.cpp,v
retrieving revision 1.48
retrieving revision 1.49
diff -u -b -r1.48 -r1.49
--- server/swf_function.cpp     19 Feb 2008 19:20:54 -0000      1.48
+++ server/swf_function.cpp     7 Mar 2008 21:07:51 -0000       1.49
@@ -85,7 +85,7 @@
        {
                arguments->push(fn.arg(i));
        }
-       arguments->set_member(NSV::PROP_CALLEE, &callee);
+       arguments->init_member(NSV::PROP_CALLEE, &callee);
 
        return arguments;
 

Index: testsuite/actionscript.all/Function.as
===================================================================
RCS file: /sources/gnash/gnash/testsuite/actionscript.all/Function.as,v
retrieving revision 1.67
retrieving revision 1.68
diff -u -b -r1.67 -r1.68
--- testsuite/actionscript.all/Function.as      7 Mar 2008 20:03:07 -0000       
1.67
+++ testsuite/actionscript.all/Function.as      7 Mar 2008 21:07:51 -0000       
1.68
@@ -417,7 +417,7 @@
         propRecorder.push(props.toString());
     }
     // no enumerable properties in default mode.
-    xcheck_equals(propRecorder.length, 0); //? shouldn't fail here
+    check_equals(propRecorder.length, 0); 
     
     ASSetPropFlags(a, null, 6, 1 );
     for(var props in a)