diff options
author | Ryan Pavlik <rpavlik@iastate.edu> | 2011-11-02 17:33:38 (GMT) |
---|---|---|
committer | Mathias Gumz <akira at fluxbox dot org> | 2012-01-04 06:53:54 (GMT) |
commit | 5ae8bdf49db36b2691ce545e3285f9ee64f92148 (patch) | |
tree | 5ef8285a7f6c54a35544903c05018fd0445aee30 /src/Window.cc | |
parent | f859e78cb24cf69cc06e4395c58d804537f301c7 (diff) | |
download | fluxbox-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.cc | 4 |
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 | ||