mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Rollup merge of #23351 - nagisa:rustdoc-lines-2, r=alexcrichton

Browse Source 
Previously it would fail on a trivial case like

    /// Summary line
    /// <trailing space>
    /// Regular content

Compliant markdown preprocessor would render that as two separate paragraphs, but our summary line
extractor interprets both lines as the same paragraph and includes both into the short summary resulting in
![screenshot from 2015-03-13 22 47 08](https://cloud.githubusercontent.com/assets/679122/6648596/7ef792b2-c9e4-11e4-9c19-704c288ec4de.png)
This commit is contained in:
Manish Goregaokar 2015-03-15 05:13:44 +05:30
commit 01f10dead3
1 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/librustdoc/html/render.rs
View File

@ -453,7 +453,7 @@ fn build_index(krate: &clean::Crate, cache: &mut Cache) -> io::Result<String> {
ty: shortty(item),
name: item.name.clone().unwrap(),
path: fqp[..fqp.len() - 1].connect("::"),
desc: shorter(item.doc_value()).to_string(),
desc: shorter(item.doc_value()),
parent: Some(did),
search_type: None,
});
@ -935,7 +935,7 @@ impl DocFolder for Cache {
ty: shortty(&item),
name: s.to_string(),
path: path.connect("::").to_string(),
desc: shorter(item.doc_value()).to_string(),
desc: shorter(item.doc_value()),
parent: parent,
search_type: get_index_search_type(&item, parent_basename),
});
@ -1527,13 +1527,14 @@ fn full_path(cx: &Context, item: &clean::Item) -> String {
return s
}
fn shorter<'a>(s: Option<&'a str>) -> &'a str {
fn shorter<'a>(s: Option<&'a str>) -> String {
match s {
Some(s) => match s.find("\n\n") {
Some(pos) => &s[..pos],
None => s,
},
None => ""
Some(s) => s.lines().take_while(|line|{
(*line).chars().any(|chr|{
!chr.is_whitespace()
})
}).collect::<Vec<_>>().connect("\n"),
None => "".to_string()
}
}
@ -1663,7 +1664,7 @@ fn item_module(w: &mut fmt::Formatter, cx: &Context,
</tr>
",
*myitem.name.as_ref().unwrap(),
Markdown(shorter(myitem.doc_value())),
Markdown(&shorter(myitem.doc_value())[..]),
class = shortty(myitem),
href = item_path(myitem),
title = full_path(cx, myitem),