From e915ab5d9f189f6f96f461d6b301210b55f99619 Mon Sep 17 00:00:00 2001 From: John Wellbelove Date: Fri, 10 Jul 2020 11:52:27 +0100 Subject: [PATCH] Remove unnecessary member functions --- include/etl/basic_string_stream.h | 16 ------------- test/test_string_stream.cpp | 39 +++++++++++-------------------- test/test_u16string_stream.cpp | 25 ++++++-------------- test/test_u32string_stream.cpp | 25 ++++++-------------- test/test_wstring_stream.cpp | 25 ++++++-------------- 5 files changed, 35 insertions(+), 95 deletions(-) diff --git a/include/etl/basic_string_stream.h b/include/etl/basic_string_stream.h index 45a4fb8b..511913e0 100644 --- a/include/etl/basic_string_stream.h +++ b/include/etl/basic_string_stream.h @@ -112,22 +112,6 @@ namespace etl text.assign(is); } - //************************************************************************* - /// Resets the stream to an empty string. - //************************************************************************* - void clear() - { - text.clear(); - } - - //************************************************************************* - /// Get the current size of the string. - //************************************************************************* - size_t size() const - { - return text.size(); - } - //************************************************************************* /// Stream operators. //************************************************************************* diff --git a/test/test_string_stream.cpp b/test/test_string_stream.cpp index bf6e713a..afef8237 100644 --- a/test/test_string_stream.cpp +++ b/test/test_string_stream.cpp @@ -138,32 +138,32 @@ namespace ss << etl::bin << value; CHECK_EQUAL(String(STR("11110001001000000---")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::oct << value; CHECK_EQUAL(String(STR("361100--------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::dec << value; CHECK_EQUAL(String(STR("123456--------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::uppercase << value; CHECK_EQUAL(String(STR("1E240---------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::nouppercase << value; CHECK_EQUAL(String(STR("1e240---------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::setw(0); ss << etl::noboolalpha << false << STR(" ") << true << STR(" ") << etl::boolalpha << false << STR(" ") << true; CHECK_EQUAL(String(STR("0 1 false true")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::setprecision(4) << 3.1415927; CHECK_EQUAL(String(STR("3.1416")), ss.str()); - ss.clear(); + ss.str().clear(); ss << STR("abcdef") << STR(" ") << etl::uppercase << STR("abcdef"); CHECK_EQUAL(String(STR("abcdef abcdef")), ss.str()); } @@ -179,31 +179,31 @@ namespace ss << etl::bin << etl::noshowbase << value; CHECK_EQUAL(String(STR("11110001001000000")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::bin << etl::showbase << value; CHECK_EQUAL(String(STR("0b11110001001000000")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::oct << etl::noshowbase << value; CHECK_EQUAL(String(STR("361100")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::oct << etl::showbase << value; CHECK_EQUAL(String(STR("0361100")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::dec << etl::noshowbase << value; CHECK_EQUAL(String(STR("123456")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::dec << etl::showbase << value; CHECK_EQUAL(String(STR("123456")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::noshowbase << value; CHECK_EQUAL(String(STR("1e240")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::showbase << value; CHECK_EQUAL(String(STR("0x1e240")), ss.str()); } @@ -304,17 +304,6 @@ namespace CHECK_EQUAL(String(STR("Hello")), istr); } - - //************************************************************************* - TEST(test_get_size) - { - String str; - Stream ss(str); - - ss.str(String(STR("Hello"))); - - CHECK_EQUAL(str.size(), ss.size()); - } }; } diff --git a/test/test_u16string_stream.cpp b/test/test_u16string_stream.cpp index f650072f..172ec364 100644 --- a/test/test_u16string_stream.cpp +++ b/test/test_u16string_stream.cpp @@ -138,32 +138,32 @@ namespace ss << etl::bin << value; CHECK_EQUAL(String(STR("11110001001000000---")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::oct << value; CHECK_EQUAL(String(STR("361100--------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::dec << value; CHECK_EQUAL(String(STR("123456--------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::uppercase << value; CHECK_EQUAL(String(STR("1E240---------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::nouppercase << value; CHECK_EQUAL(String(STR("1e240---------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::setw(0); ss << etl::noboolalpha << false << STR(" ") << true << STR(" ") << etl::boolalpha << false << STR(" ") << true; CHECK_EQUAL(String(STR("0 1 false true")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::setprecision(4) << 3.1415927; CHECK_EQUAL(String(STR("3.1416")), ss.str()); - ss.clear(); + ss.str().clear(); ss << STR("abcdef") << STR(" ") << etl::uppercase << STR("abcdef"); CHECK_EQUAL(String(STR("abcdef abcdef")), ss.str()); } @@ -264,17 +264,6 @@ namespace CHECK_EQUAL(String(STR("Hello")), istr); } - - //************************************************************************* - TEST(test_get_size) - { - String str; - Stream ss(str); - - ss.str(String(STR("Hello"))); - - CHECK_EQUAL(str.size(), ss.size()); - } }; } diff --git a/test/test_u32string_stream.cpp b/test/test_u32string_stream.cpp index d343a13d..e32d82cf 100644 --- a/test/test_u32string_stream.cpp +++ b/test/test_u32string_stream.cpp @@ -138,32 +138,32 @@ namespace ss << etl::bin << value; CHECK_EQUAL(String(STR("11110001001000000---")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::oct << value; CHECK_EQUAL(String(STR("361100--------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::dec << value; CHECK_EQUAL(String(STR("123456--------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::uppercase << value; CHECK_EQUAL(String(STR("1E240---------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::nouppercase << value; CHECK_EQUAL(String(STR("1e240---------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::setw(0); ss << etl::noboolalpha << false << STR(" ") << true << STR(" ") << etl::boolalpha << false << STR(" ") << true; CHECK_EQUAL(String(STR("0 1 false true")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::setprecision(4) << 3.1415927; CHECK_EQUAL(String(STR("3.1416")), ss.str()); - ss.clear(); + ss.str().clear(); ss << STR("abcdef") << STR(" ") << etl::uppercase << STR("abcdef"); CHECK_EQUAL(String(STR("abcdef abcdef")), ss.str()); } @@ -264,17 +264,6 @@ namespace CHECK_EQUAL(String(STR("Hello")), istr); } - - //************************************************************************* - TEST(test_get_size) - { - String str; - Stream ss(str); - - ss.str(String(STR("Hello"))); - - CHECK_EQUAL(str.size(), ss.size()); - } }; } diff --git a/test/test_wstring_stream.cpp b/test/test_wstring_stream.cpp index f5f57981..623913b7 100644 --- a/test/test_wstring_stream.cpp +++ b/test/test_wstring_stream.cpp @@ -138,32 +138,32 @@ namespace ss << etl::bin << value; CHECK_EQUAL(String(STR("11110001001000000---")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::oct << value; CHECK_EQUAL(String(STR("361100--------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::dec << value; CHECK_EQUAL(String(STR("123456--------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::uppercase << value; CHECK_EQUAL(String(STR("1E240---------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::hex << etl::nouppercase << value; CHECK_EQUAL(String(STR("1e240---------------")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::setw(0); ss << etl::noboolalpha << false << STR(" ") << true << STR(" ") << etl::boolalpha << false << STR(" ") << true; CHECK_EQUAL(String(STR("0 1 false true")), ss.str()); - ss.clear(); + ss.str().clear(); ss << etl::setprecision(4) << 3.1415927; CHECK_EQUAL(String(STR("3.1416")), ss.str()); - ss.clear(); + ss.str().clear(); ss << STR("abcdef") << STR(" ") << etl::uppercase << STR("abcdef"); CHECK_EQUAL(String(STR("abcdef abcdef")), ss.str()); } @@ -264,17 +264,6 @@ namespace CHECK_EQUAL(String(STR("Hello")), istr); } - - //************************************************************************* - TEST(test_get_size) - { - String str; - Stream ss(str); - - ss.str(String(STR("Hello"))); - - CHECK_EQUAL(str.size(), ss.size()); - } }; }