diff options
author | Rian Quinn <rianquinn@gmail.com> | 2016-11-03 19:38:32 -0600 |
---|---|---|
committer | Neil MacIntosh <neilmac@microsoft.com> | 2016-11-03 18:38:32 -0700 |
commit | d641796b21cef4730efa5bac268c28662fd67d80 (patch) | |
tree | 03b160162e2ac7077381d61ffecefaae324878bc /tests/multi_span_tests.cpp | |
parent | 38f453d608d7012a8d0f93f6c8b1d424c99b9b3e (diff) | |
download | platform_external_Microsoft-GSL-d641796b21cef4730efa5bac268c28662fd67d80.tar.gz platform_external_Microsoft-GSL-d641796b21cef4730efa5bac268c28662fd67d80.tar.bz2 platform_external_Microsoft-GSL-d641796b21cef4730efa5bac268c28662fd67d80.zip |
Cleanup additional GCC warnings
When turning on the following flags, several additional warnings
were generated, which have been cleaned up in this patch. The
flags included:
-Wextra
-Wpedantic
-Wconversion
-Wsign-conversion
-Wctor-dtor-privacy
-Wshadow
-Wnon-virtual-dtor
-Wold-style-cast
-Wcast-align
-Woverloaded-virtual
Diffstat (limited to 'tests/multi_span_tests.cpp')
-rw-r--r-- | tests/multi_span_tests.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tests/multi_span_tests.cpp b/tests/multi_span_tests.cpp index c0240ea..8dbe151 100644 --- a/tests/multi_span_tests.cpp +++ b/tests/multi_span_tests.cpp @@ -1427,14 +1427,14 @@ SUITE(multi_span_tests) } } - size_t check_sum = 0; + auto check_sum = 0; for (auto i = 0; i < length; ++i) { check_sum += av[i][1]; } { auto idx = 0; - size_t sum = 0; + auto sum = 0; for (auto num : section) { CHECK(num == av[idx][1]); sum += num; @@ -1444,8 +1444,8 @@ SUITE(multi_span_tests) CHECK(sum == check_sum); } { - size_t idx = length - 1; - size_t sum = 0; + auto idx = length - 1; + auto sum = 0; for (auto iter = section.rbegin(); iter != section.rend(); ++iter) { CHECK(*iter == av[idx][1]); sum += *iter; @@ -1650,7 +1650,7 @@ SUITE(multi_span_tests) { multi_span<int, dynamic_range> av = a; auto wav = as_writeable_bytes(av); - CHECK(wav.data() == (byte*) &a[0]); + CHECK(wav.data() == reinterpret_cast<byte*>(&a[0])); CHECK(wav.length() == sizeof(a)); } } |