VitaNuo added a comment.

Thanks!



================
Comment at: 
clang-tools-extra/clangd/unittests/tweaks/OrganizeImportsTests.cpp:35-41
+               {
+                   R"cpp(
+#include "Te^stTU.h"
+)cpp",
+                   true,
+                   {}},
+               {"void foo(^) {}", false, {}}};
----------------
kadircet wrote:
> nit: you can use EXPECT_AVAILABLE and EXPECT_UNAVAILABLE directly for these 
> two cases
I think it's the other way around. I can use the macros directly when I don't 
need to pass specific requested action kinds, which is the last (third) case. 
So it's probably not worth it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153769/new/

https://reviews.llvm.org/D153769

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D153769: [clang... Viktoriia Bakalova via Phabricator via cfe-commits
    • [PATCH] D153769: [... Viktoriia Bakalova via Phabricator via cfe-commits

Reply via email to