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

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  06/10/12 21:22:33

Modified files:
        gui            : gui.cpp 
        .              : ChangeLog 

Log message:
                * gui/gui.cpp: added IF_DEBUG_REGION_UPDATES and                
                  ENABLE_REGION_UPDATES_DEBUGGIN macros, for future
                  compiletime + runtime conditional support.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/gui/gui.cpp?cvsroot=gnash&r1=1.23&r2=1.24
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.1176&r2=1.1177

Patches:
Index: gui/gui.cpp
===================================================================
RCS file: /sources/gnash/gnash/gui/gui.cpp,v
retrieving revision 1.23
retrieving revision 1.24
diff -u -b -r1.23 -r1.24
--- gui/gui.cpp 12 Oct 2006 21:11:28 -0000      1.23
+++ gui/gui.cpp 12 Oct 2006 21:22:33 -0000      1.24
@@ -50,6 +50,19 @@
 #include "render.h"  // debug
 #include "render_handler.h"
 
+/// Define this to have updated regions enclosed in a red rectangle
+/// In the future, enabling this might actually use a runtime flag
+/// as an additional conditional.
+///
+//#define ENABLE_REGION_UPDATES_DEBUGGING 1
+
+#ifdef ENABLE_REGION_UPDATES_DEBUGGING
+// a runtime check would make the { x; } block conditionally executed
+#define IF_DEBUG_REGION_UPDATES(x) { x; }
+#else
+#define IF_DEBUG_REGION_UPDATES(x) 
+#endif
+
 namespace gnash {
 
 Gui::Gui() :
@@ -233,7 +246,7 @@
     m->display();
   
     // show invalidated region using a red rectangle (Flash debug style)
-    #if 0
+    IF_DEBUG_REGION_UPDATES (
     point corners[4];
     corners[0].m_x = draw_bounds.m_x_min;      
     corners[0].m_y = draw_bounds.m_y_min;      
@@ -245,8 +258,8 @@
     corners[3].m_y = draw_bounds.m_y_max;
     matrix dummy;      
     gnash::render::set_matrix(dummy); // reset matrix
-    gnash::render::draw_poly(&corners[0], 4, rgba(0,0,0,0), rgba(255,0,0,255));
-    #endif     
+    gnash::render::draw_poly(&corners[0], 4, rgba(0,255,0,0), 
rgba(255,0,0,255));
+    );
 
     // show frame on screen
        gui->renderBuffer();

Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.1176
retrieving revision 1.1177
diff -u -b -r1.1176 -r1.1177
--- ChangeLog   12 Oct 2006 21:11:28 -0000      1.1176
+++ ChangeLog   12 Oct 2006 21:22:33 -0000      1.1177
@@ -1,5 +1,8 @@
 2006-10-13 Sandro Santilli <address@hidden>
 
+       * gui/gui.cpp: added IF_DEBUG_REGION_UPDATES and
+         ENABLE_REGION_UPDATES_DEBUGGIN macros, for future
+         compiletime + runtime conditional support.
        * gui/gui.{cpp,h}: advance_movie reverted to a static, to make
          current gtk gui work fine (there's no point of being non-static
          *and* take a 'this' argument).