[PATCH] D117593: [clang-tidy] Change google-explicit-constructor to ignore conversions and operators marked `explicit(false)`

Luka Govedič via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 12 13:00:14 PDT 2023


ProExpertProg added a comment.

Is there an update on this? I agree adding a flag to control this behavior would be good but the Google style guide clearly does not take the `explicit(false)` option into consideration. Currently the only alternatives are a disabling the check completely or adding a bunch of NOLINT statements everywhere, neither of which is great.

Perhaps an alternative would be to add a separate check, either called `explicit-constructor` or `google-explicit-constructor-permissive` with this functionality.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117593



More information about the cfe-commits mailing list