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.

Sign up for GitHub

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

Jump to bottom

threads: some cleanup #15599

Merged
merged 10 commits into from
May 18, 2024
Merged

threads: some cleanup #15599

merged 10 commits into from
May 18, 2024

Conversation

Megamouse
Copy link
Contributor

@Megamouse Megamouse commented May 16, 2024 •
edited
Loading

  • Replace magic number id_type with thread_class
  • Initialize some uninitialized rsx thread members
  • Fix spu_thread audit check
  • Add comments to reference old scheduler at least somewhere in the code

@Megamouse Megamouse added the Refactoring Refactors or simplifies existing code label May 16, 2024
elad335
elad335 requested changes May 16, 2024
View reviewed changes
rpcs3/Emu/CPU/CPUThread.h Outdated Show resolved Hide resolved
elad335
elad335 reviewed May 16, 2024
View reviewed changes
Utilities/Thread.h Outdated Show resolved Hide resolved
@Megamouse Megamouse requested a review from elad335 May 16, 2024 10:38
elad335
elad335 requested changes May 16, 2024
View reviewed changes
rpcs3/Emu/Io/GunCon3.cpp Outdated Show resolved Hide resolved
rpcs3/util/serialization_ext.cpp Show resolved Hide resolved
rpcs3/rpcs3qt/memory_viewer_panel.cpp Outdated Show resolved Hide resolved
@RPCS3 RPCS3 deleted a comment from Megamouse May 18, 2024
@Megamouse Megamouse merged commit 9a8ed80 into RPCS3:master May 18, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elad335 elad335 elad335 requested changes

Assignees
No one assigned
Labels
Refactoring Refactors or simplifies existing code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Megamouse @elad335