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

Fix endian error. allows loading of current games save even if the memca... #1221

Merged
merged 1 commit into from Oct 7, 2014

Conversation

LPFaint99
Copy link
Contributor

No description provided.

@JMC47
Copy link
Contributor

JMC47 commented Oct 5, 2014

This fixes my issues with using a GCI folder. Btw @LPFaint99 apparently that was messing up the medabots save too in GCI folders, derp! I guess it had nothing to do with alphabetical order like I has assumed. Sorry for the misleading info.

@Sonicadvance1
Copy link
Contributor

Seems a bit dirty to do a cast like this.
Could we not just change the GameCode from a u8[4] array to just a regular u32?
We only ever use this as a u32, not sure why it was chosen to be an array in the first place.
There is at least one other place in the code that is casting it to a u32 as well.

@LPFaint99
Copy link
Contributor Author

Agreed, it was originally an array (along with every other field in gcmemorycard space) since before I started working on dolphin. I worked to convert some of the other fields, never got to all of them.

I realize looking at the code again, there is one other place where we compare the passed in m_GameCode (LE) to a (BE) gamecode.

So I changed it so we pass m_GameCode in as BE.

The type of the fields in the memorycard handling should be dealt with in a different PR,
(more effort needed to regression test, this defect is not really related to the type)

@Sonicadvance1
Copy link
Contributor

I'm fine with that resolution for now.
LGTM

@comex
Copy link
Contributor

comex commented Oct 7, 2014

Sure would be nice if the OS X buildbot didn't randomly go down.

comex added a commit that referenced this pull request Oct 7, 2014
Fix endian error. allows loading of current games save even if the memca...
@comex comex merged commit 9d059d1 into dolphin-emu:master Oct 7, 2014
@Icekhaos
Copy link

Icekhaos commented Oct 7, 2014

The buildbox is not a Hackintosh?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants