shafik added inline comments.
================ Comment at: clang/unittests/libclang/LibclangTest.cpp:1232 ASSERT_TRUE(staticAssertCsr.has_value()); - size_t argCnt = 0; + int argCnt = 0; Traverse(*staticAssertCsr, [&argCnt](CXCursor cursor, CXCursor parent) { ---------------- Above you used `0u` but here you changed the type to `int`, is there a reason why not to use `0u` here as well? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156224/new/ https://reviews.llvm.org/D156224 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits