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

Replace some unnamed tick constants with GetTicksPerSecond() #3458

Merged
merged 1 commit into from Aug 10, 2016

Conversation

JosJuice
Copy link
Member

@JosJuice JosJuice commented Jan 6, 2016

These operations should always take the same amount of time, not the same amount of ticks. The number of ticks per second is different for GameCube and Wii.

Review on Reviewable

@degasus
Copy link
Member

degasus commented Jan 6, 2016

This changes the behavior on the Wii a bit, but I doubt it matters.

@degasus
Copy link
Member

degasus commented Jan 24, 2016

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

@BhaaLseN
Copy link
Member

Reviewed 4 of 4 files at r1.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from the review on Reviewable.io

These operations should always take the same amount of time,
not the same amount of ticks. The number of ticks per second
is different for GameCube and Wii.
@degasus degasus merged commit 7914e4d into dolphin-emu:master Aug 10, 2016
@JosJuice JosJuice deleted the seconds-not-ticks branch August 10, 2016 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants