hokein added inline comments.

================
Comment at: clangd/index/SymbolCollector.cpp:89
+    // violations.
+    if (ND->isInAnonymousNamespace())
       return true;
----------------
ilya-biryukov wrote:
> Why don't we include symbols from anonymous namespaces too?
> They are very similar to static symbols.
Yeah, these symbols need a special handling (the qualified name is like 
`foo::<anonymous>::bar`), we don't support them well enough. I think it is fine 
to ignore them at the moment.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D41759



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

Reply via email to