[Gnash-commit] gnash ChangeLog gui/gtk.cpp

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/09/19 10:32:52

Modified files:
        .              : ChangeLog 
        gui            : gtk.cpp 

Log message:
                * gui/gtk.cpp (setInterval): add a note about the fact that the
                  current way of setting up the FPS timer is bogus. Note that 
the
                  problem is related to A/V syncing...

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.4350&r2=1.4351
http://cvs.savannah.gnu.org/viewcvs/gnash/gui/gtk.cpp?cvsroot=gnash&r1=1.109&r2=1.110

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.4350
retrieving revision 1.4351
diff -u -b -r1.4350 -r1.4351
--- ChangeLog   19 Sep 2007 10:17:02 -0000      1.4350
+++ ChangeLog   19 Sep 2007 10:32:52 -0000      1.4351
@@ -1,5 +1,11 @@
 2007-09-19 Sandro Santilli <address@hidden>
 
+       * gui/gtk.cpp (setInterval): add a note about the fact that the 
+         current way of setting up the FPS timer is bogus. Note that the
+         problem is related to A/V syncing...
+
+2007-09-19 Sandro Santilli <address@hidden>
+
        * server/character.{cpp,h}: provide a generic
          _name getter-setter.
        * server/sprite_instance.cpp: use character's version

Index: gui/gtk.cpp
===================================================================
RCS file: /sources/gnash/gnash/gui/gtk.cpp,v
retrieving revision 1.109
retrieving revision 1.110
diff -u -b -r1.109 -r1.110
--- gui/gtk.cpp 12 Sep 2007 10:57:05 -0000      1.109
+++ gui/gtk.cpp 19 Sep 2007 10:32:52 -0000      1.110
@@ -17,7 +17,7 @@
 // Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 //
 
-/* $Id: gtk.cpp,v 1.109 2007/09/12 10:57:05 bwy Exp $ */
+/* $Id: gtk.cpp,v 1.110 2007/09/19 10:32:52 strk Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
@@ -369,6 +369,18 @@
 GtkGui::setInterval(unsigned int interval)
 {
     _interval = interval;
+
+       // From 
http://www.idt.mdh.se/kurser/cd5040/ht02/gtk/glib/glib-the-main-event-loop.html#G-TIMEOUT-ADD-FULL
+       //
+       // Note that timeout functions may be delayed, due to the
+       // processing of other event sources. Thus they should not be
+       // relied on for precise timing. After each call to the timeout
+       // function, the time of the next timeout is recalculated based
+       // on the current time and the given interval (it does not try to
+       // 'catch up' time lost in delays).
+       //
+       // TODO: this is not what we need here, we want instead to 'catch up' !!
+       //
     g_timeout_add_full (G_PRIORITY_LOW, interval, (GSourceFunc)advance_movie,
                         this, NULL);
 }