hokein added inline comments.

================
Comment at: clang-tools-extra/clangd/unittests/TweakTests.cpp:1556
+
+    [[void [[Bar::[[b^a^z]]]]() [[{
+      return;
----------------
Sorry for not spotting it earlier.

Moving out-of-line methods is different than general functions, `void 
Bar::bar();` is illegal in C++ (C++ requires the out-of-line member declaration 
must be a definition). I think for this case, we could 1) delete the original 
method definition 2) disable the tweak. I slightly prefer 2) as out-of-line 
member definitions are rare in header files. WDYT?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69266



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to