Skip to content

Commit

Permalink
Support {{#shiftinclude auto}}
Browse files Browse the repository at this point in the history
As well as allowing explicitly-specified shift amounts, also
support an "auto" option that strips common leftmost whitespace
from an inclusion.
  • Loading branch information
daviddrysdale committed Mar 2, 2024
1 parent 6c7b259 commit 4b1b97b
Show file tree
Hide file tree
Showing 5 changed files with 345 additions and 24 deletions.
6 changes: 6 additions & 0 deletions guide/src/format/mdbook.md
Original file line number Diff line number Diff line change
Expand Up @@ -223,6 +223,12 @@ using the following syntax:
A positive number for the shift will prepend spaces to all lines; a negative number will remove
the corresponding number of characters from the beginning of each line.

The special `auto` value will remove common initial whitespace from all lines.

```hbs
\{{#shiftinclude auto:file.rs:indentedanchor}}
```

## Including a file but initially hiding all except specified lines

The `rustdoc_include` helper is for including code from external Rust files that contain complete
Expand Down
46 changes: 38 additions & 8 deletions src/preprocess/links.rs
Original file line number Diff line number Diff line change
Expand Up @@ -266,14 +266,18 @@ fn parse_include_path(path: &str) -> LinkType<'static> {
fn parse_shift_include_path(params: &str) -> LinkType<'static> {
let mut params = params.splitn(2, ':');
let param0 = params.next().unwrap();
let shift: isize = param0.parse().unwrap_or_else(|e| {
log::error!("failed to parse shift amount: {e:?}");
0
});
let shift = match shift.cmp(&0) {
Ordering::Greater => Shift::Right(shift as usize),
Ordering::Equal => Shift::None,
Ordering::Less => Shift::Left(-shift as usize),
let shift = if param0 == "auto" {
Shift::Auto
} else {
let shift: isize = param0.parse().unwrap_or_else(|e| {
log::error!("failed to parse shift amount: {e:?}");
0
});
match shift.cmp(&0) {
Ordering::Greater => Shift::Right(shift as usize),
Ordering::Equal => Shift::None,
Ordering::Less => Shift::Left(-shift as usize),
}
};
let mut parts = params.next().unwrap().splitn(2, ':');

Expand Down Expand Up @@ -1002,6 +1006,19 @@ mod tests {
);
}

#[test]
fn parse_with_auto_shifted_anchor() {
let link_type = parse_shift_include_path("auto:arbitrary:some-anchor");
assert_eq!(
link_type,
LinkType::Include(
PathBuf::from("arbitrary"),
RangeOrAnchor::Anchor("some-anchor".to_string()),
Shift::Auto
)
);
}

#[test]
fn parse_with_more_than_three_colons_ignores_everything_after_third_colon() {
let link_type = parse_include_path("arbitrary:5:10:17:anything:");
Expand Down Expand Up @@ -1053,4 +1070,17 @@ mod tests {
)
);
}

#[test]
fn parse_start_and_end_auto_shifted_range() {
let link_type = parse_shift_include_path("auto:arbitrary:5:10");
assert_eq!(
link_type,
LinkType::Include(
PathBuf::from("arbitrary"),
RangeOrAnchor::Range(LineRange::from(4..10)),
Shift::Auto
)
);
}
}
115 changes: 103 additions & 12 deletions src/utils/string.rs
Original file line number Diff line number Diff line change
Expand Up @@ -10,16 +10,54 @@ pub enum Shift {
None,
Left(usize),
Right(usize),
/// Strip leftmost whitespace that is common to all lines.
Auto,
}

fn shift_line(l: &str, shift: Shift) -> Cow<'_, str> {
#[derive(PartialEq, Eq, Debug, Clone, Copy)]
enum ExplicitShift {
None,
Left(usize),
Right(usize),
}

fn common_leading_ws(lines: &[String]) -> String {
let mut common_ws: Option<String> = None;
for line in lines {
let ws = line.chars().take_while(|c| c.is_whitespace());
if let Some(common) = common_ws {
common_ws = Some(
common
.chars()
.zip(ws)
.take_while(|(a, b)| a == b)
.map(|(a, _b)| a)
.collect(),
);
} else {
common_ws = Some(ws.collect())
}
}
common_ws.unwrap_or_else(String::new)
}

fn calculate_shift(lines: &[String], shift: Shift) -> ExplicitShift {
match shift {
Shift::None => ExplicitShift::None,
Shift::Left(l) => ExplicitShift::Left(l),
Shift::Right(r) => ExplicitShift::Right(r),
Shift::Auto => ExplicitShift::Left(common_leading_ws(lines).len()),
}
}

fn shift_line(l: &str, shift: ExplicitShift) -> Cow<'_, str> {
match shift {
Shift::None => Cow::Borrowed(l),
Shift::Right(shift) => {
ExplicitShift::None => Cow::Borrowed(l),
ExplicitShift::Right(shift) => {
let indent = " ".repeat(shift);
Cow::Owned(format!("{indent}{l}"))
}
Shift::Left(skip) => {
ExplicitShift::Left(skip) => {
if l.chars().take(skip).any(|c| !c.is_whitespace()) {
log::error!("left-shifting away non-whitespace");
}
Expand All @@ -30,6 +68,7 @@ fn shift_line(l: &str, shift: Shift) -> Cow<'_, str> {
}

fn shift_lines(lines: &[String], shift: Shift) -> Vec<Cow<'_, str>> {
let shift = calculate_shift(lines, shift);
lines.iter().map(|l| shift_line(l, shift)).collect()
}

Expand Down Expand Up @@ -160,20 +199,44 @@ pub fn take_rustdoc_include_anchored_lines(s: &str, anchor: &str) -> String {
#[cfg(test)]
mod tests {
use super::{
shift_line, take_anchored_lines, take_anchored_lines_with_shift, take_lines,
take_lines_with_shift, take_rustdoc_include_anchored_lines, take_rustdoc_include_lines,
Shift,
common_leading_ws, shift_line, take_anchored_lines, take_anchored_lines_with_shift,
take_lines, take_lines_with_shift, take_rustdoc_include_anchored_lines,
take_rustdoc_include_lines, ExplicitShift, Shift,
};

#[test]
fn common_leading_ws_test() {
let tests = [
([" line1", " line2", " line3"], " "),
([" line1", " line2", "line3"], ""),
(["\t\tline1", "\t\t line2", "\t\tline3"], "\t\t"),
(["\t line1", " \tline2", " \t\tline3"], ""),
];
for (lines, want) in tests {
let lines = lines.into_iter().map(|l| l.to_string()).collect::<Vec<_>>();
let got = common_leading_ws(&lines);
assert_eq!(got, want, "for input {lines:?}");
}
}

#[test]
fn shift_line_test() {
let s = " Line with 4 space intro";
assert_eq!(shift_line(s, Shift::None), s);
assert_eq!(shift_line(s, Shift::Left(4)), "Line with 4 space intro");
assert_eq!(shift_line(s, Shift::Left(2)), " Line with 4 space intro");
assert_eq!(shift_line(s, Shift::Left(6)), "ne with 4 space intro");
assert_eq!(shift_line(s, ExplicitShift::None), s);
assert_eq!(
shift_line(s, ExplicitShift::Left(4)),
"Line with 4 space intro"
);
assert_eq!(
shift_line(s, ExplicitShift::Left(2)),
" Line with 4 space intro"
);
assert_eq!(
shift_line(s, Shift::Right(2)),
shift_line(s, ExplicitShift::Left(6)),
"ne with 4 space intro"
);
assert_eq!(
shift_line(s, ExplicitShift::Right(2)),
" Line with 4 space intro"
);
}
Expand Down Expand Up @@ -207,6 +270,10 @@ mod tests {
take_lines_with_shift(s, 1..3, Shift::Right(2)),
" ipsum\n dolor"
);
assert_eq!(
take_lines_with_shift(s, 1..3, Shift::Auto),
"ipsum\n dolor"
);
assert_eq!(take_lines_with_shift(s, 3.., Shift::None), " sit\n amet");
assert_eq!(
take_lines_with_shift(s, 3.., Shift::Right(1)),
Expand All @@ -217,6 +284,10 @@ mod tests {
take_lines_with_shift(s, ..3, Shift::None),
" Lorem\n ipsum\n dolor"
);
assert_eq!(
take_lines_with_shift(s, ..3, Shift::Auto),
"Lorem\nipsum\n dolor"
);
assert_eq!(
take_lines_with_shift(s, ..3, Shift::Right(4)),
" Lorem\n ipsum\n dolor"
Expand All @@ -226,6 +297,10 @@ mod tests {
"rem\nsum\ndolor"
);
assert_eq!(take_lines_with_shift(s, .., Shift::None), s);
assert_eq!(
take_lines_with_shift(s, .., Shift::Auto),
"Lorem\nipsum\n dolor\nsit\namet"
);
// corner cases
assert_eq!(take_lines_with_shift(s, 4..3, Shift::None), "");
assert_eq!(take_lines_with_shift(s, 4..3, Shift::Left(2)), "");
Expand Down Expand Up @@ -307,6 +382,10 @@ mod tests {
take_anchored_lines_with_shift(s, "test", Shift::Left(2)),
"dolor\nsit\namet"
);
assert_eq!(
take_anchored_lines_with_shift(s, "test", Shift::Auto),
"dolor\nsit\namet"
);
assert_eq!(
take_anchored_lines_with_shift(s, "something", Shift::None),
""
Expand All @@ -333,6 +412,10 @@ mod tests {
take_anchored_lines_with_shift(s, "test", Shift::Left(2)),
"dolor\nsit\namet"
);
assert_eq!(
take_anchored_lines_with_shift(s, "test", Shift::Auto),
"dolor\nsit\namet"
);
assert_eq!(
take_anchored_lines_with_shift(s, "test", Shift::Left(4)),
"lor\nt\net"
Expand All @@ -359,6 +442,10 @@ mod tests {
take_anchored_lines_with_shift(s, "test", Shift::Left(2)),
"ipsum\ndolor\nsit\namet"
);
assert_eq!(
take_anchored_lines_with_shift(s, "test", Shift::Auto),
"ipsum\ndolor\nsit\namet"
);
assert_eq!(
take_anchored_lines_with_shift(s, "something", Shift::None),
""
Expand All @@ -371,6 +458,10 @@ mod tests {
take_anchored_lines_with_shift(s, "something", Shift::Left(2)),
""
);
assert_eq!(
take_anchored_lines_with_shift(s, "something", Shift::Auto),
""
);

// Include non-ASCII.
let s = " Lorem\n ANCHOR: test2\n ípsum\n ANCHOR: test\n dôlor\n sit\n amet\n ANCHOR_END: test\n lorem\n ANCHOR_END:test2\n ipsum";
Expand Down
4 changes: 4 additions & 0 deletions tests/dummy_book/src/first/nested.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,10 @@ assert!($TEST_STATUS);
{{#shiftinclude +2:nested-test-with-anchors.rs:myanchor}}
```

```rust
{{#shiftinclude auto:nested-test-with-anchors.rs:indentedanchor}}
```

## Rustdoc include adds the rest of the file as hidden

```rust
Expand Down

0 comments on commit 4b1b97b

Please sign in to comment.