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 <nat...@acm.org> PR c++/79369 * g++.dg/cpp1z/nested-namespace-def1.C: Adjust. Index: testsuite/g++.dg/cpp1z/nested-namespace-def1.C =================================================================== --- testsuite/g++.dg/cpp1z/nested-namespace-def1.C (revision 248066) +++ testsuite/g++.dg/cpp1z/nested-namespace-def1.C (working copy) @@ -11,7 +11,7 @@ A::B::C::T::U::V::Y y; inline namespace D::E {} // { dg-error "cannot be inline" } -namespace F::G:: {} // { dg-error "nested identifier required" } +namespace F::G:: {} // { dg-error "namespace name required" } namespace G __attribute ((visibility ("default"))) ::H {} // { dg-error "cannot have attributes" }