Pajesz requested a review of this revision.
Pajesz added reviewers: dkrupp, xazax.hun, nlewycky, etienne.bergeron, etienneb.
Pajesz marked an inline comment as done.
Pajesz added a comment.
Hello!
Gonna fix the tests ASAP! Any other suggestions, fixes, improvements
considering the checker?
htt
Pajesz marked 12 inline comments as done.
Pajesz added a comment.
so i have to submit K good to know
Comment at: test/clang-tidy/readability-misleading-indentation.cpp:48
@@ +47,3 @@
+ }
+ foo2(); // ok
+
nlewycky wrote:
> This is arguably misleading ind
Pajesz created this revision.
Pajesz added a reviewer: alexfh.
Pajesz added subscribers: cfe-commits, xazax.hun.
This is a clang-tidy check for llvm. It checks for dangling else and for
possible misleading indentation due to omitted braces.
http://reviews.llvm.org/D19586
Files:
clang-tidy/r
Pajesz updated this revision to Diff 55247.
Pajesz added a comment.
Probably fixed the broken patch.
http://reviews.llvm.org/D19586
Files:
clang-tidy/readability/CMakeLists.txt
clang-tidy/readability/MisleadingIndentationCheck.cpp
clang-tidy/readability/MisleadingIndentationCheck.h
clan
Pajesz updated this revision to Diff 55389.
Pajesz marked 5 inline comments as done.
Pajesz added a comment.
Both dangling else and the other check now ignore one-line if-else statements.
Corrected other reviews as well.
http://reviews.llvm.org/D19586
Files:
clang-tidy/readability/CMakeLists
Pajesz updated this revision to Diff 61651.
Pajesz added a comment.
Checker now works with for and while statements as well, new tests were added,
other syntactical and logical updates have been made.
http://reviews.llvm.org/D19586
Files:
clang-tidy/readability/MisleadingIndentationCheck.h
Pajesz updated this revision to Diff 63925.
Pajesz added a comment.
Minor changes in tests and doc and diff should be full now.
http://reviews.llvm.org/D19586
Files:
clang-tidy/readability/CMakeLists.txt
clang-tidy/readability/MisleadingIndentationCheck.cpp
clang-tidy/readability/Misleadi
