Re: [Gnash-commit] gnash ChangeLog server/video_stream_instance.cpp

[Sandro Santilli]

On Fri, May 04, 2007 at 04:23:38PM +0200, Sandro Santilli wrote:
> On Fri, May 04, 2007 at 04:19:26PM +0200, Udo Giacomozzi wrote:
> > Hello Sandro,
> > 
> > Friday, May 4, 2007, 2:16:08 PM, you wrote:
> > SS> Log message:
> > SS>                 * server/video_stream_instance.cpp (display): call
> > SS>                   clear_invalidated().
> > 
> > I think this fixes re-rendering of the same video frame, right?
> 
> Supposedly, but I couldn't find a single occurrence in which ::display
> would report m_invalidated == true.

I meant == false. Her'es a patch to debug this:

Index: server/video_stream_instance.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/video_stream_instance.cpp,v
retrieving revision 1.23
diff -U2 -r1.23 video_stream_instance.cpp
--- server/video_stream_instance.cpp    4 May 2007 12:16:07 -0000       1.23
+++ server/video_stream_instance.cpp    4 May 2007 14:27:02 -0000
@@ -81,4 +81,7 @@
 video_stream_instance::display()
 {
+       // DEBUGGING
+       if ( ! m_invalidated ) log_debug("video_stream_instance NOT invalidated 
at ::display time, we like this !!!");
+
        matrix m = get_world_matrix();
        rect bounds(0.0f, 0.0f, PIXELS_TO_TWIPS(m_def->m_width), 
PIXELS_TO_TWIPS(m_def->m_height));

--strk;