From fa056f67e8c0c3ef27df5da241c714cc0f45d71a Mon Sep 17 00:00:00 2001 From: Neil MacIntosh Date: Fri, 13 Nov 2015 03:27:53 +0000 Subject: [PATCH] Enabled -Wall for gcc and clang. --- tests/CMakeLists.txt | 1 + tests/bounds_tests.cpp | 10 +++++++--- tests/notnull_tests.cpp | 3 ++- tests/span_tests.cpp | 33 +++++++++++++++++++++------------ tests/string_span_tests.cpp | 1 + 5 files changed, 32 insertions(+), 16 deletions(-) diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt index d49cf93..b16821b 100644 --- a/tests/CMakeLists.txt +++ b/tests/CMakeLists.txt @@ -26,6 +26,7 @@ else() else() message(STATUS "The compiler ${CMAKE_CXX_COMPILER} has no C++11 support. Please use a different C++ compiler.") endif() + add_compile_options(-Wall -Wno-missing-braces) endif() if (NOT EXISTS ${CMAKE_CURRENT_SOURCE_DIR}/unittest-cpp) diff --git a/tests/bounds_tests.cpp b/tests/bounds_tests.cpp index 53c44d1..0665260 100644 --- a/tests/bounds_tests.cpp +++ b/tests/bounds_tests.cpp @@ -32,7 +32,9 @@ SUITE(bounds_test) { for (auto point : static_bounds { 2 }) { - for (decltype(point)::size_type j = 0; j < decltype(point)::rank; j++) + for (decltype(point)::size_type j = 0; + j < static_cast(decltype(point)::rank); + j++) { use(j); use(point[j]); @@ -44,6 +46,7 @@ SUITE(bounds_test) { static_bounds<3, 4, 5> b; auto a = b.slice(); + (void)a; static_bounds<4, dynamic_range, 2> x{ 4 }; x.slice().slice(); } @@ -53,7 +56,7 @@ SUITE(bounds_test) static_bounds<4, dynamic_range, 2> bounds{ 3 }; auto itr = bounds.begin(); - + (void)itr; #ifdef CONFIRM_COMPILATION_ERRORS span av(nullptr, bounds); @@ -70,13 +73,14 @@ SUITE(bounds_test) { static_bounds<7, 4, 2> b1; static_bounds<7, dynamic_range, 2> b2 = b1; - + (void)b2; #ifdef CONFIRM_COMPILATION_ERRORS static_bounds<7, dynamic_range, 1> b4 = b2; #endif static_bounds b3 = b1; static_bounds<7, 4, 2> b4 = b3; + (void)b4; static_bounds b11; diff --git a/tests/notnull_tests.cpp b/tests/notnull_tests.cpp index a9624b8..67b478a 100644 --- a/tests/notnull_tests.cpp +++ b/tests/notnull_tests.cpp @@ -69,7 +69,8 @@ SUITE(NotNullTests) MyDerived derived; Unrelated unrelated; not_null u = &unrelated; - not_null p = &derived; + (void)u; + not_null p = &derived; not_null q = &base; q = p; // allowed with heterogeneous copy ctor CHECK(q == p); diff --git a/tests/span_tests.cpp b/tests/span_tests.cpp index ae14f20..6a67b02 100644 --- a/tests/span_tests.cpp +++ b/tests/span_tests.cpp @@ -27,7 +27,6 @@ using namespace gsl; namespace { - void use(int&) {} struct BaseClass {}; struct DerivedClass : BaseClass {}; } @@ -82,6 +81,7 @@ SUITE(span_tests) span avb = avd; #endif span avcd = avd; + (void)avcd; } TEST(boundary_checks) @@ -214,9 +214,11 @@ SUITE(span_tests) const std::array arr = {0.0, 0.0, 0.0}; auto cv = as_span(arr); + (void)cv; vector vec(3); auto dv = as_span(vec); + (void)dv; #ifdef CONFIRM_COMPILATION_ERRORS auto dv2 = as_span(std::move(vec)); @@ -262,6 +264,7 @@ SUITE(span_tests) auto av = as_span(a); auto sub = av.section({15, 0, 0}, gsl::index<3>{2, 2, 2}); auto subsub = sub.section({1, 0, 0}, gsl::index<3>{1, 1, 1}); + (void)subsub; } TEST(span_section) @@ -1417,7 +1420,7 @@ SUITE(span_tests) CHECK_THROW(av1[10][3][4], fail_fast); span av2 = av1.as_span(dim<>(5), dim<6>(), dim<4>()); - + (void)av2; } TEST(span_sub) @@ -1429,7 +1432,7 @@ SUITE(span_tests) CHECK((av.sub<2,2>().bounds() == static_bounds<2>())); CHECK((av.sub<2,2>().length() == 2)); CHECK(av.sub(2,2).length() == 2); - CHECK(av.sub(2,3).length() == 3); + CHECK(av.sub(2,3).length() == 3); } @@ -1445,16 +1448,16 @@ SUITE(span_tests) CHECK((av.sub<0,5>().bounds() == static_bounds<5>())); CHECK((av.sub<0,5>().length() == 5)); CHECK(av.sub(0,5).length() == 5); - CHECK_THROW(av.sub(0,6).length(), fail_fast); - CHECK_THROW(av.sub(1,5).length(), fail_fast); + CHECK_THROW(av.sub(0,6).length(), fail_fast); + CHECK_THROW(av.sub(1,5).length(), fail_fast); } { span av = arr; CHECK((av.sub<5,0>().bounds() == static_bounds<0>())); - CHECK((av.sub<5, 0>().length() == 0)); - CHECK(av.sub(5,0).length() == 0); - CHECK_THROW(av.sub(6,0).length(), fail_fast); + CHECK((av.sub<5, 0>().length() == 0)); + CHECK(av.sub(5,0).length() == 0); + CHECK_THROW(av.sub(6,0).length(), fail_fast); } { @@ -1462,7 +1465,7 @@ SUITE(span_tests) CHECK((av.sub<0,0>().bounds() == static_bounds<0>())); CHECK((av.sub<0,0>().length() == 0)); CHECK(av.sub(0,0).length() == 0); - CHECK_THROW((av.sub<1,0>().length()), fail_fast); + CHECK_THROW((av.sub<1,0>().length()), fail_fast); } { @@ -1559,9 +1562,11 @@ SUITE(span_tests) // converting to dynamic_range a_v is always ok { span av = av4; + (void)av; } { span av = arr; + (void)av; } // initialization or assignment to static span that REDUCES size is NOT ok @@ -1577,6 +1582,7 @@ SUITE(span_tests) { span av = arr; span av2 = av; + (void)av2; } #ifdef CONFIRM_COMPILATION_ERRORS @@ -1588,7 +1594,7 @@ SUITE(span_tests) { span av = arr; - auto f = [&]() {span av2 = av.as_span(dim<>(2), dim<>(2));}; + auto f = [&]() {span av2 = av.as_span(dim<>(2), dim<>(2)); (void)av2; }; CHECK_THROW(f(), fail_fast); } @@ -1597,15 +1603,18 @@ SUITE(span_tests) // you can convert statically { span av2 = {arr, 2}; + (void)av2; } { span av2 = av4.first<1>(); + (void)av2; } // ...or dynamically { // NB: implicit conversion to span from span span av2 = av4.first(1); + (void)av2; } // initialization or assignment to static span that requires size INCREASE is not ok. @@ -1621,13 +1630,13 @@ SUITE(span_tests) } #endif { - auto f = [&]() {span av4 = {arr2, 2};}; + auto f = [&]() {span av4 = {arr2, 2}; (void)av4; }; CHECK_THROW(f(), fail_fast); } // this should fail - we are trying to assign a small dynamic a_v to a fixed_size larger one span av = arr2; - auto f = [&](){ span av2 = av; }; + auto f = [&](){ span av2 = av; (void)av2; }; CHECK_THROW(f(), fail_fast); } diff --git a/tests/string_span_tests.cpp b/tests/string_span_tests.cpp index 5d4681a..efdf0ff 100644 --- a/tests/string_span_tests.cpp +++ b/tests/string_span_tests.cpp @@ -100,6 +100,7 @@ SUITE(string_span_tests) { char stack_string[] = "Hello"; cstring_span<> v = ensure_z(stack_string); + (void)v; #ifdef CONFIRM_COMPILATION_ERRORS string_span<> v2 = v; string_span<> v3 = "Hello";