We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 1318 - Couldn't load font file when use the latest version of SDL 1.3 on Android
Summary: Couldn't load font file when use the latest version of SDL 1.3 on Android
Status: RESOLVED DUPLICATE of bug 1301
Alias: None
Product: SDL_ttf
Classification: Unclassified
Component: misc (show other bugs)
Version: unspecified
Hardware: ARM Android (All)
: P2 critical
Assignee: Sam Lantinga
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-22 07:00 UTC by Mike Chen
Modified: 2011-10-25 11:40 UTC (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Chen 2011-10-22 07:00:24 UTC 
the latest SDL 1.3 download from HG allow user read resource from assets by JNI. I test it is OK in SDL_Image, but it does not compliant with SDL_ttf. When use the function of TTF_OpenFont*() it reports error of "Couldn't load font file". I trace the source code find "SDL_RWops *rw = SDL_RWFromFile(file, "rb");" seems no problem. 
but in TTF_Font* TTF_OpenFontIndexRW( SDL_RWops *src, int freesrc, int ptsize, long index )
...
error = FT_Open_Face( library, &font->args, index, &font->face );
	if( error ) {
		TTF_SetFTError( "Couldn't load font file", error );
		TTF_CloseFont( font );
		return NULL;
	}
it returns NULL
I have no idea what wrong in FT_Open_Face.
Comment 1 Gabriel Jacobo 2011-10-24 17:37:20 UTC 
Your problem sounds exactly like the bug in #1301, which was fixed a few days ago (changeset c9cb52d6d864), can you confirm you have that patch incorporated?
Comment 2 Mike Chen 2011-10-25 06:53:06 UTC 
(In reply to comment #1)
> Your problem sounds exactly like the bug in #1301, which was fixed a few days
> ago (changeset c9cb52d6d864), can you confirm you have that patch incorporated?

Yes,it's what I really want! Thanks a lot.
Comment 3 Gabriel Jacobo 2011-10-25 11:40:42 UTC 
You're welcome, I'll close the ticket.

*** This bug has been marked as a duplicate of bug 1301 ***