aboutsummaryrefslogtreecommitdiff
path: root/src/Keys.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/Keys.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/Keys.cc')
-rw-r--r--src/Keys.cc2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/Keys.cc b/src/Keys.cc
index fc53353..f91c92e 100644
--- a/src/Keys.cc
+++ b/src/Keys.cc
@@ -146,7 +146,7 @@ public:
146 // t_key ctor sets context_ of 0 to GLOBAL, so we must here too 146 // t_key ctor sets context_ of 0 to GLOBAL, so we must here too
147 context_ = context_ ? context_ : GLOBAL; 147 context_ = context_ ? context_ : GLOBAL;
148 keylist_t::iterator it = keylist.begin(), it_end = keylist.end(); 148 keylist_t::iterator it = keylist.begin(), it_end = keylist.end();
149 for (; it != it_end; it++) { 149 for (; it != it_end; ++it) {
150 if (*it && (*it)->type == type_ && (*it)->key == key_ && 150 if (*it && (*it)->type == type_ && (*it)->key == key_ &&
151 ((*it)->context & context_) > 0 && 151 ((*it)->context & context_) > 0 &&
152 isdouble_ == (*it)->isdouble && (*it)->mod == 152 isdouble_ == (*it)->isdouble && (*it)->mod ==