-
Notifications
You must be signed in to change notification settings - Fork 1.7k
PY: deprecate a bunch of unused code #10317
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@@ -347,7 +351,9 @@ | |||
* Holds if `bit` is a binary expression node with a bitwise operator. | |||
* Helper for `this_binary_expr_points_to`. | |||
*/ | |||
predicate bitwise_expression_node(BinaryExprNode bit, ControlFlowNode left, ControlFlowNode right) { | |||
deprecated predicate bitwise_expression_node( |
Check warning
Code scanning / CodeQL
Missing QLDoc for parameter
466656b
to
ebb0aa3
Compare
ebb0aa3
to
283c711
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good to me. I'm debating whether this needs a change note or not. Some of these predicates are in principle visible externally, though I wouldn't expect anyone to actually depend on them.
I've added a change-note. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Thank you! ❤️
I'm looking through some of the results of this yet-to-be-merged QL-for-QL query: #8454
I know the Python teams wants to deprecate the points-to code, so I tried to see if I could deprecate some unused code in Python.
No query/test depend on any of the code I've deprecated.
There was some more code I initially tried to deprecate. However, it was being used in internal queries (that QL-for-QL can't see).