Skip to content

Commit 10d1999

Browse files
authored
Merge pull request #831 from epage/span
refactor(parser): Rely on winnow for error span
2 parents 999824a + e5278bf commit 10d1999

File tree

3 files changed

+4
-17
lines changed

3 files changed

+4
-17
lines changed

Cargo.lock

+2-2
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

crates/toml_edit/Cargo.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ unbounded = []
4141

4242
[dependencies]
4343
indexmap = { version = "2.3.0", features = ["std"] }
44-
winnow = { version = "0.7.0", optional = true }
44+
winnow = { version = "0.7.3", optional = true }
4545
serde = { version = "1.0.145", optional = true }
4646
kstring = { version = "2.0.0", features = ["max_inline"], optional = true }
4747
toml_datetime = { version = "0.6.8", path = "../toml_datetime" }

crates/toml_edit/src/error.rs

+1-14
Original file line numberDiff line numberDiff line change
@@ -25,20 +25,7 @@ impl TomlError {
2525
let raw = raw.finish();
2626
let raw = String::from_utf8(raw.to_owned()).expect("original document was utf8");
2727

28-
let offset = error.offset();
29-
let offset = (0..=offset)
30-
.rev()
31-
.find(|index| raw.is_char_boundary(*index))
32-
.unwrap_or(0);
33-
34-
let mut indices = raw[offset..].char_indices();
35-
indices.next();
36-
let len = if let Some((index, _)) = indices.next() {
37-
index
38-
} else {
39-
raw.len() - offset
40-
};
41-
let span = offset..(offset + len);
28+
let span = error.char_span();
4229

4330
Self {
4431
message,

0 commit comments

Comments
 (0)