Skip to content

Commit 3fad732

Browse files
committed
tools/docgen: pull out prev_prev once
We also only care about named (not anonymous) sibling nodes, so use just `prev_named_sibling()` instead of the mix of `prev_sibling()` and `prev_named_sibling()`.
1 parent ad82ec9 commit 3fad732

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

tools/src/bin/docgen/main.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -201,13 +201,14 @@ fn comment_and_requirement(
201201
prev: Node,
202202
src: &[u8],
203203
) -> Result<(Option<Comment>, Option<Feature>), Box<dyn Error>> {
204+
let prev_prev = prev.prev_named_sibling();
204205
let mut maybe_comment = Comment::new(prev, src).ok();
205206

206207
// If node wasn't a comment, see if it was an expression_statement
207208
// that itself was preceded by a comment. This skips over
208209
// expression-like preprocessor attributes on function decls.
209210
if let (None, "expression_statement", Some(prev_prev)) =
210-
(&maybe_comment, prev.kind(), prev.prev_sibling())
211+
(&maybe_comment, prev.kind(), prev_prev)
211212
{
212213
maybe_comment = Comment::new(prev_prev, src).ok();
213214
}
@@ -221,7 +222,7 @@ fn comment_and_requirement(
221222
}
222223

223224
// Otherwise, check the prev of the comment for a feature requirement
224-
let Some(prev_prev) = prev.prev_named_sibling() else {
225+
let Some(prev_prev) = prev_prev else {
225226
return Ok((maybe_comment, None));
226227
};
227228

0 commit comments

Comments
 (0)