[PATCH] D111041: [clang-tidy] Remove 'IgnoreDestructors = true' from cppcoreguidelines-explicit-virtual-functions

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 12 11:09:45 PDT 2021


carlosgalvezp added a comment.

> Eh, I personally don't care all that much one way or the other (so sure, I can do that!), but I recall there being a push to keep the "title" line under something very small (50 chars?) because of the way tools sometimes display this information to users, so I've always stripped the bit within [] to ensure we keep a sufficiently short title.

Yeah I understand, it makes sense. I just find that in general that's not respected so I wonder if people choose consistency in using [] over short commit messages. I can also find this in the Docs:

> When the changes are restricted to a specific part of the code (e.g. a back-end or optimization pass), it is customary to add a tag to the beginning of the line in square brackets. For example, “[SCEV] …” or “[OpenMP] …”. This helps email filters and searches for post-commit reviews.

Anyhow, minor detail :)

> That said, I'm wondering if you're planning to stick around in the clang-tidy community? If so, given that you've got a few good patches accepted already, it might be time to consider getting you commit privileges of your own. https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access has more details on what that entails.

That'd be great, thanks for the support! Since I'm mostly working on this on my spare time I can't promise much involvement, but rather occasional bug fixes and minor improvements. Still would be good to lift the commit burden from you haha. I really enjoy the repro structure, build system and processes, it's a very nice codebase to work with. If time allows I could potentially look into larger pieces of work, like adding new clang-tidy modules (e.g. Misra/Autosar checks). I find some local forks here and there that have done it but never pushed upstream, which I find a bit sad.


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

https://reviews.llvm.org/D111041



More information about the cfe-commits mailing list