xazax.hun marked 13 inline comments as done.
xazax.hun added inline comments.
================
Comment at: clang-tidy/readability/MisleadingIndentationCheck.cpp:20
+
+void MisleadingIndentationCheck::danglingElseCheck(
+ const MatchFinder::MatchResult &Result) {
----------------
danielmarjamaki wrote:
> There is no handling of tabs and spaces by danglingElseCheck as far as I see.
>
> The "if" might for example be indented with spaces. And then the
> corresponding "else" is indented with a tab. Then I guess there is false
> positive.
>
> If the "if" is indented with 2 spaces and the "else" is indented with 2 tabs.
> then I assume there is false negative.
>
> It's unfortunate. Is it ok?
Documented this limitation.
https://reviews.llvm.org/D19586
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
