[PATCH] D73090: [clang-tidy] Fix PR#44528 'modernize-use-using and enums'

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 2 13:42:59 PST 2020


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM! I'm happy to commit on your behalf once the minor nits are fixed up.



================
Comment at: clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp:73
+  // But also we need to check that ranges belongs the same file because
+  // different files may contain overlap ranges.
   std::string Using = "using ";
----------------
that ranges -> that the ranges
belongs -> belong to
overlap -> overlapping


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73090





More information about the cfe-commits mailing list