[PATCH] D159436: [clang-tidy] Add support for optional parameters in config.

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 5 11:12:36 PDT 2023


carlosgalvezp added a comment.

Thanks for the patch! I must admit I don't fully understand what problem it solves, i.e. parameters are already optional today (one just simply doesn't specify them in the config file). Why would we want to explicitly spell out parameters with some default value that doesn't correspond to what actually gets applied in the check?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159436



More information about the cfe-commits mailing list