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.

Sign up for GitHub

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

Jump to bottom

Unresponsive LF on <tab> autocomplete long cyrillic filenames #1090

Closed
krem opened this issue Jan 23, 2023 · 0 comments · Fixed by #1183
Closed

Unresponsive LF on <tab> autocomplete long cyrillic filenames #1090

krem opened this issue Jan 23, 2023 · 0 comments · Fixed by #1183

Comments

@krem
Copy link

krem commented Jan 23, 2023 •
edited
Loading

When current dir contains filenames longer than 63 Cyrillic letters, TAB for autocomplete makes LF unresponsive, eating tons of CPU and memory.
To reproduce:
$touch "Вступ України до НАТО підтримують рекордні 86% українців - результати опитування"
$ls [TAB]
Remove last word from the filename - and it works.

Also, regardless of filename, LF does NO suggestion by TAB for SOLE file in the directory, which is normal for, say, bash completions.
Thank you for great FM!
p-ouellette added a commit to p-ouellette/lf that referenced this issue Mar 31, 2023
@p-ouellette
Fix infinite loop in listMatches
797f12c 
This fixes an issue where listMatches could get stuck in an infinite
loop if one of the matches was longer than the window width.

Fixes gokcehan#1090
@p-ouellette p-ouellette mentioned this issue Mar 31, 2023
Merged
p-ouellette added a commit to p-ouellette/lf that referenced this issue Mar 31, 2023
@p-ouellette
Completion: always insert longest match
5cc7fe3 
Previously, longest common match completion would only apply if there
was a word before the cursor.

Suggested in gokcehan#1090
@p-ouellette p-ouellette mentioned this issue Mar 31, 2023
Merged
gokcehan pushed a commit that referenced this issue Apr 1, 2023
@p-ouellette
Fix infinite loop in listMatches (#1183)
70171ad 
This fixes an issue where listMatches could get stuck in an infinite
loop if one of the matches was longer than the window width.

Fixes #1090
gokcehan pushed a commit that referenced this issue Apr 1, 2023
@p-ouellette
Completion: always insert longest match (#1184)
f563263 
Previously, longest common match completion would only apply if there
was a word before the cursor.

Suggested in #1090
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix infinite loop in listMatches p-ouellette/lf
1 participant
@krem