Share via

Spot the bug - control flow macro

  • Article

This blog post has moved to https://matthewvaneerde.wordpress.com/2011/04/05/spot-the-bug-control-flow-macro/

Comments

  • Anonymous
    April 05, 2011
    Ugh. if (foo_supported() ? S_OK : (E_FOO_NOT_SUPPORTED != hr)) ...

  • Anonymous
    April 05, 2011
    As always, missing parentheses.  I think this should become a preprocessor warning.

  • Anonymous
    April 13, 2011
    How did the code review tool catch it?  What criterion did it invoke?

  • Anonymous
    April 13, 2011
    The comment has been removed