This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH, C++] PR43824 pedantic with inline namespaces


> That looks OK, but it needs a ChangeLog entry. ÂAnd I'm surprised that the
> testsuite didn't need any changes as a result; did you run make check-c++?

Ops! Indeed.

As commented in PR43824 I've defined this warning as a -pedantic only
warning (OPT_pedantic). That is different from the other
maybe_warn_cxx0x warnings.
That's needed because there are a few testcases
(g++.dg/lookup/strong-using-?.C) that use inline namespaces outside
the cpp0x directory.

Also, I've given a try to the changelog below.

Regards.
Rodrigo.

--
gcc/cp

2010-08-24  Rodrigo Rivas <rodrigorivascosta@gmail.com>

	PR c++/43824
	* error.c (maybe_warn_cpp0x): Add new warning
	CPP0X_INLINE_NAMESPACES.
	* parser.c (cp_parser_namespace_definition): Likewise.
	* cp-tree.h (cpp0x_warn_str): Likewise.

gcc/testsuite/

2010-08-24  Rodrigo Rivas <rodrigorivascosta@gmail.com>

	PR c++/43824
	* g++.dg/lookup/strong-using-1.C: Set dg-options to "" to avoid
	the -pedantic errors.
	* g++.dg/lookup/strong-using-2.C: Likewise.
	* g++.dg/lookup/strong-using-3.C: Likewise.
	* g++.dg/lookup/strong-using-5.C: Likewise.

	* g++.dg/cpp0x/inline-ns3.C: Add -std=c++0x.
	* g++.dg/cpp0x/inline-ns4.C: New.
	* g++.dg/cpp0x/inline-ns5.C: New.

Attachment: warn.txt
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]