Skip to content

Commit

Permalink
Fix rustoc item summaries that are headers
Browse files Browse the repository at this point in the history
Rustoc item summaries that are headers were not displayed at all because
they started with whitespace.

This PR fixes this and now removes the whitespace and then displays the
block.
  • Loading branch information
chrisduerr committed Nov 29, 2017
1 parent 0a2e9ad commit 91a4106
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/librustdoc/html/render.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1819,7 +1819,9 @@ fn full_path(cx: &Context, item: &clean::Item) -> String {

fn shorter<'a>(s: Option<&'a str>) -> String {
match s {
Some(s) => s.lines().take_while(|line|{
Some(s) => s.lines()
.skip_while(|s| s.chars().all(|c| c.is_whitespace()))
.take_while(|line|{
(*line).chars().any(|chr|{
!chr.is_whitespace()
})
Expand Down
13 changes: 13 additions & 0 deletions src/test/rustdoc/issue-46377.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// Copyright 2015 The Rust Project Developers. See the COPYRIGHT
// file at the top-level directory of this distribution and at
// http://rust-lang.org/COPYRIGHT.
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

// @has 'issue_46377/index.html' '//*[@class="docblock-short"]' 'Check out this struct!'
/// # Check out this struct!
pub struct SomeStruct;

0 comments on commit 91a4106

Please sign in to comment.