Skip to content

Add MemCard Pro GC Support#1125

Open
DDRBoxman wants to merge 1 commit intoFIX94:masterfrom
DDRBoxman:memcardpro
Open

Add MemCard Pro GC Support#1125
DDRBoxman wants to merge 1 commit intoFIX94:masterfrom
DDRBoxman:memcardpro

Conversation

@DDRBoxman
Copy link
Copy Markdown

Uses code from the swiss implementation:
emukidid/swiss-gc@3ee5c03

#1045

Uses code from the swiss implementation
emukidid/swiss-gc@3ee5c03
@bikerspade
Copy link
Copy Markdown

Any reason why this PR hasn't been accepted and merged into master with a new official build yet?

@bikerspade
Copy link
Copy Markdown

@carnage702 do you have the ability to merge this?

@bikerspade
Copy link
Copy Markdown

@FIX94 bump

@rparenti256
Copy link
Copy Markdown

@DDRBoxman Has this been tested recently? I compiled it and it generates a new memory card file on the memcardpro but the filename is corrupt and it throws an error on the memory card

@carnage702
Copy link
Copy Markdown
Collaborator

@DDRBoxman Has this been tested recently? I compiled it and it generates a new memory card file on the memcardpro but the filename is corrupt and it throws an error on the memory card

if this is true i wont merge it, i dont have it to test it.

Comment thread loader/source/menu.c
strncpy(ncfg->GamePath, StartChar, sizeof(ncfg->GamePath));
ncfg->GamePath[sizeof(ncfg->GamePath)-1] = 0;
memcpy(&(ncfg->GameID), gi[SelectedGame].ID, 4);
memcpy(&(ncfg->GameMakerCode), gi[SelectedGame].MakerCode, 2);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@DDRBoxman Before this memcpy of two bytes to GameMakerCode, should the full four bytes of this unsigned int be set to 0 so that the other two bytes aren't random uninitialized bytes?

ncfg->GameMakerCode = 0
memcpy(&(ncfg->GameMakerCode), gi[SelectedGame].MakerCode, 2);

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.

4 participants