-
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
Cache/MµPDF shenanigans, the base edition #1365
Conversation
This works around numerous issues (leak, corruption) occuring on overcommit on the old version we're currently using.
We've stopped using in front, in favor of our own Persist and its bitser or LuaJIT backends. (We still keep the buildsystem around, as we currently pickup its luacompat52 build on Android for the Evernote plugin. Which also builds/ships its own copy of it).
Make it return the number of words, too. Helps cache size estimations in front.
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.
The commit title me think you randomly included a fix for koreader/koreader#7631
Not that I know of, unless the two are related, which I doubt, but koreader/koreader#7627, yeah ;). |
Yeah, no, the commit titled |
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.
Reviewed 6 of 7 files at r1, 2 of 2 files at r2.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @NiLuJe)
Bits and bobs needed by koreader/koreader#7624 related to koreader/koreader#7627
This change is![Reviewable](https://camo.githubusercontent.com/23b05f5fb48215c989e92cc44cf6512512d083132bd3daf689867c8d9d386888/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)