[PATCH] D146842: [clang-tidy] Fix crash when handling invalid config values

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Apr 15 05:21:27 PDT 2023


njames93 requested changes to this revision.
njames93 added a comment.
This revision now requires changes to proceed.

Ok, taking a step back here, this patch is trying to address 2 unrelated things at once:

- using `--dump-config` with bad check options results in a crash
- using `--verify-config` won't actually verify that the checks are able to parse values from their respective `CheckOptions`

I think this should be split up into 2 separate patches, one for each of the proposed changes and each can be reviewed on their own.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146842



More information about the cfe-commits mailing list