rdwampler added a comment. Thanks! Can you update the documentation too?
================ Comment at: lib/Tooling/Inclusions/HeaderIncludes.cpp:211 + Ret = 0; + } + return Ret; ---------------- I think you can drop the else block and just return immediately from the for loop. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64695/new/ https://reviews.llvm.org/D64695 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits