Note
Access to this page requires authorization. You can try signing in or changing directories.
Access to this page requires authorization. You can try changing directories.
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