Use preincrement to advance iterators. (Patch by Ville Skyttä)

2 files changed, 3 insertions, 3 deletions

diff --git a/logo.c b/logo.c
index 119554b..470c5b1 100644
--- a/logo.c
+++ b/logo.c
@@ -244,7 +244,7 @@ bool cEnigmaLogoCache::Flush(void)
   if (!cacheMapM.empty()) {
     debug("cPluginSkinEnigma::Flush() NON-EMPTY");
     // delete bitmaps and clear map
-    for (std::map<std::string, cBitmap *>::iterator i = cacheMapM.begin(); i != cacheMapM.end(); i++) {
+    for (std::map<std::string, cBitmap *>::iterator i = cacheMapM.begin(); i != cacheMapM.end(); ++i) {
       delete((*i).second);
     }
     cacheMapM.clear();
diff --git a/texteffects.c b/texteffects.c
index 04a14a6..c4bb25c 100644
--- a/texteffects.c
+++ b/texteffects.c
@@ -295,7 +295,7 @@ void cEnigmaTextEffects::Action(void)
 
     TE_LOCK; //This causes an initial wait until the first Flush() is called (which TE_UNKOCKs)
     bool changed = false;
-    for (Effects_t::iterator effect = vecEffects.begin(); (effect != vecEffects.end()) && osd; effect++) {
+    for (Effects_t::iterator effect = vecEffects.begin(); (effect != vecEffects.end()) && osd; ++effect) {
       cEffect *e = (*effect);
       if (e == NULL)
         continue;
@@ -366,7 +366,7 @@ void cEnigmaTextEffects::Clear(void)
 
   //Must be TE_LOCKed by caller
 
-  for (Effects_t::iterator effect = vecEffects.begin(); effect != vecEffects.end(); effect++) {
+  for (Effects_t::iterator effect = vecEffects.begin(); effect != vecEffects.end(); ++effect) {
     delete(*effect);
   }