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

No change, improve wxDataViewMainWindow::ItemDeleted() readability.

Browse Source 
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@68845 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Václav Slavík
2011-08-22 14:14:50 +00:00
parent fa93d732e2
commit b23de238c8
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/generic/datavgen.cpp
View File

@@ -2039,13 +2039,16 @@ bool wxDataViewMainWindow::ItemDeleted(const wxDataViewItem& parent,
else // general case else // general case
{ {
wxDataViewTreeNode * node = FindNode(parent); wxDataViewTreeNode * node = FindNode(parent);
int itemPosInNode = node ? node->GetChildren().Index(item.GetID()) : wxNOT_FOUND;
// Notice that it is possible that the item being deleted is not in the // Notice that it is possible that the item being deleted is not in the
// tree at all, for example we could be deleting a never shown (because // tree at all, for example we could be deleting a never shown (because
// collapsed) item in a tree model. So it's not an error if we don't know // collapsed) item in a tree model. So it's not an error if we don't know
// about this item, just return without doing anything then. // about this item, just return without doing anything then.
if ( !node || itemPosInNode == wxNOT_FOUND ) if ( !node )
return false;
int itemPosInNode = node->GetChildren().Index(item.GetID());
if ( itemPosInNode == wxNOT_FOUND )
return false; return false;
bool isContainer = false; bool isContainer = false;