[PATCH] D62736: [clang-tidy] Fix make-unique check to work in C++17 mode.

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 31 16:13:27 PDT 2019


Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/modernize/MakeSmartPtrCheck.cpp:306
+        if (CEArg->isElidable()) {
+          if (const auto *TempExp = CEArg->getArg(0)) {
+            if (const auto *UnwrappedCE =
----------------
gribozavr wrote:
> Eugene.Zelenko wrote:
> > gribozavr wrote:
> > > Eugene.Zelenko wrote:
> > > > gribozavr wrote:
> > > > > Eugene.Zelenko wrote:
> > > > > > Return type is not mentioned explicitly, so auto should not be used.
> > > > > An explicit type is not needed for readability here.  The rule is to use auto when it improves readability, not when the type is not spelled in immediate vicinity.
> > > > I think it's reasonable to follow modernize-use-auto.
> > > modernize-use-auto is only a heuristic.
> > But set of processed situations are very reasonable.
> In abstract it might sound reasonable.  In practice it is still a heuristic and not a law.
I think it's reasonable to keep in memory that not everybody keeps functions/methods' return types in memory.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62736





More information about the cfe-commits mailing list