Re: [C++ PATCH] push_namespace cleanup

2017-05-15 Thread Nathan Sidwell
On 05/15/2017 03:38 PM, Nathan Sidwell wrote: This cleanup patch from the modules branch fixes pr 79369, where we would accept inlining of an already existing namespace. missed this testcase tweak. now committed. -- Nathan Sidwell 2017-05-15 Nathan Sidwell PR c++/79369 * g++.dg/cpp1z/ne

[C++ PATCH] push_namespace cleanup

2017-05-15 Thread Nathan Sidwell
This cleanup patch from the modules branch fixes pr 79369, where we would accept inlining of an already existing namespace. I changed push_namespace to return a count of the depth pushed, because I also have a fix for DR 2061, where we can end up pushing more than one namespace. That'll be ap