user0/rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Disable error log for position clamping, its too noisy due to ease of triggering

Browse source
This commit is contained in:
Lukas Wirth 2025-08-07 15:14:22 +02:00
parent efbe625e35
commit 1a73720b80
1 changed files with 7 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/tools/rust-analyzer/crates/rust-analyzer/src/lsp/from_proto.rs
View file

@ -40,12 +40,13 @@ pub(crate) fn offset(
})?;
let col = TextSize::from(line_col.col);
let clamped_len = col.min(line_range.len());
if clamped_len < col {
tracing::error!(
"Position {line_col:?} column exceeds line length {}, clamping it",
u32::from(line_range.len()),
);
}
// FIXME: The cause for this is likely our request retrying. Commented out as this log is just too chatty and very easy to trigger.
// if clamped_len < col {
// tracing::error!(
// "Position {line_col:?} column exceeds line length {}, clamping it",
// u32::from(line_range.len()),
// );
// }
Ok(line_range.start() + clamped_len)
}