aboutsummaryrefslogtreecommitdiff
path: root/src/FbCommands.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/FbCommands.cc
parentf859e78cb24cf69cc06e4395c58d804537f301c7 (diff)
downloadfluxbox_paul-5ae8bdf49db36b2691ce545e3285f9ee64f92148.zip
fluxbox_paul-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/FbCommands.cc')
-rw-r--r--src/FbCommands.cc4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/FbCommands.cc b/src/FbCommands.cc
index 63aa79d..b0289b8 100644
--- a/src/FbCommands.cc
+++ b/src/FbCommands.cc
@@ -512,7 +512,7 @@ void DeiconifyCmd::execute() {
512 512
513 case ALL: 513 case ALL:
514 case ALLWORKSPACE: 514 case ALLWORKSPACE:
515 for(; it != itend; it++) { 515 for(; it != itend; ++it) {
516 old_workspace_num= (*it)->workspaceNumber(); 516 old_workspace_num= (*it)->workspaceNumber();
517 if (m_mode == ALL || old_workspace_num == workspace_num || 517 if (m_mode == ALL || old_workspace_num == workspace_num ||
518 (*it)->isStuck()) { 518 (*it)->isStuck()) {
@@ -526,7 +526,7 @@ void DeiconifyCmd::execute() {
526 case LAST: 526 case LAST:
527 case LASTWORKSPACE: 527 case LASTWORKSPACE:
528 default: 528 default:
529 for (; it != itend; it++) { 529 for (; it != itend; ++it) {
530 old_workspace_num= (*it)->workspaceNumber(); 530 old_workspace_num= (*it)->workspaceNumber();
531 if(m_mode == LAST || old_workspace_num == workspace_num || 531 if(m_mode == LAST || old_workspace_num == workspace_num ||
532 (*it)->isStuck()) { 532 (*it)->isStuck()) {