Skip to content

Sio0 rework/pocketstation command additions #995

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

Merged
merged 39 commits into from
Oct 9, 2022

Conversation

johnbaumann
Copy link
Collaborator

WIP. Refactored quite a lot of SIO code.

Also experimenting with state machine from OpenMC to attempt expanding memory card command set, to potentially allow for basic pocket station support.

@codecov
Copy link

codecov bot commented Aug 28, 2022

Codecov Report

Base: 17.08% // Head: 17.09% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (5244a19) compared to base (da16ded).
Patch coverage: 19.42% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #995      +/-   ##
==========================================
+ Coverage   17.08%   17.09%   +0.01%     
==========================================
  Files         345      348       +3     
  Lines      106856   106990     +134     
==========================================
+ Hits        18256    18293      +37     
- Misses      88600    88697      +97     
Impacted Files Coverage Δ
src/core/pad.cc 26.84% <0.00%> (-0.29%) ⬇️
src/core/pad.h 0.00% <0.00%> (ø)
src/core/psxemulator.h 100.00% <ø> (ø)
src/core/sstate.cc 0.00% <0.00%> (ø)
src/gui/widgets/memcard_manager.cc 3.39% <0.00%> (-0.34%) ⬇️
src/gui/widgets/memcard_manager.h 14.28% <0.00%> (ø)
src/core/sio.cc 6.45% <9.42%> (-9.61%) ⬇️
src/core/sio.h 10.63% <20.00%> (+10.63%) ⬆️
src/core/memorycard.cc 30.37% <30.37%> (ø)
src/core/memorycard.h 55.55% <55.55%> (ø)
... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@johnbaumann johnbaumann marked this pull request as ready for review September 17, 2022 23:22
@johnbaumann
Copy link
Collaborator Author

Bah, got side tracked. Save state settings need finalized, rest is ok for review.

@johnbaumann johnbaumann merged commit b6d0752 into grumpycoders:main Oct 9, 2022
@johnbaumann johnbaumann deleted the sio0_pocketstation_stuff branch October 9, 2022 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants