Skip to content

ffi/mupdf: fix metadata strings#1921

Merged
Frenzie merged 1 commit into
koreader:masterfrom
benoit-pierre:pr/fix_mupdf_metadata_strings
Sep 5, 2024
Merged

ffi/mupdf: fix metadata strings#1921
Frenzie merged 1 commit into
koreader:masterfrom
benoit-pierre:pr/fix_mupdf_metadata_strings

Conversation

@benoit-pierre

@benoit-pierre benoit-pierre commented Sep 5, 2024

Copy link
Copy Markdown
Member

Don't create Lua strings with an embedded null terminator.

Cf. koreader/koreader#12468.


This change is Reviewable

Don't create Lua strings with an embedded null terminator.
@Frenzie Frenzie merged commit 3560f72 into koreader:master Sep 5, 2024
@benoit-pierre benoit-pierre deleted the pr/fix_mupdf_metadata_strings branch September 5, 2024 07:41
Frenzie pushed a commit to koreader/koreader that referenced this pull request Sep 23, 2024
(1) Followup to koreader/koreader-base#1921, #12503: when getting a pdf book id in the stats db, check for old records with null terminated authors and title.
Discussed in #11318. There are some other reasons for disappearing old stats for pdf (md5 changes when converting/writing to pdf or something else), hence not closing the issue.
(2) Add Dispatcher action for toggling statistics. Closes #11422.
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
…2547)

(1) Followup to koreader/koreader-base#1921, koreader#12503: when getting a pdf book id in the stats db, check for old records with null terminated authors and title.
Discussed in koreader#11318. There are some other reasons for disappearing old stats for pdf (md5 changes when converting/writing to pdf or something else), hence not closing the issue.
(2) Add Dispatcher action for toggling statistics. Closes koreader#11422.
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