[PATCH] D36149: [Tooling] Add LLVM_NODISCARD to Replacements::merge

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 1 07:59:36 PDT 2017


This revision was automatically updated to reflect the committed changes.
Closed by commit rL309689: [Tooling] Add LLVM_NODISCARD to Replacements::merge (authored by krasimir).

Repository:
  rL LLVM

https://reviews.llvm.org/D36149

Files:
  cfe/trunk/include/clang/Tooling/Core/Replacement.h


Index: cfe/trunk/include/clang/Tooling/Core/Replacement.h
===================================================================
--- cfe/trunk/include/clang/Tooling/Core/Replacement.h
+++ cfe/trunk/include/clang/Tooling/Core/Replacement.h
@@ -255,7 +255,7 @@
 
   /// \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;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36149.109122.patch
Type: text/x-patch
Size: 636 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170801/3ff3f0d5/attachment.bin>


More information about the cfe-commits mailing list