[PATCH] D80531: [clang-tidy]: Added modernize-replace-disallow-copy-and-assign-macro

Konrad Kleine via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 26 05:13:00 PDT 2020


kkleine added a comment.

@njames93 I could do that but the original Macros had were defined without a semicolon at the end and one had to add it manually. See this revision in which I replaced some occurrences of `DISALLOW_COPY_AND_ASSIGN`: eaebcbc67926a18befaa297f1778edde63baec9b <https://reviews.llvm.org/rGeaebcbc67926a18befaa297f1778edde63baec9b>. What do you suggest? Keep the semicolon to more closely match the original macros or remove it to make the test happy?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80531/new/

https://reviews.llvm.org/D80531



More information about the cfe-commits mailing list