[PATCH] D33354: [clang-tidy] readability-braces-around-statements false positive with char literals

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 19 08:35:46 PDT 2017


alexfh added a comment.

I'll repeat my comment from https://reviews.llvm.org/D25558: "I'm thoroughly confused as to why the code in your test was not handled correctly and why this is the right fix. Can you explain?" The "For some reason ..." part doesn't really explain anything. I guess, we're papering over a more generic problem, and someone has to figure out what it is and how to fix it properly.


https://reviews.llvm.org/D33354





More information about the cfe-commits mailing list