diff options
| author | Henry Wilson <m3henry@googlemail.com> | 2019-03-13 20:55:31 +0000 |
|---|---|---|
| committer | PeterN <peter@fuzzle.org> | 2019-03-26 20:15:57 +0000 |
| commit | 03ca3190c9b73de3e43fe1e7e2293b8c1e3aa3d5 (patch) | |
| tree | 7f7a4f88e80592b98c15465f01d9b8eb19096aa8 | |
| parent | cc62f4163f230ed82ef3b04187987d3e380cd570 (diff) | |
| download | openttd-03ca3190c9b73de3e43fe1e7e2293b8c1e3aa3d5.tar.xz | |
Codechange: Use range-based for-loop in Auto[Free|Delete]SmallVector
| -rw-r--r-- | src/core/smallvec_type.hpp | 8 | ||||
| -rw-r--r-- | src/script/squirrel_helper.hpp | 2 |
2 files changed, 5 insertions, 5 deletions
diff --git a/src/core/smallvec_type.hpp b/src/core/smallvec_type.hpp index e71edf89f..f43265ebe 100644 --- a/src/core/smallvec_type.hpp +++ b/src/core/smallvec_type.hpp @@ -91,8 +91,8 @@ public: */ inline void Clear() { - for (uint i = 0; i < std::vector<T>::size(); i++) { - free(std::vector<T>::operator[](i)); + for (T p : *this) { + free(p); } std::vector<T>::clear(); @@ -121,8 +121,8 @@ public: */ inline void Clear() { - for (uint i = 0; i < std::vector<T>::size(); i++) { - delete std::vector<T>::operator[](i); + for (T p : *this) { + delete p; } std::vector<T>::clear(); diff --git a/src/script/squirrel_helper.hpp b/src/script/squirrel_helper.hpp index add6b49c1..10f04f945 100644 --- a/src/script/squirrel_helper.hpp +++ b/src/script/squirrel_helper.hpp @@ -32,7 +32,7 @@ namespace SQConvert { struct SQAutoFreePointers : std::vector<void *> { ~SQAutoFreePointers() { - for (uint i = 0; i < std::vector<void *>::size(); i++) free(std::vector<void *>::operator[](i)); + for (void * p : *this) free(p); } }; |