[PATCH] D62437: [clang-tidy] Splits fuchsia-default-arguments
Eugene Zelenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 24 16:50:55 PDT 2019
Eugene.Zelenko added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/fuchsia/DefaultArgumentsCallsCheck.cpp:1
+//===--- DefaultArgumentsCallsCheck.cpp - clang-tidy-----------------------===//
+//
----------------
Please add space after clang-tidy.
================
Comment at: clang-tools-extra/clang-tidy/fuchsia/DefaultArgumentsCallsCheck.h:1
-//===--- DefaultArgumentsCheck.h - clang-tidy--------------------*- C++ -*-===//
+//===--- DefaultArgumentsCallsCheck.h - clang-tidy--------------------*- C++ -*-===//
//
----------------
Please narrow to 80 symbols and add space after clang-tidy.
================
Comment at: clang-tools-extra/clang-tidy/fuchsia/DefaultArgumentsDeclarationsCheck.h:1
+//===--- DefaultArgumentsDeclarationsCheck.h - clang-tidy ------------------*- C++ -*-===//
+//
----------------
Please narrow to 80 symbols.
================
Comment at: clang-tools-extra/clang-tidy/fuchsia/FuchsiaTidyModule.cpp:1
//===--- FuchsiaTidyModule.cpp - clang-tidy--------------------------------===//
//
----------------
Please add space after clang-tidy.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62437/new/
https://reviews.llvm.org/D62437
More information about the cfe-commits
mailing list