[PATCH] [Commented On] D25024: [clang-tidy] Add check for detecting declarations with multiple names

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 30 03:17:05 PDT 2016


omtcyfz added inline comments.


> omtcyfz wrote in OneNamePerDeclarationCheck.cpp:30
> Sure it can. I just thought `declCountIsGreaterThan` might be useful at some point.

Actually, for the sake of simplicity I'll use `unless(declCountIs(1))`, when some other part of code would need similar stuff we'd add it. For now it's only one use case, so I should be fine.

Thank you very much for the suggestion!

> Eugene.Zelenko wrote in ReleaseNotes.rst:62
> Please add one sentence description. Probably //Checks for declarations with multiple names.// should be enough.

Aw, yes, totally forgot about that. Thank you very much!

https://reviews.llvm.org/D25024





More information about the cfe-commits mailing list