tonerdo
Improve screen repaint speed
Ideally, I'd like to see conversion to Generic Lists over arrays everywhere, but these mods made this library much easier to integrate into one of my other projects.
The other issue is that by keeping track of the cursor pointer and changing one char at a time, it's slow and the repaint factor is frustrating for a user used to the native GNU tools on *nix. I modified the logic to cut to the chase and quickly repaint the affected text whole strings at a time instead of per char.
Neat project overall-- thanks for sharing. I hope these changes are helpful to your goals.
@malcomvetter thanks for the PR. Can you please look into the test failures? I'll also appreciate some data on how much your changes improve repaint speed
Hey @tonerdo, I haven't tried out @malcomvetter's changes, but I have been noticing some screen re-paint issues for tab completion in a project I'm working on that uses your readline library.
I think the issue stems from the fact that when tabbing through options, they are erased one character at a time. It's not always an issue, but I've noticed it particularly when I'm using my application over an SSH connection.
I think erasing the string all at once while tabbing through options would make a big difference. Anyway, thanks for the awesome project, I've been getting a ton of usage out of it!
That’s essentially what my changes do. They break some unit tests (due to refactor) but I haven’t gotten around to fixing that.
On Thu, Sep 6, 2018 at 1:54 PM Ryan Cobb [email protected] wrote:
Hey @tonerdo https://github.com/tonerdo, I haven't tried out @malcomvetter https://github.com/malcomvetter's changes, but I have been noticing some screen re-paint issues for tab completion in a project I'm working on that uses your readline library.
I think the issue stems from the fact that when tabbing through options, they are erased one character at a time. It's not always an issue, but I've noticed it particularly when I'm using my application over an SSH connection.
I think erasing the string all at once while tabbing through options would make a big difference. Anyway, thanks for the awesome project, I've been getting a ton of usage out of it!
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/tonerdo/readline/pull/45#issuecomment-419203889, or mute the thread https://github.com/notifications/unsubscribe-auth/AMJRZUn-vJ_5OWMIN1fcCQn--4pPfwAjks5uYW-BgaJpZM4U_UvB .
Open a PR and make sure it doesn't fail like I am seeing currently. Test it out.. if it works better than the previous ReadLine build, it should be good to go.
