Mailing-List: contact stdcxx-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: stdcxx-dev@incubator.apache.org
Received-SPF: pass (herse.apache.org: local policy)
Message-ID: <45EEFEC8.1000404@roguewave.com>
Date: Wed, 07 Mar 2007 11:04:56 -0700
From: Martin Sebor <sebor@roguewave.com>
Organization: Rogue Wave Software
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US;
 rv:1.8.1.2pre) Gecko/20070111 SeaMonkey/1.1
MIME-Version: 1.0
To: stdcxx-dev@incubator.apache.org
Subject: Re: svn commit: r511015 - in /incubator/stdcxx/trunk/tests:
 include/21.strings.h
 include/23.containers.h src/23.containers.cpp
References: <20070223165755.C93E71A981A@eris.apache.org>
In-Reply-To: <20070223165755.C93E71A981A@eris.apache.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

faridz@apache.org wrote:
> Author: faridz
> Date: Fri Feb 23 08:57:54 2007
> New Revision: 511015
> 
> URL: http://svn.apache.org/viewvc?view=rev&rev=511015
> Log:
> 2007-02-23 Farid Zaripov <faridz@kyiv.vdiweb.com>
> 
> 	ChangeLog:
> 	* 21.strings.h: #included 23.containers.h;
> 	struct StringIds inherited from struct ContainerIds;
> 	removed definitions of the AllocId and IteratorId.
> 	* 23.containers.h: New file with definitions of helpers used
> 	in clause 23 tests.
> 	* 23.containers.cpp: Ditto.
[...]
> Added: incubator/stdcxx/trunk/tests/src/23.containers.cpp
> URL: http://svn.apache.org/viewvc/incubator/stdcxx/trunk/tests/src/23.containers.cpp?view=auto&rev=511015
> ==============================================================================
> --- incubator/stdcxx/trunk/tests/src/23.containers.cpp (added)
> +++ incubator/stdcxx/trunk/tests/src/23.containers.cpp Fri Feb 23 08:57:54 2007
> @@ -0,0 +1,1180 @@
[...]
> +
> +// temporary, shuld be defined in 23.list.h
> +struct ListIds : ContainerIds
> +{
> +    enum OverloadId {};
> +};

As an FYI, the (well-formed) empty enumeration is causing aCC
to issue a warning. There are other warnings in the test driver
and the test suite so it's not a huge deal but it would be nice
to clean them up and have warning-free builds. To that end we
should probably add a dummy enumerator to OverloadId.

aCC -c -I$(TOPDIR)/include/ansi -I/usr/include  -D_RWSTDDEBUG   -mt 
-D_RWSTD_USE_CONFIG -I$(TOPDIR)/include -I$(BUILDDIR)/include 
-I$(TOPDIR)/../rwtest -I$(TOPDIR)/tests/include  -Aa +nostl  -g +d  +w 
+W392 +W655 +W684 +W818 +W819 +W849   $(TOPDIR)/tests/src/23.containers.cpp
Warning 279: "$(TOPDIR)/tests/src/23.containers.cpp", line 523 # 
Enumeration was defined with an empty enumerator list.
         enum OverloadId {};
         ^^^^^^^^^^^^^^^

Martin