fnmain/jotdown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

prepass: fix lookup of headings

Browse source 
bug caused by confusing binary_search_by_key to be element instead of
index, both of which were usize
This commit is contained in:
Noah Hellman 2023-05-08 21:19:47 +02:00
parent 9efd4e4448
commit 329207689b
1 changed files with 60 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

61
src/lib.rs
View file

@ -734,7 +734,7 @@ impl<'s> PrePass<'s> {
self.headings_lex
.binary_search_by_key(&tag, |i| &self.headings[*i].text)
.ok()
.map(|i| self.heading_id(i))
.map(|i| self.heading_id(self.headings_lex[i]))
}
}
@ -1204,6 +1204,65 @@ mod test {
id: "Some-Section".into()
}),
);
test_parse!(
concat!(
"[a][]\n", //
"[b][]\n", //
"\n", //
"# b\n", //
"\n", //
"# a\n", //
),
Start(Paragraph, Attributes::new()),
Start(
Link("#a".into(), LinkType::Span(SpanLinkType::Reference)),
Attributes::new()
),
Str("a".into()),
End(Link("#a".into(), LinkType::Span(SpanLinkType::Reference))),
Softbreak,
Start(
Link("#b".into(), LinkType::Span(SpanLinkType::Reference)),
Attributes::new()
),
Str("b".into()),
End(Link("#b".into(), LinkType::Span(SpanLinkType::Reference))),
End(Paragraph),
Blankline,
Start(Section { id: "b".into() }, Attributes::new()),
Start(
Heading {
level: 1,
has_section: true,
id: "b".into(),
},
Attributes::new(),
),
Str("b".into()),
End(Heading {
level: 1,
has_section: true,
id: "b".into(),
}),
Blankline,
End(Section { id: "b".into() }),
Start(Section { id: "a".into() }, Attributes::new()),
Start(
Heading {
level: 1,
has_section: true,
id: "a".into(),
},
Attributes::new(),
),
Str("a".into()),
End(Heading {
level: 1,
has_section: true,
id: "a".into(),
}),
End(Section { id: "a".into() }),
);
}
#[test]