[PATCH] D50580: [clang-tidy] Abseil: no namespace check

[Deanna Garcia via Phabricator via cfe-commits]

deannagarcia created this revision.
deannagarcia added reviewers: alexfh, hokein.
deannagarcia added a project: clang-tools-extra.
Herald added subscribers: xazax.hun, mgorny.

This check ensures that users of Abseil do not open namespace absl in their code, as that violates our compatibility guidelines.

We are aware that this test will cause warnings on users code through their dependencies on abseil. However, from what we know it seems like these warnings are normally suppressed. If anyone has a good idea on how to avoid this/has insight on whether this will be a problem for the average user, please let me know!


https://reviews.llvm.org/D50580

Files:
  clang-tidy/abseil/AbseilTidyModule.cpp
  clang-tidy/abseil/CMakeLists.txt
  clang-tidy/abseil/NoNamespaceCheck.cpp
  clang-tidy/abseil/NoNamespaceCheck.h
  docs/ReleaseNotes.rst
  docs/clang-tidy/checks/list.rst
  test/clang-tidy/abseil-no-namespace.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50580.160173.patch
Type: text/x-patch
Size: 5527 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180810/b7472be5/attachment.bin>