[PATCH] D80514: [clang-tidy] modernize-use-trailing-return-type support for C++20 concepts and decltype

Bernhard Manfred Gruber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 25 09:05:46 PDT 2020


bernhardmgruber added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/modernize/UseTrailingReturnTypeCheck.cpp:399
 
-  return true;
+  return;
 }
----------------
Eugene.Zelenko wrote:
> Non needed. See readability-redundant-control-flow.
Thx!


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-use-trailing-return-type.rst:22
+  virtual float f3() const && = delete;    virtual auto f3() const && -> float = delete;
+======================================== ===============================================
 
----------------
Eugene.Zelenko wrote:
> bernhardmgruber wrote:
> > I tried 2 online rst editors and they failed to format the code blocks inside the tables. Will this work with the clang documentation?
> In other documentation such examples are sequential. Same below.
Ok! I will separate the code blocks again.


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

https://reviews.llvm.org/D80514





More information about the cfe-commits mailing list