-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Rust: Improve rust/ctor-initialization #18176
base: main
Are you sure you want to change the base?
Conversation
@@ -123,10 +123,10 @@ unsafe fn harmless2_11() { | |||
// --- transitive cases --- | |||
|
|||
fn call_target3_1() { | |||
_ = stderr().write_all(b"Hello, world!"); // $ MISSING: Alert=source3_1 Alert=source3_3 Alert=source3_4 | |||
_ = stderr().write_all(b"Hello, world!"); // $ Alert=source3_1 Alert=source3_3 MISSING: Alert=source3_4 |
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.
The source3_4
result is still missing because the call to bad3_3()
has no getStaticTarget()
.
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.
One last thing, otherwise LGTM. Also, remember to run DCA before merging.
Improve the
rust/ctor-initialization
query, adding so-called "transitive" results through calls where the target is in the database.