Eugene.Zelenko added inline comments.
================ Comment at: test/clang-tidy/bugprone-branch-clone.cpp:4 +void test_basic1(int in, int &out) { + if (in > 77) +// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: if with identical then and else branches [bugprone-branch-clone] ---------------- donat.nagy wrote: > JonasToth wrote: > > donat.nagy wrote: > > > JonasToth wrote: > > > > could you please add tests for ternary operators? > > > Currently the check does not handle ternary operators, but I added some > > > checks reflecting this. > > Thank you. Could you please add a short note to the user-facing > > documentation that these are not handled. > In fact, I could easily extend the functionality of the checker to cover > ternary operators. Would it be an useful addition? Sure, it'll be great extension of functionality. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54757/new/ https://reviews.llvm.org/D54757 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits