From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Sun, 7 Jan 2007 17:09:54 +0000 (+0000)
Subject: fix attribute memory leak in wxGrid::ShowCellEditControl() (patch 1629949)
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/2c03d7714c68682f6d312a2865d186467ff7d51b

fix attribute memory leak in wxGrid::ShowCellEditControl() (patch 1629949)


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@44129 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/docs/changes.txt b/docs/changes.txt
index 27060de0e4..2d6c52227f 100644
--- a/docs/changes.txt
+++ b/docs/changes.txt
@@ -98,6 +98,7 @@ All:
 - Added wxTreeCtrl::CollapseAll[Children]() and IsEmpty() (Francesco Montorsi)
 - Several RTL-related positioning fixes (Diaa Sami)
 - Fix wxConfig::DeleteGroup() for arguments with trailing slash (David Hart)
+- Fix memory leak in wxGrid::ShowCellEditControl() (Christian Sturmlechner)
 
 wxMSW:
 
diff --git a/src/generic/grid.cpp b/src/generic/grid.cpp
index 22939d4afc..a1fc2233db 100644
--- a/src/generic/grid.cpp
+++ b/src/generic/grid.cpp
@@ -8309,7 +8309,8 @@ void wxGrid::ShowCellEditControl()
             // might not cover the entire cell
             wxClientDC dc( m_gridWin );
             PrepareDC( dc );
-            dc.SetBrush(wxBrush(GetCellAttr(row, col)->GetBackgroundColour(), wxSOLID));
+            wxGridCellAttr* attr = GetCellAttr(row, col);
+            dc.SetBrush(wxBrush(attr->GetBackgroundColour(), wxSOLID));
             dc.SetPen(*wxTRANSPARENT_PEN);
             dc.DrawRectangle(rect);
 
@@ -8329,7 +8330,6 @@ void wxGrid::ShowCellEditControl()
             if (rect.y > 0)
                 rect.y--;
 
-            wxGridCellAttr* attr = GetCellAttr(row, col);
             wxGridCellEditor* editor = attr->GetEditor(this, row, col);
             if ( !editor->IsCreated() )
             {