[PATCH] Improved checking for repeated macro argument with side effects when macro contains ?:

[Daniel Marjamäki]

I have refactored my patch.

- I added more comments.
- now the handling of __builtin_constant_p is more explicit.
- I changed uppercase=>lowercase in two functionnames in accordance with the llvm style guide.

I have rerun the patch on various projects to make sure that the more explicit handling of builtins don't cause false positives.


http://reviews.llvm.org/D10653

Files:
  clang-tidy/misc/MacroRepeatedSideEffectsCheck.cpp
  test/clang-tidy/misc-repeated-side-effects-in-macro.c

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10653.28860.patch
Type: text/x-patch
Size: 7318 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150701/2bd9d2f5/attachment.bin>