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

Replace frame download in Sakurasou No Pet Na Kanojo #6880

Merged
merged 2 commits into from
Sep 13, 2014

Conversation

daniel229
Copy link
Collaborator

As hrydgard said catch the memcpy. Fixes saveicons,the function https://gist.github.com/daniel229/ee3674788f4c560e8258

@unknownbrackets
Copy link
Collaborator

Well, memcpy() should be doing this.... I guess it's too small chunks.

Just to note, we want to avoid hooking in a loop, because if it's in a loop, it will cause a performance problem. We want to hook as early in the function as possible.

I suppose hooking here with v0 would also do the trick?
https://gist.github.com/daniel229/ee3674788f4c560e8258#file-func-sakurasou-no-pet-na-kanojo-L64

That should be outside the loop.

-[Unknown]

@daniel229
Copy link
Collaborator Author

Look like get a half picture.
01

@daniel229
Copy link
Collaborator Author

Size change to 0x00088000 then it works.

@unknownbrackets
Copy link
Collaborator

Yeah, if there's no register near there with 0x00088000 in it, then changing it is probably right.

Before it was probably downloading every line below the top half in a row... slowly. Or possibly each and every row. Not sure how the function works exactly.

-[Unknown]

@daniel229
Copy link
Collaborator Author

That A0 is 0x00088000.
02

@unknownbrackets
Copy link
Collaborator

Hmm. Right after a jalr, a0 probably isn't "safe" to use. If no reg has it before the jalr, I guess it's better to hard code.

-[Unknown]

@daniel229
Copy link
Collaborator Author

That a0 has a 0x88000 afetr changing the background,and no reg has it before the jalr.

unknownbrackets added a commit to unknownbrackets/ppsspp that referenced this pull request Sep 13, 2014
Replace frame download in Sakurasou No Pet Na Kanojo

Conflicts:
	Core/HLE/ReplaceTables.cpp
@unknownbrackets unknownbrackets merged commit 5f4a95c into hrydgard:master Sep 13, 2014
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