[Gnash-commit] gnash ChangeLog server/movie_instance.cpp serve...

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/02/15 11:14:52

Modified files:
        .              : ChangeLog 
        server         : movie_instance.cpp movie_root.cpp 

Log message:
                * server/: movie_instance.cpp, movie_root.cpp: move
                  tu_random reset from movie_instance to movie_root
                  (random number generation should be taken care at global 
level);
                  Have movie_instance::advance call ::construct() rather
                  then manually executing frame tags.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.2365&r2=1.2366
http://cvs.savannah.gnu.org/viewcvs/gnash/server/movie_instance.cpp?cvsroot=gnash&r1=1.7&r2=1.8
http://cvs.savannah.gnu.org/viewcvs/gnash/server/movie_root.cpp?cvsroot=gnash&r1=1.41&r2=1.42

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.2365
retrieving revision 1.2366
diff -u -b -r1.2365 -r1.2366
--- ChangeLog   15 Feb 2007 09:45:33 -0000      1.2365
+++ ChangeLog   15 Feb 2007 11:14:51 -0000      1.2366
@@ -1,5 +1,10 @@
 2007-02-14 Sandro Santilli <address@hidden>
 
+       * server/: movie_instance.cpp, movie_root.cpp: move
+         tu_random reset from movie_instance to movie_root
+         (random number generation should be taken care at global level);
+         Have movie_instance::advance call ::construct() rather
+         then manually executing frame tags.
        * testsuite/misc-ming.all/action_execution_order_test.c:
          Add an additional complexity to the test, checking
          for proper DisplayList construction to happen *before*

Index: server/movie_instance.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/movie_instance.cpp,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -b -r1.7 -r1.8
--- server/movie_instance.cpp   6 Dec 2006 10:21:32 -0000       1.7
+++ server/movie_instance.cpp   15 Feb 2007 11:14:52 -0000      1.8
@@ -31,7 +31,6 @@
 #include "movie_instance.h"
 #include "movie_definition.h"
 #include "movie_root.h"
-#include "tu_random.h"
 
 #include <vector>
 #include <string>
@@ -69,14 +68,9 @@
        _def->ensure_frame_loaded(min(get_current_frame()+2,
                get_frame_count()));
 
-       // random should go continuously that:
-       // 1. after restart of the player the situation has not repeated
-       // 2. by different machines the random gave different numbers
-       tu_random::next_random();
-
        if (m_on_event_load_called == false)
        {
-               execute_frame_tags(0); 
+               construct();
        }
 
        advance_sprite(delta_time);

Index: server/movie_root.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/movie_root.cpp,v
retrieving revision 1.41
retrieving revision 1.42
diff -u -b -r1.41 -r1.42
--- server/movie_root.cpp       12 Feb 2007 19:24:48 -0000      1.41
+++ server/movie_root.cpp       15 Feb 2007 11:14:52 -0000      1.42
@@ -30,6 +30,7 @@
 #include "movie_instance.h" // for implicit upcast to sprite_instance
 #include "render.h"
 #include "VM.h"
+#include "tu_random.h"
 
 #include <iostream>
 #include <string>
@@ -449,6 +450,11 @@
                }
        }
                        
+       // random should go continuously that:
+       // 1. after restart of the player the situation has not repeated
+       // 2. by different machines the random gave different numbers
+       tu_random::next_random();
+                       
 #ifdef GNASH_DEBUG
        size_t totframes = _movie->get_frame_count();
        size_t prevframe = _movie->get_current_frame();