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

Remove the unused RowToTreeNodeJob::m_parent variable

Browse Source 
This variable was assigned to but never used.

Also remove the corresponding ctor argument which was only used to
initialize this unused variable.
This commit is contained in:
Vadim Zeitlin
2018-11-04 14:58:23 +01:00
parent 3d4a47a6bb
commit 739ce60552
1 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/generic/datavgen.cpp
View File

@@ -3468,8 +3468,8 @@ int wxDataViewMainWindow::GetLineHeight( unsigned int row ) const
class RowToTreeNodeJob: public DoJob class RowToTreeNodeJob: public DoJob
{ {
public: public:
RowToTreeNodeJob(int row, int current, wxDataViewTreeNode *parent) RowToTreeNodeJob(int row, int current)
: m_row(row), m_current(current), m_parent(parent), m_ret(NULL) : m_row(row), m_current(current), m_ret(NULL)
{ {
} }
@@ -3489,8 +3489,6 @@ public:
} }
else else
{ {
m_parent = node;
// If the current node has only leaf children, we can find the // If the current node has only leaf children, we can find the
// desired node directly. This can speed up finding the node // desired node directly. This can speed up finding the node
// in some cases, and will have a very good effect for list views. // in some cases, and will have a very good effect for list views.
@@ -3512,7 +3510,6 @@ public:
private: private:
const int m_row; const int m_row;
int m_current; int m_current;
wxDataViewTreeNode* m_parent;
wxDataViewTreeNode* m_ret; wxDataViewTreeNode* m_ret;
}; };
@@ -3523,7 +3520,7 @@ wxDataViewTreeNode * wxDataViewMainWindow::GetTreeNodeByRow(unsigned int row) co
if ( row == (unsigned)-1 ) if ( row == (unsigned)-1 )
return NULL; return NULL;
RowToTreeNodeJob job( static_cast<int>(row) , -2, m_root ); RowToTreeNodeJob job( static_cast<int>(row) , -2 );
Walker( m_root , job ); Walker( m_root , job );
return job.GetResult(); return job.GetResult();
} }