Re: [PATCH] PR libstdc++/81064 fix versioned namespace

2017-07-19 Thread François Dumont
Here is the revisited version passing all tests successfully except 2 that are also failing without the patch: FAIL: 21_strings/basic_string/modifiers/insert/char/1.cc execution test FAIL: 21_strings/basic_string/modifiers/insert/wchar_t/1.cc execution test This is a rather mechanical change exc

Re: [PATCH] PR libstdc++/81064 fix versioned namespace

2017-07-18 Thread François Dumont
On 18/07/2017 16:03, Ville Voutilainen wrote: On 18 July 2017 at 16:31, Jonathan Wakely wrote: This is quite a huge change, so I'd like to wait and see if anyone else has any opinion on it. Personally I think it's necessary (assuming I understand the PR correctly) and so if nobody objects I th

Re: [PATCH] PR libstdc++/81064 fix versioned namespace

2017-07-18 Thread Ville Voutilainen
On 18 July 2017 at 16:31, Jonathan Wakely wrote: > This is quite a huge change, so I'd like to wait and see if anyone > else has any opinion on it. > > Personally I think it's necessary (assuming I understand the PR > correctly) and so if nobody objects I think we should go with this > change for

Re: [PATCH] PR libstdc++/81064 fix versioned namespace

2017-07-18 Thread Jonathan Wakely
On 18/07/17 14:31 +0100, Jonathan Wakely wrote: On 17/07/17 22:54 +0200, François Dumont wrote: This is quite a huge change, so I'd like to wait and see if anyone else has any opinion on it. Personally I think it's necessary (assuming I understand the PR correctly) and so if nobody objects I thi

Re: [PATCH] PR libstdc++/81064 fix versioned namespace

2017-07-18 Thread Jonathan Wakely
On 17/07/17 22:54 +0200, François Dumont wrote: Hi Here is the patch to fix libstdc++ versioned namespace. Now versioned namespace is only at std and __gnu_cxx levels, not anymore in nested namespaces. PR libstdc++/81064 * include/bits/algorithmfwd.h: Reorganize versioned namespa