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

UI: Icon image cache #17599

Merged
merged 7 commits into from
Jun 18, 2023
Merged

UI: Icon image cache #17599

merged 7 commits into from
Jun 18, 2023

Conversation

hrydgard
Copy link
Owner

@hrydgard hrydgard commented Jun 18, 2023

Adds a simple small-image cache.

It can be stored to disk (single file, for performance on Android) and restored. Though this is not yet hooked up, and I'm not sure we want it.

Will be needed to manage all the little achievement icons for #17589 . No need to cache them on disk, but something has to own them.

This also uses the icon cache for the icons in the homebrew store, for testing the concept.

This will be needed for achievements, and has uses already now.
It doesn't try to insert or fetch missing things itself, re-fetching is up to the caller.

This will be required for handling the many achievement icons.

Saving/loading the cache to a single file on disk is implemented but not
hooked up yet. It works without it, though of course will have to
re-fetch things on the next startup.
@hrydgard hrydgard added the User Interface PPSSPP's own user interface / UX label Jun 18, 2023
@hrydgard hrydgard added this to the v1.16.0 milestone Jun 18, 2023
@hrydgard hrydgard marked this pull request as draft June 18, 2023 12:35
@hrydgard hrydgard marked this pull request as ready for review June 18, 2023 12:54
@hrydgard hrydgard merged commit 264c6d7 into master Jun 18, 2023
19 checks passed
@hrydgard hrydgard deleted the ui-icon-cache branch June 18, 2023 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
User Interface PPSSPP's own user interface / UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant