From dd694d7740f4eacd8b7bcf4183ca31810e69e9d5 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Tue, 26 Nov 2019 12:12:03 -0500 Subject: Make functions static in unittests Enabling -Wmissing-declarations points out that a bunch of function in the unittests can be made static, do that. gdb/ChangeLog: * unittests/array-view-selftests.c (check_ptr_size_ctor2): Make static. * unittests/basic_string_view/capacity/1.cc (test01): Likewise. * unittests/basic_string_view/cons/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/cons/char/2.cc (test03): Likewise. (main): Likewise. * unittests/basic_string_view/cons/char/3.cc (test05): Likewise. (main): Likewise. * unittests/basic_string_view/element_access/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/element_access/char/empty.cc (main): Likewise. * unittests/basic_string_view/element_access/char/front_back.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/inserters/char/2.cc (test05): Likewise. (main): Likewise. * unittests/basic_string_view/modifiers/remove_prefix/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/modifiers/remove_suffix/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/modifiers/swap/char/1.cc (test01): Likewise. * unittests/basic_string_view/operations/compare/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/operations/compare/char/13650.cc (test01): Likewise. * unittests/basic_string_view/operations/copy/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/operations/data/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/operations/find/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/operations/find/char/2.cc (test02): Likewise. (main): Likewise. * unittests/basic_string_view/operations/find/char/3.cc (test03): Likewise. (main): Likewise. * unittests/basic_string_view/operations/find/char/4.cc (main): Likewise. * unittests/basic_string_view/operations/rfind/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/operations/rfind/char/2.cc (test02): Likewise. (main): Likewise. * unittests/basic_string_view/operations/rfind/char/3.cc (test03): Likewise. (main): Likewise. * unittests/basic_string_view/operations/substr/char/1.cc (test01): Likewise. (main): Likewise. * unittests/basic_string_view/operators/char/2.cc (main): Likewise. * unittests/optional/assignment/1.cc (test): Likewise. * unittests/optional/assignment/2.cc (test): Likewise. * unittests/optional/assignment/3.cc (test): Likewise. * unittests/optional/assignment/4.cc (test): Likewise. * unittests/optional/assignment/5.cc (test): Likewise. * unittests/optional/assignment/6.cc (test): Likewise. * unittests/optional/assignment/7.cc (test): Likewise. * unittests/optional/cons/copy.cc (test): Likewise. * unittests/optional/cons/default.cc (test): Likewise. * unittests/optional/cons/move.cc (test): Likewise. * unittests/optional/cons/value.cc (test): Likewise. * unittests/optional/in_place.cc (test): Likewise. * unittests/optional/observers/1.cc (test): Likewise. * unittests/optional/observers/2.cc (test): Likewise. Change-Id: I66626db864cb877cacc570d4660df633530554f5 --- gdb/unittests/basic_string_view/capacity/1.cc | 6 +++--- gdb/unittests/basic_string_view/cons/char/1.cc | 8 ++++---- gdb/unittests/basic_string_view/cons/char/2.cc | 8 ++++---- gdb/unittests/basic_string_view/cons/char/3.cc | 8 ++++---- gdb/unittests/basic_string_view/element_access/char/1.cc | 8 ++++---- gdb/unittests/basic_string_view/element_access/char/empty.cc | 4 ++-- .../basic_string_view/element_access/char/front_back.cc | 8 ++++---- gdb/unittests/basic_string_view/inserters/char/2.cc | 8 ++++---- .../basic_string_view/modifiers/remove_prefix/char/1.cc | 8 ++++---- .../basic_string_view/modifiers/remove_suffix/char/1.cc | 8 ++++---- gdb/unittests/basic_string_view/modifiers/swap/char/1.cc | 4 ++-- gdb/unittests/basic_string_view/operations/compare/char/1.cc | 8 ++++---- gdb/unittests/basic_string_view/operations/compare/char/13650.cc | 6 +++--- gdb/unittests/basic_string_view/operations/copy/char/1.cc | 8 ++++---- gdb/unittests/basic_string_view/operations/data/char/1.cc | 8 ++++---- gdb/unittests/basic_string_view/operations/find/char/1.cc | 8 ++++---- gdb/unittests/basic_string_view/operations/find/char/2.cc | 8 ++++---- gdb/unittests/basic_string_view/operations/find/char/3.cc | 8 ++++---- gdb/unittests/basic_string_view/operations/find/char/4.cc | 6 +++--- gdb/unittests/basic_string_view/operations/rfind/char/1.cc | 8 ++++---- gdb/unittests/basic_string_view/operations/rfind/char/2.cc | 8 ++++---- gdb/unittests/basic_string_view/operations/rfind/char/3.cc | 9 +++++---- gdb/unittests/basic_string_view/operations/substr/char/1.cc | 8 ++++---- gdb/unittests/basic_string_view/operators/char/2.cc | 6 +++--- 24 files changed, 89 insertions(+), 88 deletions(-) (limited to 'gdb/unittests/basic_string_view') diff --git a/gdb/unittests/basic_string_view/capacity/1.cc b/gdb/unittests/basic_string_view/capacity/1.cc index a55249d..55ee05e 100644 --- a/gdb/unittests/basic_string_view/capacity/1.cc +++ b/gdb/unittests/basic_string_view/capacity/1.cc @@ -131,8 +131,8 @@ namespace selftests { namespace string_view { namespace capacity_1 { -void -test01() +static void +test01 () { gdb::basic_string_view> str02; typedef gdb::basic_string_view< A >::size_type size_type_o; @@ -161,7 +161,7 @@ test01() VERIFY( sz03 >= sz04 ); } -int +static int main() { test01(); diff --git a/gdb/unittests/basic_string_view/cons/char/1.cc b/gdb/unittests/basic_string_view/cons/char/1.cc index a7cc8f2..b5209bd 100644 --- a/gdb/unittests/basic_string_view/cons/char/1.cc +++ b/gdb/unittests/basic_string_view/cons/char/1.cc @@ -21,8 +21,8 @@ namespace cons_1 { -void -test01() +static void +test01 () { typedef gdb::string_view::size_type csize_type; @@ -56,8 +56,8 @@ test01() VERIFY( str07.length() == 10 ); } -int -main() +static int +main () { test01(); diff --git a/gdb/unittests/basic_string_view/cons/char/2.cc b/gdb/unittests/basic_string_view/cons/char/2.cc index 74af38b..bd7fb6f 100644 --- a/gdb/unittests/basic_string_view/cons/char/2.cc +++ b/gdb/unittests/basic_string_view/cons/char/2.cc @@ -21,8 +21,8 @@ namespace cons_2 { -void -test03() +static void +test03 () { const char* with_nulls = "This contains \0 a zero byte."; @@ -35,8 +35,8 @@ test03() VERIFY( s2.size() == 28 ); } -int -main() +static int +main () { test03(); diff --git a/gdb/unittests/basic_string_view/cons/char/3.cc b/gdb/unittests/basic_string_view/cons/char/3.cc index 9e1ca7e..8dbaba7 100644 --- a/gdb/unittests/basic_string_view/cons/char/3.cc +++ b/gdb/unittests/basic_string_view/cons/char/3.cc @@ -21,15 +21,15 @@ namespace cons_3 { -void -test05() +static void +test05 () { char const * s = 0; gdb::string_view zero_length_built_with_NULL(s, 0); } -int -main() +static int +main () { test05(); diff --git a/gdb/unittests/basic_string_view/element_access/char/1.cc b/gdb/unittests/basic_string_view/element_access/char/1.cc index 59a44b8..ea24cd2 100644 --- a/gdb/unittests/basic_string_view/element_access/char/1.cc +++ b/gdb/unittests/basic_string_view/element_access/char/1.cc @@ -21,8 +21,8 @@ namespace element_access_1 { -void -test01() +static void +test01 () { typedef gdb::string_view::size_type csize_type; typedef gdb::string_view::const_reference cref; @@ -60,8 +60,8 @@ test01() } } -int -main() +static int +main () { test01(); return 0; diff --git a/gdb/unittests/basic_string_view/element_access/char/empty.cc b/gdb/unittests/basic_string_view/element_access/char/empty.cc index 1073940..5c32221 100644 --- a/gdb/unittests/basic_string_view/element_access/char/empty.cc +++ b/gdb/unittests/basic_string_view/element_access/char/empty.cc @@ -20,8 +20,8 @@ namespace element_access_empty { -int -main() +static int +main () { { gdb::string_view empty; diff --git a/gdb/unittests/basic_string_view/element_access/char/front_back.cc b/gdb/unittests/basic_string_view/element_access/char/front_back.cc index 8577ee1..7470336 100644 --- a/gdb/unittests/basic_string_view/element_access/char/front_back.cc +++ b/gdb/unittests/basic_string_view/element_access/char/front_back.cc @@ -20,8 +20,8 @@ namespace element_access_front_back { -void -test01() +static void +test01 () { gdb::string_view str("ramifications"); const gdb::string_view cstr("melodien"); @@ -32,8 +32,8 @@ test01() VERIFY( cstr.back() == 'n' ); } -int -main() +static int +main () { test01(); diff --git a/gdb/unittests/basic_string_view/inserters/char/2.cc b/gdb/unittests/basic_string_view/inserters/char/2.cc index 3235525..4d92fa0 100644 --- a/gdb/unittests/basic_string_view/inserters/char/2.cc +++ b/gdb/unittests/basic_string_view/inserters/char/2.cc @@ -30,8 +30,8 @@ namespace inserters_2 { // testing basic_filebuf::xsputn via stress testing with large string_views // based on a bug report libstdc++ 9 // mode == out -void -test05(std::size_t size) +static void +test05 (std::size_t size) { bool test ATTRIBUTE_UNUSED = true; @@ -78,8 +78,8 @@ test05(std::size_t size) VERIFY( count == 2 * size ); } -int -main() +static int +main () { test05(1); test05(1000); diff --git a/gdb/unittests/basic_string_view/modifiers/remove_prefix/char/1.cc b/gdb/unittests/basic_string_view/modifiers/remove_prefix/char/1.cc index 5c34f80..7cb93e1 100644 --- a/gdb/unittests/basic_string_view/modifiers/remove_prefix/char/1.cc +++ b/gdb/unittests/basic_string_view/modifiers/remove_prefix/char/1.cc @@ -19,8 +19,8 @@ namespace modifiers_remove_prefix { -void -test01() +static void +test01 () { using gdb::string_view; @@ -52,8 +52,8 @@ test02() } #endif -int -main() +static int +main () { test01(); #ifndef GDB_STRING_VIEW diff --git a/gdb/unittests/basic_string_view/modifiers/remove_suffix/char/1.cc b/gdb/unittests/basic_string_view/modifiers/remove_suffix/char/1.cc index eb9f73b..97e887e 100644 --- a/gdb/unittests/basic_string_view/modifiers/remove_suffix/char/1.cc +++ b/gdb/unittests/basic_string_view/modifiers/remove_suffix/char/1.cc @@ -19,8 +19,8 @@ namespace modifiers_remove_suffix { -void -test01() +static void +test01 () { using gdb::string_view; @@ -52,8 +52,8 @@ test02() } #endif -int -main() +static int +main () { test01(); #ifndef GDB_STRING_VIEW diff --git a/gdb/unittests/basic_string_view/modifiers/swap/char/1.cc b/gdb/unittests/basic_string_view/modifiers/swap/char/1.cc index 3a8db92..e1b66c6 100644 --- a/gdb/unittests/basic_string_view/modifiers/swap/char/1.cc +++ b/gdb/unittests/basic_string_view/modifiers/swap/char/1.cc @@ -20,8 +20,8 @@ namespace modifiers_swap { -void -test01() +static void +test01 () { using gdb::string_view; diff --git a/gdb/unittests/basic_string_view/operations/compare/char/1.cc b/gdb/unittests/basic_string_view/operations/compare/char/1.cc index 273e01e..44b1066 100644 --- a/gdb/unittests/basic_string_view/operations/compare/char/1.cc +++ b/gdb/unittests/basic_string_view/operations/compare/char/1.cc @@ -61,8 +61,8 @@ test_value(int result, want_value expected) return 0; } -int -test01() +static int +test01 () { using gdb::string_view; @@ -121,8 +121,8 @@ test01() } -int -main() +static int +main () { test01(); diff --git a/gdb/unittests/basic_string_view/operations/compare/char/13650.cc b/gdb/unittests/basic_string_view/operations/compare/char/13650.cc index 69e5568..c50d7c2 100644 --- a/gdb/unittests/basic_string_view/operations/compare/char/13650.cc +++ b/gdb/unittests/basic_string_view/operations/compare/char/13650.cc @@ -22,8 +22,8 @@ namespace operations_compare_13650 { // libstdc++/13650 -void -test01() +static void +test01 () { using gdb::string_view; @@ -39,7 +39,7 @@ test01() VERIFY( str_b.compare(0, 3, lit_02, 5) < 0 ); } -int +static int main() { test01(); diff --git a/gdb/unittests/basic_string_view/operations/copy/char/1.cc b/gdb/unittests/basic_string_view/operations/copy/char/1.cc index 4abc283..fd82911 100644 --- a/gdb/unittests/basic_string_view/operations/copy/char/1.cc +++ b/gdb/unittests/basic_string_view/operations/copy/char/1.cc @@ -21,8 +21,8 @@ namespace operations_copy_1 { -void -test01() +static void +test01 () { typedef gdb::string_view::size_type csize_type; @@ -35,8 +35,8 @@ test01() VERIFY( '9' == buffer[0] ); } -int -main() +static int +main () { test01(); diff --git a/gdb/unittests/basic_string_view/operations/data/char/1.cc b/gdb/unittests/basic_string_view/operations/data/char/1.cc index 1d71fcc..0901e4c 100644 --- a/gdb/unittests/basic_string_view/operations/data/char/1.cc +++ b/gdb/unittests/basic_string_view/operations/data/char/1.cc @@ -21,8 +21,8 @@ namespace operations_data_1 { -int -test01() +static int +test01 () { gdb::string_view empty; @@ -33,8 +33,8 @@ test01() return 0; } -int -main() +static int +main () { test01(); diff --git a/gdb/unittests/basic_string_view/operations/find/char/1.cc b/gdb/unittests/basic_string_view/operations/find/char/1.cc index fe769eb..ee36cb0 100644 --- a/gdb/unittests/basic_string_view/operations/find/char/1.cc +++ b/gdb/unittests/basic_string_view/operations/find/char/1.cc @@ -21,8 +21,8 @@ namespace operations_find_1 { -void -test01() +static void +test01 () { typedef gdb::string_view::size_type csize_type; typedef gdb::string_view::const_reference cref; @@ -154,8 +154,8 @@ test02() } #endif -int -main() +static int +main () { test01(); #ifndef GDB_STRING_VIEW diff --git a/gdb/unittests/basic_string_view/operations/find/char/2.cc b/gdb/unittests/basic_string_view/operations/find/char/2.cc index 2a220df..72ed9c5 100644 --- a/gdb/unittests/basic_string_view/operations/find/char/2.cc +++ b/gdb/unittests/basic_string_view/operations/find/char/2.cc @@ -21,8 +21,8 @@ namespace operations_find_2 { -void -test02() +static void +test02 () { typedef gdb::string_view::size_type csize_type; csize_type npos = gdb::string_view::npos; @@ -152,8 +152,8 @@ test03() } #endif -int -main() +static int +main () { test02(); #ifndef GDB_STRING_VIEW diff --git a/gdb/unittests/basic_string_view/operations/find/char/3.cc b/gdb/unittests/basic_string_view/operations/find/char/3.cc index b47daad..3995c07 100644 --- a/gdb/unittests/basic_string_view/operations/find/char/3.cc +++ b/gdb/unittests/basic_string_view/operations/find/char/3.cc @@ -21,8 +21,8 @@ namespace operations_find_3 { -void -test03() +static void +test03 () { typedef gdb::string_view::size_type csize_type; csize_type npos = gdb::string_view::npos; @@ -152,8 +152,8 @@ test04() } #endif -int -main() +static int +main () { test03(); #ifndef GDB_STRING_VIEW diff --git a/gdb/unittests/basic_string_view/operations/find/char/4.cc b/gdb/unittests/basic_string_view/operations/find/char/4.cc index 17f1862..75b04a4 100644 --- a/gdb/unittests/basic_string_view/operations/find/char/4.cc +++ b/gdb/unittests/basic_string_view/operations/find/char/4.cc @@ -22,7 +22,7 @@ namespace operations_find_4 { // libstdc++/31401 -void +static void test01() { typedef gdb::string_view::size_type csize_type; @@ -34,8 +34,8 @@ test01() VERIFY( pos1 == npos ); } -int -main() +static int +main () { test01(); diff --git a/gdb/unittests/basic_string_view/operations/rfind/char/1.cc b/gdb/unittests/basic_string_view/operations/rfind/char/1.cc index 7a634f4..11fa0d7 100644 --- a/gdb/unittests/basic_string_view/operations/rfind/char/1.cc +++ b/gdb/unittests/basic_string_view/operations/rfind/char/1.cc @@ -21,8 +21,8 @@ namespace operations_rfind_1 { // basic_string_view rfind -void -test01() +static void +test01 () { typedef gdb::string_view::size_type csize_type; typedef gdb::string_view::const_reference cref; @@ -84,8 +84,8 @@ test01() VERIFY( csz01 == npos ); } -int -main() +static int +main () { test01(); diff --git a/gdb/unittests/basic_string_view/operations/rfind/char/2.cc b/gdb/unittests/basic_string_view/operations/rfind/char/2.cc index d4e8561..aa99d0e 100644 --- a/gdb/unittests/basic_string_view/operations/rfind/char/2.cc +++ b/gdb/unittests/basic_string_view/operations/rfind/char/2.cc @@ -21,8 +21,8 @@ namespace operations_rfind_2 { // basic_string_view::find_last_of -void -test02() +static void +test02 () { gdb::string_view z("ab"); gdb::string_view::size_type pos; @@ -42,8 +42,8 @@ test02() VERIFY( pos == gdb::string_view::npos ); } -int -main() +static int +main () { test02(); diff --git a/gdb/unittests/basic_string_view/operations/rfind/char/3.cc b/gdb/unittests/basic_string_view/operations/rfind/char/3.cc index 4d37556..989814c 100644 --- a/gdb/unittests/basic_string_view/operations/rfind/char/3.cc +++ b/gdb/unittests/basic_string_view/operations/rfind/char/3.cc @@ -21,8 +21,8 @@ namespace operations_rfind_3 { // basic_string_view::find_last_not_of -void -test03() +static void +test03 () { typedef gdb::string_view::size_type csize_type; gdb::string_view::size_type pos; @@ -56,8 +56,9 @@ test03() pos = z.find_last_not_of("Xa"); VERIFY( pos == 1 ); } -int -main() + +static int +main () { test03(); diff --git a/gdb/unittests/basic_string_view/operations/substr/char/1.cc b/gdb/unittests/basic_string_view/operations/substr/char/1.cc index b732204..a95173a 100644 --- a/gdb/unittests/basic_string_view/operations/substr/char/1.cc +++ b/gdb/unittests/basic_string_view/operations/substr/char/1.cc @@ -21,8 +21,8 @@ namespace operations_substr_1 { -void -test01() +static void +test01 () { typedef gdb::string_view::size_type csize_type; typedef gdb::string_view::const_reference cref; @@ -68,8 +68,8 @@ test01() } } -int -main() +static int +main () { test01(); diff --git a/gdb/unittests/basic_string_view/operators/char/2.cc b/gdb/unittests/basic_string_view/operators/char/2.cc index 38aa3c2..281d7da 100644 --- a/gdb/unittests/basic_string_view/operators/char/2.cc +++ b/gdb/unittests/basic_string_view/operators/char/2.cc @@ -111,7 +111,7 @@ template namespace operators_2 { -void +static void test01() { gdb::string_view str_0("costa rica"); @@ -360,8 +360,8 @@ test02() } #endif -int -main() +static int +main () { test01(); #ifndef GDB_STRING_VIEW -- cgit v1.1