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

Improve trophy context loader #10000

Merged
merged 2 commits into from May 21, 2021
Merged

Improve trophy context loader #10000

merged 2 commits into from May 21, 2021

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Mar 21, 2021

  • Add magic test in TROPUSR.
  • Optimize TROPUSR::Save.
  • Replace invalid TROPUSR.DAT with empty new ones. (+report an error)
  • Fix return code of Load whenever TROPUSR.DAT hadn't existed before.
  • sceNpTrophyRegisterContext will abort if TROPUSR.DAT generation failed.

@AllenR-A
Copy link

eyyy, 10K!

@AniLeo AniLeo requested a review from Megamouse April 29, 2021 15:57
@AniLeo
Copy link
Member

AniLeo commented Apr 29, 2021

@elad335 what is remaining for this one?

rpcs3/Emu/Cell/Modules/sceNpTrophy.cpp Outdated Show resolved Hide resolved
rpcs3/Loader/TROPUSR.cpp Outdated Show resolved Hide resolved
@elad335 elad335 marked this pull request as ready for review April 30, 2021 10:27
@elad335
Copy link
Contributor Author

elad335 commented Apr 30, 2021

Ready now.

* Replace invalid TROPUSR.DAT with empty new ones. (+report an error)
* Fix return code of Load whenever TROPUSR.DAT hadn't existed before.
* Add header magic test.
* sceNpTrophyRegisterContext will abort if TROPUSR.DAT generation failed.
@elad335 elad335 changed the title Improve trophy error checks Improve trophy context loader Apr 30, 2021
@AniLeo AniLeo requested a review from Megamouse April 30, 2021 15:55
@Nekotekina Nekotekina merged commit 3a2b9f8 into RPCS3:master May 21, 2021
@elad335 elad335 deleted the rap branch September 19, 2022 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants