You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@stdcxx.apache.org by Martin Sebor <se...@roguewave.com> on 2008/06/23 22:00:58 UTC

Re: svn commit: r667637 - in /stdcxx/branches/4.3.x: etc/config/src/STATIC_ASSERT.cpp include/rw/_meta_cat.h include/rw/_meta_ref.h include/rw/_meta_rel.h include/rw/_static_assert.h

elemings@apache.org wrote:
> Author: elemings
> Date: Fri Jun 13 13:20:48 2008
> New Revision: 667637
> 
> URL: http://svn.apache.org/viewvc?rev=667637&view=rev
> Log:
> 2008-06-13  Eric Lemings <er...@roguewave.com>
> 
> 	STDCXX-916
> 	* include/rw/_meta_cat.h, include/rw/_meta_ref.h,
> 	include/rw/_meta_rel.h: Include missing headers.
> 	* include/rw/_static_assert.h [_RWSTD_STATIC_ASSERT]: Reverse
> 	definitions.  Add Mesg parameter to internal definition so both
> 	definitions have same number of parameters.
> 	* etc/config/src/STATIC_ASSERT.cpp (main): Change main() to a
> 	regular function so config scripts don't find main() and
> 	(unnecessarily) link (and run?) program.
> 
[...]
> Modified: stdcxx/branches/4.3.x/include/rw/_static_assert.h
> URL: http://svn.apache.org/viewvc/stdcxx/branches/4.3.x/include/rw/_static_assert.h?rev=667637&r1=667636&r2=667637&view=diff
> ==============================================================================
> --- stdcxx/branches/4.3.x/include/rw/_static_assert.h (original)
> +++ stdcxx/branches/4.3.x/include/rw/_static_assert.h Fri Jun 13 13:20:48 2008
> @@ -53,14 +53,12 @@
>  } // namespace __rw
>  
>  #ifndef _RWSTD_NO_STATIC_ASSERT
> -
> -#  define _RWSTD_STATIC_ASSERT(Cond)                                       \
> +#  define _RWSTD_STATIC_ASSERT(Cond,Mesg) static_assert(Cond, Mesg)

There's a missing space after the comma.

> +#else
> +#  define _RWSTD_STATIC_ASSERT(Cond,Mesg)                                  \
>       typedef                                                               \
>       _RW::__rw_static_assert<sizeof (__rw_assert_failed<(Cond)>) != 0>     \
>           _RWSTD_PASTE(__static_assert,__LINE__)

Same here.

Martin