Eugene.Zelenko added inline comments.
Comment at: docs/clang-tidy/checks/misc-placement-new-target-size.rst:5
+==
+
+
Eugene.Zelenko wrote:
> Unnecessary empty line.
Somehow empty line is still there.
CHANGES SINCE LAST ACTION
http
DennisL updated this revision to Diff 193471.
DennisL added a comment.
Simplify logic
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60139/new/
https://reviews.llvm.org/D60139
Files:
clang-tidy/misc/CMakeLists.txt
clang-tidy/misc/MiscTidyModule.cpp
clang-tidy/misc/PlacementNewTarg
DennisL updated this revision to Diff 193454.
DennisL added a comment.
Remove debug output
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60139/new/
https://reviews.llvm.org/D60139
Files:
clang-tidy/misc/CMakeLists.txt
clang-tidy/misc/MiscTidyModule.cpp
clang-tidy/misc/PlacementNe
DennisL updated this revision to Diff 193449.
DennisL marked 13 inline comments as done.
DennisL added a comment.
Updated patch to address reviewer feedback
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60139/new/
https://reviews.llvm.org/D60139
Files:
clang-tidy/misc/CMakeLists.txt
Eugene.Zelenko added inline comments.
Comment at: clang-tidy/misc/PlacementNewTargetSizeCheck.cpp:10
+#include
+
+#include "PlacementNewTargetSizeCheck.h"
Unnecessary empty line. Please run Clang-format after fixing.
Comment at: clang-tidy/mis