use auto

Jan Steemann · Jan Steemann · commit 9a46fd53e67e · 2015-06-05T12:46:41.000+02:00
diff --git a/arangod/Utils/ExplicitTransaction.h b/arangod/Utils/ExplicitTransaction.h
@@ -75,13 +75,13 @@ namespace triagens {
           if (waitForSync) {
             this->setWaitForSync();
           }
-
-          for (auto it = readCollections.begin(); it != readCollections.end(); ++it) {
-            this->addCollection((*it), TRI_TRANSACTION_READ);
+          
+          for (auto const& it : readCollections) {
+            this->addCollection(it, TRI_TRANSACTION_READ);
           }
 
-          for (auto it = writeCollections.begin(); it != writeCollections.end(); ++it) {
-            this->addCollection((*it), TRI_TRANSACTION_WRITE);
+          for (auto const& it : writeCollections) {
+            this->addCollection(it, TRI_TRANSACTION_WRITE);
           }
         }
 
@@ -107,16 +107,15 @@ namespace triagens {
             this->setWaitForSync();
           }
 
-          for (auto it = readCollections.begin(); it != readCollections.end(); ++it) {
-            this->addCollection((*it), TRI_TRANSACTION_READ);
+          for (auto const& it : readCollections) {
+            this->addCollection(it, TRI_TRANSACTION_READ);
           }
 
-          for (auto it = writeCollections.begin(); it != writeCollections.end(); ++it) {
-            this->addCollection((*it), TRI_TRANSACTION_WRITE);
+          for (auto const& it : writeCollections) {
+            this->addCollection(it, TRI_TRANSACTION_WRITE);
           }
         }
 
-
 ////////////////////////////////////////////////////////////////////////////////
 /// @brief end the transaction
 ////////////////////////////////////////////////////////////////////////////////

Original file line number	Diff line number	Diff line change
`@@ -75,13 +75,13 @@ namespace triagens {`
`75`	`75`	`if (waitForSync) {`
`76`	`76`	`this->setWaitForSync();`
`77`	`77`	`}`
`78`		`-`
`79`		`- for (auto it = readCollections.begin(); it != readCollections.end(); ++it) {`
`80`		`- this->addCollection((*it), TRI_TRANSACTION_READ);`
	`78`	`+`
	`79`	`+ for (auto const& it : readCollections) {`
	`80`	`+ this->addCollection(it, TRI_TRANSACTION_READ);`
`81`	`81`	`}`
`82`	`82`
`83`		`- for (auto it = writeCollections.begin(); it != writeCollections.end(); ++it) {`
`84`		`- this->addCollection((*it), TRI_TRANSACTION_WRITE);`
	`83`	`+ for (auto const& it : writeCollections) {`
	`84`	`+ this->addCollection(it, TRI_TRANSACTION_WRITE);`
`85`	`85`	`}`
`86`	`86`	`}`
`87`	`87`
`@@ -107,16 +107,15 @@ namespace triagens {`
`107`	`107`	`this->setWaitForSync();`
`108`	`108`	`}`
`109`	`109`
`110`		`- for (auto it = readCollections.begin(); it != readCollections.end(); ++it) {`
`111`		`- this->addCollection((*it), TRI_TRANSACTION_READ);`
	`110`	`+ for (auto const& it : readCollections) {`
	`111`	`+ this->addCollection(it, TRI_TRANSACTION_READ);`
`112`	`112`	`}`
`113`	`113`
`114`		`- for (auto it = writeCollections.begin(); it != writeCollections.end(); ++it) {`
`115`		`- this->addCollection((*it), TRI_TRANSACTION_WRITE);`
	`114`	`+ for (auto const& it : writeCollections) {`
	`115`	`+ this->addCollection(it, TRI_TRANSACTION_WRITE);`
`116`	`116`	`}`
`117`	`117`	`}`
`118`	`118`
`119`		`-`
`120`	`119`	`////////////////////////////////////////////////////////////////////////////////`
`121`	`120`	`/// @brief end the transaction`
`122`	`121`	`////////////////////////////////////////////////////////////////////////////////`