[PATCH] D30841: [clang-tidy] readability-misleading-indentation: fix chained if

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 11 13:43:34 PST 2017


xazax.hun accepted this revision.
xazax.hun added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D30841#698634, @fgross wrote:

> I just assumed it would traverse in the "right" way, is there any documentation about AST / matcher traversal?


I do not know of any, but I remember having some non-determinism across platforms, but I think matching parent before children is a safe assumption for all platforms.

The patch now looks good to me, but lets wait what @alexfh says.


https://reviews.llvm.org/D30841





More information about the cfe-commits mailing list