Tyker marked an inline comment as done. Tyker added inline comments.
================ Comment at: clang/lib/Parse/ParseStmt.cpp:1376 + + MIChecker.Check(ElseStmt.isUsable()); ---------------- xbolva00 wrote: > What is wrong with code you used some rev ago? > > if (usable) check(); > > Now you uselessly instantiate MIChecker since if Usable = false, check is not > called. > > I like the older code more... > > If (usable) checkForMisleadingIndention(...) > > Was good and accepted. previous patch gave up on else if because we can't know wether there are braces. this revision can produce correct diagnostics on else if CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70638/new/ https://reviews.llvm.org/D70638 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
