open [13139] fs: Implement IProgramRegistry and add various data types #13139

Open
Accord wants to merge 11 commits from __refs_pull_13139_head into master
Owner

Branch: __refs_pull_13139_head

FearlessTobi opened issue at 2024-02-23 18:55:

The next part in the quest to improve filesystem accuracy.
This time, even more data types, utility classes, etc.
With all that work done, I also took the time to implement the fsp_pr service.
There might be some memory issues in this code, therefore please do not tag this yet.
Current To-Dos:

  • Fix MSVC compilation
  • Properly credit LibHac
  • Fix random crashes when launching games
  • Remove globals
  • Deal with ProgramRegistry
  • Fix remaining review comments

[Export of Github issue for yuzu-emu/yuzu. Generated on 2024.02.28 at 10:04:32.]

**Branch:** [__refs_pull_13139_head](https://accord.dravee.dev/Accord/yuzu-mirror/src/branch/__refs_pull_13139_head) #### <div><img width="50" height="50" src="https://avatars.githubusercontent.com/u/20753089?u=4c1e2e5b0b4b5b40b1004b7fe0948d56f0adaf50&v=4" width="50"></div>[FearlessTobi](https://github.com/FearlessTobi) opened issue at [2024-02-23 18:55](https://github.com/yuzu-emu/yuzu/pull/13139): > The next part in the quest to improve filesystem accuracy. > This time, even more data types, utility classes, etc. > With all that work done, I also took the time to implement the fsp_pr service. > There might be some memory issues in this code, therefore please do not tag this yet. > Current To-Dos: > - [x] Fix MSVC compilation > - [x] Properly credit LibHac > - [x] Fix random crashes when launching games > - [x] Remove globals > - [x] Deal with ProgramRegistry > - [x] Fix remaining review comments > ------------------------------------------------------------------------------- > [Export of Github issue for [yuzu-emu/yuzu](https://github.com/yuzu-emu/yuzu). Generated on 2024.02.28 at 10:04:32.]
The information for this has been taken from Switchbrew.
Needed for fssrv_program_info.
These will be needed to reimplement the filesystem at a later point.
fs: Use the ProgramRegistry in the FileSystemController
Some checks failed
yuzu-ci / transifex (pull_request) Has been cancelled
yuzu-ci / reuse (pull_request) Has been cancelled
codespell / Check for spelling errors (pull_request) Has been cancelled
yuzu verify / verify format (pull_request) Has been cancelled
yuzu verify / test build (pull_request) Has been cancelled
yuzu verify / test build-1 (pull_request) Has been cancelled
yuzu verify / test build-2 (pull_request) Has been cancelled
yuzu verify / test build (macos) (pull_request) Has been cancelled
yuzu verify / test build (windows, msvc) (pull_request) Has been cancelled
yuzu verify / android (pull_request) Has been cancelled
c14cf18d9f
I am not really happy how this turned out, but I don't see a better way with the limitations imposed by our current filesystem code.
Some checks failed
yuzu-ci / transifex (pull_request) Has been cancelled
yuzu-ci / reuse (pull_request) Has been cancelled
codespell / Check for spelling errors (pull_request) Has been cancelled
yuzu verify / verify format (pull_request) Has been cancelled
yuzu verify / test build (pull_request) Has been cancelled
yuzu verify / test build-1 (pull_request) Has been cancelled
yuzu verify / test build-2 (pull_request) Has been cancelled
yuzu verify / test build (macos) (pull_request) Has been cancelled
yuzu verify / test build (windows, msvc) (pull_request) Has been cancelled
yuzu verify / android (pull_request) Has been cancelled
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin __refs_pull_13139_head:__refs_pull_13139_head
git switch __refs_pull_13139_head

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff __refs_pull_13139_head
git switch __refs_pull_13139_head
git rebase master
git switch master
git merge --ff-only __refs_pull_13139_head
git switch __refs_pull_13139_head
git rebase master
git switch master
git merge --no-ff __refs_pull_13139_head
git switch master
git merge --squash __refs_pull_13139_head
git switch master
git merge --ff-only __refs_pull_13139_head
git switch master
git merge __refs_pull_13139_head
git push origin master
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Accord/yuzu-mirror!13139
No description provided.