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

lib: don't prepend mailto to url in autolink Event

Browse source 
better to provide the original url, the event is already tagged as email

also avoids a string allocation
This commit is contained in:
Noah Hellman 2023-03-19 13:32:06 +01:00
commit 05a4992d99
2 changed files with 8 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/html.rs
View file

@ -25,6 +25,7 @@
use crate::Alignment;
use crate::Container;
use crate::Event;
use crate::LinkType;
use crate::ListKind;
use crate::OrderedListNumbering::*;
use crate::Render;
@ -161,11 +162,14 @@ impl<'s, I: Iterator<Item = Event<'s>>, W: std::fmt::Write> Writer<'s, I, W> {
Container::DescriptionTerm => self.out.write_str("<dt")?,
Container::CodeBlock { .. } => self.out.write_str("<pre")?,
Container::Span | Container::Math { .. } => self.out.write_str("<span")?,
Container::Link(dst, ..) => {
Container::Link(dst, ty) => {
if dst.is_empty() {
self.out.write_str("<a")?;
} else {
self.out.write_str(r#"<a href=""#)?;
if matches!(ty, LinkType::Email) {
self.out.write_str("mailto:")?;
}
self.write_attr(dst)?;
self.out.write_char('"')?;
}

6
src/lib.rs
View file

@ -746,10 +746,10 @@ impl<'s> Parser<'s> {
}
inline::Container::Autolink => {
let url = self.inlines.src(inline.span);
let (url, ty) = if url.contains('@') {
(format!("mailto:{}", url).into(), LinkType::Email)
let ty = if url.contains('@') {
LinkType::Email
} else {
(url, LinkType::AutoLink)
LinkType::AutoLink
};
Container::Link(url, ty)
}