[PATCH] D139919: use std::optional in ClangTidyCheck

Yuao Ma via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 13 02:49:32 PST 2022


addr2line added a comment.

In D139919#3991407 <https://reviews.llvm.org/D139919#3991407>, @carlosgalvezp wrote:

> Sounds like the change to the YAML parser can potentially affect many other components other than clang-tidy - should that be done in a separate patch?

Only these left.

  ./clang-tools-extra/clang-tidy/ClangTidyCheck.cpp
  95:  if (std::optional<bool> Parsed = llvm::yaml::parseBool(Value))
  
  ./clang-tools-extra/clangd/ConfigYAML.cpp
  365:      if (auto Bool = llvm::yaml::parseBool(**Scalar))
  
  ./clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp
  427:  llvm::Optional<bool> Parsed = llvm::yaml::parseBool(Iter->getValue());


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

https://reviews.llvm.org/D139919



More information about the cfe-commits mailing list