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/FbCommands.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/FbCommands.cc')
-rw-r--r-- | src/FbCommands.cc | 4 |
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()) { |