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

Use the alloc size from real PSP for FONT_OPEN_INTERNAL_STINGY mode. #13180

Merged
merged 1 commit into from Jul 22, 2020

Conversation

shenweip
Copy link
Contributor

@shenweip shenweip commented Jul 22, 2020

These values are from pspautotest, see hrydgard/pspautotests#206 . Should helps #13166 , this game uses FONT_OPEN_INTERNAL_STINGY mode to open font files(index 0, 1 and 11), the alloc size seems incorrect before.

@hrydgard hrydgard added the PGF / sceFont Issue involves PGF fonts. label Jul 22, 2020
@hrydgard hrydgard added this to the v1.11.0 milestone Jul 22, 2020
@hrydgard
Copy link
Owner

Oh, nice find!

@hrydgard hrydgard merged commit 3ec54d3 into hrydgard:master Jul 22, 2020
@@ -613,7 +618,7 @@ class FontLib {

u32 allocSize = 12;
if (mode == FONT_OPEN_INTERNAL_STINGY) {
allocSize = 0x239B4;
allocSize = loadedFont->GetFont()->getStingySize();
Copy link
Owner

Choose a reason for hiding this comment

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

a little curious that this one is = and the other one +=. Maybe the bias of 12 is builtin to the non-stingy sizes?

@sum2012
Copy link
Collaborator

sum2012 commented Jul 22, 2020

Fix pspautotests link:
hrydgard/pspautotests#206
Anyway I will fix myself for jpcsp or tell gid15
Thanks again @shenweip

edit:I need gid15 to find gb3s1518.bwfon's StingySize

@shenweip shenweip deleted the Font-stingy branch July 22, 2020 23:00
gid15 added a commit to jpcsp/jpcsp that referenced this pull request Jul 24, 2020
memory size as on a real PSP. Tested for all the PSP fonts.
Based on hrydgard/ppsspp#13180 by shenweip.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PGF / sceFont Issue involves PGF fonts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants