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

Strange corrupt screen when editing Rust - LSP position out of bounds error / cyclic deps: helix_view #3617

Closed
David-Else opened this issue Aug 31, 2022 · 2 comments
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug R-duplicate Duplicated issue: please refer to the linked issue

Comments

@David-Else
Copy link
Contributor

David-Else commented Aug 31, 2022

Summary

Screencast.from.31-08-22.15.09.30.webm

As I press enter and backspace in insert mode the ghostly ". screen... appears, if I then go into normal mode it acts as if that line is not there and jumps over it when I navigate up and down.

Everything seems borked, but I can keep undoing until things return to how they were.

Reproduction Steps

Not sure how to repeat it, but hopefully the video and log will be enough.

Helix log

I included the whole log, what is in the screen shot should be covered by what is at the end of the log:

helix.log

Platform

Linux

Terminal Emulator

Kitty

Helix Version

helix 22.05 (83f177d)

@David-Else David-Else added the C-bug Category: This is a bug label Aug 31, 2022
@kirawi kirawi added the A-helix-term Area: Helix term improvements label Aug 31, 2022
@David-Else David-Else changed the title Strange corrupt screen when editing Rust - LSP out of bounds error / cyclic deps: helix_view Strange corrupt screen when editing Rust - LSP position out of bounds error / cyclic deps: helix_view Sep 1, 2022
@the-mikedavis
Copy link
Member

I think this is #2248 since it looks like it's the end of the file. The "lsp position out of bounds" looks like it may be the EOF character so it could be a off-by-one in rust-analyzer or we might not be handling it correctly. The cyclic deps message is expected - it's a false positive from the way that rust-analyzer analyzes the workspace

@the-mikedavis
Copy link
Member

I reproduced this and it does end up being #2248 so I'll close in favor of that. I'll have some notes to share about a fix there too

@the-mikedavis the-mikedavis closed this as not planned Won't fix, can't repro, duplicate, stale Sep 4, 2022
@the-mikedavis the-mikedavis added the R-duplicate Duplicated issue: please refer to the linked issue label Sep 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug R-duplicate Duplicated issue: please refer to the linked issue
Projects
None yet
Development

No branches or pull requests

3 participants