16

On my implementation of C++ (Visual Studio 2008 implementation) I see the following line in <cassert>

#ifdef  NDEBUG
#define assert(_Expression) ((void)0)

I do not understand the need to cast 0 to void. It seems to me that

#ifdef  NDEBUG
#define assert(_Expression) (0)

or even simply

#ifdef  NDEBUG
#define assert(_Expression) 0

would do, considering the contexts in which assert(expr) can be used.

So, what's the danger of 0 of type int instead of 0 of type void in this case? Any realistic examples?

1
  • As an aside, while this might seem like a trivial question to many, this is a topic that I've actually had less experienced programmers ask about occasionally. It's similar to (but not a duplicate of) questions about why macros are often wrapped in do ... while(0) constructs. Sometimes one person's 'too obvious to be bothered with' is another person's 'WTF?!?'. Commented Aug 11, 2011 at 16:14

1 Answer 1

24

The only purpose of the complicated expression (void)0 is to avoid compiler warnings. If you just had a naked, useless expression, the compiler might warn about an expression that has no effect. But by explicitly casting something to void you indicate that you mean to do this.

(Think about how confusing it would be to the user if the compiler suddenly said, "Warning: expression 0; has no effect.", when all you've done is switched to release mode.)

This was also common practice in C, where you'd say (void)printf("Hello"); to tell the compiler that you intentionally chose to ignore the return value of the function.

The (void) cast is not merely a choice by a particular implementation; it's required by the C standard. Quoting the 2011 ISO C standard (similar wording appears in the 1990 and 1999 editions):

If NDEBUG is defined as a macro name at the point in the source file where <assert.h> is included, the assert macro is defined simply as

#define assert(ignore) ((void)0)

The C++ standard requires the contents of the <cassert> header to be the same as the Standard C <assert.h> header.

11
  • 11
    There is another reason to do this in the context of assert: it ensures that "foo = assert(blurf);" provokes a compile-time error in both debug and release modes.
    – zwol
    Commented Aug 9, 2011 at 20:50
  • 1
    Curiosum: In GCC in C++ mode, we have static_cast<void>. Tidy.
    – Kerrek SB
    Commented Aug 9, 2011 at 20:53
  • It's correct that it generates no code and also suppresses warning, but the real question is why not just #define assert(ignore). My only guess is that some compilers don't like empty statements. Commented Aug 9, 2011 at 20:54
  • @Gene: Same reason as what Zack said - it prevents misuse of the assert statement, and it avoids warnings about naked semicolons.
    – Kerrek SB
    Commented Aug 9, 2011 at 20:55
  • 4
    Don't forget that assert() is required to expand "to a void expression" by the standard. I'm sure the requirement is in the standard for reasons that align with many of the comments here. But as far as a particular implementation is concerned, that's all the rationale that's necessary. Commented Aug 9, 2011 at 21:19

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.