[PATCH] D27054: Introducing clang::tooling::AtomicChange for refactoring tools.

Alexander Shaposhnikov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 28 05:18:20 PST 2017


alexshap added inline comments.


================
Comment at: include/clang/Tooling/Refactoring/AtomicChange.h:59
+  /// \brief Returns the path of the file containing this atomic change.
+  std::string getFilePath() const { return FilePath; }
+
----------------
i assume i might be missing smth - why here and above (in getKey, getFilePath, getError) std::string is returned by value  ?


https://reviews.llvm.org/D27054





More information about the cfe-commits mailing list