[PATCH] D83053: [clang-tidy] OptionsView::store specialized on bool

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 2 08:05:44 PDT 2020


njames93 marked an inline comment as done.
njames93 added inline comments.
Herald added a subscriber: wuzish.


================
Comment at: clang-tools-extra/clang-tidy/cppcoreguidelines/ProBoundsConstantArrayIndexCheck.cpp:31
   Options.store(Opts, "GslHeader", GslHeader);
-  Options.store(Opts, "IncludeStyle", IncludeStyle);
+  Options.store(Opts, "IncludeStyle", IncludeStyle,
+                utils::IncludeSorter::getMapping());
----------------
Had to force this to explicitly call the correct function to avoid a compile error


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83053





More information about the cfe-commits mailing list