[Gnash-commit] gnash ChangeLog testsuite/misc-ming.all/loop_te...

[Zou Lunkai]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Zou Lunkai <zoulunkai>  07/06/18 02:40:33

Modified files:
        .              : ChangeLog 
        testsuite/misc-ming.all: loop_test7.c 

Log message:
        add missing head file and comments

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.3572&r2=1.3573
http://cvs.savannah.gnu.org/viewcvs/gnash/testsuite/misc-ming.all/loop_test7.c?cvsroot=gnash&r1=1.2&r2=1.3

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.3572
retrieving revision 1.3573
diff -u -b -r1.3572 -r1.3573
--- ChangeLog   18 Jun 2007 02:39:26 -0000      1.3572
+++ ChangeLog   18 Jun 2007 02:40:32 -0000      1.3573
@@ -1,6 +1,5 @@
 2007-06-18 Zou Lunkai <address@hidden>
 
-       * libbase/GC.h: add missing head file
        * testsuite/misc-ming.all/loop_test7: review and comments
        
 2007-06-18 Sandro Santilli <address@hidden>

Index: testsuite/misc-ming.all/loop_test7.c
===================================================================
RCS file: /sources/gnash/gnash/testsuite/misc-ming.all/loop_test7.c,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -b -r1.2 -r1.3
--- testsuite/misc-ming.all/loop_test7.c        24 May 2007 08:48:04 -0000      
1.2
+++ testsuite/misc-ming.all/loop_test7.c        18 Jun 2007 02:40:33 -0000      
1.3
@@ -39,7 +39,6 @@
  *
  * Expected behaviour:
  *
- *   TODO: verify
  *   After jump back, the onConstruct event handler for the red square has 
been invoked twice.
  *
  * run as ./loop_test7
@@ -117,6 +116,12 @@
 
   // Frame3: Remove red square
   SWFDisplayItem_remove(it1);
+  // After compile, the RemoveObject2 tag is *after* the DoAction tag which 
+  // contains the following check. So it's not surprise that we can still 
access
+  // "movieClip1" here when considering the gloabal ActionQueue model! If the 
+  // RemoveObject2 is *before* the DoAction, then typeof(movieClip1) will 
reurn 'undefined'.
+  // So Gnash fails here because of action execution order!
+  // TODO: add testcase for this(DoAction placed *before* RemoveObject2 within 
the same frame).
   xcheck_equals(mo, "typeof(movieClip1)", "'movieclip'"); // kept alive for 
calling onUnload!
   check_equals(mo, "_root.mc1Constructed", "1");
   SWFMovie_nextFrame(mo);