From: Daniel Winzen <d@winzen4.de>
Date: Sat, 2 May 2015 07:13:56 +0000 (+0200)
Subject: Some optimizations
X-Git-Tag: release-1.4.3~1^2~1
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=bab4d91a78be87065c6a2fc010b986b209acb4b5;p=libmatroska

Some optimizations
---

diff --git a/src/FileKax.cpp b/src/FileKax.cpp
index 5f88d4f..337ce1a 100644
--- a/src/FileKax.cpp
+++ b/src/FileKax.cpp
@@ -280,7 +280,7 @@ inline bool FileMatroska::IsMyTrack(const Track * aTrack) const
   if (aTrack == 0)
     throw 0;
 
-  for (std::vector<Track*>::const_iterator i = myTracks.begin(); i != myTracks.end(); i ++) {
+  for (std::vector<Track*>::const_iterator i = myTracks.begin(); i != myTracks.end(); ++i) {
     if (*i == aTrack)
       break;
   }
@@ -296,7 +296,7 @@ void FileMatroska::SelectReadingTrack(Track * aTrack, bool select)
   if (IsMyTrack(aTrack)) {
     // here we have the right track
     // check if it's not already selected
-    for (std::vector<uint8>::iterator j = mySelectedTracks.begin(); j != mySelectedTracks.end(); j ++) {
+    for (std::vector<uint8>::iterator j = mySelectedTracks.begin(); j != mySelectedTracks.end(); ++j) {
       if (*j == aTrack->TrackNumber())
         break;
     }
@@ -314,7 +314,7 @@ inline bool FileMatroska::IsReadingTrack(const uint8 aTrackNumber) const
 {
   for (std::vector<uint8>::const_iterator trackIdx = mySelectedTracks.begin();
        trackIdx != mySelectedTracks.end() && *trackIdx < aTrackNumber;
-       trackIdx++) {}
+       ++trackIdx) {}
 
   if (trackIdx == mySelectedTracks.end())
     return false;
diff --git a/src/KaxBlock.cpp b/src/KaxBlock.cpp
index 883f641..b59538c 100644
--- a/src/KaxBlock.cpp
+++ b/src/KaxBlock.cpp
@@ -89,7 +89,7 @@ KaxInternalBlock::KaxInternalBlock(const KaxInternalBlock & ElementToClone)
   std::vector<DataBuffer *>::iterator myItr = myBuffers.begin();
   while (Itr != ElementToClone.myBuffers.end()) {
     *myItr = (*Itr)->Clone();
-    Itr++; myItr++;
+    ++Itr; ++myItr;
   }
 }