53

I'd like to implement an "assert" that prevents compilation, rather than failing at runtime, in the error case.

I currently have one defined like this, which works great, but which increases the size of the binaries.

#define MY_COMPILER_ASSERT(EXPRESSION) switch (0) {case 0: case (EXPRESSION):;}

Sample code (which fails to compile).

#define DEFINE_A 1
#define DEFINE_B 1
MY_COMPILER_ASSERT(DEFINE_A == DEFINE_B);

How can I implement this so that it does not generate any code (in order to minimize the size of the binaries generated)?

Trevor Hickey
  • 36,288
  • 32
  • 162
  • 271
NickB
  • 1,471
  • 4
  • 14
  • 20

11 Answers11

54

A compile-time assert in pure standard C is possible, and a little bit of preprocessor trickery makes its usage look just as clean as the runtime usage of assert().

The key trick is to find a construct that can be evaluated at compile time and can cause an error for some values. One answer is the declaration of an array cannot have a negative size. Using a typedef prevents the allocation of space on success, and preserves the error on failure.

The error message itself will cryptically refer to declaration of a negative size (GCC says "size of array foo is negative"), so you should pick a name for the array type that hints that this error really is an assertion check.

A further issue to handle is that it is only possible to typedef a particular type name once in any compilation unit. So, the macro has to arrange for each usage to get a unique type name to declare.

My usual solution has been to require that the macro have two parameters. The first is the condition to assert is true, and the second is part of the type name declared behind the scenes. The answer by plinth hints at using token pasting and the __LINE__ predefined macro to form a unique name possibly without needing an extra argument.

Unfortunately, if the assertion check is in an included file, it can still collide with a check at the same line number in a second included file, or at that line number in the main source file. We could paper over that by using the macro __FILE__, but it is defined to be a string constant and there is no preprocessor trick that can turn a string constant back into part of an identifier name; not to mention that legal file names can contain characters that are not legal parts of an identifier.

So, I would propose the following code fragment:

/** A compile time assertion check.
 *
 *  Validate at compile time that the predicate is true without
 *  generating code. This can be used at any point in a source file
 *  where typedef is legal.
 *
 *  On success, compilation proceeds normally.
 *
 *  On failure, attempts to typedef an array type of negative size. The
 *  offending line will look like
 *      typedef assertion_failed_file_h_42[-1]
 *  where file is the content of the second parameter which should
 *  typically be related in some obvious way to the containing file
 *  name, 42 is the line number in the file on which the assertion
 *  appears, and -1 is the result of a calculation based on the
 *  predicate failing.
 *
 *  \param predicate The predicate to test. It must evaluate to
 *  something that can be coerced to a normal C boolean.
 *
 *  \param file A sequence of legal identifier characters that should
 *  uniquely identify the source file in which this condition appears.
 */
#define CASSERT(predicate, file) _impl_CASSERT_LINE(predicate,__LINE__,file)

#define _impl_PASTE(a,b) a##b
#define _impl_CASSERT_LINE(predicate, line, file) \
    typedef char _impl_PASTE(assertion_failed_##file##_,line)[2*!!(predicate)-1];

A typical usage might be something like:

#include "CAssert.h"
...
struct foo { 
    ...  /* 76 bytes of members */
};
CASSERT(sizeof(struct foo) == 76, demo_c);

In GCC, an assertion failure would look like:

$ gcc -c demo.c
demo.c:32: error: size of array `assertion_failed_demo_c_32' is negative
$
RBerteig
  • 41,948
  • 7
  • 88
  • 128
  • 2
    If you don't care about portability, in GCC \_\_COUNTER\_\_ can be used to give a unique identifier to paste onto the typedef name. It was added fairly recently (4.3) – Greg Rogers Jul 17 '09 at 15:52
  • 1
    I've been amazed that the C preprocessor has never had something like __COUNTER__. Macro assemblers have had similar constructs as long as there have been macro assemblers, not to mention the ability for one macro to define another which can be used to build any kind of unique symbol one might need. Unfortunately for me, my embedded systems projects are generally stuck with gcc 3.4.5 or so, if not some proprietary compiler that (mostly) complies with C89. – RBerteig Jul 17 '09 at 20:03
  • @RBerteig Hello, I would like to use this code for an Open Source project. What would you licence it under? – Hannes Landeholm Aug 19 '14 at 12:26
  • 2
    @HannesLandeholm If I were publishing it myself, I would normally apply the MIT license. Published as it is here without any further discussion, it is fair to assume that the general claim that user content is licensed as [CC-BY-SA](http://creativecommons.org/licenses/by-sa/3.0/) applies. To that end, you could simply add "CC-BY-SA 3.0 from http://stackoverflow.com/a/809465/68204" to the comment block. Personally, I prefer not to impose the "share alike" restriction on code, but attribution and a link here is always appropriate. – RBerteig Aug 19 '14 at 20:14
  • Definitely quite cool. glib does the same with [G_STATIC_ASSERT()](https://developer.gnome.org/glib/stable/glib-Miscellaneous-Macros.html#G-STATIC-ASSERT:CAPS) – Marco A. Aug 31 '18 at 09:02
  • 3
    This approach now triggers GCC warnings about unused typedefs, unfrotunately. – Kaz Mar 04 '22 at 19:20
9

The following COMPILER_VERIFY(exp) macro works fairly well.

// combine arguments (after expanding arguments)
#define GLUE(a,b) __GLUE(a,b)
#define __GLUE(a,b) a ## b

#define CVERIFY(expr, msg) typedef char GLUE (compiler_verify_, msg) [(expr) ? (+1) : (-1)]

#define COMPILER_VERIFY(exp) CVERIFY (exp, __LINE__)

It works for both C and C++ and can be used anywhere a typedef would be allowed. If the expression is true, it generates a typedef for an array of 1 char (which is harmless). If the expression is false, it generates a typedef for an array of -1 chars, which will generally result in an error message. The expression given as an arugment can be anything that evaluates to a compile-time constant (so expressions involving sizeof() work fine). This makes it much more flexible than

#if (expr)
#error
#endif

where you are restricted to expressions that can be evaluated by the preprocessor.

Stephen C. Steel
  • 4,380
  • 1
  • 20
  • 21
  • I like this but it causes a warning for an unused declared variable: main.c:22:47: warning: typedef ‘compiler_verify_39’ locally defined but not used [-Wunused-local-typedefs]. Since my project treats warnings as errors I can not use this. – Alex Jul 12 '19 at 10:44
  • 1
    Things have moved on in the decade since I posted this answer. If your compiler supports C11, you can use the _Static_assert that is now part of the language. – Stephen C. Steel Jul 12 '19 at 21:50
  • @Alex, this warning can be silenced with `(void)(sizeof(TYPEDEF_NAME));`. You might need an extra layer of dereference to pass GLUE as a parameter to the CVERIFY() macro, so that you have a TYPEDEF_NAME to work with. And `(void)ENTITY;` is a general way of dismissing 'unused' warnings. – Jetski S-type Nov 08 '21 at 23:45
7

As Leander said, static assertions are being added to C++11, and now they have.

static_assert(exp, message)

For example

#include "myfile.hpp"

static_assert(sizeof(MyClass) == 16, "MyClass is not 16 bytes!")

void doStuff(MyClass object) { }

See the cppreference page on it.

Dylan
  • 572
  • 5
  • 10
  • 9
    The question is about C not C++. – Trevor Hickey Feb 17 '17 at 18:57
  • 1
    C11 provides the similarly named `_Static_assert` (see https://stackoverflow.com/a/7287341/446106). Additionally, `assert.h` contains a `static_assert` macro that maps to it (https://en.cppreference.com/w/c/error/static_assert) – mwfearnley Feb 18 '19 at 23:05
5

The best writeup that I could find on static assertions in C is at pixelbeat. Note that static assertions are being added to C++ 0X, and may make it in to C1X, but that's not going to be for a while. I do not know if the macros in the link I gave will increase the size of your binaries. I would suspect they would not, at least if you compile at a reasonable level of optimisation, but your mileage may vary.

ChrisInEdmonton
  • 4,470
  • 5
  • 33
  • 48
4

If your compiler sets a preprocessor macro like DEBUG or NDEBUG you can make something like this (otherwise you could set this up in a Makefile):

#ifdef DEBUG
#define MY_COMPILER_ASSERT(EXPRESSION)   switch (0) {case 0: case (EXPRESSION):;}
#else
#define MY_COMPILER_ASSERT(EXPRESSION)
#endif

Then, your compiler asserts only for debug builds.

dreamlax
  • 93,976
  • 29
  • 161
  • 209
4

Using '#error' is a valid preprocessor definition that causes compilation to stop on most compilers. You can just do it like this, for example, to prevent compilation in debug:


#ifdef DEBUG
#error Please don't compile now
#endif
Steve Wranovsky
  • 5,503
  • 4
  • 34
  • 52
  • 6
    Unfortunately this aborts at the preprocessor level, so it's unable to handle things like `assert(sizeof(long) == sizeof(void *))`. – ephemient Apr 30 '09 at 15:17
4

I know you're interested in C, but take a look at boost's C++ static_assert. (Incidentally, this is likely becoming available in C++1x.)

We've done something similar, again for C++:

#define COMPILER_ASSERT(expr)  enum { ARG_JOIN(CompilerAssertAtLine, __LINE__) = sizeof( char[(expr) ? +1 : -1] ) }

This works only in C++, apparently. This article discusses a way to modify it for use in C.

leander
  • 8,527
  • 1
  • 30
  • 43
2

When you compile your final binaries, define MY_COMPILER_ASSERT to be blank, so that its output isn't included in the result. Only define it the way you have it for debugging.

But really, you aren't going to be able to catch every assertion this way. Some just don't make sense at compile time (like the assertion that a value is not null). All you can do is verify the values of other #defines. I'm not really sure why you'd want to do that.

Welbog
  • 59,154
  • 9
  • 110
  • 123
  • 5
    It is useful because anything that can be verified at compile time is something that didn't require a test case at run time. When building a portable protocol implementation it is useful to validate assumptions about structure size and layout. Since sizes and offsets are known at compile time, testing them then is preferred. Also, implementing a compile-time assert without code generation means that there is no reason to take it out of release builds. At worst it clutters the symbol table with orphan type names. – RBerteig Jul 17 '09 at 20:07
0

Since C11, static_assert is available via <assert.h>. Since C23, static_assert is itself a keyword. https://en.cppreference.com/w/c/error/static_assert

static_assert(2 + 2 == 4, "2+2 isn't 4"); // OK
static_assert(2 + 2 == 5, "2+2 isn't 4"); // Compile-time error

If for some reason you are forced to use old C standards, check this article for inspiration: https://www.pixelbeat.org/programming/gcc/static_assert.html

gresolio
  • 966
  • 10
  • 16
-1

I found this to give the least confusing error message for GCC. Everything else had some suffix about a negative size or some other confusing thing:

#define STATIC_ASSERT(expr, msg)   \
typedef char ______Assertion_Failed_____##msg[1];  __unused \
typedef char ______Assertion_Failed_____##msg[(expr)?1:2] __unused

example usage:

 unsigned char testvar;
 STATIC_ASSERT(sizeof(testvar) >= 8, testvar_is_too_small);

And the error message in gcc (ARM/GNU C Compiler : 6.3.1):

conflicting types for '______Assertion_Failed_____testvar_is_too_small'
Joe
  • 625
  • 5
  • 3
-2

Well, you could use the static asserts in the boost library.

What I believe they do there, is to define an array.

 #define MY_COMPILER_ASSERT(EXPRESSION) char x[(EXPRESSION)];

If EXPRESSION is true, it defines char x[1];, which is OK. If false, it defines char x[0]; which is illegal.

James Curran
  • 101,701
  • 37
  • 181
  • 258
  • But in C89 that would break compiling no matter what since variables can only be declared at the top of the scope. Maybe wrapping it in curly brackets might fix it? Also, zero-sized arrays are only forbidden in ISO C I think. You would also get a slew of warnings about multiple declarations and unused variables if not in its own scope. – dreamlax Apr 30 '09 at 15:22
  • 1
    `#define MY_COMPILER_ASSERT(EXPRESSION) do {char x[(EXPRESSION)?1:-1];} while (0)` would be better: inside braces, so declaration is legal and scoped, sizes 1 and -1 are clearly valid/invalid on all C versions, and forces you to `MY_COMPILER_ASSERT(...);` with a trailing semicolon, for visual consistency with all other function-like things in C. – ephemient Apr 30 '09 at 15:48
  • Declaring a typedef is preferable to delaring an unused variable. An unused typedef is harmless, but an unused variable may itself generate a warning on many compilers. – Stephen C. Steel Apr 30 '09 at 16:00