[Lldb-commits] [PATCH] D120718: Qualify DataExtractor with lldb_private

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Mar 1 13:21:06 PST 2022


labath added a comment.

In D120718#3352635 <https://reviews.llvm.org/D120718#3352635>, @shafik wrote:

> In D120718#3352450 <https://reviews.llvm.org/D120718#3352450>, @JDevlieghere wrote:
>
>> I think the better solution here is to get rid of the `using namespace llvm;` in the implementation file instead.
>
> That is a good point. I did a quick look at the file `DataFileCache.h` and none of the files it includes directly use `using namespace llvm;`.
>
> It looks like across llvm-project we have 54 headers that use `using namespace llvm;` I thought it was a lot more.

It looks like at least some of those are inside functions, which is kind of OK. But what's worse, some of those are not inside functions, which is  😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱 😱. using directives in headers completely mangle the symbol lookups in any file which includes them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120718



More information about the lldb-commits mailing list