[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