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

slow bookmarks? #3655

Closed
Eduardomb22 opened this issue Feb 1, 2018 · 9 comments
Closed

slow bookmarks? #3655

Eduardomb22 opened this issue Feb 1, 2018 · 9 comments
Assignees
Labels

Comments

@Eduardomb22
Copy link

  • KOReader version: lastest version
  • Device: ppw3

Issue

Is the bookmark menu kind slow? It is taking some time to enter here, PDF or EPUB files.

@poire-z
Copy link
Contributor

poire-z commented Feb 1, 2018

If your bookmarks are highlights with long text, yes, it may have become slow since a few nightlys, see #3589 (comment). Hopefully it will be fixed soon.

@Eduardomb22
Copy link
Author

Thanks. It is probably this.

@Frenzie
Copy link
Member

Frenzie commented Feb 1, 2018

Yep, some semi-random comments in a closed issue/PR are a bit hard to find. ;-)

@robert00s
Copy link
Contributor

@Eduardomb22
Can you check your bookmark menu after my patch #3657?

@Eduardomb22
Copy link
Author

Eduardomb22 commented Feb 2, 2018

@robert00s, I'm not a programmer so I've tried to copy the menu.lua (or add and remove the lines from the file) to mine menu.lua. But I'm getting this error when I open a book and click in the bookmarks menu (It doesn't crash if the book doesn't have any bookmark saved)

02/02/18-14:43:23 WARN sizeUtf8Text called without text
02/02/18-14:43:23 WARN sizeUtf8Text called without text
02/02/18-14:43:23 WARN sizeUtf8Text called without text
02/02/18-14:43:23 WARN sizeUtf8Text called without text
lipc-wait-event exited normally with status: 0
Segmentation fault

I don't know if I'm doing right (probably it is my fault), so I think it is better to wait someone that could test it.

@robert00s
Copy link
Contributor

robert00s commented Feb 2, 2018

Strange.
Unzip this file and replace menu.lua in frontend/ui/widget
menu.lua.zip
Edit: I have the same error on my kindle.

@robert00s
Copy link
Contributor

Problem is with this line:
self.text = table.concat(table.move(item_name.charlist, 1, offset - 4, 1, {})) .. "…"

@Eduardomb22
Copy link
Author

Eduardomb22 commented Feb 2, 2018

You can ping me so I can test it again. Thanks for looking into the problem.

@poire-z
Copy link
Contributor

poire-z commented Feb 4, 2018

Fixed by #3657.

@poire-z poire-z closed this as completed Feb 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants