ilya-biryukov added inline comments.

================
Comment at: clang-tools-extra/clangd/refactor/tweaks/DefineInline.cpp:200
+    std::string Qualifier;
+    // FIXME: Also take using directives and namespace aliases inside function
+    // body into account.
----------------
kadircet wrote:
> ilya-biryukov wrote:
> > NIT: it looks like this FIXME belongs to the `getQualification` function 
> > itself.
> I don't think so, as qualification function has no idea about the function 
> body, right?
But it does handle using namespace directives and namespace aliases, so if we 
ever add this logic it better live in the same place, i.e. inside 
`getQualification` function or inside another overload handling that case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69033



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

Reply via email to