Eugene.Zelenko added inline comments.
================ Comment at: clang-tidy/abseil/WrapUniqueCheck.cpp:85 + + std::string Left = (ConsDecl) ? "auto " + NameRef.str() + " = " : ""; + std::string NewText = ---------------- I don't think that you need round brackets around ConsDecl. Same below. ================ Comment at: docs/clang-tidy/checks/abseil-wrap-unique.rst:5 +================== +Looks for instances of factory functions which uses a non-public constructor +that returns a ``std::unqiue_ptr<T>`` then recommends using ``absl::wrap_unique(new T(...))``. ---------------- Please separate with empty line. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57435/new/ https://reviews.llvm.org/D57435 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits