Skip to content
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

Fixes to precise_time_ns #10403

Closed
wants to merge 2 commits into from
Closed

Fixes to precise_time_ns #10403

wants to merge 2 commits into from

Conversation

poiru
Copy link
Contributor

@poiru poiru commented Nov 10, 2013

Closes #2675.

precise_time_ns

The QueryPerformance* functions take a LARGE_INTEGER, which is a signed
64-bit integer rather than an unsigned 64-bit integer. `ts.tv_sec`, too,
is a signed integer so `ns_per_s` has been changed to a int64_t.
@poiru
Copy link
Contributor Author

poiru commented Nov 11, 2013

@alexcrichton Can you r+ yet again? The last force push didn't seem to get here for some reason. Sorry for the annoyance!

bors added a commit that referenced this pull request Nov 11, 2013
@bors bors closed this Nov 11, 2013
@poiru poiru deleted the issue-2675 branch January 30, 2014 17:01
flip1995 pushed a commit to flip1995/rust that referenced this pull request Mar 10, 2023
Fix rust-lang#107877, etc.

Fix rust-lang#10009
Fix rust-lang#10387
Fix rust-lang#107877

The fix is to verify that the associated item's trait is implemented before trying to project the item's type.

r? `@Jarcho`

---

changelog: ICE: [`needless_borrow`]: No longer panics on ambiguous projections
[rust-lang#10403](rust-lang/rust-clippy#10403)
<!-- changelog_checked -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

QueryPerformanceFrequency failure mode not handled
3 participants