[PATCH] D49890: Clang-Tidy Export Problem

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 26 18:40:17 PDT 2018


Eugene.Zelenko added inline comments.


================
Comment at: ClangTidy.cpp:612
+
+  FileManager *Files = new FileManager(FileSystemOptions());
+  vfs::FileSystem &FileSystem = *Files->getVirtualFileSystem();
----------------
You could use auto here, because type is in new statement.


================
Comment at: ClangTidy.cpp:614
+  vfs::FileSystem &FileSystem = *Files->getVirtualFileSystem();
+  auto InitialWorkingDir = FileSystem.getCurrentWorkingDirectory();
+  if (!InitialWorkingDir)
----------------
Type is not obvious, so please don't use auto.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D49890





More information about the cfe-commits mailing list