[PATCH] D117409: [clang-tidy] Move IncludeInserter into ClangTidyContext

[Nathan James via Phabricator via cfe-commits]

njames93 marked an inline comment as done.
njames93 added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/IncludeInserter.h:20
 namespace tidy {
-namespace utils {
 
----------------
LegalizeAdulthood wrote:
> njames93 wrote:
> > LegalizeAdulthood wrote:
> > > What's the guidance on what belongs in `clang::tidy`
> > > and what belongs in `clang::tidy::utils`?
> > Well as the file moved out of utils, its no longer in the utils namespace.
> Yes, if you move the file out of utils then it shouldn't be in the
> utils namespace.  But why move the file in the first place?
> 
> IOW, what is the guideline for what files belong under `utils`
> and what files belong under `clang-tidy`?
Utils is for utilities that checks can make use of, but aren't required for the base clang-tidy infra to work.
By having the Inserter as a dependancy for ClangTidyDiagnosticConsumer it is now required for clang-tidy infrastructure, hence it was moved out of utils.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117409