[Gnash-commit] gnash ChangeLog server/vm/ActionExec.cpp [release_0_8_1]

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Branch:         release_0_8_1
Changes by:     Sandro Santilli <strk>  07/08/10 14:24:53

Modified files:
        .              : ChangeLog 
        server/vm      : ActionExec.cpp 

Log message:
                * server/vm/ActionExec.cpp (cleanupAfterRun): don't assume the 
call
                  stack should be empty at the end of global code execution. 
Still
                  print an error as we need further investigation about it. 
Fixes
                  bug #20740.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&only_with_tag=release_0_8_1&r1=1.3971.2.9&r2=1.3971.2.10
http://cvs.savannah.gnu.org/viewcvs/gnash/server/vm/ActionExec.cpp?cvsroot=gnash&only_with_tag=release_0_8_1&r1=1.39&r2=1.39.2.1

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.3971.2.9
retrieving revision 1.3971.2.10
diff -u -b -r1.3971.2.9 -r1.3971.2.10
--- ChangeLog   10 Aug 2007 05:58:42 -0000      1.3971.2.9
+++ ChangeLog   10 Aug 2007 14:24:51 -0000      1.3971.2.10
@@ -1,3 +1,10 @@
+2007-08-10 Sandro Santilli <address@hidden>
+
+       * server/vm/ActionExec.cpp (cleanupAfterRun): don't assume the call
+         stack should be empty at the end of global code execution. Still
+         print an error as we need further investigation about it. Fixes
+         bug #20740.
+
 2007-08-10 Zou Lunkai <address@hidden>
        
        * server/dlist.cpp: revert mask related code, fix bug#20655, which is

Index: server/vm/ActionExec.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/vm/ActionExec.cpp,v
retrieving revision 1.39
retrieving revision 1.39.2.1
diff -u -b -r1.39 -r1.39.2.1
--- server/vm/ActionExec.cpp    1 Jul 2007 10:54:36 -0000       1.39
+++ server/vm/ActionExec.cpp    10 Aug 2007 14:24:52 -0000      1.39.2.1
@@ -17,7 +17,7 @@
 // Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 //
 
-/* $Id: ActionExec.cpp,v 1.39 2007/07/01 10:54:36 bjacques Exp $ */
+/* $Id: ActionExec.cpp,v 1.39.2.1 2007/08/10 14:24:52 strk Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
@@ -288,7 +288,15 @@
     if ( ! isFunction() && env.callStackDepth() > 0 )
     {
        log_error(_("Call stack non-empty at end of ExecutableCode run (limits 
hit?)"));
-       env.clearCallFrames();
+       // TOOD:
+       // bug #20740 contains a movie that fails an assertion if we clean the 
call stack here
+       // maybe global code was executed as effect of a function code (think 
gotoFrame, attachMovie ?)
+       // We'll keep the verbose error for now, but let's not clean the call 
frame, shouldn't
+       // hurt anyway..
+       // A better implementation would likely be taking note of the 
callStackDepth at startup
+       // (like for _initial_stack_size) and check it here for consistency 
(and proper cleanup
+       // on limits hit).
+       //env.clearCallFrames();
     }
 
     // check if the stack was smashed