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

Cache IFrameStableClock in Player for easier access #22273

Merged
merged 3 commits into from
Jan 18, 2023

Conversation

peppy
Copy link
Member

@peppy peppy commented Jan 18, 2023

Allows directly referencing rather than going through DrawableRuleset. Helps with testing and implementation of the new song progress display (#22144).

Allows directly referencing rather than going through `DrawableRuleset`.
Helps with testing and implementation of the new song progress display
(ppy#22144).
@frenzibyte
Copy link
Member

The fact that the frame stable clock is cached from player as an IFrameStableClock but also caches itself inside FrameStabilityContainer as an IGameplayClock as well feels quite awkward, but don't think that's worth blocking this PR for, given the testing benefit it comes with.

@frenzibyte frenzibyte enabled auto-merge January 18, 2023 11:16
@peppy peppy disabled auto-merge January 18, 2023 13:31
@peppy peppy merged commit 0f3a586 into ppy:master Jan 18, 2023
@peppy peppy deleted the frame-stable-cache branch January 23, 2023 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants