[PATCH] D101721: [clang-tidy][NFC] Update tests and Default options to use boolean value

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 3 10:58:52 PDT 2021


njames93 added a comment.

In D101721#2733279 <https://reviews.llvm.org/D101721#2733279>, @aaron.ballman wrote:

> That makes sense to me. Should we file a bug to suggest adding the deprecation warning in Clang 14(?) and planned removal in Clang 16(?) so that we don't lose track of this? (I have no firm opinion about which versions we decide to start deprecating and remove so long as they're not disruptive.)

I was thinking later tbh. clang-tidy 11 is where the spelling support was adopted and, more importantly, emitting a warning when a present key could not be parsed. Giving 3 years from there puts a deprecation notice in Clang17 (Assuming we stick to the same semiannual release cycle).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101721



More information about the cfe-commits mailing list