Hello again, Here is another tiny patch, this one to fix a memory leak:
diff -r 7578c8ba1630 drivers/wxwidgets_gc.cpp
--- a/drivers/wxwidgets_gc.cpp Wed Mar 11 14:37:03 2009 +0100
+++ b/drivers/wxwidgets_gc.cpp Wed Mar 11 14:40:59 2009 +0100
@@ -68,6 +68,8 @@
if( m_font )
delete m_font;
+
+ delete m_context;
}
This needs to be done because m_context is returned from
wxGraphicsContext::Create() and its return value must be deleted by caller,
without it each call to wxPLDevGC::SetExternalBuffer() leaks memory.
Notice that I didn't add a "if( m_context )" check before the delete as it
is unnecessary (the result of calling delete NULL is defined as a NOP) but,
of course, please feel free to add it for consistency with the lines above
if you prefer (or, better, remove the checks above too :-).
Thanks again,
VZ
pgpGAZnMbH7gB.pgp
Description: PGP signature
------------------------------------------------------------------------------ Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com
_______________________________________________ Plplot-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/plplot-devel
