diff options
author | Tobias Markmann <tm@ayena.de> | 2016-04-04 14:41:44 (GMT) |
---|---|---|
committer | Tobias Markmann <tm@ayena.de> | 2016-04-04 14:41:44 (GMT) |
commit | 3c560e31b0f168da917e8d566db01fd1cd997d86 (patch) | |
tree | a6d1c5c276d3571c6303a31af9f3cefd34b13d52 /QA/Checker | |
parent | 4f3821a090931499b9c68b3b3ad785a98ea9d742 (diff) | |
download | swift-3c560e31b0f168da917e8d566db01fd1cd997d86.zip swift-3c560e31b0f168da917e8d566db01fd1cd997d86.tar.bz2 |
Modernize code to use range based for loops using clang-tidy
Run 'clang-tidy -fix -checks=modernize-loop-convert' on all
source code files on OS X. This does not modernize platform
specific code on Linux and Windows
Test-Information:
Code builds and unit tests pass on OS X 10.11.4.
Change-Id: I65b99e0978cfab8ca6de2a3e5342e7a81416c12c
Diffstat (limited to 'QA/Checker')
-rw-r--r-- | QA/Checker/IO.cpp | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/QA/Checker/IO.cpp b/QA/Checker/IO.cpp index d8fcc96..4b43635 100644 --- a/QA/Checker/IO.cpp +++ b/QA/Checker/IO.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2010 Isode Limited. + * Copyright (c) 2010-2016 Isode Limited. * All rights reserved. * See the COPYING file for more information. */ @@ -12,12 +12,12 @@ std::ostream& operator<<(std::ostream& os, const Swift::ByteArray& s) { std::ios::fmtflags oldFlags = os.flags(); os << std::hex; - for (Swift::ByteArray::const_iterator i = s.begin(); i != s.end(); ++i) { - if (*i >= 32 && *i < 127) { - os << *i; + for (unsigned char i : s) { + if (i >= 32 && i < 127) { + os << i; } else { - os << "\\x" << static_cast<unsigned int>(static_cast<unsigned char>(*i)); + os << "\\x" << static_cast<unsigned int>(static_cast<unsigned char>(i)); } } os << std::endl; @@ -28,12 +28,12 @@ std::ostream& operator<<(std::ostream& os, const Swift::ByteArray& s) { std::ostream& operator<<(std::ostream& os, const Swift::SafeByteArray& s) { std::ios::fmtflags oldFlags = os.flags(); os << std::hex; - for (Swift::SafeByteArray::const_iterator i = s.begin(); i != s.end(); ++i) { - if (*i >= 32 && *i < 127) { - os << *i; + for (unsigned char i : s) { + if (i >= 32 && i < 127) { + os << i; } else { - os << "\\x" << static_cast<unsigned int>(static_cast<unsigned char>(*i)); + os << "\\x" << static_cast<unsigned int>(static_cast<unsigned char>(i)); } } os << std::endl; @@ -42,16 +42,16 @@ std::ostream& operator<<(std::ostream& os, const Swift::SafeByteArray& s) { } std::ostream& operator<<(std::ostream& os, const std::vector<int>& s) { - for (std::vector<int>::const_iterator i = s.begin(); i != s.end(); ++i) { - os << *i << " "; + for (int i : s) { + os << i << " "; } os << std::endl; return os; } std::ostream& operator<<(std::ostream& os, const std::vector<size_t>& s) { - for (std::vector<size_t>::const_iterator i = s.begin(); i != s.end(); ++i) { - os << *i << " "; + for (size_t i : s) { + os << i << " "; } os << std::endl; return os; |