Communities

Writing
Writing
Codidact Meta
Codidact Meta
The Great Outdoors
The Great Outdoors
Photography & Video
Photography & Video
Scientific Speculation
Scientific Speculation
Cooking
Cooking
Electrical Engineering
Electrical Engineering
Judaism
Judaism
Languages & Linguistics
Languages & Linguistics
Software Development
Software Development
Mathematics
Mathematics
Christianity
Christianity
Code Golf
Code Golf
Music
Music
Physics
Physics
Linux Systems
Linux Systems
Power Users
Power Users
Tabletop RPGs
Tabletop RPGs
Community Proposals
Community Proposals
tag:snake search within a tag
answers:0 unanswered questions
user:xxxx search by author id
score:0.5 posts with 0.5+ score
"snake oil" exact phrase
votes:4 posts with 4+ votes
created:<1w created < 1 week ago
post_type:xxxx type of post
Search help
Notifications
Mark all as read See all your notifications »
Q&A

Welcome to Software Development on Codidact!

Will you help us build our independent community of developers helping developers? We're small and trying to grow. We welcome questions about all aspects of software development, from design to code to QA and more. Got questions? Got answers? Got code you'd like someone to review? Please join us.

Comments on What is do { } while(0) in macros and should we use it?

Parent

What is do { } while(0) in macros and should we use it?

+9
−1

Background

I can see the need to use { } when implementing a function-like macro such as this one:

#define HCF(code) fprintf(stderr, "halt and catch fire"); exit(code);

Because if we use the following calling code

bool bad = false;
if(bad)
  HCF(0xDEADBEEF);
printf("good");

Then it will expand to

bool bad = false;
if(bad)
  fprintf(stderr, "halt and catch fire"); 
exit(0xDEADBEEF);
printf("good");

which was not the intention - it will now exit with an error code even though the program is working as intended.

So the correct way would be to write the macro like this instead:

#define HCF(code) { fprintf(stderr, "halt and catch fire"); exit(code); }

Question

But I've come across macros that instead of just plain braces { } uses this:

#define HCF(code) do { fprintf(stderr, "halt and catch fire"); \
                       exit(code); } while(0)

This seems similar to { } but more obscure. What's the purpose of this do { } while(0) and why should it be used instead of { }?

History
Why does this post require attention from curators or moderators?
You might want to add some details to your flag.
Why should this post be closed?

0 comment threads

Post
+5
−0

Reasons to use the construct #define FOO(x) do{...} while(0);:

  1. As mentioned above, doing so solves the problem of

    if (...) FOO(y);
    
  2. You can declare variables inside the block, and they harmlessly go out of scope at the end of the block. Without the do{} block, the same symbol would cause a warning (or worse) if it were declared elsewhere in the block in which FOO(x) gets expanded.

    Why declare variables inside a macro? In case a macro argument appears more than once in the body, and someone writes FOO(func(y)), where func has side effects. This includes FOO(y++) etc. After the macro expands, the side effect would happen as many times as the macro argument appears in the body!

    Declaring a variable inside a do {} block prevents this, as follows:

    #define FOO(x) do { long xx = x; func2(xx); func3(xx); } while (0);
    

    Obviously there is still an assumption about type of x, but that is the case anyway if you are going to be calling func2(x) etc.

  3. Without the do{} block, there might be ambiguity as to whether FOO(x) expands into a statement or an expression. do{} makes it clearly a statement, which can help catch someone trying to use FOO(x) where an expression is expected. See also: "statement-expressions", aka ({...}), which is a construct in case you want the block to be an expression rather than a statement.

History
Why does this post require attention from curators or moderators?
You might want to add some details to your flag.

1 comment thread

General comments (2 comments)
General comments
Lundin‭ wrote almost 4 years ago

The question was why to pick do { } while(0) over { }. Your examples work just as fine with { }.

Pete W‭ wrote almost 4 years ago

Fair enough, I missed that.

I learned it from looking at the way headers in some system macros were written, and have been doing it ever since. I also got into the habit of using do{} to call attention to there being a block, but this comes from having done a bunch of Perl, where curlies have multiple meanings.

One final possible reason:

#define FOO(x) {...}

If you write (FOO(y)) , it will expand into a statement-expression rather than a statement. That's getting a bit contrived, I admit.