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

Allow load sceLibFont module #19115

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

sum2012
Copy link
Collaborator

@sum2012 sum2012 commented May 7, 2024

It is using custom font library.
Fix #19111
Also remove zh_gb.pgf hack (Now no need)
Test on chinese patched game of Zettai Meikyuu Grimm Nanatsu no Kagi to Rakuen no Otome (ULJM05632)

It is using custom font library.
Fix hrydgard#19111
Now no need.
Test on chinese patched game of Zettai Meikyuu Grimm Nanatsu no Kagi to Rakuen no Otome (ULJM05632)
@sum2012 sum2012 added PGF / sceFont Issue involves PGF fonts. Patched-game Occurs only on patched or modified version of a game. labels May 7, 2024
Don't know it is worked before
@sum2012 sum2012 changed the title Allow load fontfuck module Allow load sceLibfont module May 7, 2024
Comment on lines 813 to 814
if (strcmp(func.moduleName, "sceLibfont") == 0) {
if (FuncImportIsSyscall(func.moduleName, func.nid)) {
Copy link
Owner

Choose a reason for hiding this comment

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

Can you explain this logic? Seems to me that now, FuncImportIsSyscall will only be checked if func.moduleName == "sceLibFont". Couldn't this change behavior or break things for other modules? I don't really understand.

And spell mistake sceLibFont
@sum2012
Copy link
Collaborator Author

sum2012 commented May 8, 2024

@hrydgard fixed

@sum2012 sum2012 changed the title Allow load sceLibfont module Allow load sceLibFont module May 8, 2024
@hrydgard
Copy link
Owner

Ah okay, I understand now.

I still wonder if this could somehow break other games that might have previously not loaded sceLibFont but now will, might do some testing myself after I send the iOS release for review. Although seems more likely to fix than to break stuff, if we can successfully load the library and use it...

@xeonliu
Copy link

xeonliu commented May 17, 2024

Many Chinese Localization Games uses "fontfuck" moudule, which is basically a wrapper around the original sceFontLib. The sceLibFont is renamed to sceLibFttt, and then can be loaded by the "fontfuck" module.

A particular fontfuck module I found recently will load fonts.pgf located in USRDIR.

Copy link

@xeonliu xeonliu left a comment

Choose a reason for hiding this comment

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

I think this will solve a lot of problem.
But sceFont HLE seems to be useless then.
sceFttt & sceFont will be loaded as normal modules.
I think that HLE still has a lot of problem on loading PGF fonts compared to jpcsp. Not sure if it's good to leave a buggy implementation here.

@hrydgard
Copy link
Owner

Yeah I suppose ideally we should skip the HLE here entirely. I'm not willing to mess with this until I've gotten first a good stable iOS build out, and then 1.18. Then we can go experimental for a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Patched-game Occurs only on patched or modified version of a game. PGF / sceFont Issue involves PGF fonts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature Request: Add an option switch to use font files from the game ROM.
3 participants