Eugene.Zelenko added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/zircon/FblMoveCheck.cpp:61 + // Add in the <utility> header. + if (auto IncludeFixit = + Inserter->CreateIncludeInsertion(SM.getFileID(Start), "utility", ---------------- Please don't use auto when type could not be deduced from statement. ================ Comment at: clang-tools-extra/docs/ReleaseNotes.rst:188 + + This check is part of the migration checks for moving Zircon user code to use + the C++ standard library. It suggests converting uses of ``fbl::move`` to ---------------- I think first statement is not necessary. Second should start from //Suggests//. Will be good idea to synchronize it with documentation. https://reviews.llvm.org/D54168 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits