diff --git a/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.assign/move.pass.cpp b/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.assign/move.pass.cpp --- a/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.assign/move.pass.cpp +++ b/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.assign/move.pass.cpp @@ -6,6 +6,8 @@ // //===----------------------------------------------------------------------===// +// UNSUPPORTED: c++03 + // // template , class Allocator = allocator > @@ -15,49 +17,75 @@ #include #include +#include +#include "make_string.h" #include "test_macros.h" -int main(int, char**) -{ - { - std::stringbuf buf1("testing"); - std::stringbuf buf; - buf = std::move(buf1); - assert(buf.str() == "testing"); - } +#define STR(S) MAKE_STRING(CharT, S) + +template +struct test_stringbuf : std::basic_stringbuf { + using std::basic_stringbuf::basic_stringbuf; + + // Checks the following requirement after being moved from: + // The six pointers of std::basic_streambuf in *this are guaranteed to be different + // from the corresponding pointers in the moved-from rhs unless null. + void check_different_pointers(test_stringbuf const& other) const { + assert(this->eback() == nullptr || this->eback() != other.eback()); + assert(this->gptr() == nullptr || this->gptr() != other.gptr()); + assert(this->egptr() == nullptr || this->egptr() != other.egptr()); + assert(this->pbase() == nullptr || this->pbase() != other.pbase()); + assert(this->pptr() == nullptr || this->pptr() != other.pptr()); + assert(this->epptr() == nullptr || this->epptr() != other.epptr()); + } +}; + +template +void test() { + std::basic_string strings[] = {STR(""), STR("short"), STR("loooooooooooooooooooong")}; + for (std::basic_string const& s : strings) { { - std::stringbuf buf1("testing", std::ios_base::in); - std::stringbuf buf; - buf = std::move(buf1); - assert(buf.str() == "testing"); + test_stringbuf buf1(s); + test_stringbuf buf; + buf = std::move(buf1); + assert(buf.str() == s); + buf.check_different_pointers(buf1); } { - std::stringbuf buf1("testing", std::ios_base::out); - std::stringbuf buf; - buf = std::move(buf1); - assert(buf.str() == "testing"); + test_stringbuf buf1(s, std::ios_base::in); + test_stringbuf buf; + buf = std::move(buf1); + assert(buf.str() == s); + buf.check_different_pointers(buf1); } -#ifndef TEST_HAS_NO_WIDE_CHARACTERS { - std::wstringbuf buf1(L"testing"); - std::wstringbuf buf; - buf = std::move(buf1); - assert(buf.str() == L"testing"); + test_stringbuf buf1(s, std::ios_base::out); + test_stringbuf buf; + buf = std::move(buf1); + assert(buf.str() == s); + buf.check_different_pointers(buf1); } { - std::wstringbuf buf1(L"testing", std::ios_base::in); - std::wstringbuf buf; - buf = std::move(buf1); - assert(buf.str() == L"testing"); + test_stringbuf buf1; + test_stringbuf buf; + buf = std::move(buf1); + buf.check_different_pointers(buf1); } + // Use the assignment operator on an actual std::stringbuf, not test_stringbuf { - std::wstringbuf buf1(L"testing", std::ios_base::out); - std::wstringbuf buf; - buf = std::move(buf1); - assert(buf.str() == L"testing"); + std::basic_stringbuf buf1(s); + std::basic_stringbuf buf; + buf = std::move(buf1); + assert(buf.str() == s); } -#endif // TEST_HAS_NO_WIDE_CHARACTERS + } +} +int main(int, char**) { + test(); +#ifndef TEST_HAS_NO_WIDE_CHARACTERS + test(); +#endif return 0; } diff --git a/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/move.alloc.pass.cpp b/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/move.alloc.pass.cpp --- a/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/move.alloc.pass.cpp +++ b/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/move.alloc.pass.cpp @@ -17,6 +17,7 @@ #include #include +#include #include "make_string.h" #include "test_allocator.h" @@ -26,12 +27,73 @@ #define SV(S) MAKE_STRING_VIEW(CharT, S) template -static void test() { - std::basic_stringbuf, test_allocator> buf1(STR("testing")); - const test_allocator a(2); - const std::basic_stringbuf, test_allocator> buf(std::move(buf1), a); - assert(buf.get_allocator() == a); - assert(buf.view() == SV("testing")); +struct test_stringbuf : std::basic_stringbuf, test_allocator> { + using std::basic_stringbuf, test_allocator>::basic_stringbuf; + + // Checks the following requirement after being moved from: + // The six pointers of std::basic_streambuf in *this are guaranteed to be different + // from the corresponding pointers in the moved-from rhs unless null. + void check_different_pointers(test_stringbuf const& other) const { + assert(this->eback() == nullptr || this->eback() != other.eback()); + assert(this->gptr() == nullptr || this->gptr() != other.gptr()); + assert(this->egptr() == nullptr || this->egptr() != other.egptr()); + assert(this->pbase() == nullptr || this->pbase() != other.pbase()); + assert(this->pptr() == nullptr || this->pptr() != other.pptr()); + assert(this->epptr() == nullptr || this->epptr() != other.epptr()); + } +}; + +template +void test() { + std::basic_string strings[] = {STR(""), STR("short"), STR("loooooooooooooooooooong")}; + for (std::basic_string const& s : strings) { + using StringBuf = std::basic_stringbuf, test_allocator>; + { + test_stringbuf buf1(s); + const test_allocator a(2); + test_stringbuf buf(std::move(static_cast(buf1)), a); + assert(buf.get_allocator() == a); + assert(buf.view() == s); + assert(buf1.view().empty()); + buf.check_different_pointers(buf1); + } + { + test_stringbuf buf1(s, std::ios_base::in); + const test_allocator a(2); + test_stringbuf buf(std::move(static_cast(buf1)), a); + assert(buf.get_allocator() == a); + assert(buf.view() == s); + assert(buf1.view().empty()); + buf.check_different_pointers(buf1); + } + { + test_stringbuf buf1(s, std::ios_base::out); + const test_allocator a(2); + test_stringbuf buf(std::move(static_cast(buf1)), a); + assert(buf.get_allocator() == a); + assert(buf.view() == s); + assert(buf1.view().empty()); + buf.check_different_pointers(buf1); + } + { + test_stringbuf buf1; + const test_allocator a(2); + test_stringbuf buf(std::move(static_cast(buf1)), a); + assert(buf.get_allocator() == a); + assert(buf.view().empty()); + assert(buf1.view().empty()); + buf.check_different_pointers(buf1); + } + // Use the constructor from an actual std::stringbuf, not test_stringbuf + { + StringBuf buf1(s); + const test_allocator a(2); + StringBuf buf(std::move(buf1), a); + assert(buf.get_allocator() == a); + assert(buf.view() == s); + assert(buf1.view().empty()); + } + } } int main(int, char**) { diff --git a/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/move.pass.cpp b/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/move.pass.cpp --- a/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/move.pass.cpp +++ b/libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/move.pass.cpp @@ -6,6 +6,8 @@ // //===----------------------------------------------------------------------===// +// UNSUPPORTED: c++03 + // // template , class Allocator = allocator > @@ -15,43 +17,79 @@ #include #include +#include +#include "make_string.h" #include "test_macros.h" -int main(int, char**) -{ - { - std::stringbuf buf1("testing"); - std::stringbuf buf(std::move(buf1)); - assert(buf.str() == "testing"); - } +#define STR(S) MAKE_STRING(CharT, S) + +template +struct test_stringbuf : std::basic_stringbuf { + using std::basic_stringbuf::basic_stringbuf; + + test_stringbuf(std::basic_stringbuf&& other) : std::basic_stringbuf(std::move(other)) {} + + // Checks the following requirement after being moved from: + // The six pointers of std::basic_streambuf in *this are guaranteed to be different + // from the corresponding pointers in the moved-from rhs unless null. + void check_different_pointers(test_stringbuf const& other) const { + assert(this->eback() == nullptr || this->eback() != other.eback()); + assert(this->gptr() == nullptr || this->gptr() != other.gptr()); + assert(this->egptr() == nullptr || this->egptr() != other.egptr()); + assert(this->pbase() == nullptr || this->pbase() != other.pbase()); + assert(this->pptr() == nullptr || this->pptr() != other.pptr()); + assert(this->epptr() == nullptr || this->epptr() != other.epptr()); + } +}; + +template +void test() { + std::basic_string strings[] = {STR(""), STR("short"), STR("loooooooooooooooooooong")}; + for (std::basic_string const& s : strings) { + using StringBuf = std::basic_stringbuf; { - std::stringbuf buf1("testing", std::ios_base::in); - std::stringbuf buf(std::move(buf1)); - assert(buf.str() == "testing"); + test_stringbuf buf1(s); + test_stringbuf buf(std::move(static_cast(buf1))); + assert(buf.str() == s); + assert(buf1.str().empty()); + buf.check_different_pointers(buf1); } { - std::stringbuf buf1("testing", std::ios_base::out); - std::stringbuf buf(std::move(buf1)); - assert(buf.str() == "testing"); + test_stringbuf buf1(s, std::ios_base::in); + test_stringbuf buf(std::move(static_cast(buf1))); + assert(buf.str() == s); + assert(buf1.str().empty()); + buf.check_different_pointers(buf1); } -#ifndef TEST_HAS_NO_WIDE_CHARACTERS { - std::wstringbuf buf1(L"testing"); - std::wstringbuf buf(std::move(buf1)); - assert(buf.str() == L"testing"); + test_stringbuf buf1(s, std::ios_base::out); + test_stringbuf buf(std::move(static_cast(buf1))); + assert(buf.str() == s); + assert(buf1.str().empty()); + buf.check_different_pointers(buf1); } { - std::wstringbuf buf1(L"testing", std::ios_base::in); - std::wstringbuf buf(std::move(buf1)); - assert(buf.str() == L"testing"); + test_stringbuf buf1; + test_stringbuf buf(std::move(static_cast(buf1))); + assert(buf.str().empty()); + assert(buf1.str().empty()); + buf.check_different_pointers(buf1); } + // Use the constructor from an actual std::stringbuf, not test_stringbuf { - std::wstringbuf buf1(L"testing", std::ios_base::out); - std::wstringbuf buf(std::move(buf1)); - assert(buf.str() == L"testing"); + StringBuf buf1(s); + StringBuf buf(std::move(buf1)); + assert(buf.str() == s); + assert(buf1.str().empty()); } -#endif + } +} +int main(int, char**) { + test(); +#ifndef TEST_HAS_NO_WIDE_CHARACTERS + test(); +#endif return 0; } diff --git a/libcxx/utils/data/ignore_format.txt b/libcxx/utils/data/ignore_format.txt --- a/libcxx/utils/data/ignore_format.txt +++ b/libcxx/utils/data/ignore_format.txt @@ -3478,10 +3478,8 @@ libcxx/test/std/input.output/string.streams/ostringstream/ostringstream.members/str.pass.cpp libcxx/test/std/input.output/string.streams/ostringstream/types.pass.cpp libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.assign/member_swap.pass.cpp -libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.assign/move.pass.cpp libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.assign/nonmember_swap.pass.cpp libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/default.pass.cpp -libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/move.pass.cpp libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.cons/string.pass.cpp libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.members/str.pass.cpp libcxx/test/std/input.output/string.streams/stringbuf/stringbuf.virtuals/overflow.pass.cpp