[PATCH] D55433: [clang-tidy] Adding a new modernize use nodiscard checker

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 18 06:18:01 PST 2018


MyDeveloperDay added a comment.

In D55433#1334150 <https://reviews.llvm.org/D55433#1334150>, @curdeius wrote:

> LGTM.
>  Any ideas for improvement, @JonasToth?
>  I'd rather have it merged and improve on it later if there are ideas on how to do it better.


Thanks for the LGTM, as you know I don't have commit access, but I'm happy to wait for more people to review if there are concerns.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55433/new/

https://reviews.llvm.org/D55433





More information about the cfe-commits mailing list