[Gnash-commit] gnash ChangeLog backend/render_handler_agg.cpp

[Udo Giacomozzi]

CVSROOT:        /cvsroot/gnash
Module name:    gnash
Changes by:     Udo Giacomozzi <udog>   06/11/04 12:34:55

Modified files:
        .              : ChangeLog 
        backend        : render_handler_agg.cpp 

Log message:
        opzimized alpha mask clearing

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.1514&r2=1.1515
http://cvs.savannah.gnu.org/viewcvs/gnash/backend/render_handler_agg.cpp?cvsroot=gnash&r1=1.38&r2=1.39

Patches:
Index: ChangeLog
===================================================================
RCS file: /cvsroot/gnash/gnash/ChangeLog,v
retrieving revision 1.1514
retrieving revision 1.1515
diff -u -b -r1.1514 -r1.1515
--- ChangeLog   4 Nov 2006 12:22:53 -0000       1.1514
+++ ChangeLog   4 Nov 2006 12:34:55 -0000       1.1515
@@ -1,7 +1,7 @@
 2006-11-04 Udo Giacomozzi <address@hidden>
 
        * backend/render_handler_agg.cpp: removed compatibility include as
-       it is not enough anyway; optimized screen clearing
+       it is not enough anyway; optimized screen and mask clearing
 
 2006-11-04 Markus Gothe <address@hidden>
 

Index: backend/render_handler_agg.cpp
===================================================================
RCS file: /cvsroot/gnash/gnash/backend/render_handler_agg.cpp,v
retrieving revision 1.38
retrieving revision 1.39
diff -u -b -r1.38 -r1.39
--- backend/render_handler_agg.cpp      4 Nov 2006 12:22:54 -0000       1.38
+++ backend/render_handler_agg.cpp      4 Nov 2006 12:34:55 -0000       1.39
@@ -16,7 +16,7 @@
 
  
 
-/* $Id: render_handler_agg.cpp,v 1.38 2006/11/04 12:22:54 udog Exp $ */
+/* $Id: render_handler_agg.cpp,v 1.39 2006/11/04 12:34:55 udog Exp $ */
 
 // Original version by Udo Giacomozzi and Hannes Mayr, 
 // INDUNET GmbH (www.indunet.it)
@@ -260,7 +260,10 @@
     
     m_rbuf.attach(m_buffer, width, height, width);
     
-    m_rbase.clear(agg::gray8(0));
+    // NOTE: The buffer is *not* cleared. The clear() function must be called
+    // to clear the buffer (alpha=0). The reason is to avoid clearing the 
+    // whole mask when only a small portion is really used.
+
   }
   
   ~agg_alpha_mask() 
@@ -268,6 +271,19 @@
     delete [] m_buffer;
   }
   
+  void clear(unsigned int left, unsigned int top, unsigned int width, 
+    unsigned int height) {
+    
+         if (!width) return;
+         
+         unsigned int y;
+         const unsigned int max_y = top+height; // to be exact, it's one off 
the max.
+         const agg::gray8 black(0);
+                 
+    for (y=top; y<max_y; y++) 
+      m_pixf.copy_hline(left, y, width, black);
+  }
+  
   renderer_base& get_rbase() {
     return m_rbase;
   }
@@ -632,6 +648,9 @@
     
     agg_alpha_mask* new_mask = new agg_alpha_mask(xres, yres); 
     
+    new_mask->clear(m_clip_xmin, m_clip_ymin, 
+      m_clip_xmax-m_clip_xmin+1, m_clip_ymax-m_clip_ymin+1); 
+    
     m_alpha_mask.push_back(new_mask);
     
        }