X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/93a800a95e67a38284a2980543c4b0b1a792573c..ff8af61ce07803bd86cafea25a06915dfed2ce36:/tests/strings/strings.cpp diff --git a/tests/strings/strings.cpp b/tests/strings/strings.cpp index f937121091..a0485c9007 100644 --- a/tests/strings/strings.cpp +++ b/tests/strings/strings.cpp @@ -35,6 +35,7 @@ private: CPPUNIT_TEST( String ); CPPUNIT_TEST( PChar ); CPPUNIT_TEST( Format ); + CPPUNIT_TEST( FormatUnicode ); CPPUNIT_TEST( Constructors ); CPPUNIT_TEST( StaticConstructors ); CPPUNIT_TEST( Extraction ); @@ -53,6 +54,7 @@ private: CPPUNIT_TEST( ToULongLong ); #endif // wxLongLong_t CPPUNIT_TEST( ToDouble ); + CPPUNIT_TEST( FromDouble ); CPPUNIT_TEST( StringBuf ); CPPUNIT_TEST( UTF8Buf ); CPPUNIT_TEST( CStrDataTernaryOperator ); @@ -61,11 +63,13 @@ private: CPPUNIT_TEST( ExplicitConversion ); CPPUNIT_TEST( IndexedAccess ); CPPUNIT_TEST( BeforeAndAfter ); + CPPUNIT_TEST( ScopedBuffers ); CPPUNIT_TEST_SUITE_END(); void String(); void PChar(); void Format(); + void FormatUnicode(); void Constructors(); void StaticConstructors(); void Extraction(); @@ -84,6 +88,7 @@ private: void ToULongLong(); #endif // wxLongLong_t void ToDouble(); + void FromDouble(); void StringBuf(); void UTF8Buf(); void CStrDataTernaryOperator(); @@ -93,6 +98,7 @@ private: void ExplicitConversion(); void IndexedAccess(); void BeforeAndAfter(); + void ScopedBuffers(); DECLARE_NO_COPY_CLASS(StringTestCase) }; @@ -100,7 +106,7 @@ private: // register in the unnamed registry so that these tests are run by default CPPUNIT_TEST_SUITE_REGISTRATION( StringTestCase ); -// also include in it's own registry so that these tests can be run alone +// also include in its own registry so that these tests can be run alone CPPUNIT_TEST_SUITE_NAMED_REGISTRATION( StringTestCase, "StringTestCase" ); StringTestCase::StringTestCase() @@ -117,12 +123,12 @@ void StringTestCase::String() for (int i = 0; i < 2; ++i) { - a = _T("Hello"); - b = _T(" world"); - c = _T("! How'ya doin'?"); + a = wxT("Hello"); + b = wxT(" world"); + c = wxT("! How'ya doin'?"); a += b; a += c; - c = _T("Hello world! What's up?"); + c = wxT("Hello world! What's up?"); CPPUNIT_ASSERT( c != a ); } } @@ -135,12 +141,12 @@ void StringTestCase::PChar() for (int i = 0; i < 2; ++i) { - wxStrcpy (a, _T("Hello")); - wxStrcpy (b, _T(" world")); - wxStrcpy (c, _T("! How'ya doin'?")); + wxStrcpy (a, wxT("Hello")); + wxStrcpy (b, wxT(" world")); + wxStrcpy (c, wxT("! How'ya doin'?")); wxStrcat (a, b); wxStrcat (a, c); - wxStrcpy (c, _T("Hello world! What's up?")); + wxStrcpy (c, wxT("Hello world! What's up?")); CPPUNIT_ASSERT( wxStrcmp (c, a) != 0 ); } } @@ -148,19 +154,40 @@ void StringTestCase::PChar() void StringTestCase::Format() { wxString s1,s2; - s1.Printf(_T("%03d"), 18); - CPPUNIT_ASSERT( s1 == wxString::Format(_T("%03d"), 18) ); - s2.Printf(_T("Number 18: %s\n"), s1.c_str()); - CPPUNIT_ASSERT( s2 == wxString::Format(_T("Number 18: %s\n"), s1.c_str()) ); + s1.Printf(wxT("%03d"), 18); + CPPUNIT_ASSERT( s1 == wxString::Format(wxT("%03d"), 18) ); + s2.Printf(wxT("Number 18: %s\n"), s1.c_str()); + CPPUNIT_ASSERT( s2 == wxString::Format(wxT("Number 18: %s\n"), s1.c_str()) ); static const size_t lengths[] = { 1, 512, 1024, 1025, 2048, 4096, 4097 }; for ( size_t n = 0; n < WXSIZEOF(lengths); n++ ) { const size_t len = lengths[n]; - wxString s(_T('Z'), len); - CPPUNIT_ASSERT_EQUAL( len, wxString::Format(_T("%s"), s.c_str()).length()); + wxString s(wxT('Z'), len); + CPPUNIT_ASSERT_EQUAL( len, wxString::Format(wxT("%s"), s.c_str()).length()); } + + + CPPUNIT_ASSERT_EQUAL + ( + "two one", + wxString::Format(wxT("%2$s %1$s"), wxT("one"), wxT("two")) + ); +} + +void StringTestCase::FormatUnicode() +{ +#if wxUSE_UNICODE + const char *UNICODE_STR = "Iestat\xC4\xAB %i%i"; + //const char *UNICODE_STR = "Iestat\xCC\x84 %i%i"; + + wxString fmt = wxString::FromUTF8(UNICODE_STR); + wxString s = wxString::Format(fmt, 1, 1); + wxString expected(fmt); + expected.Replace("%i", "1"); + CPPUNIT_ASSERT_EQUAL( expected, s ); +#endif // wxUSE_UNICODE } void StringTestCase::Constructors() @@ -179,6 +206,20 @@ void StringTestCase::Constructors() CPPUNIT_ASSERT_EQUAL( L"Hello", wxString(L"Hello", 5) ); #endif // wxUSE_UNICODE + CPPUNIT_ASSERT_EQUAL( 0, wxString(wxString(), 17).length() ); + +#if wxUSE_UNICODE_UTF8 + // This string has 3 characters (, and ), not 4 when using UTF-8 + // locale! + if ( wxConvLibc.IsUTF8() ) + { + wxString s3("h\xc3\xa9llo", 4); + CPPUNIT_ASSERT_EQUAL( 3, s3.length() ); + CPPUNIT_ASSERT_EQUAL( 'l', (char)s3[2] ); + } +#endif // wxUSE_UNICODE_UTF8 + + static const char *s = "?really!"; const char *start = wxStrchr(s, 'r'); const char *end = wxStrchr(s, '!'); @@ -206,20 +247,23 @@ void StringTestCase::StaticConstructors() CPPUNIT_ASSERT_EQUAL( "Hello", wxString::FromUTF8("Hello", 5) ); CPPUNIT_ASSERT_EQUAL( "Hello", wxString::FromUTF8("Hello") ); + CPPUNIT_ASSERT_EQUAL( 2, wxString::FromUTF8("h\xc3\xa9llo", 3).length() ); + + //CPPUNIT_ASSERT_EQUAL( 1, wxString::FromUTF8("", 1).length() ); } void StringTestCase::Extraction() { - wxString s(_T("Hello, world!")); + wxString s(wxT("Hello, world!")); - CPPUNIT_ASSERT( wxStrcmp( s.c_str() , _T("Hello, world!") ) == 0 ); - CPPUNIT_ASSERT( wxStrcmp( s.Left(5).c_str() , _T("Hello") ) == 0 ); - CPPUNIT_ASSERT( wxStrcmp( s.Right(6).c_str() , _T("world!") ) == 0 ); - CPPUNIT_ASSERT( wxStrcmp( s(3, 5).c_str() , _T("lo, w") ) == 0 ); - CPPUNIT_ASSERT( wxStrcmp( s.Mid(3).c_str() , _T("lo, world!") ) == 0 ); - CPPUNIT_ASSERT( wxStrcmp( s.substr(3, 5).c_str() , _T("lo, w") ) == 0 ); - CPPUNIT_ASSERT( wxStrcmp( s.substr(3).c_str() , _T("lo, world!") ) == 0 ); + CPPUNIT_ASSERT( wxStrcmp( s.c_str() , wxT("Hello, world!") ) == 0 ); + CPPUNIT_ASSERT( wxStrcmp( s.Left(5).c_str() , wxT("Hello") ) == 0 ); + CPPUNIT_ASSERT( wxStrcmp( s.Right(6).c_str() , wxT("world!") ) == 0 ); + CPPUNIT_ASSERT( wxStrcmp( s(3, 5).c_str() , wxT("lo, w") ) == 0 ); + CPPUNIT_ASSERT( wxStrcmp( s.Mid(3).c_str() , wxT("lo, world!") ) == 0 ); + CPPUNIT_ASSERT( wxStrcmp( s.substr(3, 5).c_str() , wxT("lo, w") ) == 0 ); + CPPUNIT_ASSERT( wxStrcmp( s.substr(3).c_str() , wxT("lo, world!") ) == 0 ); #if wxUSE_UNICODE static const char *germanUTF8 = "Oberfl\303\244che"; @@ -236,13 +280,13 @@ void StringTestCase::Extraction() if ( result ) \ CPPUNIT_ASSERT_EQUAL(correct_rest, rest) - TEST_STARTS_WITH( _T("Hello"), _T(", world!"), true ); - TEST_STARTS_WITH( _T("Hello, "), _T("world!"), true ); - TEST_STARTS_WITH( _T("Hello, world!"), _T(""), true ); - TEST_STARTS_WITH( _T("Hello, world!!!"), _T(""), false ); - TEST_STARTS_WITH( _T(""), _T("Hello, world!"), true ); - TEST_STARTS_WITH( _T("Goodbye"), _T(""), false ); - TEST_STARTS_WITH( _T("Hi"), _T(""), false ); + TEST_STARTS_WITH( wxT("Hello"), wxT(", world!"), true ); + TEST_STARTS_WITH( wxT("Hello, "), wxT("world!"), true ); + TEST_STARTS_WITH( wxT("Hello, world!"), wxT(""), true ); + TEST_STARTS_WITH( wxT("Hello, world!!!"), wxT(""), false ); + TEST_STARTS_WITH( wxT(""), wxT("Hello, world!"), true ); + TEST_STARTS_WITH( wxT("Goodbye"), wxT(""), false ); + TEST_STARTS_WITH( wxT("Hi"), wxT(""), false ); #undef TEST_STARTS_WITH @@ -255,15 +299,15 @@ void StringTestCase::Extraction() if ( result ) \ CPPUNIT_ASSERT_EQUAL(correct_rest, rest) - TEST_ENDS_WITH( _T(""), _T("Hello, world!"), true ); - TEST_ENDS_WITH( _T("!"), _T("Hello, world"), true ); - TEST_ENDS_WITH( _T(", world!"), _T("Hello"), true ); - TEST_ENDS_WITH( _T("ello, world!"), _T("H"), true ); - TEST_ENDS_WITH( _T("Hello, world!"), _T(""), true ); - TEST_ENDS_WITH( _T("very long string"), _T(""), false ); - TEST_ENDS_WITH( _T("?"), _T(""), false ); - TEST_ENDS_WITH( _T("Hello, world"), _T(""), false ); - TEST_ENDS_WITH( _T("Gello, world!"), _T(""), false ); + TEST_ENDS_WITH( wxT(""), wxT("Hello, world!"), true ); + TEST_ENDS_WITH( wxT("!"), wxT("Hello, world"), true ); + TEST_ENDS_WITH( wxT(", world!"), wxT("Hello"), true ); + TEST_ENDS_WITH( wxT("ello, world!"), wxT("H"), true ); + TEST_ENDS_WITH( wxT("Hello, world!"), wxT(""), true ); + TEST_ENDS_WITH( wxT("very long string"), wxT(""), false ); + TEST_ENDS_WITH( wxT("?"), wxT(""), false ); + TEST_ENDS_WITH( wxT("Hello, world"), wxT(""), false ); + TEST_ENDS_WITH( wxT("Gello, world!"), wxT(""), false ); #undef TEST_ENDS_WITH } @@ -273,15 +317,15 @@ void StringTestCase::Trim() #define TEST_TRIM( str , dir , result ) \ CPPUNIT_ASSERT( wxString(str).Trim(dir) == result ) - TEST_TRIM( _T(" Test "), true, _T(" Test") ); - TEST_TRIM( _T(" "), true, _T("") ); - TEST_TRIM( _T(" "), true, _T("") ); - TEST_TRIM( _T(""), true, _T("") ); + TEST_TRIM( wxT(" Test "), true, wxT(" Test") ); + TEST_TRIM( wxT(" "), true, wxT("") ); + TEST_TRIM( wxT(" "), true, wxT("") ); + TEST_TRIM( wxT(""), true, wxT("") ); - TEST_TRIM( _T(" Test "), false, _T("Test ") ); - TEST_TRIM( _T(" "), false, _T("") ); - TEST_TRIM( _T(" "), false, _T("") ); - TEST_TRIM( _T(""), false, _T("") ); + TEST_TRIM( wxT(" Test "), false, wxT("Test ") ); + TEST_TRIM( wxT(" "), false, wxT("") ); + TEST_TRIM( wxT(" "), false, wxT("") ); + TEST_TRIM( wxT(""), false, wxT("") ); #undef TEST_TRIM } @@ -289,11 +333,11 @@ void StringTestCase::Trim() void StringTestCase::Find() { #define TEST_FIND( str , start , result ) \ - CPPUNIT_ASSERT( wxString(str).find(_T("ell"), start) == result ); + CPPUNIT_ASSERT( wxString(str).find(wxT("ell"), start) == result ); - TEST_FIND( _T("Well, hello world"), 0, 1 ); - TEST_FIND( _T("Well, hello world"), 6, 7 ); - TEST_FIND( _T("Well, hello world"), 9, wxString::npos ); + TEST_FIND( wxT("Well, hello world"), 0, 1 ); + TEST_FIND( wxT("Well, hello world"), 6, 7 ); + TEST_FIND( wxT("Well, hello world"), 9, wxString::npos ); #undef TEST_FIND } @@ -307,37 +351,41 @@ void StringTestCase::Replace() CPPUNIT_ASSERT_EQUAL( result, s ); \ } - TEST_REPLACE( _T("012-AWORD-XYZ"), 4, 5, _T("BWORD"), _T("012-BWORD-XYZ") ); - TEST_REPLACE( _T("increase"), 0, 2, _T("de"), _T("decrease") ); - TEST_REPLACE( _T("wxWindow"), 8, 0, _T("s"), _T("wxWindows") ); - TEST_REPLACE( _T("foobar"), 3, 0, _T("-"), _T("foo-bar") ); - TEST_REPLACE( _T("barfoo"), 0, 6, _T("foobar"), _T("foobar") ); + TEST_REPLACE( wxT("012-AWORD-XYZ"), 4, 5, wxT("BWORD"), wxT("012-BWORD-XYZ") ); + TEST_REPLACE( wxT("increase"), 0, 2, wxT("de"), wxT("decrease") ); + TEST_REPLACE( wxT("wxWindow"), 8, 0, wxT("s"), wxT("wxWindows") ); + TEST_REPLACE( wxT("foobar"), 3, 0, wxT("-"), wxT("foo-bar") ); + TEST_REPLACE( wxT("barfoo"), 0, 6, wxT("foobar"), wxT("foobar") ); #define TEST_NULLCHARREPLACE( o , olen, pos , len , replacement , r, rlen ) \ { \ wxString s(o,olen); \ s.replace( pos , len , replacement ); \ - CPPUNIT_ASSERT( s == wxString(r,rlen) ); \ + CPPUNIT_ASSERT_EQUAL( wxString(r,rlen), s ); \ } - TEST_NULLCHARREPLACE( _T("null\0char"), 9, 5, 1, _T("d"), - _T("null\0dhar"), 9 ); + TEST_NULLCHARREPLACE( wxT("null\0char"), 9, 5, 1, wxT("d"), + wxT("null\0dhar"), 9 ); #define TEST_WXREPLACE( o , olen, olds, news, all, r, rlen ) \ { \ wxString s(o,olen); \ s.Replace( olds, news, all ); \ - CPPUNIT_ASSERT( s == wxString(r,rlen) ); \ + CPPUNIT_ASSERT_EQUAL( wxString(r,rlen), s ); \ } - TEST_WXREPLACE( _T("null\0char"), 9, _T("c"), _T("de"), true, - _T("null\0dehar"), 10 ); + TEST_WXREPLACE( wxT("null\0char"), 9, wxT("c"), wxT("de"), true, + wxT("null\0dehar"), 10 ); - TEST_WXREPLACE( _T("null\0dehar"), 10, _T("de"), _T("c"), true, - _T("null\0char"), 9 ); + TEST_WXREPLACE( wxT("null\0dehar"), 10, wxT("de"), wxT("c"), true, + wxT("null\0char"), 9 ); TEST_WXREPLACE( "life", 4, "f", "", false, "lie", 3 ); + TEST_WXREPLACE( "life", 4, "f", "", true, "lie", 3 ); + TEST_WXREPLACE( "life", 4, "fe", "ve", true, "live", 4 ); + TEST_WXREPLACE( "xx", 2, "x", "yy", true, "yyyy", 4 ); + TEST_WXREPLACE( "xxx", 3, "xx", "z", true, "zx", 2 ); #undef TEST_WXREPLACE #undef TEST_NULLCHARREPLACE @@ -349,17 +397,17 @@ void StringTestCase::Match() #define TEST_MATCH( s1 , s2 , result ) \ CPPUNIT_ASSERT( wxString(s1).Matches(s2) == result ) - TEST_MATCH( _T("foobar"), _T("foo*"), true ); - TEST_MATCH( _T("foobar"), _T("*oo*"), true ); - TEST_MATCH( _T("foobar"), _T("*bar"), true ); - TEST_MATCH( _T("foobar"), _T("??????"), true ); - TEST_MATCH( _T("foobar"), _T("f??b*"), true ); - TEST_MATCH( _T("foobar"), _T("f?b*"), false ); - TEST_MATCH( _T("foobar"), _T("*goo*"), false ); - TEST_MATCH( _T("foobar"), _T("*foo"), false ); - TEST_MATCH( _T("foobarfoo"), _T("*foo"), true ); - TEST_MATCH( _T(""), _T("*"), true ); - TEST_MATCH( _T(""), _T("?"), false ); + TEST_MATCH( wxT("foobar"), wxT("foo*"), true ); + TEST_MATCH( wxT("foobar"), wxT("*oo*"), true ); + TEST_MATCH( wxT("foobar"), wxT("*bar"), true ); + TEST_MATCH( wxT("foobar"), wxT("??????"), true ); + TEST_MATCH( wxT("foobar"), wxT("f??b*"), true ); + TEST_MATCH( wxT("foobar"), wxT("f?b*"), false ); + TEST_MATCH( wxT("foobar"), wxT("*goo*"), false ); + TEST_MATCH( wxT("foobar"), wxT("*foo"), false ); + TEST_MATCH( wxT("foobarfoo"), wxT("*foo"), true ); + TEST_MATCH( wxT(""), wxT("*"), true ); + TEST_MATCH( wxT(""), wxT("?"), false ); #undef TEST_MATCH } @@ -367,14 +415,14 @@ void StringTestCase::Match() void StringTestCase::CaseChanges() { - wxString s1(_T("Hello!")); + wxString s1(wxT("Hello!")); wxString s1u(s1); wxString s1l(s1); s1u.MakeUpper(); s1l.MakeLower(); - CPPUNIT_ASSERT_EQUAL( _T("HELLO!"), s1u ); - CPPUNIT_ASSERT_EQUAL( _T("hello!"), s1l ); + CPPUNIT_ASSERT_EQUAL( wxT("HELLO!"), s1u ); + CPPUNIT_ASSERT_EQUAL( wxT("hello!"), s1l ); wxString s2u, s2l; s2u.MakeUpper(); @@ -447,6 +495,10 @@ void StringTestCase::Compare() CPPUNIT_ASSERT( s1 != neq2 ); CPPUNIT_ASSERT( s1 != neq3 ); CPPUNIT_ASSERT( s1 != neq4 ); + + CPPUNIT_ASSERT( wxString("\n").Cmp(" ") < 0 ); + CPPUNIT_ASSERT( wxString("'").Cmp("!") > 0 ); + CPPUNIT_ASSERT( wxString("!").Cmp("z") < 0 ); } void StringTestCase::CompareNoCase() @@ -494,6 +546,10 @@ void StringTestCase::CompareNoCase() CPPUNIT_CNCNEQ_ASSERT( s1, neq ); CPPUNIT_CNCNEQ_ASSERT( s1, neq2 ); CPPUNIT_CNCNEQ_ASSERT( s1, neq3 ); + + CPPUNIT_ASSERT( wxString("\n").CmpNoCase(" ") < 0 ); + CPPUNIT_ASSERT( wxString("'").CmpNoCase("!") > 0); + CPPUNIT_ASSERT( wxString("!").Cmp("Z") < 0 ); } void StringTestCase::Contains() @@ -505,15 +561,15 @@ void StringTestCase::Contains() bool contains; } containsData[] = { - { _T(""), _T(""), true }, - { _T(""), _T("foo"), false }, - { _T("foo"), _T(""), true }, - { _T("foo"), _T("f"), true }, - { _T("foo"), _T("o"), true }, - { _T("foo"), _T("oo"), true }, - { _T("foo"), _T("ooo"), false }, - { _T("foo"), _T("oooo"), false }, - { _T("foo"), _T("fooo"), false }, + { wxT(""), wxT(""), true }, + { wxT(""), wxT("foo"), false }, + { wxT("foo"), wxT(""), true }, + { wxT("foo"), wxT("f"), true }, + { wxT("foo"), wxT("o"), true }, + { wxT("foo"), wxT("oo"), true }, + { wxT("foo"), wxT("ooo"), false }, + { wxT("foo"), wxT("oooo"), false }, + { wxT("foo"), wxT("fooo"), false }, }; for ( size_t n = 0; n < WXSIZEOF(containsData); n++ ) @@ -543,6 +599,7 @@ static const struct ToLongData long value; #endif // wxLongLong_t int flags; + int base; long LValue() const { return value; } unsigned long ULValue() const { return value; } @@ -554,25 +611,41 @@ static const struct ToLongData bool IsOk() const { return !(flags & Number_Invalid); } } longData[] = { - { _T("1"), 1, Number_Ok }, - { _T("0"), 0, Number_Ok }, - { _T("a"), 0, Number_Invalid }, - { _T("12345"), 12345, Number_Ok }, - { _T("--1"), 0, Number_Invalid }, + { wxT("1"), 1, Number_Ok }, + { wxT("0"), 0, Number_Ok }, + { wxT("a"), 0, Number_Invalid }, + { wxT("12345"), 12345, Number_Ok }, + { wxT("--1"), 0, Number_Invalid }, - { _T("-1"), -1, Number_Signed | Number_Long }, - // this is surprizing but consistent with strtoul() behaviour - { _T("-1"), ULONG_MAX, Number_Unsigned | Number_Long }, + { wxT("-1"), -1, Number_Signed | Number_Long }, + // this is surprising but consistent with strtoul() behaviour + { wxT("-1"), ULONG_MAX, Number_Unsigned | Number_Long }, // this must overflow, even with 64 bit long - { _T("922337203685477580711"), 0, Number_Invalid }, + { wxT("922337203685477580711"), 0, Number_Invalid }, #ifdef wxLongLong_t - { _T("2147483648"), wxLL(2147483648), Number_LongLong }, - { _T("-2147483648"), wxLL(-2147483648), Number_LongLong | Number_Signed }, - { _T("9223372036854775808"), wxULL(9223372036854775808), Number_LongLong | + { wxT("2147483648"), wxLL(2147483648), Number_LongLong }, + { wxT("-2147483648"), wxLL(-2147483648), Number_LongLong | Number_Signed }, + { wxT("9223372036854775808"), wxULL(9223372036854775808), Number_LongLong | Number_Unsigned }, #endif // wxLongLong_t + + // Base tests. + { wxT("010"), 10, Number_Ok, 10 }, + { wxT("010"), 8, Number_Ok, 0 }, + { wxT("010"), 8, Number_Ok, 8 }, + { wxT("010"), 16, Number_Ok, 16 }, + + { wxT("0010"), 10, Number_Ok, 10 }, + { wxT("0010"), 8, Number_Ok, 0 }, + { wxT("0010"), 8, Number_Ok, 8 }, + { wxT("0010"), 16, Number_Ok, 16 }, + + { wxT("0x11"), 0, Number_Invalid, 10 }, + { wxT("0x11"), 17, Number_Ok, 0 }, + { wxT("0x11"), 0, Number_Invalid, 8 }, + { wxT("0x11"), 17, Number_Ok, 16 }, }; void StringTestCase::ToLong() @@ -585,10 +658,30 @@ void StringTestCase::ToLong() if ( ld.flags & (Number_LongLong | Number_Unsigned) ) continue; - CPPUNIT_ASSERT_EQUAL( ld.IsOk(), wxString(ld.str).ToLong(&l) ); + // NOTE: unless you're using some exotic locale, ToCLong and ToLong + // should behave the same for our test data set: + + CPPUNIT_ASSERT_EQUAL( ld.IsOk(), + wxString(ld.str).ToCLong(&l, ld.base) ); + if ( ld.IsOk() ) + CPPUNIT_ASSERT_EQUAL( ld.LValue(), l ); + + CPPUNIT_ASSERT_EQUAL( ld.IsOk(), + wxString(ld.str).ToLong(&l, ld.base) ); if ( ld.IsOk() ) CPPUNIT_ASSERT_EQUAL( ld.LValue(), l ); } + + // special case: check that the output is not modified if the parsing + // failed completely + l = 17; + CPPUNIT_ASSERT( !wxString("foo").ToLong(&l) ); + CPPUNIT_ASSERT_EQUAL( 17, l ); + + // also check that it is modified if we did parse something successfully in + // the beginning of the string + CPPUNIT_ASSERT( !wxString("9 cats").ToLong(&l) ); + CPPUNIT_ASSERT_EQUAL( 9, l ); } void StringTestCase::ToULong() @@ -601,7 +694,16 @@ void StringTestCase::ToULong() if ( ld.flags & (Number_LongLong | Number_Signed) ) continue; - CPPUNIT_ASSERT_EQUAL( ld.IsOk(), wxString(ld.str).ToULong(&ul) ); + // NOTE: unless you're using some exotic locale, ToCLong and ToLong + // should behave the same for our test data set: + + CPPUNIT_ASSERT_EQUAL( ld.IsOk(), + wxString(ld.str).ToCULong(&ul, ld.base) ); + if ( ld.IsOk() ) + CPPUNIT_ASSERT_EQUAL( ld.ULValue(), ul ); + + CPPUNIT_ASSERT_EQUAL( ld.IsOk(), + wxString(ld.str).ToULong(&ul, ld.base) ); if ( ld.IsOk() ) CPPUNIT_ASSERT_EQUAL( ld.ULValue(), ul ); } @@ -619,7 +721,8 @@ void StringTestCase::ToLongLong() if ( ld.flags & (Number_Long | Number_Unsigned) ) continue; - CPPUNIT_ASSERT_EQUAL( ld.IsOk(), wxString(ld.str).ToLongLong(&l) ); + CPPUNIT_ASSERT_EQUAL( ld.IsOk(), + wxString(ld.str).ToLongLong(&l, ld.base) ); if ( ld.IsOk() ) CPPUNIT_ASSERT_EQUAL( ld.LLValue(), l ); } @@ -635,7 +738,8 @@ void StringTestCase::ToULongLong() if ( ld.flags & (Number_Long | Number_Signed) ) continue; - CPPUNIT_ASSERT_EQUAL( ld.IsOk(), wxString(ld.str).ToULongLong(&ul) ); + CPPUNIT_ASSERT_EQUAL( ld.IsOk(), + wxString(ld.str).ToULongLong(&ul, ld.base) ); if ( ld.IsOk() ) CPPUNIT_ASSERT_EQUAL( ld.ULLValue(), ul ); } @@ -653,71 +757,155 @@ void StringTestCase::ToDouble() bool ok; } doubleData[] = { - { _T("1"), 1, true }, - { _T("1.23"), 1.23, true }, - { _T(".1"), .1, true }, - { _T("1."), 1, true }, - { _T("1.."), 0, false }, - { _T("0"), 0, true }, - { _T("a"), 0, false }, - { _T("12345"), 12345, true }, - { _T("-1"), -1, true }, - { _T("--1"), 0, false }, + { wxT("1"), 1, true }, + { wxT("1.23"), 1.23, true }, + { wxT(".1"), .1, true }, + { wxT("1."), 1, true }, + { wxT("1.."), 0, false }, + { wxT("0"), 0, true }, + { wxT("a"), 0, false }, + { wxT("12345"), 12345, true }, + { wxT("-1"), -1, true }, + { wxT("--1"), 0, false }, + { wxT("-3E-5"), -3E-5, true }, + { wxT("-3E-abcde5"), 0, false }, }; - // we need to use decimal point, not comma or whatever is its value for the - // current locale - wxSetlocale(LC_ALL, "C"); + // test ToCDouble() first: size_t n; for ( n = 0; n < WXSIZEOF(doubleData); n++ ) { const ToDoubleData& ld = doubleData[n]; + CPPUNIT_ASSERT_EQUAL( ld.ok, wxString(ld.str).ToCDouble(&d) ); + if ( ld.ok ) + CPPUNIT_ASSERT_EQUAL( ld.value, d ); + } + + + // test ToDouble() now: + // NOTE: for the test to be reliable, we need to set the locale explicitly + // so that we know the decimal point character to use + + if (!wxLocale::IsAvailable(wxLANGUAGE_FRENCH)) + return; // you should have french support installed to continue this test! + + wxLocale locale; + + // don't load default catalog, it may be unavailable: + CPPUNIT_ASSERT( locale.Init(wxLANGUAGE_FRENCH, wxLOCALE_DONT_LOAD_DEFAULT) ); + + static const struct ToDoubleData doubleData2[] = + { + { wxT("1"), 1, true }, + { wxT("1,23"), 1.23, true }, + { wxT(",1"), .1, true }, + { wxT("1,"), 1, true }, + { wxT("1,,"), 0, false }, + { wxT("0"), 0, true }, + { wxT("a"), 0, false }, + { wxT("12345"), 12345, true }, + { wxT("-1"), -1, true }, + { wxT("--1"), 0, false }, + { wxT("-3E-5"), -3E-5, true }, + { wxT("-3E-abcde5"), 0, false }, + }; + + for ( n = 0; n < WXSIZEOF(doubleData2); n++ ) + { + const ToDoubleData& ld = doubleData2[n]; CPPUNIT_ASSERT_EQUAL( ld.ok, wxString(ld.str).ToDouble(&d) ); if ( ld.ok ) CPPUNIT_ASSERT_EQUAL( ld.value, d ); } } +void StringTestCase::FromDouble() +{ + static const struct FromDoubleTestData + { + double value; + int prec; + const char *str; + } testData[] = + { + { 1.23, -1, "1.23" }, + // NB: there are no standards about the minimum exponent width + // and newer MSVC versions use 3 digits as minimum exponent + // width while GNU libc uses 2 digits as minimum width... +#ifdef wxUSING_VC_CRT_IO + { -3e-10, -1, "-3e-010" }, +#else + { -3e-10, -1, "-3e-10" }, +#endif + { -0.45678, -1, "-0.45678" }, + { 1.2345678, 0, "1" }, + { 1.2345678, 1, "1.2" }, + { 1.2345678, 2, "1.23" }, + { 1.2345678, 3, "1.235" }, + }; + + for ( unsigned n = 0; n < WXSIZEOF(testData); n++ ) + { + const FromDoubleTestData& td = testData[n]; + CPPUNIT_ASSERT_EQUAL( td.str, wxString::FromCDouble(td.value, td.prec) ); + } + + if ( !wxLocale::IsAvailable(wxLANGUAGE_FRENCH) ) + return; + + wxLocale locale; + CPPUNIT_ASSERT( locale.Init(wxLANGUAGE_FRENCH, wxLOCALE_DONT_LOAD_DEFAULT) ); + + for ( unsigned m = 0; m < WXSIZEOF(testData); m++ ) + { + const FromDoubleTestData& td = testData[m]; + + wxString str(td.str); + str.Replace(".", ","); + CPPUNIT_ASSERT_EQUAL( str, wxString::FromDouble(td.value, td.prec) ); + } +} + void StringTestCase::StringBuf() { // check that buffer can be used to write into the string wxString s; - wxStrcpy(wxStringBuffer(s, 10), _T("foo")); + wxStrcpy(wxStringBuffer(s, 10), wxT("foo")); CPPUNIT_ASSERT_EQUAL(3, s.length()); - CPPUNIT_ASSERT(_T('f') == s[0u]); - CPPUNIT_ASSERT(_T('o') == s[1]); - CPPUNIT_ASSERT(_T('o') == s[2]); + CPPUNIT_ASSERT(wxT('f') == s[0u]); + CPPUNIT_ASSERT(wxT('o') == s[1]); + CPPUNIT_ASSERT(wxT('o') == s[2]); { // also check that the buffer initially contains the original string // contents wxStringBuffer buf(s, 10); - CPPUNIT_ASSERT_EQUAL( _T('f'), buf[0] ); - CPPUNIT_ASSERT_EQUAL( _T('o'), buf[1] ); - CPPUNIT_ASSERT_EQUAL( _T('o'), buf[2] ); - CPPUNIT_ASSERT_EQUAL( _T('\0'), buf[3] ); + CPPUNIT_ASSERT_EQUAL( wxT('f'), buf[0] ); + CPPUNIT_ASSERT_EQUAL( wxT('o'), buf[1] ); + CPPUNIT_ASSERT_EQUAL( wxT('o'), buf[2] ); + CPPUNIT_ASSERT_EQUAL( wxT('\0'), buf[3] ); } { wxStringBufferLength buf(s, 10); - CPPUNIT_ASSERT_EQUAL( _T('f'), buf[0] ); - CPPUNIT_ASSERT_EQUAL( _T('o'), buf[1] ); - CPPUNIT_ASSERT_EQUAL( _T('o'), buf[2] ); - CPPUNIT_ASSERT_EQUAL( _T('\0'), buf[3] ); + CPPUNIT_ASSERT_EQUAL( wxT('f'), buf[0] ); + CPPUNIT_ASSERT_EQUAL( wxT('o'), buf[1] ); + CPPUNIT_ASSERT_EQUAL( wxT('o'), buf[2] ); + CPPUNIT_ASSERT_EQUAL( wxT('\0'), buf[3] ); // and check that it can be used to write only the specified number of // characters to the string - wxStrcpy(buf, _T("barrbaz")); + wxStrcpy(buf, wxT("barrbaz")); buf.SetLength(4); } CPPUNIT_ASSERT_EQUAL(4, s.length()); - CPPUNIT_ASSERT(_T('b') == s[0u]); - CPPUNIT_ASSERT(_T('a') == s[1]); - CPPUNIT_ASSERT(_T('r') == s[2]); - CPPUNIT_ASSERT(_T('r') == s[3]); + CPPUNIT_ASSERT(wxT('b') == s[0u]); + CPPUNIT_ASSERT(wxT('a') == s[1]); + CPPUNIT_ASSERT(wxT('r') == s[2]); + CPPUNIT_ASSERT(wxT('r') == s[3]); // check that creating buffer of length smaller than string works, i.e. at // least doesn't crash (it would if we naively copied the entire original @@ -788,11 +976,14 @@ void StringTestCase::CStrDataOperators() CPPUNIT_ASSERT( s.c_str()[0] == 'h' ); CPPUNIT_ASSERT( s.c_str()[1] == 'e' ); - CPPUNIT_ASSERT( s.c_str()[5] == '\0' ); + + // IMPORTANT: at least with the CRT coming with MSVC++ 2008 trying to access + // the final character results in an assert failure (with debug CRT) + //CPPUNIT_ASSERT( s.c_str()[5] == '\0' ); CPPUNIT_ASSERT( *s.c_str() == 'h' ); CPPUNIT_ASSERT( *(s.c_str() + 2) == 'l' ); - CPPUNIT_ASSERT( *(s.c_str() + 5) == '\0' ); + //CPPUNIT_ASSERT( *(s.c_str() + 5) == '\0' ); } bool CheckStrChar(const wxString& expected, char *s) @@ -839,28 +1030,90 @@ void StringTestCase::IndexedAccess() // this tests for a possible bug in UTF-8 based wxString implementation: // the 3rd character of the underlying byte string is going to change, but // the 3rd character of wxString should remain the same - s[0] = L'\u00e9'; + s[0] = L'\xe9'; CPPUNIT_ASSERT_EQUAL( 'r', (char)s[2] ); } void StringTestCase::BeforeAndAfter() { - const wxString s(L"letter=\u00e9;\u00e7a=l\u00e0"); + // Construct a string with 2 equal signs in it by concatenating its three + // parts: before the first "=", in between the two "="s and after the last + // one. This allows to avoid duplicating the string contents (which has to + // be different for Unicode and ANSI builds) in the tests below. +#if wxUSE_UNICODE + #define FIRST_PART L"letter" + #define MIDDLE_PART L"\xe9;\xe7a" + #define LAST_PART L"l\xe0" +#else // !wxUSE_UNICODE + #define FIRST_PART "letter" + #define MIDDLE_PART "e;ca" + #define LAST_PART "la" +#endif // wxUSE_UNICODE/!wxUSE_UNICODE - CPPUNIT_ASSERT_EQUAL( "letter", s.BeforeFirst('=') ); - CPPUNIT_ASSERT_EQUAL( s, s.BeforeFirst('!') ); - CPPUNIT_ASSERT_EQUAL( L"letter=\u00e9", s.BeforeFirst(';') ); + const wxString s(FIRST_PART wxT("=") MIDDLE_PART wxT("=") LAST_PART); - CPPUNIT_ASSERT_EQUAL( L"letter=\u00e9;\u00e7a", s.BeforeLast('=') ); - CPPUNIT_ASSERT_EQUAL( "", s.BeforeLast('!') ); - CPPUNIT_ASSERT_EQUAL( L"letter=\u00e9", s.BeforeLast(';') ); + wxString r; - CPPUNIT_ASSERT_EQUAL( L"\u00e9;\u00e7a=l\u00e0", s.AfterFirst('=') ); + CPPUNIT_ASSERT_EQUAL( FIRST_PART, s.BeforeFirst('=', &r) ); + CPPUNIT_ASSERT_EQUAL( MIDDLE_PART wxT("=") LAST_PART, r ); + + CPPUNIT_ASSERT_EQUAL( s, s.BeforeFirst('!', &r) ); + CPPUNIT_ASSERT_EQUAL( "", r ); + + + CPPUNIT_ASSERT_EQUAL( FIRST_PART wxT("=") MIDDLE_PART, s.BeforeLast('=', &r) ); + CPPUNIT_ASSERT_EQUAL( LAST_PART, r ); + + CPPUNIT_ASSERT_EQUAL( "", s.BeforeLast('!', &r) ); + CPPUNIT_ASSERT_EQUAL( s, r ); + + + CPPUNIT_ASSERT_EQUAL( MIDDLE_PART wxT("=") LAST_PART, s.AfterFirst('=') ); CPPUNIT_ASSERT_EQUAL( "", s.AfterFirst('!') ); - CPPUNIT_ASSERT_EQUAL( L"\u00e7a=l\u00e0", s.AfterFirst(';') ); - CPPUNIT_ASSERT_EQUAL( L"l\u00e0", s.AfterLast('=') ); + + CPPUNIT_ASSERT_EQUAL( LAST_PART, s.AfterLast('=') ); CPPUNIT_ASSERT_EQUAL( s, s.AfterLast('!') ); - CPPUNIT_ASSERT_EQUAL( L"\u00e7a=l\u00e0", s.AfterLast(';') ); + + #undef LAST_PART + #undef MIDDLE_PART + #undef FIRST_PART } +void StringTestCase::ScopedBuffers() +{ + // wxString relies on efficient buffers, verify they work as they should + + const char *literal = "Hello World!"; + + // non-owned buffer points to the string passed to it + wxScopedCharBuffer sbuf = wxScopedCharBuffer::CreateNonOwned(literal); + CPPUNIT_ASSERT( sbuf.data() == literal ); + + // a copy of scoped non-owned buffer still points to the same string + wxScopedCharBuffer sbuf2(sbuf); + CPPUNIT_ASSERT( sbuf.data() == sbuf2.data() ); + + // but assigning it to wxCharBuffer makes a full copy + wxCharBuffer buf(sbuf); + CPPUNIT_ASSERT( buf.data() != literal ); + CPPUNIT_ASSERT_EQUAL( literal, buf.data() ); + + wxCharBuffer buf2 = sbuf; + CPPUNIT_ASSERT( buf2.data() != literal ); + CPPUNIT_ASSERT_EQUAL( literal, buf.data() ); + + // Check that extending the buffer keeps it NUL-terminated. + size_t len = 10; + + wxCharBuffer buf3(len); + CPPUNIT_ASSERT_EQUAL('\0', buf3.data()[len]); + + wxCharBuffer buf4; + buf4.extend(len); + CPPUNIT_ASSERT_EQUAL('\0', buf4.data()[len]); + + wxCharBuffer buf5(5); + buf5.extend(len); + CPPUNIT_ASSERT_EQUAL('\0', buf5.data()[len]); +}