stdcxx-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Martin Sebor <>
Subject Re: svn commit: r399914 - in /incubator/stdcxx/trunk/tests/strings: 21.string.append.cpp 21.string.assign.cpp 21.string.insert.cpp 21.string.replace.cpp
Date Fri, 05 May 2006 16:50:48 GMT
Anton Pevtsov wrote:
> Martin Sebor wrote:
>>Anton, this simplifies the exception safety setup. Another change
> [...]
>>I'll try to cook one up over the weekend or sometime next week.
> The changes looks good. I'll use them porting the rest of the tests. 
> I plan to port tests for access, ctor, update the test for swap and
> accommodate already ported tests to your planned changes.

Sounds like a plan!

> Martin Sebor wrote:
>>Btw., I noticed that the 21.string.copy.cpp hasn't been converted to
> the new simplified format yet. This is just a reminder that we should >
> remember to do so before we move on to other (non-string) tests.
> Yes, today I updated this test. The changes are here:

Great, thanks for taking care of it! Committed thus:

> I think we need to port tests for 
> 21.string.ctor
> 21.string.access (I am working on this now)
> And implement tests for
> 21.string.swap (it is ready, but needed to be converted to the
> simplified format)


> string methods c_str(), data(), begin(), end(), rbegin(), rend(),
> get_allocator().

Most (all?) of these should probably go in the same test.

> Also I noticed that we have missed push_back method in the append test. 
> Btw, shall we have special MemberId enum element for this method and
> separate elements for operator[] and at ?

I guess we might as well. It will give us centralized control
over all string member functions.


View raw message