[PATCH] D45163: [Sema] -Wunused-value: diagnose unused std::move() call results.

Marshall Clow via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 2 11:07:21 PDT 2018


mclow.lists added a comment.

So.
https://bugs.llvm.org/show_bug.cgi?id=10011 was resolved by https://wg21.link/P0600 (which added `[[nodiscard]]` to `string.empty()`

We can do the same for `move`.
However, I have been promised a comprehensive paper listing all the (100s?) of places in the standard library which should be so marked, and I'm inclined to wait for it (but not for too long).
[ The next WG21 meeting is in June ]


Repository:
  rC Clang

https://reviews.llvm.org/D45163





More information about the cfe-commits mailing list