[PATCH] D63324: [clang-tidy] Replace memcpy by std::copy
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 14 06:35:12 PDT 2019
lebedev.ri added a comment.
Missing tests.
================
Comment at: clang-tools-extra/clang-tidy/modernize/ReplaceMemcpyByStdCopy.cpp:120
+} // namespace clang
\ No newline at end of file
----------------
please add all the missing newlines
================
Comment at: clang-tools-extra/clang-tidy/modernize/ReplaceMemcpyByStdCopy.h:1-2
+//===--- ReplaceMemcpyByStdCopy.h - clang-tidy--------------------------*- C++
+//-*-===//
+//
----------------
should be something like
```
//===--- ReplaceMemcpyByStdCopy.h - clang-tidy -----------------*- C++-*-===//
```
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63324/new/
https://reviews.llvm.org/D63324
More information about the cfe-commits
mailing list