[Tooling] Add LLVM_NODISCARD to Replacements::merge

krasimirgg · krasimirgg · commit 43d8fd38be0f · 2017-08-01T14:58:14.000Z
Summary: This patch adds LLVM_NODISCARD to Replacements::merge. I've hit this several times already. Reviewers: ioeric Reviewed By: ioeric Subscribers: klimek, cfe-commits Differential Revision: https://reviews.llvm.org/D36149 llvm-svn: 309689
diff --git a/clang/include/clang/Tooling/Core/Replacement.h b/clang/include/clang/Tooling/Core/Replacement.h
@@ -255,7 +255,7 @@ class Replacements {
 
   /// \brief Merges \p Replaces into the current replacements. \p Replaces
   /// refers to code after applying the current replacements.
-  Replacements merge(const Replacements &Replaces) const;
+  LLVM_NODISCARD Replacements merge(const Replacements &Replaces) const;
 
   // Returns the affected ranges in the changed code.
   std::vector<Range> getAffectedRanges() const;
