aboutsummaryrefslogtreecommitdiff
path: root/src/Window.cc
diff options
context:
space:
mode:
authorRyan Pavlik <rpavlik@iastate.edu>2011-11-02 17:33:38 (GMT)
committerMathias Gumz <akira at fluxbox dot org>2012-01-04 06:53:54 (GMT)
commit5ae8bdf49db36b2691ce545e3285f9ee64f92148 (patch)
tree5ef8285a7f6c54a35544903c05018fd0445aee30 /src/Window.cc
parentf859e78cb24cf69cc06e4395c58d804537f301c7 (diff)
downloadfluxbox-5ae8bdf49db36b2691ce545e3285f9ee64f92148.zip
fluxbox-5ae8bdf49db36b2691ce545e3285f9ee64f92148.tar.bz2
Pre-increment non-primitive types.
Found with cppcheck: "Prefix ++/-- operators should be preferred for non-primitive types. Pre-increment/decrement can be more efficient than post-increment/decrement. Post-increment/decrement usually involves keeping a copy of the previous value around and adds a little extra code."
Diffstat (limited to 'src/Window.cc')
-rw-r--r--src/Window.cc4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/Window.cc b/src/Window.cc
index db5bacc..a4eef44 100644
--- a/src/Window.cc
+++ b/src/Window.cc
@@ -1345,7 +1345,7 @@ void FluxboxWindow::iconify() {
1345 WinClient &client = *(*client_it); 1345 WinClient &client = *(*client_it);
1346 WinClient::TransientList::iterator it = client.transientList().begin(); 1346 WinClient::TransientList::iterator it = client.transientList().begin();
1347 WinClient::TransientList::iterator it_end = client.transientList().end(); 1347 WinClient::TransientList::iterator it_end = client.transientList().end();
1348 for (; it != it_end; it++) 1348 for (; it != it_end; ++it)
1349 if ((*it)->fbwindow()) 1349 if ((*it)->fbwindow())
1350 (*it)->fbwindow()->iconify(); 1350 (*it)->fbwindow()->iconify();
1351 } 1351 }
@@ -2966,7 +2966,7 @@ void FluxboxWindow::doSnapping(int &orig_left, int &orig_top) {
2966 Workspace::Windows::iterator it_end = wins.end(); 2966 Workspace::Windows::iterator it_end = wins.end();
2967 2967
2968 unsigned int bw; 2968 unsigned int bw;
2969 for (; it != it_end; it++) { 2969 for (; it != it_end; ++it) {
2970 if ((*it) == this) 2970 if ((*it) == this)
2971 continue; // skip myself 2971 continue; // skip myself
2972 2972