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/FbTk/STLUtil.hh | |
parent | f859e78cb24cf69cc06e4395c58d804537f301c7 (diff) | |
download | fluxbox_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/FbTk/STLUtil.hh')
-rw-r--r-- | src/FbTk/STLUtil.hh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/FbTk/STLUtil.hh b/src/FbTk/STLUtil.hh index b1007f7..abc5472 100644 --- a/src/FbTk/STLUtil.hh +++ b/src/FbTk/STLUtil.hh | |||
@@ -85,7 +85,7 @@ F forAllIf(C& c, I i, F f) { | |||
85 | typedef typename C::iterator iterator; | 85 | typedef typename C::iterator iterator; |
86 | iterator it = c.begin(); | 86 | iterator it = c.begin(); |
87 | iterator end = c.end(); | 87 | iterator end = c.end(); |
88 | for (; it != end; it++) { | 88 | for (; it != end; ++it) { |
89 | if (i(*it)) | 89 | if (i(*it)) |
90 | f(*it); | 90 | f(*it); |
91 | } | 91 | } |