Skip to content

Optimize s:Render()#594

Closed
mattn wants to merge 1 commit intomasterfrom
optimize3
Closed

Optimize s:Render()#594
mattn wants to merge 1 commit intomasterfrom
optimize3

Conversation

@mattn
Copy link
Copy Markdown
Member

@mattn mattn commented Mar 16, 2022

As far as I can see the code of s:Render(), many lines should be possible to be omitted over the height.

@tacahiroy Could you please review this?

If my analysis is correct, we will get great performance improvements. Below is demo under the directory which have 155732 files.

screenshot

@tacahiroy
Copy link
Copy Markdown
Member

tacahiroy commented Mar 16, 2022 via email

@mattn
Copy link
Copy Markdown
Member Author

mattn commented Mar 16, 2022

Ah, sorry. This is wrong. Can not navigate with cursor up/down more than the number over the height.

@mattn mattn closed this Mar 16, 2022
@mattn mattn deleted the optimize3 branch March 16, 2022 16:02
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