Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix what seems to be a big memory leaks in SetPageImage()

Browse Source 
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@25305 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin
2004-01-22 14:39:29 +00:00
parent 1312d8f6e0
commit b870b105ae
2 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/gtk/notebook.cpp
View File

@@ -430,17 +430,18 @@ bool wxNotebook::SetPageImage( size_t page, int image )
{ {
/* Case 2) or 4). There is already an image in the gtkhbox. Let's find it */ /* Case 2) or 4). There is already an image in the gtkhbox. Let's find it */
GList *child = gtk_container_children(GTK_CONTAINER(nb_page->m_box)); GList *children = gtk_container_children(GTK_CONTAINER(nb_page->m_box));
while (child) for ( child = children; child; child = child->next )
{ {
if (GTK_IS_PIXMAP(child->data)) if (GTK_IS_PIXMAP(child->data))
{ {
pixmapwid = GTK_WIDGET(child->data); pixmapwid = GTK_WIDGET(child->data);
break; break;
} }
child = child->next;
} }
g_list_free(children);
/* We should have the pixmap widget now */ /* We should have the pixmap widget now */
wxASSERT(pixmapwid != NULL); wxASSERT(pixmapwid != NULL);

7
src/gtk1/notebook.cpp
View File

@@ -430,17 +430,18 @@ bool wxNotebook::SetPageImage( size_t page, int image )
{ {
/* Case 2) or 4). There is already an image in the gtkhbox. Let's find it */ /* Case 2) or 4). There is already an image in the gtkhbox. Let's find it */
GList *child = gtk_container_children(GTK_CONTAINER(nb_page->m_box)); GList *children = gtk_container_children(GTK_CONTAINER(nb_page->m_box));
while (child) for ( child = children; child; child = child->next )
{ {
if (GTK_IS_PIXMAP(child->data)) if (GTK_IS_PIXMAP(child->data))
{ {
pixmapwid = GTK_WIDGET(child->data); pixmapwid = GTK_WIDGET(child->data);
break; break;
} }
child = child->next;
} }
g_list_free(children);
/* We should have the pixmap widget now */ /* We should have the pixmap widget now */
wxASSERT(pixmapwid != NULL); wxASSERT(pixmapwid != NULL);