[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