Charusso accepted this revision.
Charusso added a comment.
This revision is now accepted and ready to land.
Thanks you!
================
Comment at: clang/lib/StaticAnalyzer/Core/RegionStore.cpp:215
+ << "{ \"cluster\": \"" << I.getKey() << "\", \"pointer\": \""
+ << ((const void *)I.getKey()) << "\", \"items\": [" << NL;
----------------
People say extra parens are bad. So as they are the root of the evil, could you
remove it, please?
(I also like to emphasize same stuff packed into parens for better
understanding/readability, but we are the minority.)
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63362/new/
https://reviews.llvm.org/D63362
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
