[v3] define string::pop_back()

[Jonathan Wakely]

On 7 November 2011 09:17, Eric Botcazou wrote:
>>         * include/bits/basic_string.h (basic_string::at): Move adjacent to
>> other overload.
>>         (basic_string::pop_back): Define.
>>         * include/debug/string (__gnu_debug::basic_string::pop_back):
>> Likewise. * include/ext/vstring.h (__versa_string::pop_back): Likewise. *
>> config/abi/pre/gnu.ver: Add new symbols.
>>         * testsuite/21_strings/basic_string/modifiers/char/pop_back.cc:
>> New. * testsuite/21_strings/basic_string/modifiers/wchar_t/pop_back.cc:
>> New. * testsuite/21_strings/basic_string/range_access.cc: Split to ... *
>> testsuite/21_strings/basic_string/range_access/char/1.cc: Here and ... *
>> testsuite/21_strings/basic_string/range_access/wchar_t/1.cc: Here. *
>> testsuite/ext/vstring/modifiers/char/pop_back.cc: New.
>>         * testsuite/ext/vstring/modifiers/wchar_t/pop_back.cc: New.
>
> This breaks bootstrap on Solaris:
>
> ld: fatal: libstdc++-symbols.ver-sun: 4806: symbol `std::basic_string<char,
> std::char_traits<char>, std::allocator<char> >::pop_back()' is already defined
> in file: libstdc++-symbols.ver-sun
>
> _ZNSs8pop_backEv is referenced twice in the version file.

Thanks, I'll try to work out the right adjustment for the solaris
symbols file but can't test it.