Fix horizontal trim for block doc comments

This commit is contained in:
Guillaume Gomez 2022-02-05 14:45:29 +01:00
parent a00e130dae
commit e3942874a0
2 changed files with 38 additions and 6 deletions

View File

@ -43,7 +43,7 @@ fn get_vertical_trim(lines: &[&str]) -> Option<(usize, usize)> {
if i != 0 || j != lines.len() { Some((i, j)) } else { None }
}
fn get_horizontal_trim(lines: &[&str], kind: CommentKind) -> Option<usize> {
fn get_horizontal_trim<'a>(lines: &'a [&str], kind: CommentKind) -> Option<String> {
let mut i = usize::MAX;
let mut first = true;
@ -51,7 +51,8 @@ fn get_horizontal_trim(lines: &[&str], kind: CommentKind) -> Option<usize> {
// present. However, we first need to strip the empty lines so they don't get in the middle
// when we try to compute the "horizontal trim".
let lines = if kind == CommentKind::Block {
let mut i = 0;
// Whatever happens, we skip the first line.
let mut i = if lines[0].trim_start().starts_with('*') { 0 } else { 1 };
let mut j = lines.len();
while i < j && lines[i].trim().is_empty() {
@ -84,7 +85,7 @@ fn get_horizontal_trim(lines: &[&str], kind: CommentKind) -> Option<usize> {
return None;
}
}
Some(i)
if lines.is_empty() { None } else { Some(lines[0][..i].into()) }
}
let data_s = data.as_str();
@ -102,8 +103,13 @@ fn get_horizontal_trim(lines: &[&str], kind: CommentKind) -> Option<usize> {
changes = true;
// remove a "[ \t]*\*" block from each line, if possible
for line in lines.iter_mut() {
if horizontal + 1 < line.len() {
*line = &line[horizontal + 1..];
if let Some(tmp) = line.strip_prefix(&horizontal) {
*line = tmp;
if kind == CommentKind::Block
&& (*line == "*" || line.starts_with("* ") || line.starts_with("**"))
{
*line = &line[1..];
}
}
}
}

View File

@ -24,7 +24,7 @@ fn test_block_doc_comment_3() {
create_default_session_globals_then(|| {
let comment = "\n let a: *i32;\n *a = 5;\n";
let stripped = beautify_doc_string(Symbol::intern(comment), CommentKind::Block);
assert_eq!(stripped.as_str(), " let a: *i32;\n *a = 5;");
assert_eq!(stripped.as_str(), "let a: *i32;\n*a = 5;");
})
}
@ -41,3 +41,29 @@ fn test_line_doc_comment() {
assert_eq!(stripped.as_str(), "!test");
})
}
#[test]
fn test_doc_blocks() {
create_default_session_globals_then(|| {
let stripped = beautify_doc_string(
Symbol::intern(
" # Returns
*
",
),
CommentKind::Block,
);
assert_eq!(stripped.as_str(), " # Returns\n\n");
let stripped = beautify_doc_string(
Symbol::intern(
"
* # Returns
*
",
),
CommentKind::Block,
);
assert_eq!(stripped.as_str(), " # Returns\n\n");
})
}