Remove testsuite/23_containers/map/operators/1.cc ?

Paolo Carlini paolo.carlini@oracle.com
Wed Jun 4 20:35:00 GMT 2014


Hi,

On 06/04/2014 10:22 PM, François Dumont wrote:
> Hello
>
>     Do you see any objection in removing this file ? It doesn't look 
> like a test at all to me, just send some datas to std::cerr. There is 
> also a weird comment talking about multiset while code is using map.
The comment seems a pasto, which should be fixed. Likewise the uses of 
the inserters should be either turned into VERIFYs or alternately the 
testcase changed to do-compile (the original issue was a compile-time 
issue, but years ago we used to mostly add to the library testsuite 
do-run testcases). Indeed, we could also move it to where the other 
insert tests are (I think the reason of the weird directory has to do 
with the semantics of the operator[] in C++98).

Paolo.



More information about the Libstdc++ mailing list