Fix false positive on concept declaration#288
Fix false positive on concept declaration#288aaronliu0130 merged 3 commits intocpplint:developfrom armandas:develop
Conversation
aaronliu0130
left a comment
There was a problem hiding this comment.
This is a massive step up, though there is the very rare case of an identifier being named "requires" and the user linting a pre-C++20 file. It would be great if we could work out a way to also require the "concept" keyword, but I would be happy to merge this as-is if you can't figure it out.
|
If I understand correctly, the only issue with someone using One way to go about fixing that is to go through the preceding lines and look for |
|
Yeah, that's what I meant. Like I said, it's probably not really an issue if we don't see a good way.. |
My attempt to fix #225, since #229 seems to be dead.
I used a
requireskeyword for concept detection, as theconceptkeyword could be a few lines above the opening brace.Not 100% on the regex, so if someone could check it, I would appreciate it.