Hello, i've had an issue recently panicking on slice out of bounds, i think this line

oh yeah pos + old.len() should do it too :) <div class="snippet-clipboard-content

Resolved by <a class="issue-link js-issue-link" data-error-text="Failed to load title"

Panicking on for_each_mut_string about distant HOT 3 CLOSED

Polish-Civil commented on June 9, 2024

Panicking on for_each_mut_string

from distant.

Comments (3)

chipsenkbeil commented on June 9, 2024 1

Hm, that call eventually triggers this:

distant/distant-core/src/client/lsp/msg.rs

Line 313 in ea2e128

s.replace_range(pos..old.len(), new);

I'm not available to test it yet, but I think I meant to write s.replace_range(pos..pos + old.len(), new);.

from distant.

Polish-Civil commented on June 9, 2024

oh yeah pos + old.len() should do it too :)

unable to swap prefix old: [file:] new: [distant:] string: [Expands to the file name in which it was invoked.\n\nWith [`line!`] and [`column!`], these macros provide debugging information for\ndevelopers about the location within the source.\n\nThe expanded expression has type `&'static str`, and the returned file\nis not the invocation of the `file!` macro itself, but rather the\nfirst macro invocation leading up to the invocation of the `file!`\nmacro.\n\n# Examples\n\n```rust\nlet this_file = file!();\nprintln!(\"defined in file: {this_file}\");]

Not sure what that gibberish is though, got it while running rust-analyzer

from distant.

chipsenkbeil commented on June 9, 2024

Resolved by #146

from distant.

Recommend Projects