C++: Update @kind for metric queries#522
Conversation
|
I have no idea what difference this might make to PI, TI, and LGTM. Are there corresponding PRs for other languages? Who is competent to review this? |
|
Depending on how this PR went, I was planning to make further PRs for other languages. As far as I know, this is behavior preserving for all products but @aibaars and @adityasharad would be best placed to comment on the safety of this change. |
|
With Arthur's changes to the toolchain, this is entirely behaviour-preserving for 1.19 onwards, but 1.18 toolchains will not recognise Another concern I raised on the internal PRs is that the metric property tags still involve |
|
On balance it sounds as if I should close this PR and raise a couple of Jira issues for consideration in a future release. |
@Semmle/cpp-analysis this PR updates the metric queries to use the alternative
@kind metricthat was introduced by @aibaars in #28231 and #28480. I omitted any deprecated queries.This seems as if it would be worth doing for 1.19, but if there may be wider implications and you plan to do this later feel free to close the PR.