This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc 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] support: Define support_static_assert for use from C and C++

From: Paul Eggert <eggert at cs dot ucla dot edu>
To: Florian Weimer <fweimer at redhat dot com>, libc-alpha at sourceware dot org
Date: Sun, 31 Dec 2017 12:25:18 -0800
Subject: Re: [PATCH] support: Define support_static_assert for use from C and C++
Authentication-results: sourceware.org; auth=none
References: <20171231192358.7F97E401BC22B@oldenburg.str.redhat.com>

Florian Weimer wrote:

+/* Static assertion, under a common name for both C++ and C11.  */
+#ifdef __cplusplus
+# define support_static_assert static_assert
+#else
+# define support_static_assert _Static_assert
+#endif


Wouldn't it be simpler to do this?

#ifdef __cplusplus
# define _Static_assert static_assert
#endif

and then use _Static_assert everywhere? Or do it the other way around. It seemsodd to introduce yet another name "support_static_assert" for this concept.

Follow-Ups:
- Re: [PATCH] support: Define support_static_assert for use from C and C++
  - From: Florian Weimer

References:
- [PATCH] support: Define support_static_assert for use from C and C++
  - From: Florian Weimer

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