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

Use pre-increment/decrement operators to move iterator in the loop (propgrid sample).

Browse Source 
When the return value is ignored, the ++it/--it is never less efficient than the it++/it--.
This commit is contained in:
Artur Wieczorek 2015-06-26 18:15:45 +02:00
parent 7c0b52671d
commit 4228e3210f
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
samples/propgrid/propgrid.cpp
View File

@ -2406,7 +2406,7 @@ void FormMain::OnIterate3Click( wxCommandEvent& WXUNUSED(event) )
for ( it = m_pPropGridManager->GetCurrentPage()-> for ( it = m_pPropGridManager->GetCurrentPage()->
GetIterator( wxPG_ITERATE_DEFAULT, wxBOTTOM ); GetIterator( wxPG_ITERATE_DEFAULT, wxBOTTOM );
!it.AtEnd(); !it.AtEnd();
it-- ) --it )
{ {
wxPGProperty* p = *it; wxPGProperty* p = *it;

2
samples/propgrid/tests.cpp
View File

@ -470,7 +470,7 @@ bool FormMain::RunTests( bool fullTest, bool interactive )
wxArrayPGProperty::reverse_iterator it2; wxArrayPGProperty::reverse_iterator it2;
for ( it2 = array.rbegin(); it2 != array.rend(); it2++ ) for ( it2 = array.rbegin(); it2 != array.rend(); ++it2 )
{ {
wxPGProperty* p = (wxPGProperty*)*it2; wxPGProperty* p = (wxPGProperty*)*it2;
RT_MSG(wxString::Format(wxT("Deleting '%s' ('%s')"),p->GetLabel().c_str(),p->GetName().c_str())); RT_MSG(wxString::Format(wxT("Deleting '%s' ('%s')"),p->GetLabel().c_str(),p->GetName().c_str()));