bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

Inline implementation of lookup_line

Browse source 
to simplify the implementation and avoid unnecessary
conversions from `Option<usize>` to `isize` and back.
This commit is contained in:
Tomasz Miąsko 2021-07-05 00:00:00 +00:00
parent aa65b08b1d
commit 1719d45013
2 changed files with 16 additions and 25 deletions
Download patch file Download diff file Expand all files Collapse all files

20
compiler/rustc_span/src/lib.rs
View file

@ -1552,13 +1552,11 @@ impl SourceFile {
/// number. If the source_file is empty or the position is located before the
/// first line, `None` is returned.
pub fn lookup_line(&self, pos: BytePos) -> Option<usize> {
if self.lines.is_empty() {
return None;
match self.lines.binary_search(&pos) {
Ok(idx) => Some(idx),
Err(0) => None,
Err(idx) => Some(idx - 1),
}
let line_index = lookup_line(&self.lines[..], pos);
assert!(line_index < self.lines.len() as isize);
if line_index >= 0 { Some(line_index as usize) } else { None }
}
pub fn line_bounds(&self, line_index: usize) -> Range<BytePos> {
@ -1957,16 +1955,6 @@ impl InnerSpan {
}
}
// Given a slice of line start positions and a position, returns the index of
// the line the position is on. Returns -1 if the position is located before
// the first line.
fn lookup_line(lines: &[BytePos], pos: BytePos) -> isize {
match lines.binary_search(&pos) {
Ok(line) => line as isize,
Err(line) => line as isize - 1,
}
}
/// Requirements for a `StableHashingContext` to be used in this crate.
///
/// This is a hack to allow using the [`HashStable_Generic`] derive macro