-
Notifications
You must be signed in to change notification settings - Fork 104
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] MuPDF Android font dir patch update #971
Conversation
+char * | ||
+get_font_file_android(char *name) | ||
+{ | ||
+ char *fontdir; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const char fontdir[] = "/system/fonts";
size_t len = sizeof(fontdir) + strlen(name) + 1; // sizeof on a string literal includes the NUL
char *filename = malloc(len);
if (filename == NULL) {
return NULL;
}
(Fairly minor, this works ;)).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure I tested with what I thought of as a stress test (EPUB) but I guess that just used the Times (or whatever) that's bundled with MuPDF. In any case I didn't have any HTML dictionaries installed on the emulator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're fine with the basic principle besides this nit? :-P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep!
@@ -68,7 +68,7 @@ index f6951ba2..b1fdd981 100644 | |||
/* | |||
Base 14 PDF fonts from URW. | |||
Noto fonts from Google. | |||
@@ -367,3 +371,185 @@ fz_lookup_noto_emoji_font(fz_context *ctx, int *size) | |||
@@ -367,3 +371,188 @@ fz_lookup_noto_emoji_font(fz_context *ctx, int *size) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NiLuJe Apparently I thought your suggestion took off more lines than it did? :-P Oh well, luckily there's the CI.
koreader/koreader-base#971 Fixes <koreader#5347 (comment)>. Also includes: * [feat] Add FFI RTC interface (koreader-base#969) * Bump thirdparty/djvulibre to current master (koreader-base#970)
koreader/koreader-base#971 Fixes <#5347 (comment)>. Also includes: * [feat] Add FFI RTC interface (<koreader/koreader-base#969>) * Bump thirdparty/djvulibre to current master (<koreader/koreader-base#970>)
This reverts commit b7dd4c1. Closes <koreader/koreader#5617>.
This reverts commit b7dd4c1. Closes <koreader/koreader#5617>.
koreader/koreader-base#971 Fixes <koreader#5347 (comment)>. Also includes: * [feat] Add FFI RTC interface (<koreader/koreader-base#969>) * Bump thirdparty/djvulibre to current master (<koreader/koreader-base#970>)
Cf. koreader/koreader#5347 (comment).